kriegaex edited a comment on pull request #343: URL: https://github.com/apache/maven-surefire/pull/343#issuecomment-836122555
Concerning `E2ETest::endToEndTest`, it hangs at `assertThat( awaitHandlerFinished.await( 30L, TimeUnit.SECONDS ) ).isTrue();`. This is true even if I set `final int totalCalls = 1;`, i.e. assuming that whatever the test does is actually happening, the countdown latch is never set correctly. It looks like a concurrency issue at first glance. Maybe later I have time to take another look. For now, my impression is that maybe nothing is slow but the test needs to be adjusted to your code changes. I would lie to tell you that I understand what the tests in `E2ETest` do exactly, so my hunch could be wrong. What I do see, though, is that the code inside `EventHandler<Event> h` is never ever called. I added both breakpoints and debug logs and nothing ever happens there. --- **Update:** It seems, in `SurefireForkChannel`, you set `private final Bindings bindings = new Bindings( 3 );`, but in `E2ETest::endToEndTest` you only call `bindEventHandler`, not `bindCommandReader`, i.e. the binding never counts down to 0 and `job()` is never called. What that all means, I have no clue. I just see that things do not happen which probably are supposed to. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org
