core.async exception rewriting can result in an incorrect return value

Description

The following test case passes in core.async 0.2.395, but fails on 0.3.443 due to an incorrect return value from an exception catch. See comments in the test for details.

Environment

None

Activity

Show:
import
February 1, 2019, 11:27 AM

Comment made by: alesguzik

I have encountered a very similar case, and it is broken even with the patch above:

The result is

It doesn't matter whether I catch clojure.lang.ExceptionInfo or Throwable in any of those places in the test.

Kevin Downey
June 11, 2019, 9:59 PM

I suspect the above failing test is the same as which has a patch which builds on this one

Kevin Downey
January 4, 2020, 12:20 AM

rebased my local branch and generated a new patch

Alex Miller
January 9, 2020, 12:47 PM

Patch applied

Alex Miller
January 10, 2020, 6:48 PM

Released in 0.7.559

Assignee

Kevin Downey

Reporter

import

Labels

Approval

None

Patch

Code and Test

Priority

Major
Configure