CHORE[DEPENDENCIES]: bump swagger-parser => 2.0.22 (#7597)

this fixes a bug where the parser would return
null when missing parameters rather than reporting
some useful error.

Fixes N/A

Author: codymikol

modules/openapi-generator/src/main/java/org/openapitools/codegen/InlineModelResolver.java

@@ -472,14 +472,22 @@ private void flattenComponents(OpenAPI openAPI) {
      * @param m Schema implementation
      */
     private void fixStringModel(Schema m) {
-        if (m.getType() != null && m.getType().equals("string") && m.getExample() != null) {
+        if (schemaIsOfType(m, "string") && schemaContainsExample(m)) {
             String example = m.getExample().toString();
-            if (example.substring(0, 1).equals("\"") && example.substring(example.length() - 1).equals("\"")) {
+            if (example.startsWith("\"") && example.endsWith("\"")) {
                 m.setExample(example.substring(1, example.length() - 1));
             }
         }
     }
 
+    private boolean schemaIsOfType(Schema m, String type) {
+        return m.getType() != null && m.getType().equals(type);
+    }
+
+    private boolean schemaContainsExample(Schema m) {
+        return m.getExample() != null && m.getExample() != "";
+    }
+
     /**
      * Generates a unique model name. Non-alphanumeric characters will be replaced
      * with underscores

modules/openapi-generator/src/test/java/org/openapitools/codegen/InlineModelResolverTest.java

@@ -28,6 +28,7 @@
 import io.swagger.v3.oas.models.responses.ApiResponses;
 import io.swagger.v3.parser.core.models.ParseOptions;
 import org.openapitools.codegen.utils.ModelUtils;
+import org.testng.Assert;
 import org.testng.annotations.Test;
 
 import java.util.HashMap;
@@ -927,7 +928,7 @@ public void emptyExampleOnStringTypeModels() {
         assertEquals("#/components/schemas/EmptyExampleOnStringTypeModels", schema.getItems().get$ref());
 
         assertTrue(ModelUtils.getReferencedSchema(openAPI, schema.getItems()) instanceof StringSchema);
-        assertNull(ModelUtils.getReferencedSchema(openAPI, schema.getItems()).getExample());
+        Assert.assertSame(ModelUtils.getReferencedSchema(openAPI, schema.getItems()).getExample(), "");
     }
 
     @Test

pom.xml

@@ -1516,7 +1516,7 @@
         <maven.compiler.target>1.8</maven.compiler.target>
         <swagger-core-version>2.1.2</swagger-core-version>
         <swagger-parser-groupid>io.swagger.parser.v3</swagger-parser-groupid>
-        <swagger-parser-version>2.0.20</swagger-parser-version>
+        <swagger-parser-version>2.0.22</swagger-parser-version>
         <felix-version>3.3.1</felix-version>
         <commons-io-version>2.4</commons-io-version>
         <commons-cli-version>1.2</commons-cli-version>

samples/client/petstore/go/go-petstore/api/openapi.yaml

@@ -2126,3 +2126,4 @@ components:
     http_basic_test:
       scheme: basic
       type: http
+x-original-swagger-version: "2.0"

samples/client/petstore/haskell-http-client/openapi.yaml

@@ -2126,3 +2126,4 @@ components:
     http_basic_test:
       scheme: basic
       type: http
+x-original-swagger-version: "2.0"

samples/client/petstore/java/feign-no-nullable/api/openapi.yaml

@@ -2187,4 +2187,5 @@ components:
     http_basic_test:
       scheme: basic
       type: http
+x-original-swagger-version: "2.0"
 

samples/client/petstore/java/feign/api/openapi.yaml

@@ -2187,4 +2187,5 @@ components:
     http_basic_test:
       scheme: basic
       type: http
+x-original-swagger-version: "2.0"
 

samples/client/petstore/java/google-api-client/api/openapi.yaml

@@ -2187,4 +2187,5 @@ components:
     http_basic_test:
       scheme: basic
       type: http
+x-original-swagger-version: "2.0"
 

samples/client/petstore/java/jersey1/api/openapi.yaml

@@ -2187,4 +2187,5 @@ components:
     http_basic_test:
       scheme: basic
       type: http
+x-original-swagger-version: "2.0"
 

samples/client/petstore/java/jersey2-java8/api/openapi.yaml

@@ -2187,4 +2187,5 @@ components:
     http_basic_test:
       scheme: basic
       type: http
+x-original-swagger-version: "2.0"