diff --git a/src/main/java/com/retailsvc/http/OpenApiServer.java b/src/main/java/com/retailsvc/http/OpenApiServer.java index bd0e069..550ecff 100644 --- a/src/main/java/com/retailsvc/http/OpenApiServer.java +++ b/src/main/java/com/retailsvc/http/OpenApiServer.java @@ -307,9 +307,9 @@ private static void validateSecurityWiring(Spec spec, Map - OpenApiServer.builder() - .spec(spec) - .handlers(Map.of("getX", req -> Response.ok(Map.of()))) - .port(0) - .build()) + assertThatThrownBy(builder::build) .isInstanceOf(IllegalStateException.class) .hasMessageContaining("bearerAuth"); } @@ -71,14 +66,9 @@ void unsupportedSchemeThrowsWhenReferenced() { List.of(), List.of(Map.of("oauth", List.of()))); Spec spec = Spec.from(r); + OpenApiServer.Builder builder = handlerBuilder(spec); - assertThatThrownBy( - () -> - OpenApiServer.builder() - .spec(spec) - .handlers(Map.of("getX", req -> Response.ok(Map.of()))) - .port(0) - .build()) + assertThatThrownBy(builder::build) .isInstanceOf(IllegalStateException.class) .hasMessageContaining("unsupported"); } @@ -91,18 +81,20 @@ void unknownSchemeReferenceThrows() { List.of(), List.of(Map.of("missingScheme", List.of()))); Spec spec = Spec.from(r); + OpenApiServer.Builder builder = handlerBuilder(spec); - assertThatThrownBy( - () -> - OpenApiServer.builder() - .spec(spec) - .handlers(Map.of("getX", req -> Response.ok(Map.of()))) - .port(0) - .build()) + assertThatThrownBy(builder::build) .isInstanceOf(IllegalStateException.class) .hasMessageContaining("missingScheme"); } + private static OpenApiServer.Builder handlerBuilder(Spec spec) { + return OpenApiServer.builder() + .spec(spec) + .handlers(Map.of("getX", req -> Response.ok(Map.of()))) + .port(0); + } + @Test void externalAuthSkipsAllChecks() throws Exception { Map r =