Merge pull request #138 from dadowns/master

Make hasEntity() work with a closed stream.

Author: Adam Lindenthal

core-common/src/main/java/org/glassfish/jersey/message/internal/InboundMessageContext.java

@@ -743,8 +743,6 @@ public void setWorkers(MessageBodyWorkers workers) {
      *         {@code false} otherwise.
      */
     public boolean hasEntity() {
-        entityContent.ensureNotClosed();
-
         try {
             return !entityContent.isEmpty();
         } catch (IllegalStateException ex) {

tests/e2e/src/test/java/org/glassfish/jersey/tests/e2e/client/ResponseCloseTest.java

@@ -49,6 +49,8 @@
 import org.glassfish.jersey.test.JerseyTest;
 
 import org.junit.Test;
+
+import static org.junit.Assert.assertFalse;
 import static org.junit.Assert.fail;
 
 /**
@@ -160,10 +162,10 @@ public void testHasEntityAfterClose() {
         final Response response = target().path("simple").request().get(Response.class);
         response.close();
         try {
-            response.hasEntity();
-            fail("IllegalStateException expected when reading a buffered entity after response has been closed.");
+            boolean hasEntity = response.hasEntity();
+            assertFalse("Should return false when the connection is already closed", hasEntity);
         } catch (IllegalStateException ex) {
-            // expected
+            fail("IllegalStateException should have been caught inside hasEntity.");
         }
     }
 }