test(jaxrs-spec/quarkus): add missing ResponseStatus annotation test coverage

Add negative tests for returnJBossResponse=true and non-Quarkus libraries to
verify @ResponseStatus is suppressed, and a positive test for interfaceOnly=false
to cover the apiMethod.mustache code path. Also fixes the response filter to
restrict to pure numeric 2xx codes only, removing 3xx support and wildcard
code guard, and drops the redundant global additionalProperties flag.

Author: Ignacio-Vidal

modules/openapi-generator/src/main/java/org/openapitools/codegen/languages/JavaJAXRSSpecServerCodegen.java

@@ -346,14 +346,13 @@ public OperationsMap postProcessOperationsWithModels(OperationsMap objs, List<Mo
         if (QUARKUS_LIBRARY.equals(library) && !returnResponse && !returnJbossResponse) {
             for (CodegenOperation op : objs.getOperations().getOperation()) {
                 op.responses.stream()
-                        .filter(r -> r.is2xx || r.is3xx)
+                        .filter(r -> r.is2xx && r.code.matches("\\d+"))
                         .findFirst()
                         .ifPresent(r -> op.vendorExtensions.put("x-java-success-response-code", r.code));
             }
             if (objs.getOperations().getOperation().stream()
                     .anyMatch(op -> op.vendorExtensions.containsKey("x-java-success-response-code"))) {
                 objs.put("hasResponseStatusAnnotations", true);
-                additionalProperties.put("hasResponseStatusAnnotations", true);
             }
         }
         return objs;

modules/openapi-generator/src/test/java/org/openapitools/codegen/java/jaxrs/JavaJAXRSSpecServerCodegenTest.java

@@ -1444,20 +1444,86 @@ public void generateQuarkusInterfaceDoesNotAddResponseStatusAnnotationWhenReturn
     }
 
     /**
-     * Verify that when using the quarkus library with interfaceOnly=true and a 3xx response,
-     * the generated interface method is annotated with {@code @ResponseStatus(<code>)}.
+     * Verify that the {@code @ResponseStatus} annotation is NOT emitted when returnJBossResponse=true,
+     * because the caller controls the status code via the {@code RestResponse} wrapper in that mode.
      */
     @Test
-    public void generateQuarkusInterfaceAddsResponseStatusAnnotationFor3xxResponseCode() throws Exception {
+    public void generateQuarkusInterfaceDoesNotAddResponseStatusAnnotationWhenReturnJBossResponse() throws Exception {
         final File output = Files.createTempDirectory("test").toFile().getCanonicalFile();
         output.deleteOnExit();
 
         final OpenAPI openAPI = new OpenAPIParser()
-                .readLocation("src/test/resources/3_0/jaxrs-spec/petstore-quarkus.yaml", null, new ParseOptions()).getOpenAPI();
+                .readLocation("src/test/resources/3_0/ping.yaml", null, new ParseOptions()).getOpenAPI();
 
         codegen.setOutputDir(output.getAbsolutePath());
         codegen.setLibrary(QUARKUS_LIBRARY);
         codegen.additionalProperties().put(INTERFACE_ONLY, true);
+        codegen.additionalProperties().put(USE_JAKARTA_EE, true); //Required by returnJBossResponse
+        codegen.additionalProperties().put(RETURN_JBOSS_RESPONSE, true); //Given returnJBossResponse is true
+
+        final ClientOptInput input = new ClientOptInput()
+                .openAPI(openAPI)
+                .config(codegen);
+
+        final DefaultGenerator generator = new DefaultGenerator();
+        final List<File> files = generator.opts(input).generate();
+
+        validateJavaSourceFiles(files);
+
+        //Then the annotation must NOT appear
+        TestUtils.ensureContainsFile(files, output, "src/gen/java/org/openapitools/api/PingApi.java");
+        assertFileNotContains(output.toPath().resolve("src/gen/java/org/openapitools/api/PingApi.java"),
+                "@ResponseStatus",
+                "import org.jboss.resteasy.reactive.ResponseStatus");
+    }
+
+    /**
+     * Verify that {@code @ResponseStatus} is NOT emitted when using a non-Quarkus jaxrs-spec library,
+     * since {@code org.jboss.resteasy.reactive.ResponseStatus} is a RESTEasy Reactive / Quarkus-specific annotation.
+     */
+    @Test
+    public void generateNonQuarkusInterfaceDoesNotAddResponseStatusAnnotation() throws Exception {
+        final File output = Files.createTempDirectory("test").toFile().getCanonicalFile();
+        output.deleteOnExit();
+
+        final OpenAPI openAPI = new OpenAPIParser()
+                .readLocation("src/test/resources/3_0/ping.yaml", null, new ParseOptions()).getOpenAPI();
+
+        codegen.setOutputDir(output.getAbsolutePath());
+        // No setLibrary call — uses the default jaxrs-spec library
+        codegen.additionalProperties().put(INTERFACE_ONLY, true);
+
+        final ClientOptInput input = new ClientOptInput()
+                .openAPI(openAPI)
+                .config(codegen);
+
+        final DefaultGenerator generator = new DefaultGenerator();
+        final List<File> files = generator.opts(input).generate();
+
+        validateJavaSourceFiles(files);
+
+        //Then the annotation must NOT appear
+        TestUtils.ensureContainsFile(files, output, "src/gen/java/org/openapitools/api/PingApi.java");
+        assertFileNotContains(output.toPath().resolve("src/gen/java/org/openapitools/api/PingApi.java"),
+                "@ResponseStatus",
+                "import org.jboss.resteasy.reactive.ResponseStatus");
+    }
+
+    /**
+     * Verify that when using the quarkus library with interfaceOnly=false (concrete stub class),
+     * the generated implementation method is also annotated with {@code @ResponseStatus(<code>)}.
+     */
+    @Test
+    public void generateQuarkusConcreteClassAddsResponseStatusAnnotation() throws Exception {
+        final File output = Files.createTempDirectory("test").toFile().getCanonicalFile();
+        output.deleteOnExit();
+
+        final OpenAPI openAPI = new OpenAPIParser()
+                .readLocation("src/test/resources/3_0/ping.yaml", null, new ParseOptions()).getOpenAPI();
+
+        codegen.setOutputDir(output.getAbsolutePath());
+        codegen.setLibrary(QUARKUS_LIBRARY);
+        codegen.additionalProperties().put(INTERFACE_ONLY, false); //Given the concrete class is generated
 
         final ClientOptInput input = new ClientOptInput()
                 .openAPI(openAPI)
@@ -1468,10 +1534,10 @@ public void generateQuarkusInterfaceAddsResponseStatusAnnotationFor3xxResponseCo
 
         validateJavaSourceFiles(files);
 
-        //Then the generated interface contains the ResponseStatus import and annotation with code 302
-        TestUtils.ensureContainsFile(files, output, "src/gen/java/org/openapitools/api/RedirectApi.java");
-        assertFileContains(output.toPath().resolve("src/gen/java/org/openapitools/api/RedirectApi.java"),
+        //Then the generated class contains the ResponseStatus import and annotation with code 201
+        TestUtils.ensureContainsFile(files, output, "src/gen/java/org/openapitools/api/PingApi.java");
+        assertFileContains(output.toPath().resolve("src/gen/java/org/openapitools/api/PingApi.java"),
                 "import org.jboss.resteasy.reactive.ResponseStatus;",
-                "@ResponseStatus(302)");
+                "@ResponseStatus(201)");
     }
 }