fix(editor): retrieve parameter default value from dereferenced spec (#…

…213) closes #212
NeuraLegion · Sep 20, 2023 · 18888f0 · 18888f0
1 parent 749b6d8
commit 18888f0
Show file tree

Hide file tree

Showing 5 changed files with 96 additions and 2 deletions.
diff --git a/packages/editor/src/editor/oas/BaseOasParameterObjectsParser.ts b/packages/editor/src/editor/oas/BaseOasParameterObjectsParser.ts
@@ -18,7 +18,7 @@ export abstract class BaseOasParameterObjectsParser<
 {
   protected constructor(
     protected readonly doc: D,
-    private readonly dereferencedDoc: D
+    protected readonly dereferencedDoc: D
   ) {}
 
   protected abstract getParameterValue(

diff --git a/packages/editor/src/editor/oas/v2/OasV2ParameterObjectsParser.ts b/packages/editor/src/editor/oas/v2/OasV2ParameterObjectsParser.ts
@@ -70,7 +70,7 @@ export class OasV2ParameterObjectsParser extends BaseOasParameterObjectsParser<
     parameter: OpenAPIV2.Parameter
   ): SpecTreeRequestBodyParam[] {
     const operationObject: OpenAPIV2.OperationObject = jsonPointer.get(
-      this.doc,
+      this.dereferencedDoc,
       jsonPointer.compile(jsonPointer.parse(pointer).slice(0, -2))
     );
     const mimeTypes = operationObject.consumes || ['application/json'];

diff --git a/packages/editor/tests/fixtures/oas2-default-param-value.result.json b/packages/editor/tests/fixtures/oas2-default-param-value.result.json
@@ -0,0 +1,36 @@
+{
+  "children": [
+    {
+      "children": [
+        {
+          "jsonPointer": "/paths/~1pet/post",
+          "method": "POST",
+          "parameters": [
+            {
+              "bodyType": "application/json",
+              "paramType": "requestBody",
+              "value": {
+                "name": "Jack"
+              },
+              "valueJsonPointer": "/paths/~1pet/post/parameters/0/schema/default"
+            }
+          ],
+          "path": "/pet"
+        }
+      ],
+      "jsonPointer": "/paths/~1pet",
+      "path": "/pet"
+    }
+  ],
+  "jsonPointer": "/",
+  "name": "Swagger Petstore",
+  "parameters": [
+    {
+      "name": "host",
+      "paramType": "variable",
+      "value": "petstore.swagger.io",
+      "valueJsonPointer": "/host"
+    }
+  ],
+  "path": "/"
+}
diff --git a/packages/editor/tests/fixtures/oas2-default-param-value.yaml b/packages/editor/tests/fixtures/oas2-default-param-value.yaml
@@ -0,0 +1,39 @@
+swagger: '2.0'
+info:
+  title: Swagger Petstore
+  version: 1.0.0
+host: petstore.swagger.io
+basePath: /v2
+schemes:
+  - https
+paths:
+  /pet:
+    post:
+      consumes:
+        - application/json
+      parameters:
+        - $ref: '#/parameters/ReferencedPostBodyParam'
+      responses:
+        200:
+          description: successful operation
+parameters:
+  ReferencedPostBodyParam:
+    description: A JSON object containing pet information
+    in: body
+    name: body
+    required: true
+    schema:
+      $ref: '#/definitions/Pet'
+definitions:
+  Pet:
+    properties:
+      name:
+        example: doggie
+        type: string
+    required:
+      - name
+    type: object
+    xml:
+      name: Pet
+    default:
+      name: Jack
diff --git a/packages/editor/tests/oas2.spec.ts b/packages/editor/tests/oas2.spec.ts
@@ -73,6 +73,25 @@ describe('OasV2Editor', () => {
       expect(result).toEqual(expected);
     });
 
+    it('should retrieve default parameter value', async () => {
+      const sourceYaml = readFileSync(
+        resolve(__dirname, './fixtures/oas2-default-param-value.yaml'),
+        'utf-8'
+      );
+      await openApiParser.setup(sourceYaml);
+
+      const expected = JSON.parse(
+        readFileSync(
+          resolve(__dirname, './fixtures/oas2-default-param-value.result.json'),
+          'utf-8'
+        )
+      );
+
+      const result = openApiParser.parse();
+
+      expect(result).toEqual(expected);
+    });
+
     it('should be exception on call "parse" before "setup"', () =>
       expect(() => openApiParser.parse()).toThrowError(
         'You have to call "setup" to initialize the document'