Skip to content

Commit

Permalink
Fix assertion in AbstractSimpleTransportTestCase (#32991)
Browse files Browse the repository at this point in the history
This is a follow-up to #32956. That commit incorrectly used assertBusy
which led to a possible race in the test. This commit fixes it.
  • Loading branch information
Tim-Brooks committed Aug 21, 2018
1 parent 2c75504 commit 23035de
Showing 1 changed file with 1 addition and 1 deletion.
Expand Up @@ -2653,7 +2653,7 @@ public void testChannelCloseWhileConnecting() {
public void onConnectionOpened(final Transport.Connection connection) {
closeConnectionChannel(connection);
try {
assertBusy(connection::isClosed);
assertBusy(() -> assertTrue(connection.isClosed()));
} catch (Exception e) {
throw new AssertionError(e);
}
Expand Down

0 comments on commit 23035de

Please sign in to comment.