phaller / reactive-async

Expressive deterministic concurrency in Scala

BSD 2-Clause "Simplified" License

15 stars 9 forks source link

Track RejectedExecutionExceptions #127

Closed phaller closed 6 years ago

phaller commented 6 years ago

Adjust problematic tests to first wait for quiescence before shutting down.
Add asserts to check that no RejectedExecutionExceptions were encountered.

JanKoelzer commented 6 years ago

118 is related to RejectedExecutionExceptions as well.

phaller commented 6 years ago

@JanKoelzer thanks for the pointer! I should have seen that earlier. ;-)

But, yeah, currently we have the big issue that the test suite is broken and CI is essentially useless, because it may succeed even in the presence of RejectedExecutionExceptions.

phaller commented 6 years ago

@JanKoelzer your PR appears to have fixed the test suite- excellent! Thanks!

JanKoelzer commented 6 years ago

This has been fixed by #118 and can be closed.