Apply pull request suggestions

Author: mcruzdev

moqu/core/src/main/java/io/quarkiverse/openapi/moqu/marshall/ObjectMapperFactory.java

@@ -7,12 +7,9 @@
  */
 public class ObjectMapperFactory {
 
-    private static ObjectMapper objectMapper;
+    private static ObjectMapper objectMapper = new ObjectMapper();
 
     public static ObjectMapper getInstance() {
-        if (objectMapper == null) {
-            objectMapper = new ObjectMapper();
-        }
         return objectMapper;
     }
 }

moqu/core/src/main/java/io/quarkiverse/openapi/moqu/model/Header.java

@@ -1,12 +1,12 @@
 package io.quarkiverse.openapi.moqu.model;
 
-import java.util.Set;
+import java.util.List;
 
 /**
  * Represents an HTTP header with a name and a set of associated values.
  *
  * @param name the name of the HTTP header (e.g., "Accept", "Content-Type").
  * @param value the set of values associated with the header, allowing multiple values (e.g., "application/json", "text/html").
  */
-public record Header(String name, Set<String> value) {
+public record Header(String name, List<String> value) {
 }

moqu/core/src/main/java/io/quarkiverse/openapi/moqu/model/Request.java

@@ -1,5 +1,6 @@
 package io.quarkiverse.openapi.moqu.model;
 
+import java.util.Collection;
 import java.util.List;
 
 /**
@@ -12,5 +13,5 @@
  * @param accept the "Accept" header, which specifies the expected response format.
  * @param parameters the list of parameters to be included in the request.
  */
-public record Request(String url, String httpMethod, String exampleName, Header accept, List<Parameter> parameters) {
+public record Request(String url, String httpMethod, String exampleName, Header accept, Collection<Parameter> parameters) {
 }

moqu/core/src/test/java/io/quarkiverse/openapi/moqu/OpenAPIMoquImporterTest.java

@@ -1,6 +1,6 @@
 package io.quarkiverse.openapi.moqu;
 
-import static io.quarkiverse.openapi.moqu.Testing.readContentFromFile;
+import static io.quarkiverse.openapi.moqu.TestUtils.readContentFromFile;
 
 import java.util.List;
 import java.util.Map;

moqu/core/src/test/java/io/quarkiverse/openapi/moqu/Testing.java renamed to moqu/core/src/test/java/io/quarkiverse/openapi/moqu/TestUtils.java

@@ -6,7 +6,7 @@
 import java.nio.file.Files;
 import java.nio.file.Path;
 
-public class Testing {
+public class TestUtils {
 
     public static String readContentFromFile(String resourcePath) {
         URL url = Thread.currentThread().getContextClassLoader().getResource((resourcePath));

moqu/core/src/test/java/io/quarkiverse/openapi/moqu/wiremock/mapper/WiremockMapperTest.java

@@ -1,6 +1,6 @@
 package io.quarkiverse.openapi.moqu.wiremock.mapper;
 
-import static io.quarkiverse.openapi.moqu.Testing.readContentFromFile;
+import static io.quarkiverse.openapi.moqu.TestUtils.readContentFromFile;
 
 import java.util.List;
 

moqu/core/src/test/java/io/quarkiverse/openapi/moqu/wiremock/mapper/WiremockPathParamTest.java

@@ -9,7 +9,7 @@
 
 import io.quarkiverse.openapi.moqu.Moqu;
 import io.quarkiverse.openapi.moqu.OpenAPIMoquImporter;
-import io.quarkiverse.openapi.moqu.Testing;
+import io.quarkiverse.openapi.moqu.TestUtils;
 import io.quarkiverse.openapi.moqu.wiremock.model.WiremockMapping;
 
 public class WiremockPathParamTest {
@@ -18,7 +18,7 @@ public class WiremockPathParamTest {
     @DisplayName("Should convert a OpenAPI with a single path param correctly")
     void shouldMapOneWiremockDefinition() {
 
-        String content = Testing.readContentFromFile("wiremock/path_param_one_path_param.yml");
+        String content = TestUtils.readContentFromFile("wiremock/path_param_one_path_param.yml");
         if (content == null) {
             Assertions.fail("Was not possible to read the file!");
         }
@@ -50,7 +50,7 @@ void shouldMapOneWiremockDefinition() {
     @DisplayName("Should convert with a two OpenAPI#paths each one with one path param")
     void shouldMapTwoWiremockDefinitions() {
 
-        String content = Testing.readContentFromFile("wiremock/path_param_two_params_but_different_path.yml");
+        String content = TestUtils.readContentFromFile("wiremock/path_param_two_params_but_different_path.yml");
         if (content == null) {
             Assertions.fail("Was not possible to read the file!");
         }
@@ -91,7 +91,7 @@ void shouldMapTwoWiremockDefinitions() {
     @DisplayName("Should convert with a combination of path param")
     void shouldConvertWithACombinationOfPathParam() {
 
-        String content = Testing.readContentFromFile("wiremock/path_param_two_path_params_combination.yml");
+        String content = TestUtils.readContentFromFile("wiremock/path_param_two_path_params_combination.yml");
         if (content == null) {
             Assertions.fail("Was not possible to read the file!");
         }
@@ -133,7 +133,7 @@ void shouldConvertWithACombinationOfPathParam() {
     @DisplayName("Should convert with a combination but only one with example")
     void shouldConvertPathParamCombinationOnlyOneWithExample() {
 
-        String content = Testing.readContentFromFile("wiremock/path_param_two_path_params_only_one_with_example.yml");
+        String content = TestUtils.readContentFromFile("wiremock/path_param_two_path_params_only_one_with_example.yml");
         if (content == null) {
             Assertions.fail("Was not possible to read the file!");
         }

moqu/moqu-wiremock/deployment/pom.xml renamed to moqu/deployment/pom.xml

@@ -17,11 +17,13 @@
             <groupId>io.quarkus</groupId>
             <artifactId>quarkus-core-deployment</artifactId>
         </dependency>
+
         <dependency>
             <groupId>io.quarkiverse.openapi.generator</groupId>
             <artifactId>quarkus-openapi-generator-moqu-wiremock</artifactId>
             <version>${project.version}</version>
         </dependency>
+
         <dependency>
             <groupId>io.quarkiverse.openapi.generator</groupId>
             <artifactId>quarkus-openapi-generator-moqu-core</artifactId>