andreacavalli/netty5
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Fix sporadic assertion failure in SocketSslEchoTest

Browse Source 
Motivation:

SocketSslEchoTest.testSslEcho() has a race condition where a renegotiation future can be done before:

    assertThat(renegoFuture.isDone(), is(false));

Modifications:

Remove the offending assertion.

Result:

More build stability
This commit is contained in:
Trustin Lee 2015-06-01 13:55:20 +09:00
parent 70e3d17620
commit c7b164b409
1 changed files with 0 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
testsuite/src/main/java/io/netty/testsuite/transport/socket/SocketSslEchoTest.java
View File

@ -300,7 +300,6 @@ public class SocketSslEchoTest extends AbstractSocketTest {
renegoFuture = clientSslHandler.renegotiate(); renegoFuture = clientSslHandler.renegotiate();
logStats("CLIENT RENEGOTIATES"); logStats("CLIENT RENEGOTIATES");
assertThat(renegoFuture, is(not(sameInstance(clientHandshakeFuture)))); assertThat(renegoFuture, is(not(sameInstance(clientHandshakeFuture))));
assertThat(renegoFuture.isDone(), is(false));
} }
} }