fix: context in pm.require (#1083)

Author: appurva21

lib/sandbox/pm-require.js

@@ -2,7 +2,7 @@ const { LEGACY_GLOBS } = require('./postman-legacy-interface'),
 
     MODULE_KEY = '__module_obj', // why not use `module`?
     MODULE_WRAPPER = [
-        '(function (exports, module) {\n',
+        '((exports, module) => {\n',
         `\n})(${MODULE_KEY}.exports, ${MODULE_KEY});`
     ];
 

test/unit/sandbox-libraries/pm-require.test.js

@@ -1,3 +1,5 @@
+const { LEGACY_GLOBS } = require('../../../lib/sandbox/postman-legacy-interface');
+
 describe('sandbox library - pm.require api', function () {
     this.timeout(1000 * 60);
     var Sandbox = require('../../../'),
@@ -573,4 +575,24 @@ describe('sandbox library - pm.require api', function () {
             done();
         });
     });
+
+    it('should have access to complete context (expect legacy globals)', function (done) {
+        context.execute(`
+            const assert = require('assert');
+            const test = pm.require('test');
+
+            outsideContext =  Object.getOwnPropertyNames(this);
+
+            const diff = outsideContext.filter(x => !insideContext.includes(x));
+            assert.ok(diff.every((key) => '${LEGACY_GLOBS.toString()}'.split(',').includes(key) ));
+
+        `, {
+            context: sampleContextData,
+            resolvedPackages: {
+                test: {
+                    data: 'insideContext = Object.getOwnPropertyNames(this);'
+                }
+            }
+        }, done);
+    });
 });