Skip to content

Commit c78091d

Browse files
committed
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: #3157 Reviewed-By: Rod Vagg <rod@vagg.org> Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
1 parent dbce29a commit c78091d

File tree

10 files changed

+11
-0
lines changed

10 files changed

+11
-0
lines changed

β€Žtest/addons/async-hello-world/test.jsβ€Ž

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,5 @@
11
'use strict';
2+
require('../../common');
23
var assert = require('assert');
34
var binding = require('./build/Release/binding');
45
var called = false;

β€Žtest/addons/at-exit/test.jsβ€Ž

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,2 +1,3 @@
11
'use strict';
2+
require('../../common');
23
var binding = require('./build/Release/binding');

β€Žtest/addons/heap-profiler/test.jsβ€Ž

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,7 @@
11
'use strict';
22

3+
require('../../common');
4+
35
const binding = require('./build/Release/binding');
46

57
// Create an AsyncWrap object.

β€Žtest/addons/hello-world-function-export/test.jsβ€Ž

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,5 @@
11
'use strict';
2+
require('../../common');
23
var assert = require('assert');
34
var binding = require('./build/Release/binding');
45
assert.equal('world', binding());

β€Žtest/addons/hello-world/test.jsβ€Ž

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,5 @@
11
'use strict';
2+
require('../../common');
23
var assert = require('assert');
34
var binding = require('./build/Release/binding');
45
assert.equal('world', binding.hello());

β€Žtest/addons/repl-domain-abort/test.jsβ€Ž

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,5 @@
11
'use strict';
2+
require('../../common');
23
var assert = require('assert');
34
var repl = require('repl');
45
var stream = require('stream');

β€Žtest/debugger/test-debugger-repl-break-in-module.jsβ€Ž

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,5 @@
11
'use strict';
2+
require('../common');
23
var repl = require('./helper-debugger-repl.js');
34

45
repl.startDebugger('break-in-module/main.js');

β€Žtest/debugger/test-debugger-repl-restart.jsβ€Ž

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,5 @@
11
'use strict';
2+
require('../common');
23
var repl = require('./helper-debugger-repl.js');
34

45
repl.startDebugger('breakpoints.js');

β€Žtest/debugger/test-debugger-repl-term.jsβ€Ž

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,5 @@
11
'use strict';
2+
require('../common');
23
process.env.NODE_FORCE_READLINE = 1;
34

45
var repl = require('./helper-debugger-repl.js');

β€Žtest/debugger/test-debugger-repl.jsβ€Ž

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,5 @@
11
'use strict';
2+
require('../common');
23
var repl = require('./helper-debugger-repl.js');
34

45
repl.startDebugger('breakpoints.js');

0 commit comments

Comments
Β (0)