Fixed incorrect usage of semicolons in nodejs-request

Author: shreys7

codegens/nodejs-request/lib/request.js

@@ -16,7 +16,7 @@ function makeSnippet (request, indentString, options) {
   var snippet = 'var request = require(\'request\');\n',
     optionsArray = [];
 
-  snippet += 'var fs = require(\'fs\')\n';
+  snippet += 'var fs = require(\'fs\');\n';
   snippet += 'var options = {\n';
 
   /**
@@ -45,7 +45,7 @@ function makeSnippet (request, indentString, options) {
     optionsArray.push(indentString + 'followRedirect: false');
   }
   snippet += optionsArray.join(',\n') + '\n';
-  snippet += '}\n';
+  snippet += '};\n';
 
   snippet += 'request(options, function (error, response) { \n';
   snippet += indentString + 'if (error) throw new Error(error);\n';

codegens/nodejs-request/test/unit/snippet.test.js

@@ -263,6 +263,46 @@ describe('nodejs-request convert function', function () {
       });
     });
 
+    it('should return snippet with proper semicolon placed where required', function () {
+      // testing for the below snippet
+      /*
+       var request = require('request');
+       var fs = require('fs');
+       var options = {
+         'method': 'GET',
+         'url': 'https://postman-echo.com/headers',
+         'headers': {
+           'my-sample-header': 'Lorem ipsum dolor sit amet',
+           'not-disabled-header': 'ENABLED'
+         }
+       };
+       request(options, function (error, response) {
+         if (error) throw new Error(error);
+         console.log(response.body);
+       }); */
+      request = new sdk.Request(mainCollection.item[0].request);
+      options = {};
+      convert(request, options, function (error, snippet) {
+        if (error) {
+          expect.fail(null, null, error);
+        }
+        console.log(snippet);
+        expect(snippet).to.be.a('string');
+        var snippetArray = snippet.split('\n');
+        snippetArray.forEach(function (line, index) {
+          if (line.charAt(line.length - 2) === ')') {
+            expect(line.charAt(line.length - 1)).to.equal(';');
+          }
+          expect(line.charAt(line.length - 1)).to.not.equal(')');
+          // check for the closing curly bracket of options object
+          if (line.startsWith('request')) {
+            var previousLine = snippetArray[index - 1];
+            expect(previousLine.charAt(previousLine.length - 1)).to.equal(';');
+          }
+        });
+      });
+    });
+
     describe('getOptions function', function () {
 
       it('should return an array of specific options', function () {