Merge pull request swagger-api#977 from fujigon/feature/validation-recognize-ref

fix bug of validation failure when path parameter contains ref

Author: gracekarina

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

@@ -560,7 +560,7 @@ private boolean isPathParamDefined(String pathParam, List<Parameter> parameters)
             return false;
         } else {
             Parameter pathParamDefined = parameters.stream()
-                            .filter(parameter -> pathParam.equals(parameter.getName()) && "path".equals(parameter.getIn()))
+                            .filter(parameter -> (parameter.get$ref() != null) || (pathParam.equals(parameter.getName()) && "path".equals(parameter.getIn())))
                             .findFirst()
                             .orElse(null);
             if (pathParamDefined == null) {

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

@@ -37,6 +37,7 @@
 import io.swagger.v3.parser.core.models.SwaggerParseResult;
 import mockit.Injectable;
 import org.apache.commons.io.FileUtils;
+import org.hamcrest.CoreMatchers;
 import org.testng.Assert;
 import org.testng.annotations.AfterClass;
 import org.testng.annotations.BeforeClass;
@@ -51,6 +52,7 @@
 import java.util.*;
 
 import static com.github.tomakehurst.wiremock.client.WireMock.*;
+import static org.junit.Assert.assertThat;
 import static org.testng.Assert.*;
 
 
@@ -70,7 +72,7 @@ public void testIssueIntegerDefault() {
         Assert.assertNull ( openAPI.getPaths().get("/issue-125").getGet().getResponses().get("200").getContent().get("*/*").getSchema().getFormat());
         Assert.assertNull (openAPI.getPaths().get("/primitiveBody/binary").getPost().getRequestBody().getContent().get("application/octet-stream").getSchema().getFormat());
     }
-  
+
     @Test
     public void testIssue983() {
         OpenAPIV3Parser parser = new OpenAPIV3Parser();
@@ -81,7 +83,7 @@ public void testIssue983() {
         Yaml.prettyPrint(openAPI);
         Assert.assertNotNull(openAPI.getComponents().getSchemas().get("InventoryId"));
     }
-  
+
     @Test
     public void testIssue913() {
         OpenAPIV3Parser parser = new OpenAPIV3Parser();
@@ -162,7 +164,7 @@ public void testIssue837() {
         Assert.assertEquals(examples.get("local").get$ref(), "#/components/examples/LocalRef");
         Assert.assertEquals(examples.get("external").get$ref(), "#/components/examples/ExternalRef");
     }
-  
+
     @Test
     public void testIssue834() {
         ParseOptions options = new ParseOptions();
@@ -1755,6 +1757,14 @@ public void testIssue975_allOf() {
         assertEquals(composed.getAllOf().get(0).get$ref(), "#/components/schemas/Image");
     }
 
+    @Test
+    public void testValidationIssue() {
+        ParseOptions parseOptions = new ParseOptions();
+        parseOptions.setResolveFully(true);
+        SwaggerParseResult result = new OpenAPIV3Parser().readLocation("src/test/resources/validation/path-parameter-validation.yaml", null, parseOptions);
+        assertThat(result.getMessages().size(), CoreMatchers.is(0));
+    }
+
     private static int getDynamicPort() {
         return new Random().ints(10000, 20000).findFirst().getAsInt();
     }

modules/swagger-parser-v3/src/test/resources/validation/path-parameter-validation.yaml

@@ -0,0 +1,25 @@
+openapi: 3.0.0
+info:
+  title: Sample APIs
+  version: 1.0.0
+  description:
+    Sample API
+
+components:
+  parameters:
+    accountId:
+      name: accountId
+      schema:
+        type: string
+      in: path
+      required: true
+
+paths:
+  /users/{accountId}:
+    get:
+      operationId: ReadUser
+      parameters:
+      - $ref: '#/components/parameters/accountId'
+      responses:
+        '200':
+          description: 200 response