postman-runtime 7.26.3-beta.1 → 7.26.5

package/test/integration/requester-spec/redirect.test.js

@@ -1,7 +1,7 @@
 var sinon = require('sinon'),
     expect = require('chai').expect;
 
-describe('Requester Spec: redirect', function () {
+(typeof window === 'undefined' ? describe : describe.skip)('Requester Spec: redirect', function () {
     var testrun;
 
     describe('with followOriginalHttpMethod: false', function () {

package/test/integration/requester-spec/timing.test.js

@@ -1,6 +1,6 @@
 var expect = require('chai').expect;
 
-describe('Requester Spec: timings', function () {
+(typeof window === 'undefined' ? describe : describe.skip)('Requester Spec: timings', function () {
     var testrun,
         URL = 'https://postman-echo.com/get',
         collection = {

package/test/integration/requester-spec/useWhatWGUrlParser.test.js

@@ -1,6 +1,6 @@
 var expect = require('chai').expect;
 
-describe('Requester Spec: useWhatWGUrlParser', function () {
+(typeof window === 'undefined' ? describe : describe.skip)('Requester Spec: useWhatWGUrlParser', function () {
     var testrun,
         URL = 'https://postman-echo.com/get?q=(%{^*^}%)',
         collection = {

package/test/integration/runner-spec/ignoreProxySystemVariables.test.js

@@ -1,7 +1,7 @@
 var ProxyConfigList = require('postman-collection').ProxyConfigList,
     expect = require('chai').expect;
 
-describe('Runner Spec: ignoreProxyEnvironmentVariables', function () {
+(typeof window === 'undefined' ? describe : describe.skip)('Runner Spec: ignoreProxyEnvironmentVariables', function () {
     var testrun;
 
     before(function () {

package/test/integration/sandbox-libraries/pm.test.js

@@ -1,6 +1,7 @@
 var expect = require('chai').expect,
     sinon = require('sinon'),
-    postmanRequest = require('postman-request');
+    postmanRequest = require('postman-request'),
+    IS_BROWSER = typeof window !== 'undefined';
 
 describe('sandbox library - pm api', function () {
     var testrun;
@@ -60,7 +61,7 @@ describe('sandbox library - pm api', function () {
         });
     });
 
-    describe('chai', function () {
+    (IS_BROWSER ? describe.skip : describe)('chai', function () {
         before(function (done) {
             this.run({
                 collection: {
@@ -140,7 +141,8 @@ describe('sandbox library - pm api', function () {
         });
     });
 
-    describe('sendRequest', function () {
+    (IS_BROWSER ? describe.skip : describe)('sendRequest', function () {
+        // eslint-disable-next-line mocha/no-sibling-hooks
         before(function (done) {
             this.run({
                 collection: {
@@ -173,6 +175,7 @@ describe('sandbox library - pm api', function () {
             });
         });
 
+        // eslint-disable-next-line mocha/no-identical-title
         it('should complete the run', function () {
             expect(testrun).to.be.ok;
             sinon.assert.calledOnce(testrun.start);
@@ -186,6 +189,7 @@ describe('sandbox library - pm api', function () {
             sinon.assert.calledWith(testrun.response.getCall(0), null);
         });
 
+        // eslint-disable-next-line mocha/no-identical-title
         it('should run the test script successfully', function () {
             sinon.assert.calledOnce(testrun.script);
             sinon.assert.calledWith(testrun.script.getCall(0), null);
@@ -206,7 +210,7 @@ describe('sandbox library - pm api', function () {
     });
 
 
-    describe('cookies.jar', function () {
+    (IS_BROWSER ? describe.skip : describe)('cookies.jar', function () {
         describe('get', function () {
             before(function (done) {
                 this.run({

package/test/integration/sanity/certificate.test.js

@@ -3,7 +3,7 @@ var fs = require('fs'),
     expect = require('chai').expect,
     CertificateList = require('postman-collection').CertificateList;
 
-describe('certificates', function () {
+(typeof window === 'undefined' ? describe : describe.skip)('certificates', function () {
     var certDataPath = path.join(__dirname, '..', '..', 'fixtures', 'certificates'),
         certificateId = 'test-certificate',
         testrun;

package/test/integration/sanity/content-type-in-form-data.test.js

@@ -2,7 +2,7 @@ var fs = require('fs'),
     sinon = require('sinon'),
     expect = require('chai').expect;
 
-describe('content-type', function () {
+(typeof window === 'undefined' ? describe : describe.skip)('content-type', function () {
     var URL,
         testrun;
 

package/test/integration/sanity/cookie-handling.test.js

@@ -1,6 +1,6 @@
 var expect = require('chai').expect;
 
-describe('cookies', function () {
+(typeof window === 'undefined' ? describe : describe.skip)('cookies', function () {
     var _ = require('lodash'),
         testrun;
 

package/test/integration/sanity/dns-lookup.test.js

@@ -1,7 +1,7 @@
 var dns = require('dns'),
     expect = require('chai').expect;
 
-describe('DNS lookup', function () {
+(typeof window === 'undefined' ? describe : describe.skip)('DNS lookup', function () {
     var testrun;
 
     before(function (done) {

package/test/integration/sanity/econnrefused.test.js

@@ -1,6 +1,6 @@
 var expect = require('chai').expect;
 
-describe('econnrefused', function () {
+(typeof window === 'undefined' ? describe : describe.skip)('econnrefused', function () {
     var _ = require('lodash'),
         testrun;
 

package/test/integration/sanity/file-uploads.test.js

@@ -3,7 +3,7 @@ var fs = require('fs'),
     expect = require('chai').expect,
     sinon = require('sinon');
 
-describe('File uploads', function () {
+(typeof window === 'undefined' ? describe : describe.skip)('File uploads', function () {
     var testrun;
 
     before(function (done) {

package/test/integration/sanity/form-data-order.test.js

@@ -2,7 +2,7 @@ var fs = require('fs'),
     sinon = require('sinon'),
     expect = require('chai').expect;
 
-describe('form-data with numeric keys', function () {
+(typeof window === 'undefined' ? describe : describe.skip)('form-data with numeric keys', function () {
     var testrun;
 
     before(function (done) {

package/test/integration/sanity/http-methods.test.js

@@ -1,6 +1,7 @@
 var fs = require('fs'),
     sinon = require('sinon'),
-    expect = require('chai').expect;
+    expect = require('chai').expect,
+    IS_BROWSER = typeof window !== 'undefined';
 
 describe('http methods', function () {
     var testrun,
@@ -143,7 +144,7 @@ describe('http methods', function () {
                 });
             });
 
-            describe('urlencoded', function () {
+            (IS_BROWSER ? describe.skip : describe)('urlencoded', function () {
                 before(function (done) {
                     this.run({
                         collection: {
@@ -243,7 +244,8 @@ describe('http methods', function () {
                 });
             });
 
-            describe('file', function () {
+            (IS_BROWSER ? describe.skip : describe)('file', function () {
+                // eslint-disable-next-line mocha/no-sibling-hooks
                 before(function (done) {
                     this.run({
                         fileResolver: fs,
@@ -267,6 +269,7 @@ describe('http methods', function () {
                     });
                 });
 
+                // eslint-disable-next-line mocha/no-identical-title
                 it('should complete the run', function () {
                     expect(testrun).to.be.ok;
                     sinon.assert.calledOnce(testrun.start);

package/test/integration/sanity/http-reason-phrases.test.js

@@ -1,6 +1,6 @@
 var expect = require('chai').expect;
 
-describe('http reasons', function () {
+(typeof window === 'undefined' ? describe : describe.skip)('http reasons', function () {
     var testrun;
 
     before(function (done) {

package/test/integration/sanity/multi-value-response-headers.test.js

@@ -1,7 +1,7 @@
 var _ = require('lodash'),
     expect = require('chai').expect;
 
-describe('multi valued headers', function () {
+(typeof window === 'undefined' ? describe : describe.skip)('multi valued headers', function () {
     var testrun;
 
     before(function (done) {

package/test/integration/sanity/proxy-http-vars.test.js

@@ -1,6 +1,6 @@
 var expect = require('chai').expect;
 
-describe('proxy configuration vars', function () {
+(typeof window === 'undefined' ? describe : describe.skip)('proxy configuration vars', function () {
     var testrun;
 
     describe('lowercase', function () {

package/test/integration/sanity/proxy-http.test.js

@@ -1,7 +1,7 @@
 var ProxyConfigList = require('postman-collection').ProxyConfigList,
     expect = require('chai').expect;
 
-describe('proxy', function () {
+(typeof window === 'undefined' ? describe : describe.skip)('proxy', function () {
     var testrun;
 
     describe('sanity', function () {

package/test/integration/sanity/redirect.test.js

@@ -1,6 +1,7 @@
 var fs = require('fs'),
     sinon = require('sinon'),
-    expect = require('chai').expect;
+    expect = require('chai').expect,
+    IS_BROWSER = typeof window !== 'undefined';
 
 describe('redirects', function () {
     var testrun;
@@ -87,7 +88,7 @@ describe('redirects', function () {
         });
     });
 
-    describe('307 redirect', function () {
+    (IS_BROWSER ? describe.skip : describe)('307 redirect', function () {
         before(function (done) {
             this.run({
                 fileResolver: fs,
@@ -142,7 +143,8 @@ describe('redirects', function () {
         });
     });
 
-    describe('308 redirect', function () {
+    (IS_BROWSER ? describe.skip : describe)('308 redirect', function () {
+        // eslint-disable-next-line mocha/no-sibling-hooks
         before(function (done) {
             this.run({
                 fileResolver: fs,
@@ -172,6 +174,7 @@ describe('redirects', function () {
             });
         });
 
+        // eslint-disable-next-line mocha/no-identical-title
         it('should complete the run', function () {
             expect(testrun).to.be.ok;
             sinon.assert.calledOnce(testrun.start);

package/test/integration/sanity/request-size.test.js

@@ -2,7 +2,7 @@ var net = require('net'),
     expect = require('chai').expect,
     enableServerDestroy = require('server-destroy');
 
-describe('request size', function () {
+(typeof window === 'undefined' ? describe : describe.skip)('request size', function () {
     var server,
         testrun,
         POSTMAN = 'postman',

package/test/integration/sanity/response-status-message.test.js

@@ -1,6 +1,6 @@
 var expect = require('chai').expect;
 
-describe('response status message', function () {
+(typeof window === 'undefined' ? describe : describe.skip)('response status message', function () {
     var testrun;
 
     before(function (done) {

package/test/integration/sanity/restricted-addresses.test.js

@@ -1,6 +1,6 @@
 var expect = require('chai').expect;
 
-describe('restricted addresses', function () {
+(typeof window === 'undefined' ? describe : describe.skip)('restricted addresses', function () {
     var testrun;
 
     before(function (done) {

package/test/integration/sanity/script-result.test.js

@@ -88,7 +88,7 @@ describe('script result format', function () {
             expect(test).to.have.deep.property('data', {});
         });
 
-        it('should have cookies in the result', function () {
+        (typeof window === 'undefined' ? it : it.skip)('should have cookies in the result', function () {
             var prerequest = testrun.script.firstCall.args[2],
                 test = testrun.script.secondCall.args[2];
 

package/test/integration/sanity/server-ssl.test.js

@@ -1,6 +1,6 @@
 var expect = require('chai').expect;
 
-describe('SSL', function () {
+(typeof window === 'undefined' ? describe : describe.skip)('SSL', function () {
     var testrun;
 
     before(function (done) {

package/test/integration/sanity/systemproxy.test.js

@@ -1,7 +1,7 @@
 var expect = require('chai').expect,
     sdk = require('postman-collection');
 
-describe('systemProxy', function () {
+(typeof window === 'undefined' ? describe : describe.skip)('systemProxy', function () {
     var testrun;
 
     describe('valid output config', function () {

package/test/integration/sanity/timeouts/request.test.js

@@ -1,6 +1,6 @@
 var expect = require('chai').expect;
 
-describe('request timeout', function () {
+(typeof window === 'undefined' ? describe : describe.skip)('request timeout', function () {
     var testrun;
 
     before(function (done) {

package/test/integration/sanity/timeouts/sync-scripts.test.js

@@ -167,7 +167,7 @@ describe('synchronous script timeouts', function () {
             });
         });
 
-        describe('script timeout', function () {
+        (typeof window === 'undefined' ? describe : describe.skip)('script timeout', function () {
             var testrun;
 
             before(function (done) {

package/test/integration/sanity/uvm-globals-this.test.js

@@ -1,6 +1,6 @@
 var expect = require('chai').expect;
 
-describe('UVM allowed globals', function () {
+(typeof window === 'undefined' ? describe : describe.skip)('UVM allowed globals', function () {
     var testrun,
         globals = ['isNaN', 'SyntaxError', 'ArrayBuffer', 'JSON', 'unescape', 'URIError', 'TypeError', 'WeakSet',
             'Array', 'parseFloat', 'EvalError', 'parseInt', 'Error', 'Object', 'Int16Array',

package/test/integration/sanity/v2-regression.test.js

@@ -1,7 +1,7 @@
 var request = require('postman-request'),
     expect = require('chai').expect;
 
-describe('V2 regressions', function () {
+(typeof window === 'undefined' ? describe : describe.skip)('V2 regressions', function () {
     var testrun;
 
     before(function (done) {

package/test/karma.conf.js

@@ -22,8 +22,8 @@ module.exports = function (config) {
         // preprocess matching files before serving them to the browser
         // available preprocessors: https://npmjs.org/browse/keyword/karma-preprocessor
         preprocessors: {
-            '../index.js': ['browserify'], // Mention path as per your test js folder
-            '../test/unit/**/*.js': ['browserify'], // Mention path as per your library js folder
+            '../index.js': ['browserify'],
+            '../test/unit/**/*.js': ['browserify'],
             '../test/integration/**/*.js': ['browserify']
         },
         // test results reporter to use

package/test/system/repository.test.js

@@ -133,6 +133,10 @@ describe('project repository', function () {
             it('should have valid content', function () {
                 expect(gitignore).to.not.be.empty;
             });
+
+            it('should ignore the dist directory', function () {
+                expect(gitignore).to.include('dist/**');
+            });
         });
 
         describe(npmignorePath, function () {
@@ -143,10 +147,15 @@ describe('project repository', function () {
             it('should have valid content', function () {
                 expect(npmignore).to.not.be.empty;
             });
+
+            it('should not ignore the dist directory', function () {
+                expect(npmignore).not.to.include('dist/**');
+            });
         });
 
-        it('should have .gitignore coverage must be a subset of .npmignore coverage', function () {
-            expect(_.intersection(gitignore, npmignore)).to.eql(gitignore);
+        // eslint-disable-next-line mocha/valid-test-description
+        it('.gitignore coverage must be a subset of .npmignore coverage (except dist directory)', function () {
+            expect(_.intersection(gitignore, _.union(npmignore, ['dist/**']))).to.eql(gitignore);
         });
     });
 

package/test/unit/auth-handlers.test.js

@@ -1305,6 +1305,70 @@ describe('Auth Handler:', function () {
             });
         });
 
+        // issue: https://github.com/postmanlabs/postman-app-support/issues/8737
+        it('should generate correct signature for empty callback and addEmptyParamsToSign:true', function () {
+            var rawReq = {
+                    url: 'https://postman-echo.com/oauth1',
+                    auth: {
+                        type: 'oauth1',
+                        oauth1: {
+                            consumerKey: 'RKCGzna7bv9YD57c',
+                            consumerSecret: 'D+EdQ-gs$-%@2Nu7',
+                            token: 'foo',
+                            tokenSecret: 'bar',
+                            signatureMethod: 'HMAC-SHA1',
+                            timestamp: '1461319769',
+                            nonce: 'ik3oT5',
+                            version: '1.0',
+                            verifier: 'bar',
+                            callback: '',
+                            addParamsToHeader: false,
+                            addEmptyParamsToSign: true
+                        }
+                    }
+                },
+                request = new Request(rawReq),
+                auth = request.auth,
+                authInterface = createAuthInterface(auth),
+                handler = AuthLoader.getHandler(auth.type);
+
+            handler.sign(authInterface, request, function () {
+                expect(request.url.query.get('oauth_signature')).to.eql('w8WS1SXfe/dtJu/4tH5DaD7qZgM=');
+            });
+        });
+
+        // issue: https://github.com/postmanlabs/postman-app-support/issues/8737
+        it('should generate correct signature for empty verifier and addEmptyParamsToSign:true', function () {
+            var rawReq = {
+                    url: 'https://postman-echo.com/oauth1',
+                    auth: {
+                        type: 'oauth1',
+                        oauth1: {
+                            consumerKey: 'RKCGzna7bv9YD57c',
+                            consumerSecret: 'D+EdQ-gs$-%@2Nu7',
+                            token: 'foo',
+                            tokenSecret: 'bar',
+                            signatureMethod: 'HMAC-SHA1',
+                            timestamp: '1461319769',
+                            nonce: 'ik3oT5',
+                            version: '1.0',
+                            verifier: '',
+                            callback: 'http://postman.com',
+                            addParamsToHeader: false,
+                            addEmptyParamsToSign: true
+                        }
+                    }
+                },
+                request = new Request(rawReq),
+                auth = request.auth,
+                authInterface = createAuthInterface(auth),
+                handler = AuthLoader.getHandler(auth.type);
+
+            handler.sign(authInterface, request, function () {
+                expect(request.url.query.get('oauth_signature')).to.eql('WO1RMBRLIM5Anfxxt8P7Kbt82b4=');
+            });
+        });
+
         it('should generate correct signature for RSA based signature method', function () {
             // eslint-disable-next-line max-len
             var privateKey = '-----BEGIN RSA PRIVATE KEY-----\nMIICWwIBAAKBgFKLvzM9zbm3I0+HWcHlBSqpfRY/bKs6NDLclERrzfnReFV4utjkhjaEQPPT6tHVHKrZkcxmIgwe3XrkJkUjcuingXIF+Fc3KpY61qJ4HSM50qIuHdi+C5YfuXwNrh6OOeZAhhqgSw2e2XqPfATbkYYwpIFpdVdcH/Pb2ynpd6VXAgMBAAECgYAbQE+LFyhH25Iou0KCpJ0kDHhjU+UIUlrRP8kjHYQOqXzUmtr0p903OkpHNPsc8wJX1SQxGra60aXE4HVR9fYFQNliAnSmA/ztGR4ddnirK1Gzog4y2OOkicTdSqJ/1XXtTEDSRkA0Z2DIqcWgudeSDzVjUpreYwQ/rCEZbi50AQJBAJcf9wi5bU8tdZUCg3/8MNDwHhr4If4V/9kmhsgNp+M/9tHwCbD05hCbiGS7g58DPF+6V2K30qQYq7yvBP8Te4ECQQCL1GhX/YwkD6rexi0E1bjz+RqhNLTR9kexkTfSYmL6zHeeIFSH8ROioGOJMU51lUtMNkkrKEeki5SZpkfaQOzXAkAvBnJPU6vQ7HtfH8YdiDMEgQNNLxMcxmmzf4qHK8CnNRsvnnrVho8kcdFSTwsY6t/Zhdl1TXANQeQGtYtfeAeBAkEAhUB351JSWJMtrHqCsFbTmHxNKk7F+kiObeMLpUvpM0PiwifhJmNQ6Oubr0Pzlw4c4ZXiCGSsUVxK0lmpo423pQJATYDoxVhZrKA3xDAifWoyxbyxf/WXtUGDaAOuZc/naVN5TKiqaEO6G+k3NpmOXNKsYU/Zd9e6P/TnfU74TyDDDA==\n-----END RSA PRIVATE KEY-----',
@@ -1568,6 +1632,56 @@ describe('Auth Handler:', function () {
             });
         });
 
+        it('should separate custom header prefix and token with a space', function () {
+            var clonedRequestObj,
+                request,
+                auth,
+                authInterface,
+                handler;
+
+            clonedRequestObj = _.cloneDeep(requestObj);
+            clonedRequestObj.auth.oauth2.headerPrefix = 'Postman';
+
+            request = new Request(clonedRequestObj);
+            auth = request.auth;
+            authInterface = createAuthInterface(auth);
+            handler = AuthLoader.getHandler(auth.type);
+
+            handler.sign(authInterface, request, _.noop);
+
+            expect(request.headers.all()).to.be.an('array').that.has.lengthOf(1);
+            expect(request.headers.toJSON()[0]).to.eql({
+                key: 'Authorization',
+                value: 'Postman ' + requestObj.auth.oauth2.accessToken,
+                system: true
+            });
+        });
+
+        it('should trim extra spaces in custom header prefix', function () {
+            var clonedRequestObj,
+                request,
+                auth,
+                authInterface,
+                handler;
+
+            clonedRequestObj = _.cloneDeep(requestObj);
+            clonedRequestObj.auth.oauth2.headerPrefix = '     Postman     ';
+
+            request = new Request(clonedRequestObj);
+            auth = request.auth;
+            authInterface = createAuthInterface(auth);
+            handler = AuthLoader.getHandler(auth.type);
+
+            handler.sign(authInterface, request, _.noop);
+
+            expect(request.headers.all()).to.be.an('array').that.has.lengthOf(1);
+            expect(request.headers.toJSON()[0]).to.eql({
+                key: 'Authorization',
+                value: 'Postman ' + requestObj.auth.oauth2.accessToken,
+                system: true
+            });
+        });
+
         it('should add empty header prefix when headerPrefix = ""', function () {
             var clonedRequestObj,
                 request,

package/test/unit/version.test.js

@@ -4,7 +4,7 @@ var expect = require('chai').expect,
     collectionPackage = require('../../node_modules/postman-collection/package.json'),
     runtime = require('../../');
 
-describe('version', function () {
+(typeof window === 'undefined' ? describe : describe.skip)('version', function () {
     var moduleData = {
             version: runtimePackage.version,
             dependencies: {