Merge pull request #1434 from phiz71/1433-resolve-schema-without-type

fix ResolverFully for schema without type

Author: gracekarina

modules/swagger-parser-v3/src/main/java/io/swagger/v3/parser/util/ResolverFully.java

@@ -395,18 +395,15 @@ public Schema resolveSchema(Schema schema) {
             for (String key : updated.keySet()) {
                 Schema property = updated.get(key);
 
-                if(property instanceof ObjectSchema) {
-                    ObjectSchema op = (ObjectSchema) property;
-                    if (op.getProperties() != model.getProperties()) {
-                        if (property.getType() == null) {
-                            property.setType("object");
-                        }
-                        model.addProperties(key, property);
-                    } else {
-                        LOGGER.debug("not adding recursive properties, using generic object");
-                        ObjectSchema newSchema = new ObjectSchema();
-                        model.addProperties(key, newSchema);
+                if (property.getProperties() != model.getProperties()) {
+                    if (property.getType() == null) {
+                        property.setType("object");
                     }
+                    model.addProperties(key, property);
+                } else {
+                    LOGGER.debug("not adding recursive properties, using generic object");
+                    ObjectSchema newSchema = new ObjectSchema();
+                    model.addProperties(key, newSchema);
                 }
 
             }

modules/swagger-parser/src/test/java/io/swagger/parser/OpenAPIParserTest.java

@@ -1,13 +1,13 @@
 package io.swagger.parser;
 
-import io.swagger.util.Yaml;
 import io.swagger.v3.oas.models.Components;
 import io.swagger.v3.oas.models.OpenAPI;
 import io.swagger.v3.oas.models.media.ArraySchema;
 import io.swagger.v3.oas.models.media.ObjectSchema;
 import io.swagger.v3.oas.models.media.Schema;
 import io.swagger.v3.oas.models.PathItem;
 
+import io.swagger.v3.oas.models.media.StringSchema;
 import io.swagger.v3.oas.models.parameters.Parameter;
 import io.swagger.v3.oas.models.parameters.RequestBody;
 
@@ -561,6 +561,32 @@ public void testIssue1086() {
         Schema score = schema.getProperties().get("score");
         assertEquals(score.getMultipleOf().intValue(), 1);
     }
-  
+
+    @Test
+    public void testIssue1433_ResolveSchemaWithoutType() {
+        OpenAPIParser openApiParser = new OpenAPIParser();
+        ParseOptions options = new ParseOptions();
+        options.setResolveFully(true);
+
+        OpenAPI openAPI = openApiParser.readLocation("issue_1433-resolve-schema-without-type.yaml", null, options).getOpenAPI();
+        final Schema requestBodySchema = openAPI.getPaths().get("/foo").getPost().getRequestBody().getContent().get("application/json").getSchema();
+        assertNotNull(requestBodySchema);
+
+        final Map properties = requestBodySchema.getProperties();
+        assertEquals(properties.size(), 2);
+
+        final Object bar = properties.get("bar");
+        assertEquals(bar.getClass(), StringSchema.class);
+
+        final Object input = properties.get("input");
+        assertEquals(input.getClass(), Schema.class);
+
+        final Map inputProperties = ((Schema) input).getProperties();
+        assertNotNull(inputProperties);
+        assertEquals(inputProperties.size(),1);
+
+        final Object baz = inputProperties.get("baz");
+        assertEquals(baz.getClass(), StringSchema.class);
+    }
 }
 

modules/swagger-parser/src/test/resources/issue_1433-resolve-schema-without-type.yaml

@@ -0,0 +1,28 @@
+openapi: 3.0.0
+info:
+  title: no type resolution
+  version: 1.0.0
+paths:
+  /foo:
+    post:
+      requestBody:
+        required: true
+        content:
+          application/json:
+            schema:
+              $ref: '#/components/schemas/FooInput'
+      responses:
+        '200':
+          description: subscription successfully created
+components:
+  schemas:
+    FooInput:
+      properties:
+        bar:
+          type: string
+        input:
+          $ref: '#/components/schemas/BazInput'
+    BazInput:
+      properties:
+        baz:
+          type: string