Fix for issue 911

Not able to identify the missed path variables

Author: ymohdriz

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

@@ -52,15 +52,7 @@
 import java.net.URI;
 import java.net.URISyntaxException;
 import java.net.URL;
-import java.util.ArrayList;
-import java.util.Arrays;
-import java.util.Iterator;
-import java.util.LinkedHashMap;
-import java.util.LinkedHashSet;
-import java.util.List;
-import java.util.Map;
-import java.util.Optional;
-import java.util.Set;
+import java.util.*;
 import java.util.stream.Collectors;
 import java.util.stream.Stream;
 
@@ -526,13 +518,64 @@ public Paths getPaths(ObjectNode obj, String location, ParseResult result) {
                 } else {
                     ObjectNode path = (ObjectNode) pathValue;
                     PathItem pathObj = getPathItem(path,String.format("%s.'%s'", location,pathName), result);
+                    String[] eachPart = pathName.split("/");
+                    Arrays.stream(eachPart)
+                            .filter(part -> part.startsWith("{") && part.endsWith("}") && part.length() > 2)
+                            .forEach(part -> {
+                                String pathParam = part.substring(1, part.length() - 1);
+                                boolean definedInPathLevel = isPathParamDefined(pathParam, pathObj.getParameters());
+                                if (!definedInPathLevel) {
+                                    List<Operation> operationsInAPath = getAllOperationsInAPath(pathObj);
+                                    operationsInAPath.forEach(operation -> {
+                                        if (!isPathParamDefined(pathParam, operation.getParameters())) {
+                                            result.warning(location + ".'" + pathName + "'"," Declared path parameter " + pathParam + " needs to be defined as a path parameter in path or operation level");
+                                            return;
+                                        }
+                                    });
+                                }
+                            });
                     paths.put(pathName, pathObj);
                 }
             }
         }
         return paths;
     }
 
+    private boolean isPathParamDefined(String pathParam, List<Parameter> parameters) {
+        if (parameters == null || parameters.isEmpty()) {
+            return false;
+        } else {
+            Parameter pathParamDefined = parameters.stream()
+                            .filter(parameter -> pathParam.equals(parameter.getName()))
+                            .findFirst()
+                            .orElse(null);
+            if (pathParamDefined == null) {
+                return false;
+            }
+        }
+        return true;
+    }
+
+    private void addToOperationsList(List<Operation> operationsList, Operation operation) {
+        if (operation == null) {
+            return;
+        }
+        operationsList.add(operation);
+    }
+
+    public List<Operation> getAllOperationsInAPath(PathItem pathObj) {
+        List<Operation> operations = new ArrayList<>();
+        addToOperationsList(operations, pathObj.getGet());
+        addToOperationsList(operations, pathObj.getPut());
+        addToOperationsList(operations, pathObj.getPost());
+        addToOperationsList(operations, pathObj.getPatch());
+        addToOperationsList(operations, pathObj.getDelete());
+        addToOperationsList(operations, pathObj.getTrace());
+        addToOperationsList(operations, pathObj.getOptions());
+        addToOperationsList(operations, pathObj.getHead());
+        return operations;
+    }
+
     public PathItem getPathItem(ObjectNode obj, String location, ParseResult result) {
 
 
@@ -2669,6 +2712,7 @@ protected static class ParseResult {
         private Map<Location, JsonNode> extra = new LinkedHashMap<>();
         private Map<Location, JsonNode> unsupported = new LinkedHashMap<>();
         private Map<Location, String> invalidType = new LinkedHashMap<>();
+        private List<Location> warnings = new ArrayList<>();
         private List<Location> missing = new ArrayList<>();
 
         public ParseResult() {
@@ -2686,6 +2730,10 @@ public void missing(String location, String key) {
             missing.add(new Location(location, key));
         }
 
+        public void warning(String location, String key) {
+            warnings.add(new Location(location, key));
+        }
+
         public void invalidType(String location, String key, String expectedType, JsonNode value) {
             invalidType.put(new Location(location, key), expectedType);
         }
@@ -2715,6 +2763,11 @@ public List<String> getMessages() {
                 String message = "attribute " + location + l.key + " is missing";
                 messages.add(message);
             }
+            for (Location l : warnings) {
+                String location = l.location.equals("") ? "" : l.location + ".";
+                String message = "attribute " + location +l.key;
+                messages.add(message);
+            }
             for (Location l : unsupported.keySet()) {
                 String location = l.location.equals("") ? "" : l.location + ".";
                 String message = "attribute " + location + l.key + " is unsupported";

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

@@ -7,6 +7,7 @@
 import org.testng.annotations.Test;
 
 import static org.testng.Assert.assertNotNull;
+import static org.testng.AssertJUnit.assertEquals;
 
 public class OAIDeserializationTest {
     @Test
@@ -34,4 +35,12 @@ public void testDeserializeSimpleDefinition() throws Exception {
 
         assertNotNull(result.getOpenAPI());
     }
+
+    @Test
+    public void testIssue911() {
+        SwaggerParseResult result = new OpenAPIV3Parser().readLocation("issue_911.yaml", null, null);
+        System.out.println(result.getMessages());
+        assertEquals(result.getMessages().size(),1);
+        assertNotNull(result.getOpenAPI());
+    }
 }

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

@@ -0,0 +1,48 @@
+openapi: 3.0.0
+servers:
+- url: 'http://localhost:8000/v2/api'
+info:
+  description: >-
+    This is a sample server Petstore server.  You can find out more about
+    Swagger at http://swagger.io or on
+    irc.freenode.net, #swagger.  For this sample, you can use the api key
+    "special-key" to test the authorization filters
+  version: 1.0.0
+  title: Swagger Petstore
+  termsOfService: 'http://swagger.io/terms/'
+  contact:
+    name: [email protected]
+  license:
+    name: Apache 2.0
+    url: 'http://www.apache.org/licenses/LICENSE-2.0.html'
+externalDocs:
+  description: Find more info here
+  url: 'https://swagger.io'
+tags:
+- name: pet
+  description: Pet Operations
+  externalDocs:
+    url: 'http://swagger.io'
+- name: user
+  description: All about the Users
+paths:
+  '/pet/{petId}/{pathParamNotDefined}':
+    get:
+      tags:
+      - pet
+      summary: Find pet by ID
+      description: >-
+        Returns a pet when ID < 10.  ID > 10 or nonintegers will simulate API
+        error conditions
+      operationId: getPetById
+      responses:
+        200:
+          description: 200 ok
+      parameters:
+      - name: petId
+        in: path
+        description: ID of pet that needs to be fetched
+        required: true
+        schema:
+          type: integer
+          format: int64