Skip to content

Commit ff8202c

Browse files
committed
test: fix undeclared variable access
Fixes all cases of undeclared variable access as uncovered by the no-undef rule of eslint. PR-URL: #1794 Reviewed-By: Trevor Norris <[email protected]>
1 parent 8732977 commit ff8202c

File tree

3 files changed

+12
-20
lines changed

3 files changed

+12
-20
lines changed

test/parallel/test-cluster-worker-exit.js

+6-10
Original file line numberDiff line numberDiff line change
@@ -106,17 +106,13 @@ function checkResults(expected_results, results) {
106106
var actual = results[k],
107107
expected = expected_results[k];
108108

109-
if (typeof expected === 'function') {
110-
expected(r[k]);
111-
} else {
112-
var msg = (expected[1] || '') +
113-
(' [expected: ' + expected[0] + ' / actual: ' + actual + ']');
109+
var msg = (expected[1] || '') +
110+
(' [expected: ' + expected[0] + ' / actual: ' + actual + ']');
114111

115-
if (expected && expected.length) {
116-
assert.equal(actual, expected[0], msg);
117-
} else {
118-
assert.equal(actual, expected, msg);
119-
}
112+
if (expected && expected.length) {
113+
assert.equal(actual, expected[0], msg);
114+
} else {
115+
assert.equal(actual, expected, msg);
120116
}
121117
}
122118
}

test/parallel/test-cluster-worker-kill.js

+5-9
Original file line numberDiff line numberDiff line change
@@ -106,16 +106,12 @@ function checkResults(expected_results, results) {
106106
var actual = results[k],
107107
expected = expected_results[k];
108108

109-
if (typeof expected === 'function') {
110-
expected(r[k]);
109+
var msg = (expected[1] || '') +
110+
(' [expected: ' + expected[0] + ' / actual: ' + actual + ']');
111+
if (expected && expected.length) {
112+
assert.equal(actual, expected[0], msg);
111113
} else {
112-
var msg = (expected[1] || '') +
113-
(' [expected: ' + expected[0] + ' / actual: ' + actual + ']');
114-
if (expected && expected.length) {
115-
assert.equal(actual, expected[0], msg);
116-
} else {
117-
assert.equal(actual, expected, msg);
118-
}
114+
assert.equal(actual, expected, msg);
119115
}
120116
}
121117
}

test/pummel/test-net-timeout.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -79,7 +79,7 @@ process.on('exit', function() {
7979
assert.ok(starttime != null);
8080
assert.ok(timeouttime != null);
8181

82-
diff = timeouttime - starttime;
82+
var diff = timeouttime - starttime;
8383
console.log('diff = ' + diff);
8484

8585
assert.ok(timeout < diff);

0 commit comments

Comments
 (0)