Check for global leaks in all tests

test/common.js

@@ -24,3 +24,35 @@ exports.indirectInstanceOf = function (obj, cls) {
   var objChain = protoCtrChain(obj);
   return objChain.slice(-clsChain.length) === clsChain;
 };
+
+
+// Turn this off if the test should not check for global leaks.
+exports.globalCheck = true;
+
+process.on('exit', function () {
+  if (!exports.globalCheck) return;
+  var knownGlobals = [ setTimeout,
+                       setInterval,
+                       clearTimeout,
+                       clearInterval,
+                       console,
+                       Buffer,
+                       process,
+                       global ];
+
+  for (var x in global) {
+    var found = false;
+
+    for (var y in knownGlobals) {
+      if (global[x] === knownGlobals[y]) {
+        found = true;
+        break;
+      }
+    }
+
+    if (!found) {
+      console.error("Unknown global: %s", x);
+      exports.assert.ok(false);
+    }
+  }
+});

test/simple/test-global-leak.js

@@ -1,27 +0,0 @@
-var assert = require('assert');
-
-var knownGlobals = [ setTimeout
-                   , setInterval
-                   , clearTimeout
-                   , clearInterval
-                   , console
-                   , Buffer
-                   , process
-                   , global
-                   ];
-
-for (var x in global) {
-  var found = false;
-
-  for (var y in knownGlobals) {
-    if (global[x] === knownGlobals[y]) {
-      found = true;
-      break;
-    }
-  }
-
-  if (!found) {
-    console.error("Unknown global: %s", x);
-    assert.ok(false);
-  }
-}

test/simple/test-global.js

@@ -1,6 +1,8 @@
 var common = require('../common');
 var assert = require('assert');
 
+common.globalCheck = false;
+
 baseFoo = "foo";
 global.baseBar = "bar";
 

test/simple/test-repl.js

@@ -1,6 +1,8 @@
 var common = require('../common');
 var assert = require('assert');
 
+common.globalCheck = false;
+
 var net = require("net"),
   repl = require("repl"),
   message = "Read, Eval, Print Loop",

test/simple/test-script-new.js

@@ -1,7 +1,9 @@
 var common = require('../common');
 var assert = require('assert');
-
 var Script = require('vm').Script;
+
+common.globalCheck = false;
+
 common.debug('run a string');
 var script = new Script('"passed";');
 common.debug('script created');

test/simple/test-script-static-new.js

@@ -1,8 +1,9 @@
 var common = require('../common');
 var assert = require('assert');
-
 var Script = require('vm').Script;
 
+common.globalCheck = false;
+
 common.debug('run a string');
 var result = Script.runInNewContext('"passed";');
 assert.equal('passed', result);

test/simple/test-script-static-this.js

@@ -1,8 +1,9 @@
 var common = require('../common');
 var assert = require('assert');
-
 var Script = require('vm').Script;
 
+common.globalCheck = false;
+
 common.debug('run a string');
 var result = Script.runInThisContext('"passed";');
 assert.equal('passed', result);

test/simple/test-script-this.js

@@ -1,8 +1,9 @@
 var common = require('../common');
 var assert = require('assert');
-
 var Script = require('vm').Script;
 
+common.globalCheck = false;
+
 common.debug('run a string');
 var script = new Script('"passed";');
 var result = script.runInThisContext(script);