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

HTTP/2 shutdown cleanup miss

Browse Source 
Motiviation:
https://github.com/netty/netty/pull/3865 was merged from a machine with old code. A test case that was updates was not merged.

Modifications:
- Merge the missing test case updates

Result:
Test case no longer fails.
This commit is contained in:
Scott Mitchell 2015-06-08 07:59:36 -07:00
parent 2449760ea3
commit e58e338ffd
1 changed files with 3 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
codec-http2/src/test/java/io/netty/handler/codec/http2/Http2ConnectionHandlerTest.java
View File

@ -319,6 +319,9 @@ public class Http2ConnectionHandlerTest {
} }
}).when(future).addListener(any(GenericFutureListener.class)); }).when(future).addListener(any(GenericFutureListener.class));
handler.close(ctx, promise); handler.close(ctx, promise);
if (future.isDone()) {
when(connection.numActiveStreams()).thenReturn(0);
}
handler.closeStream(stream, future); handler.closeStream(stream, future);
// Simulate another stream close call being made after the context should already be closed. // Simulate another stream close call being made after the context should already be closed.
handler.closeStream(stream, future); handler.closeStream(stream, future);