test: load common.js in all tests

common.js contains code that detects leaked variables.

In preparation for an eslint rule that will enforce loading common.js in
test files, load it everywhere it can be loaded and use an
`eslint-disable` comment for files that intentionally leak.

PR-URL: https://github.com/nodejs/node/pull/3157
Reviewed-By: Rod Vagg <rod@vagg.org>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>

diff --git a/test/addons/async-hello-world/test.js b/test/addons/async-hello-world/test.js
index a284a019f01a8be3dcf762441921374ac9c3a2cc..14b80b711b9b35f427fb53079b445696efb66668 100644 (file)
--- a/test/addons/async-hello-world/test.js
+++ b/test/addons/async-hello-world/test.js
@@ -1,4 +1,5 @@
 'use strict';
+require('../../common');
 var assert = require('assert');
 var binding = require('./build/Release/binding');
 var called = false;

diff --git a/test/addons/at-exit/test.js b/test/addons/at-exit/test.js
index c36f59b976e983e740b1a9de935b8f80efd8c1a3..32264a6604eeb6bf4f8b59a86757e4d40a362c90 100644 (file)
--- a/test/addons/at-exit/test.js
+++ b/test/addons/at-exit/test.js
@@ -1,2 +1,3 @@
 'use strict';
+require('../../common');
 var binding = require('./build/Release/binding');

diff --git a/test/addons/heap-profiler/test.js b/test/addons/heap-profiler/test.js
index a4a316fa572f9b846abef5a7bc151f0b91e64484..3fa00e9b86afadf86b6d99944288f91305bb992d 100644 (file)
--- a/test/addons/heap-profiler/test.js
+++ b/test/addons/heap-profiler/test.js
@@ -1,5 +1,7 @@
 'use strict';
 
+require('../../common');
+
 const binding = require('./build/Release/binding');
 
 // Create an AsyncWrap object.

diff --git a/test/addons/hello-world-function-export/test.js b/test/addons/hello-world-function-export/test.js
index 1056386583faf16a6bed01c379d8c3d978cf1e91..6dcdf39ce1c4135812e5f8051b018f11cc72cc89 100644 (file)
--- a/test/addons/hello-world-function-export/test.js
+++ b/test/addons/hello-world-function-export/test.js
@@ -1,4 +1,5 @@
 'use strict';
+require('../../common');
 var assert = require('assert');
 var binding = require('./build/Release/binding');
 assert.equal('world', binding());

diff --git a/test/addons/hello-world/test.js b/test/addons/hello-world/test.js
index c0492292fd725033daf8dcea6063d9d11f67a49a..5637e9e6e93a3e2ab104928cacd38d3ca3ce81ed 100644 (file)
--- a/test/addons/hello-world/test.js
+++ b/test/addons/hello-world/test.js
@@ -1,4 +1,5 @@
 'use strict';
+require('../../common');
 var assert = require('assert');
 var binding = require('./build/Release/binding');
 assert.equal('world', binding.hello());

diff --git a/test/addons/repl-domain-abort/test.js b/test/addons/repl-domain-abort/test.js
index a520df03805ccb6bdb210eea22b31a6a497436cd..1268b61f362eedfad1cb33db9d89c2f00eb9be82 100644 (file)
--- a/test/addons/repl-domain-abort/test.js
+++ b/test/addons/repl-domain-abort/test.js
@@ -1,4 +1,5 @@
 'use strict';
+require('../../common');
 var assert = require('assert');
 var repl = require('repl');
 var stream = require('stream');

diff --git a/test/debugger/test-debugger-repl-break-in-module.js b/test/debugger/test-debugger-repl-break-in-module.js
index 995f6fe0db3a3d1a8ab52dee12ea682b3c89d0bf..e3f998eb6465d33831ebf6211c3295ee1c16d41d 100644 (file)
--- a/test/debugger/test-debugger-repl-break-in-module.js
+++ b/test/debugger/test-debugger-repl-break-in-module.js
@@ -1,4 +1,5 @@
 'use strict';
+require('../common');
 var repl = require('./helper-debugger-repl.js');
 
 repl.startDebugger('break-in-module/main.js');

diff --git a/test/debugger/test-debugger-repl-restart.js b/test/debugger/test-debugger-repl-restart.js
index d3b2c169dea0a80c15bf34b92b4229950bee0ff6..584cb098bf02e618787e7f90aca426f1c223bdfd 100644 (file)
--- a/test/debugger/test-debugger-repl-restart.js
+++ b/test/debugger/test-debugger-repl-restart.js
@@ -1,4 +1,5 @@
 'use strict';
+require('../common');
 var repl = require('./helper-debugger-repl.js');
 
 repl.startDebugger('breakpoints.js');

diff --git a/test/debugger/test-debugger-repl-term.js b/test/debugger/test-debugger-repl-term.js
index 741adf682ab1ddb96e1240f53e8a7669ff8b2f39..6872f6ce2f9c72d0365eb5353a3ed4ad5f97c6f6 100644 (file)
--- a/test/debugger/test-debugger-repl-term.js
+++ b/test/debugger/test-debugger-repl-term.js
@@ -1,4 +1,5 @@
 'use strict';
+require('../common');
 process.env.NODE_FORCE_READLINE = 1;
 
 var repl = require('./helper-debugger-repl.js');

diff --git a/test/debugger/test-debugger-repl.js b/test/debugger/test-debugger-repl.js
index 2e44da55d728597416419185bf240bc02e07d2f7..f93570c50156ceea604ff2279b859ae7c4233bf6 100644 (file)
--- a/test/debugger/test-debugger-repl.js
+++ b/test/debugger/test-debugger-repl.js
@@ -1,4 +1,5 @@
 'use strict';
+require('../common');
 var repl = require('./helper-debugger-repl.js');
 
 repl.startDebugger('breakpoints.js');