enable strict YAML parsing

Author: jmini

modules/swagger-parser-v3/src/main/java/io/swagger/v3/parser/ObjectMapperFactory.java

@@ -15,19 +15,19 @@ public static ObjectMapper createJson() {
     }
 
     protected static ObjectMapper createJson(boolean includePathDeserializer, boolean includeResponseDeserializer) {
-        return create(null, includePathDeserializer, includeResponseDeserializer);
+        return create(createJsonFactory(), includePathDeserializer, includeResponseDeserializer);
     }
 
     public static ObjectMapper createYaml() {
         return createYaml(true, true);
     }
 
     protected static ObjectMapper createYaml(boolean includePathDeserializer, boolean includeResponseDeserializer) {
-        return create(new YAMLFactory(), includePathDeserializer, includeResponseDeserializer);
+        return create(createYamlFactory(), includePathDeserializer, includeResponseDeserializer);
     }
 
     private static ObjectMapper create(JsonFactory jsonFactory, boolean includePathDeserializer, boolean includeResponseDeserializer) {
-        ObjectMapper mapper = jsonFactory == null ? new ObjectMapper(createJsonFactory()) : new ObjectMapper(jsonFactory);
+        ObjectMapper mapper = new ObjectMapper(jsonFactory);
 
         mapper.setSerializationInclusion(JsonInclude.Include.NON_NULL);
         mapper.configure(SerializationFeature.FAIL_ON_EMPTY_BEANS, false);
@@ -42,4 +42,10 @@ private static JsonFactory createJsonFactory() {
           .enable(StreamReadFeature.STRICT_DUPLICATE_DETECTION)
           .build();
     }
+
+    private static JsonFactory createYamlFactory() {
+        return YAMLFactory.builder()
+          .enable(StreamReadFeature.STRICT_DUPLICATE_DETECTION)
+          .build();
+    }
 }

modules/swagger-parser-v3/src/test/java/io/swagger/v3/parser/test/OpenAPIV3ParserTest.java

@@ -2256,7 +2256,7 @@ public void testSampleParser() {
     }
 
     @Test
-    public void testDuplicateHttpStatusCodes() {
+    public void testDuplicateHttpStatusCodesJson() {
         final String location = "src/test/resources/duplicateHttpStatusCodes.json";
 
         final ParseOptions options = new ParseOptions();
@@ -2271,6 +2271,22 @@ public void testDuplicateHttpStatusCodes() {
 
     }
 
+    @Test
+    public void testDuplicateHttpStatusCodesYaml() {
+        final String location = "src/test/resources/duplicateHttpStatusCodes.yaml";
+
+        final ParseOptions options = new ParseOptions();
+        options.setResolve(true);
+
+        final OpenAPIV3Parser parser = new OpenAPIV3Parser();
+        final SwaggerParseResult result = parser.readLocation(location, null, options);
+        assertNull(result.getOpenAPI());
+        List<String> messages = result.getMessages();
+        assertEquals(1, messages.size());
+        assertEquals(messages.get(0), "Duplicate field '200' in `src/test/resources/duplicateHttpStatusCodes.yaml`");
+
+    }
+
     private static int getDynamicPort() {
         return new Random().ints(10000, 20000).findFirst().getAsInt();
     }

modules/swagger-parser-v3/src/test/resources/duplicateHttpStatusCodes.yaml

@@ -0,0 +1,22 @@
+openapi: 3.0.1
+info:
+  title: title
+  description: This is a sample server
+  license:
+    name: Apache-2.0
+    url: http://www.apache.org/licenses/LICENSE-2.0.html
+  version: 1.0.0
+servers:
+- url: https://api.absolute.org/v2
+  description: An absolute path
+paths:
+  /whatever:
+    get:
+      summary: Some operation
+      description: Some operation
+      operationId: doWhatever
+      responses:
+        "200":
+          description: OK
+        "200":
+          description: duplicate HTTP status code

modules/swagger-parser-v3/src/test/resources/oas3.yaml.template

@@ -37,7 +37,6 @@ security:
   - tokenAuth: []
 info:
   description: 'This is a sample server Petstore'
-  version: 1.0.0
   title: Sample Pet Store App
   termsOfService: http://swagger.io/terms/
   x-info: info extension

modules/swagger-parser-v3/src/test/resources/petstore.yaml

@@ -450,7 +450,6 @@ paths:
       summary: Logs user into the system
       description: ''
       operationId: loginUser
-      security: []
       parameters:
         - name: username
           in: query