Hook HeadResponder into ResponderSupplierFactory

onlyskin · onlyskin · commit 821c6da98dc3 · 2017-12-15T13:33:54.000+02:00
diff --git a/src/main/java/httpserver/Method.java b/src/main/java/httpserver/Method.java
@@ -1,5 +1,5 @@
 package httpserver;
 
 public enum Method {
-    GET, POST, PUT, DELETE, OPTIONS
+    GET, HEAD, POST, PUT, DELETE, OPTIONS
 }
diff --git a/src/main/java/httpserver/ResponderSupplierFactory.java b/src/main/java/httpserver/ResponderSupplierFactory.java
@@ -13,6 +13,10 @@ public ResponderSupplier makeResponderSupplier() {
                 getRouteMap(),
                 new PathExaminer(),
                 new Html()));
+        responderSupplier.registerResponder(Method.HEAD, new HeadResponder(
+                getRouteMap(),
+                new PathExaminer(),
+                new Html()));
         responderSupplier.registerResponder(Method.POST, new PostResponder(
                 new PathExaminer(),
                 new FileOperator()));
diff --git a/src/test/java/httpserver/responder/OptionsResponderTest.java b/src/test/java/httpserver/responder/OptionsResponderTest.java
@@ -21,6 +21,7 @@ public void responderHasListOfAllowedMethodsInAllowHeader() throws Exception {
 
         ResponderSupplier responderSupplier = new ResponderSupplier(mock(InvalidMethodResponder.class));
         responderSupplier.registerResponder(Method.GET, trueResponderMock);
+        responderSupplier.registerResponder(Method.HEAD, trueResponderMock);
         responderSupplier.registerResponder(Method.POST, falseResponderMock);
         responderSupplier.registerResponder(Method.PUT, trueResponderMock);
         responderSupplier.registerResponder(Method.DELETE, falseResponderMock);
@@ -32,7 +33,7 @@ public void responderHasListOfAllowedMethodsInAllowHeader() throws Exception {
         Response response = optionsResponder.respond(mock(AppConfig.class), request);
 
         assertEquals(200, response.getStatusCode());
-        assertEquals(new Header("Allow", "GET,PUT,OPTIONS"), response.getHeaders()[0]);
+        assertEquals(new Header("Allow", "GET,HEAD,PUT,OPTIONS"), response.getHeaders()[0]);
     }
 
     @Test

Original file line number	Diff line number	Diff line change
`@@ -1,5 +1,5 @@`
`1`	`1`	`package httpserver;`
`2`	`2`
`3`	`3`	`public enum Method {`
`4`		`- GET, POST, PUT, DELETE, OPTIONS`
	`4`	`+ GET, HEAD, POST, PUT, DELETE, OPTIONS`
`5`	`5`	`}`