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

Fixing NPE in StreamBufferingEncoderTest

Browse Source 
Motivation:

The bufferingNewStreamFailsAfterGoAwayReceived method currently causes an NPE.

Modifications:

Fixed the test so that a valid ByteBuf is passed in.

Result:

The test no longer throws an NPE.
This commit is contained in:
nmittler 2015-07-20 12:44:52 -07:00
parent f2b445f97f
commit 3d6819623e
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
codec-http2/src/test/java/io/netty/handler/codec/http2/StreamBufferingEncoderTest.java
View File

@ -204,7 +204,7 @@ public class StreamBufferingEncoderTest {
public void bufferingNewStreamFailsAfterGoAwayReceived() { public void bufferingNewStreamFailsAfterGoAwayReceived() {
encoder.writeSettingsAck(ctx, promise); encoder.writeSettingsAck(ctx, promise);
setMaxConcurrentStreams(0); setMaxConcurrentStreams(0);
connection.goAwayReceived(1, 8, null); connection.goAwayReceived(1, 8, EMPTY_BUFFER);
promise = mock(ChannelPromise.class); promise = mock(ChannelPromise.class);
encoderWriteHeaders(3, promise); encoderWriteHeaders(3, promise);