From 3e0cbf0caa69df9960a013afc1e7776373aff9ea Mon Sep 17 00:00:00 2001 From: Trustin Lee Date: Sun, 3 Jun 2012 03:32:30 -0700 Subject: [PATCH] Fix HttpClientCodecTest.testFailsOnMissingResponse() - AbstractCodecEmbedder does not throw an exception immediately anymore. It stores the caught exceptions in the product queue and throws them on pool() or peek(). --- .../io/netty/handler/codec/http/HttpClientCodecTest.java | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/codec-http/src/test/java/io/netty/handler/codec/http/HttpClientCodecTest.java b/codec-http/src/test/java/io/netty/handler/codec/http/HttpClientCodecTest.java index 994836b3a7..e6b2cbe2a5 100644 --- a/codec-http/src/test/java/io/netty/handler/codec/http/HttpClientCodecTest.java +++ b/codec-http/src/test/java/io/netty/handler/codec/http/HttpClientCodecTest.java @@ -68,13 +68,15 @@ public class HttpClientCodecTest { HttpClientCodec codec = new HttpClientCodec(4096, 8192, 8192, true); EncoderEmbedder encoder = new EncoderEmbedder(codec); - encoder.offer(new DefaultHttpRequest(HttpVersion.HTTP_1_1, HttpMethod.GET, "http://localhost/")); + assertTrue(encoder.offer(new DefaultHttpRequest(HttpVersion.HTTP_1_1, HttpMethod.GET, "http://localhost/"))); + assertNotNull(encoder.poll()); + assertTrue(encoder.finish()); try { - encoder.finish(); + encoder.poll(); fail(); } catch (CodecException e) { - assertTrue(e.getCause() instanceof PrematureChannelClosureException); + assertTrue(e instanceof PrematureChannelClosureException); } }