fixing bug when parser trying to solve remote ref in additional properties schema #342

Author: gracekarina

modules/swagger-parser-v3/src/main/java/io/swagger/v3/parser/processors/ExternalRefProcessor.java

@@ -111,6 +111,26 @@ public String processRefToExternalSchema(String $ref, RefFormat refFormat) {
                     }
                 }
             }
+            if(schema.getAdditionalProperties() != null){
+                Schema additionalProperty = schema.getAdditionalProperties();
+                if (additionalProperty.get$ref() != null) {
+                    processRefProperty(additionalProperty, file);
+                } else if (additionalProperty instanceof ArraySchema) {
+                    ArraySchema arrayProp = (ArraySchema) additionalProperty;
+                    if (arrayProp.getItems().get$ref() != null) {
+                        processRefProperty(arrayProp.getItems(), file);
+                    }
+                } else if (additionalProperty.getAdditionalProperties() != null) {
+                    Schema mapProp =  additionalProperty;
+                    if (mapProp.getAdditionalProperties().get$ref() != null) {
+                        processRefProperty(mapProp.getAdditionalProperties(), file);
+                    } else if (mapProp.getAdditionalProperties() instanceof ArraySchema &&
+                            ((ArraySchema) mapProp.getAdditionalProperties()).getItems().get$ref() != null) {
+                        processRefProperty(((ArraySchema) mapProp.getAdditionalProperties()).getItems(), file);
+                    }
+                }
+
+            }
             if (schema instanceof ArraySchema && ((ArraySchema) schema).getItems().get$ref() != null) {
                 processRefProperty(((ArraySchema) schema).getItems(), file);
             }

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

@@ -303,6 +303,17 @@ public void testInlineModelResolverByUrl(@Injectable final List<AuthorizationVal
         assertNotNull(userAddress.getProperties().get("street"));
     }
 
+    @Test
+    public void testRefAdditionalProperties() throws Exception {
+        OpenAPI openAPI = new OpenAPIV3Parser().read("src/test/resources/additionalProperties.yaml");
+
+        Assert.assertNotNull(openAPI);
+        Assert.assertTrue(openAPI.getComponents().getSchemas().size() == 3);
+        Assert.assertNotNull(openAPI.getComponents().getSchemas().get("link-object"));
+        Assert.assertNotNull(openAPI.getComponents().getSchemas().get("rel-data"));
+        Assert.assertNotNull(openAPI.getComponents().getSchemas().get("result"));
+    }
+
 
     private static int getDynamicPort() {
         return new Random().ints(10000, 20000).findFirst().getAsInt();

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

@@ -0,0 +1,18 @@
+---
+openapi: 3.0.0
+servers:
+- url: http://swagger.io/show-bug
+info:
+  title: Test API
+  version: 1.0.0
+paths:
+  "/tests":
+    get:
+      operationId: listTests
+      responses:
+        '200':
+          description: Returns results.
+          content:
+            application/json:
+              schema:
+                "$ref": "./globals.yaml#/components/schemas/result"

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

@@ -0,0 +1,22 @@
+components:
+  schemas:
+    link-object:
+      type: object
+      additionalProperties:
+        "$ref": "#/components/schemas/rel-data"
+    rel-data:
+      type: object
+      required:
+      - href
+      properties:
+        href:
+          type: string
+        note:
+          type: string
+    result:
+      type: object
+      properties:
+        name:
+          type: string
+        _links:
+          "$ref": "#/components/schemas/link-object"