avoid using example.value

LuisTejedaS · LuisTejedaS · commit 6bb435bd75b2 · 2022-05-13T13:29:40.000-05:00
use the whole object for example instead of example.value
diff --git a/lib/schemaUtils.js b/lib/schemaUtils.js
@@ -1360,30 +1360,6 @@ module.exports = {
     return example;
   },
 
-  /**
-   * Identifies if the given schema has the property "value"
-   * @param {*} schema bodyObject (media type) schema to use
-   * @param {object} components - components defined in the OAS spec.
-  * @param {object} options - a standard list of options that's globally passed around. Check options.js for more.
-   * @returns {boolean} Wheter to use the property value of the example as the value of the
-   * body data
-   */
-  schemaHasValueProp(schema, components, options) {
-    let schemaHasValue = false,
-      schemaObject;
-    if (!schema) {
-      return false;
-    }
-    if (schema.$ref) {
-      schemaObject = this.getRefObject(schema.$ref, components, options);
-    }
-    else {
-      schemaObject = schema;
-    }
-    schemaHasValue = schemaObject.properties.hasOwnProperty('value');
-    return schemaHasValue;
-  },
-
   /**
    * converts one of the eamples or schema in Media Type object to postman data
    * @param {*} bodyObj is MediaTypeObject
@@ -1433,11 +1409,6 @@ module.exports = {
         }
       }
       bodyData = bodyObj.example;
-      // return example value if present else example is returned
-      if (bodyData.hasOwnProperty('value') &&
-        !this.schemaHasValueProp(bodyObj.schema, components, options)) {
-        bodyData = bodyData.value;
-      }
     }
     else if (!_.isEmpty(bodyObj.examples) && (resolveTo === 'example' || !bodyObj.schema)) {
       // take one of the examples as the body and not all
diff --git a/test/data/valid_openapi/petstoreParamExample.yaml b/test/data/valid_openapi/petstoreParamExample.yaml
@@ -0,0 +1,70 @@
+openapi: "3.0.0"
+info:
+  version: 1.0.0
+  title: Swagger Petstore
+  license:
+    name: MIT
+servers:
+  - url: http://petstore.swagger.io/v1
+paths:
+  /pets/{petId}:
+    get:
+      summary: Info for a specific pet
+      operationId: showPetById
+      tags:
+        - pets
+      parameters:
+        - name: petId
+          in: path
+          required: true
+          description: The id of the pet to retrieve
+          schema:
+            type: object
+            properties:
+              value:
+                type:
+                  - integer
+          example:
+            value: 1
+        - name: user
+          in: query
+          required: true
+          style: deepObject
+          schema:
+            type:
+              - object
+            properties:
+              id:
+                type:
+                  - integer
+              value:
+                type:
+                  - string
+          example:
+            id: 1
+            value: Vani
+      responses:
+        '200':
+          description: Expected response to a valid request
+          content:
+            application/json:
+              schema:
+                $ref: "#/components/schemas/Pets"
+components:
+  schemas:
+    Pet:
+      required:
+        - id
+        - name
+      properties:
+        id:
+          type: integer
+          format: int64
+        name:
+          type: string
+        tag:
+          type: string
+    Pets:
+      type: array
+      items:
+        $ref: "#/components/schemas/Pet"
diff --git a/test/unit/base.test.js b/test/unit/base.test.js
@@ -47,7 +47,8 @@ describe('CONVERT FUNCTION TESTS ', function() {
       parameterExamples = path.join(__dirname, VALID_OPENAPI_PATH + '/parameteres_with_examples.yaml'),
       issue10229 = path.join(__dirname, VALID_OPENAPI_PATH, '/issue#10229.json'),
       deepObjectLengthProperty = path.join(__dirname, VALID_OPENAPI_PATH, '/deepObjectLengthProperty.yaml'),
-      valuePropInExample = path.join(__dirname, VALID_OPENAPI_PATH, '/valuePropInExample.yaml');
+      valuePropInExample = path.join(__dirname, VALID_OPENAPI_PATH, '/valuePropInExample.yaml'),
+      petstoreParamExample = path.join(__dirname, VALID_OPENAPI_PATH, '/petstoreParamExample.yaml');
 
 
     it('Should add collection level auth with type as `bearer`' +
@@ -468,7 +469,7 @@ describe('CONVERT FUNCTION TESTS ', function() {
             expect(rootRequest.body.raw).to
               .equal('{\n  "a": "<string>",\n  "b": "<string>"\n}');
             expect(exampleRequest.body.raw).to
-              .equal('{\n  "a": "example-b",\n  "b": "example-c"\n}');
+              .equal('{\n  "value": {\n    "a": "example-b",\n    "b": "example-c"\n  }\n}');
             done();
           });
       });
@@ -1125,6 +1126,19 @@ describe('CONVERT FUNCTION TESTS ', function() {
           done();
         });
     });
+
+    it('Should convert example in parameters' +
+    petstoreParamExample, function(done) {
+      var openapi = fs.readFileSync(petstoreParamExample, 'utf8');
+      Converter.convert({ type: 'string', data: openapi },
+        { schemaFaker: true, requestParametersResolution: 'Example' }, (err, conversionResult) => {
+          expect(err).to.be.null;
+          expect(conversionResult.result).to.equal(true);
+          expect(conversionResult.output[0].data.item[0].request.url.variable[0].value).to.equal('value,1');
+          expect(conversionResult.output[0].data.item[0].request.url.query[1].key).to.equal('user[value]');
+          done();
+        });
+    });
   });
 
   describe('requestNameSource option', function() {
diff --git a/test/unit/util.test.js b/test/unit/util.test.js
@@ -690,14 +690,23 @@ describe('SCHEMA UTILITY FUNCTION TESTS ', function () {
     });
 
     it('should work for example', function() {
+      var bodyWithExample = {
+          example: 'This is a sample value'
+        },
+        retValExample = SchemaUtils.convertToPmBodyData(bodyWithExample, 'application/json');
+
+      expect(retValExample).to.equal('This is a sample value');
+    });
+
+    it('should work for example with value property', function() {
       var bodyWithExample = {
           example: {
             value: 'This is a sample value'
           }
         },
         retValExample = SchemaUtils.convertToPmBodyData(bodyWithExample, 'application/json');
 
-      expect(retValExample).to.equal('This is a sample value');
+      expect(retValExample.value).to.equal('This is a sample value');
     });
 
     it('should work for examples', function() {
@@ -1678,10 +1687,7 @@ describe('SCHEMA UTILITY FUNCTION TESTS ', function () {
               description: 'body description',
               content: {
                 'text/plain': {
-                  example: {
-                    summary: 'A list containing two items',
-                    value: 'text/plain description'
-                  }
+                  example: 'text/plain description'
                 }
               }
             },
@@ -1690,7 +1696,8 @@ describe('SCHEMA UTILITY FUNCTION TESTS ', function () {
             exampleParametersResolution: 'example'
           });
           resultBody = result.body.raw;
-          expect(resultBody).to.equal('"text/plain description"');
+          expect(resultBody)
+            .to.equal('"text/plain description"');
           expect(result.contentHeader).to.deep.include(
             { key: 'Content-Type', value: 'text/plain' });
           done();
@@ -1701,10 +1708,7 @@ describe('SCHEMA UTILITY FUNCTION TESTS ', function () {
               description: 'body description',
               content: {
                 'text/html': {
-                  example: {
-                    summary: 'A list containing two items',
-                    value: '<html><body><ul><li>item 1</li><li>item 2</li></ul></body></html>'
-                  }
+                  example: '<html><body><ul><li>item 1</li><li>item 2</li></ul></body></html>'
                 }
               }
             },
@@ -1857,10 +1861,7 @@ describe('SCHEMA UTILITY FUNCTION TESTS ', function () {
             description: 'body description',
             content: {
               'text/plain': {
-                example: {
-                  summary: 'A list containing two items',
-                  value: 'text/plain description'
-                }
+                example: 'text/plain description'
               }
             }
           },
@@ -1877,10 +1878,7 @@ describe('SCHEMA UTILITY FUNCTION TESTS ', function () {
             description: 'body description',
             content: {
               'text/html': {
-                example: {
-                  summary: 'A list containing two items',
-                  value: '<html><body><ul><li>item 1</li><li>item 2</li></ul></body></html>'
-                }
+                example: '<html><body><ul><li>item 1</li><li>item 2</li></ul></body></html>'
               }
             }
           },
