304 for assets must have an empty/missing response

jknack · jknack · commit 5c60f82abf1f · 2014-10-08T10:10:51.000-03:00
diff --git a/jooby-tests/src/test/java/jooby/AssetFeature.java b/jooby-tests/src/test/java/jooby/AssetFeature.java
@@ -23,6 +23,7 @@ public void jsAsset() throws Exception {
     response = Request.Get(uri("/assets/file.js").build())
         .addHeader("If-Modified-Since", lastModified).execute().returnResponse();
     assertEquals(304, response.getStatusLine().getStatusCode());
+    assertEquals(null, response.getEntity());
   }
 
   @Test
@@ -36,6 +37,7 @@ public void cssAsset() throws Exception {
     response = Request.Get(uri("/assets/file.css").build())
         .addHeader("If-Modified-Since", lastModified).execute().returnResponse();
     assertEquals(304, response.getStatusLine().getStatusCode());
+    assertEquals(null, response.getEntity());
   }
 
 }

Original file line number	Diff line number	Diff line change
`@@ -23,6 +23,7 @@ public void jsAsset() throws Exception {`
`23`	`23`	`response = Request.Get(uri("/assets/file.js").build())`
`24`	`24`	`.addHeader("If-Modified-Since", lastModified).execute().returnResponse();`
`25`	`25`	`assertEquals(304, response.getStatusLine().getStatusCode());`
	`26`	`+ assertEquals(null, response.getEntity());`
`26`	`27`	`}`
`27`	`28`
`28`	`29`	`@Test`
`@@ -36,6 +37,7 @@ public void cssAsset() throws Exception {`
`36`	`37`	`response = Request.Get(uri("/assets/file.css").build())`
`37`	`38`	`.addHeader("If-Modified-Since", lastModified).execute().returnResponse();`
`38`	`39`	`assertEquals(304, response.getStatusLine().getStatusCode());`
	`40`	`+ assertEquals(null, response.getEntity());`
`39`	`41`	`}`
`40`	`42`
`41`	`43`	`}`