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

Fix flaky ChannelInitializerTest.testChannelInitializerEventExecutor() (#8738)

Browse Source 
Motivation:

testChannelInitializerEventExecutor() did sometimes fail as we sometimes miss to count down the latch. This can happen when we remove the handler from the pipeline before channelUnregistered(...) was called for it.

Modifications:

Countdown the latch in handlerRemoved(...).

Result:

Fix flaky test.
This commit is contained in:
Norman Maurer 2019-01-21 09:01:04 +01:00
parent 8bcea35d2e
commit d870199c4e
1 changed files with 2 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
transport/src/test/java/io/netty/channel/ChannelInitializerTest.java
View File

@ -338,7 +338,7 @@ public class ChannelInitializerTest {
}
@Override
public void channelUnregistered(ChannelHandlerContext ctx) {
public void handlerRemoved(ChannelHandlerContext ctx) {
latch.countDown();
}
});
@ -372,6 +372,7 @@ public class ChannelInitializerTest {
client.closeFuture().sync();
server.closeFuture().sync();
// Wait until the handler is removed from the pipeline and so no more events are handled by it.
latch.await();
assertEquals(1, invokeCount.get());