Set authorization placeholder values as postman env variable format

Author: erfemega

lib/schemaUtils.js

@@ -1090,25 +1090,25 @@ module.exports = {
         else if (_.toLower(securityDef.scheme) === 'bearer') {
           helper = {
             type: 'bearer',
-            bearer: [{ key: 'token', value: '<Bearer Token>' }]
+            bearer: [{ key: 'token', value: '{{bearer_token}}' }]
           };
         }
         else if (_.toLower(securityDef.scheme) === 'digest') {
           helper = {
             type: 'digest',
             digest: [
-              { key: 'username', value: '<Digest Auth Username>' },
-              { key: 'password', value: '<Digest Auth Password>' },
-              { key: 'realm', value: '<realm>' }
+              { key: 'username', value: '{{digest_auth_username}}' },
+              { key: 'password', value: '{{digest_auth_password}}' },
+              { key: 'realm', value: '{{realm}}' }
             ]
           };
         }
         else if (_.toLower(securityDef.scheme) === 'oauth' || _.toLower(securityDef.scheme) === 'oauth1') {
           helper = {
             type: 'oauth1',
             oauth1: [
-              { key: 'consumerSecret', value: '<OAuth 1.0 Consumer Key>' },
-              { key: 'consumerKey', value: '<OAuth 1.0 Consumer Secret>' },
+              { key: 'consumerSecret', value: '{{oauth_10_consumer_key}}' },
+              { key: 'consumerKey', value: '{{oauth_10_consumer_secret}}' },
               { key: 'addParamsToHeader', value: true }
             ]
           };
@@ -1197,9 +1197,9 @@ module.exports = {
           apikey: [
             {
               key: 'key',
-              value: _.isString(securityDef.name) ? securityDef.name : '<API Key Name>'
+              value: _.isString(securityDef.name) ? securityDef.name : '{{api_key_name}}'
             },
-            { key: 'value', value: '<API Key>' },
+            { key: 'value', value: '{{api_key}}' },
             {
               key: 'in',
               value: _.includes(['query', 'header'], securityDef.in) ? securityDef.in : 'header'

test/data/valid_openapi/specWithAuth.yaml

@@ -0,0 +1,49 @@
+openapi: "3.0.0"
+info:
+  version: 1.0.0
+  title: Swagger Petstore
+  license:
+    name: MIT
+servers:
+  - url: https://postman-echo.com/get
+security:
+  - bear: []
+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: string
+      responses:
+        default:
+          description: unexpected error
+          content:
+            application/json:
+              schema:
+                $ref: "#/components/schemas/Error"
+components:
+  schemas:
+    Error:
+      type: object
+      required:
+        - code
+        - message
+      properties:
+        code:
+          type: integer
+          format: int32
+        message:
+          type: string
+  securitySchemes:
+    bear:
+      type: http
+      scheme: bearer
+      bearerFormat: JWT

test/unit/base.test.js

@@ -58,7 +58,8 @@ describe('CONVERT FUNCTION TESTS ', function() {
       allHTTPMethodsSpec = path.join(__dirname, VALID_OPENAPI_PATH, '/all-http-methods.yaml'),
       invalidNullInfo = path.join(__dirname, INVALID_OPENAPI_PATH, '/invalid-null-info.json'),
       invalidNullInfoTitle = path.join(__dirname, INVALID_OPENAPI_PATH, '/invalid-info-null-title.json'),
-      invalidNullInfoVersion = path.join(__dirname, INVALID_OPENAPI_PATH, '/invalid-info-null-version.json');
+      invalidNullInfoVersion = path.join(__dirname, INVALID_OPENAPI_PATH, '/invalid-info-null-version.json'),
+      specWithAuth = path.join(__dirname, VALID_OPENAPI_PATH + '/specWithAuth.yaml');
 
 
     it('Should add collection level auth with type as `bearer`' +
@@ -1291,6 +1292,24 @@ describe('CONVERT FUNCTION TESTS ', function() {
         done();
       });
     });
+
+    describe('[Github #643] - Generated value for corresponding' +
+      ' authorization should be as environment variable format', function(done) {
+      it('Should convert a collection and set auth placeholder as variable', function() {
+        var openapi = fs.readFileSync(specWithAuth, 'utf8');
+        Converter.convert({ type: 'string', data: openapi }, { schemaFaker: true }, (err, conversionResult) => {
+          expect(err).to.be.null;
+          expect(conversionResult.result).to.equal(true);
+          expect(conversionResult.output.length).to.equal(1);
+          expect(conversionResult.output[0].type).to.equal('collection');
+          expect(conversionResult.output[0].data).to.have.property('info');
+          expect(conversionResult.output[0].data).to.have.property('item');
+          expect(conversionResult.output[0].data.item.length).to.equal(1);
+          expect(conversionResult.output[0].data.auth.bearer[0].value).to.equal('{{bearer_token}}');
+          done();
+        });
+      });
+    });
   });
 
   describe('Converting swagger 2.0 files', function() {