Merge pull request OpenAPITools#49 from jmini/patch-4

Fix NullPointerException with oneOf discriminator change

Author: quen2404

src/main/java/com/qdesrame/openapi/diff/compare/schemadiffresult/ComposedSchemaDiffResult.java

@@ -46,6 +46,7 @@ public Optional<ChangedSchema> diff(HashSet<String> refSet, Components leftCompo
                     changedSchema.setOldSchema(left);
                     changedSchema.setNewSchema(right);
                     changedSchema.setDiscriminatorPropertyChanged(true);
+                    changedSchema.setContext(context);
                     return Optional.of(changedSchema);
                 }
 

src/test/java/com/qdesrame/openapi/test/OneOfDiffTest.java

@@ -16,6 +16,8 @@ public class OneOfDiffTest {
     private final String OPENAPI_DOC3 = "oneOf_diff_3.yaml";
     private final String OPENAPI_DOC4 = "composed_schema_1.yaml";
     private final String OPENAPI_DOC5 = "composed_schema_2.yaml";
+    private final String OPENAPI_DOC6 = "oneOf_discriminator-changed_1.yaml";
+    private final String OPENAPI_DOC7 = "oneOf_discriminator-changed_2.yaml";
 
     @Test
     public void testDiffSame() {
@@ -37,4 +39,10 @@ public void testComposedSchema() {
         assertOpenApiBackwardIncompatible(OPENAPI_DOC4, OPENAPI_DOC5);
     }
 
+    @Test
+    public void testOneOfDiscrimitatorChanged() {
+        //The oneOf 'discriminator' changed: 'realtype' -> 'othertype':
+        assertOpenApiBackwardIncompatible(OPENAPI_DOC6, OPENAPI_DOC7);
+    }
+
 }

src/test/resources/oneOf_discriminator-changed_1.yaml

@@ -0,0 +1,49 @@
+openapi: 3.0.1
+info:
+  title: oneOf test for issue 29
+  version: '1.0'
+servers:
+  - url: 'http://localhost:8000/'
+paths:
+  /state:
+    post:
+      operationId: update
+      requestBody:
+        content:
+            application/json:
+              schema:
+                oneOf:
+                  - $ref: '#/components/schemas/A'
+                  - $ref: '#/components/schemas/B'
+                discriminator:
+                  propertyName: realtype
+                  mapping:
+                    a-type: '#/components/schemas/A'
+                    b-type: '#/components/schemas/B'
+        required: true
+      responses:
+        '201':
+          description: OK
+components:
+  schemas:
+    A:
+      type: object
+      properties:
+        realtype:
+          type: string
+        othertype:
+          type: string
+        message:
+          type: string
+    B:
+      type: object
+      properties:
+        realtype:
+          type: string
+        othertype:
+          type: string
+        description:
+          type: string
+        code:
+          type: integer
+          format: int32

src/test/resources/oneOf_discriminator-changed_2.yaml

@@ -0,0 +1,49 @@
+openapi: 3.0.1
+info:
+  title: oneOf test for issue 29
+  version: '1.0'
+servers:
+  - url: 'http://localhost:8000/'
+paths:
+  /state:
+    post:
+      operationId: update
+      requestBody:
+        content:
+            application/json:
+              schema:
+                oneOf:
+                  - $ref: '#/components/schemas/A'
+                  - $ref: '#/components/schemas/B'
+                discriminator:
+                  propertyName: othertype
+                  mapping:
+                    a-type: '#/components/schemas/A'
+                    b-type: '#/components/schemas/B'
+        required: true
+      responses:
+        '201':
+          description: OK
+components:
+  schemas:
+    A:
+      type: object
+      properties:
+        realtype:
+          type: string
+        othertype:
+          type: string
+        message:
+          type: string
+    B:
+      type: object
+      properties:
+        realtype:
+          type: string
+        othertype:
+          type: string
+        description:
+          type: string
+        code:
+          type: integer
+          format: int32