test: Integration coverage for extra handlers and builder

Author: thced

src/test/java/com/retailsvc/http/ExtraHandlersIT.java

@@ -0,0 +1,107 @@
+package com.retailsvc.http;
+
+import static com.retailsvc.http.Handlers.defaultExceptionHandler;
+import static org.assertj.core.api.Assertions.assertThat;
+import static org.junit.jupiter.api.Assertions.fail;
+
+import com.sun.net.httpserver.HttpHandler;
+import java.io.IOException;
+import java.net.URI;
+import java.net.http.HttpRequest;
+import java.net.http.HttpResponse.BodyHandlers;
+import java.util.Map;
+import org.junit.jupiter.api.Test;
+
+class ExtraHandlersIT extends ServerBaseTest {
+
+  @Test
+  void aliveExtraReturns204AndBypassesValidation() throws Exception {
+    try (var s =
+            OpenApiServer.builder()
+                .spec(spec)
+                .jsonMapper(jsonMapper())
+                .handlers(Map.of())
+                .exceptionHandler(defaultExceptionHandler())
+                .port(0)
+                .addHandler("/alive", Handlers.aliveHandler())
+                .build();
+        var client = httpClient()) {
+
+      var req =
+          HttpRequest.newBuilder()
+              .uri(URI.create("http://localhost:" + s.listenPort() + "/alive"))
+              .GET()
+              .build();
+      var resp = client.send(req, BodyHandlers.ofString());
+
+      assertThat(resp.statusCode()).isEqualTo(204);
+      assertThat(resp.body()).isEmpty();
+    }
+  }
+
+  @Test
+  void specHandlerServesClasspathResource() throws Exception {
+    try (var s =
+            OpenApiServer.builder()
+                .spec(spec)
+                .jsonMapper(jsonMapper())
+                .handlers(Map.of())
+                .exceptionHandler(defaultExceptionHandler())
+                .port(0)
+                .addHandler("/openapi.yaml", Handlers.specHandler("/openapi.yaml"))
+                .build();
+        var client = httpClient()) {
+
+      var req =
+          HttpRequest.newBuilder()
+              .uri(URI.create("http://localhost:" + s.listenPort() + "/openapi.yaml"))
+              .GET()
+              .build();
+      var resp = client.send(req, BodyHandlers.ofString());
+
+      assertThat(resp.statusCode()).isEqualTo(200);
+      assertThat(resp.headers().firstValue("Content-Type")).contains("application/yaml");
+      assertThat(resp.body()).isNotEmpty();
+    }
+  }
+
+  @Test
+  void extraHandlerExceptionFlowsThroughExceptionHandler() throws Exception {
+    HttpHandler boom =
+        ex -> {
+          throw new RuntimeException("boom");
+        };
+
+    try (var s =
+            OpenApiServer.builder()
+                .spec(spec)
+                .jsonMapper(jsonMapper())
+                .handlers(Map.of())
+                .exceptionHandler(defaultExceptionHandler())
+                .port(0)
+                .addHandler("/boom", boom)
+                .build();
+        var client = httpClient()) {
+
+      var req =
+          HttpRequest.newBuilder()
+              .uri(URI.create("http://localhost:" + s.listenPort() + "/boom"))
+              .GET()
+              .build();
+      var resp = client.send(req, BodyHandlers.ofString());
+
+      assertThat(resp.statusCode()).isEqualTo(500);
+    }
+  }
+
+  @Test
+  void existingPublicConstructorStillWorks() {
+    try {
+      try (var s = new OpenApiServer(spec, jsonMapper(), Map.of(), defaultExceptionHandler(), 0)) {
+        assertThat(s.listenPort()).isGreaterThan(0);
+      }
+    } catch (IOException io) {
+      fail(io);
+    }
+  }
+}