Rename response

onlyskin · onlyskin · commit d3916209c29c · 2017-12-19T19:34:11.000+02:00
diff --git a/src/main/java/httpserver/responder/special/CoffeeResponder.java b/src/main/java/httpserver/responder/special/CoffeeResponder.java
@@ -3,13 +3,13 @@
 import httpserver.AppConfig;
 import httpserver.Request;
 import httpserver.responder.Responder;
-import httpserver.response.FourEighteenResponse;
+import httpserver.response.TeapotResponse;
 import httpserver.response.Response;
 
 public class CoffeeResponder implements Responder {
     @Override
     public Response respond(AppConfig appConfig, Request request) {
-        return new FourEighteenResponse();
+        return new TeapotResponse();
     }
 
     @Override
diff --git a/src/main/java/httpserver/response/TeapotResponse.java b/src/main/java/httpserver/response/TeapotResponse.java
@@ -1,7 +1,7 @@
 package httpserver.response;
 
-public class FourEighteenResponse extends Response {
-    public FourEighteenResponse() {
+public class TeapotResponse extends Response {
+    public TeapotResponse() {
         super.setPayload("I'm a teapot".getBytes());
     }
 
diff --git a/src/test/java/httpserver/ResponseWriterTest.java b/src/test/java/httpserver/ResponseWriterTest.java
@@ -1,7 +1,7 @@
 package httpserver;
 
 import httpserver.header.Header;
-import httpserver.response.FourEighteenResponse;
+import httpserver.response.TeapotResponse;
 import httpserver.response.NotFoundResponse;
 import httpserver.response.OkResponse;
 import httpserver.response.Response;
@@ -54,7 +54,7 @@ public void itWritesTheFirstLineFor404() throws Exception {
 
     @Test
     public void itWritesTheFirstLineFor418() throws Exception {
-        String output = outputForResponse(new FourEighteenResponse());
+        String output = outputForResponse(new TeapotResponse());
 
         assertTrue(output.contains("HTTP/1.1 418 I'm a teapot\r\n"));
     }
diff --git a/src/test/java/httpserver/response/FourEighteenResponseTest.java b/src/test/java/httpserver/response/FourEighteenResponseTest.java
diff --git a/src/test/java/httpserver/response/TeapotResponseTest.java b/src/test/java/httpserver/response/TeapotResponseTest.java
@@ -0,0 +1,33 @@
+package httpserver.response;
+
+import httpserver.header.Header;
+import org.junit.Test;
+
+import java.util.Arrays;
+
+import static junit.framework.TestCase.*;
+
+public class TeapotResponseTest {
+
+    private final TeapotResponse teapotResponse;
+
+    public TeapotResponseTest() {
+        teapotResponse = new TeapotResponse();
+    }
+
+    @Test
+    public void hasStatusCode405() throws Exception {
+        assertEquals(418, teapotResponse.getStatusCode());
+    }
+
+    @Test
+    public void hasEmptyPayload() throws Exception {
+        assertEquals("I'm a teapot", new String(teapotResponse.getPayload()));
+    }
+
+    @Test
+    public void hasNoHeaders() throws Exception {
+        assertTrue(Arrays.equals(new Header[0], teapotResponse.getHeaders()));
+    }
+
+}

Original file line number	Diff line number	Diff line change
`@@ -1,7 +1,7 @@`
`1`	`1`	`package httpserver.response;`
`2`	`2`
`3`		`-public class FourEighteenResponse extends Response {`
`4`		`- public FourEighteenResponse() {`
	`3`	`+public class TeapotResponse extends Response {`
	`4`	`+ public TeapotResponse() {`
`5`	`5`	`super.setPayload("I'm a teapot".getBytes());`
`6`	`6`	`}`
`7`	`7`