tests: undefined variable "self" should be void 0 in the vm

Author: watilde

tests/noderunner.js

@@ -17,6 +17,8 @@ function runTest(testPath) {
     var content,
         context = LIBS.vm.createContext({});
 
+    context.self = void 0;
+
     content = LIBS.fs.readFileSync(LIBS.path.resolve(TEST_DIR, testPath)).toString();
     LIBS.vm.runInContext(POLYFILL_CODE, context, POLYFILL_PATH);
     LIBS.vm.runInContext(DATA_CODE, context, DATA_PATH);

tests/polyfilling.js

@@ -12,6 +12,7 @@ function assert(value, expected, message) {
 
 const context = new vm.createContext();
 var script = new vm.Script('this');
+context.self = void 0;
 const window = script.runInContext(context);
 
 const fs = require('fs');
@@ -22,10 +23,11 @@ window.window = window;  // circular, in case the polyfill uses window
 var originalIntl = window.Intl;
 var script = new vm.Script(code);
 script.runInContext(context);
-assert(typeof global.Intl, 'object', 'for this test to function, global.Intl is required');
+assert(typeof window.Intl, 'object', 'for this test to function, window.Intl is required');
 assert(typeof window.IntlPolyfill, 'object', 'polyfill should always add the custom global IntlPolyfill');
-assert(window.Intl, originalIntl, 'validating that the polyfilling process does not touch the original Intl value');
-
+if (originalIntl) {
+  assert(window.Intl, originalIntl, 'validating that the polyfilling process does not touch the original Intl value if it exists');
+}
 // second evaluation
 window.window = window;  // circular, in case the polyfill uses window
 window.Intl = undefined; // disabling Intl