summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorRoman Reiss <me@silverwind.io>2015-05-29 19:28:01 +0200
committerRoman Reiss <me@silverwind.io>2015-06-11 20:22:51 +0200
commitff8202c6f475973c8674e61c794b77e1b072144b (patch)
tree9d5ba10768095c60881f88206dbb05f40a7c4560
parent873297753642e77fb556d13a0e4bc228f34171bd (diff)
downloadandroid-node-v8-ff8202c6f475973c8674e61c794b77e1b072144b.tar.gz
android-node-v8-ff8202c6f475973c8674e61c794b77e1b072144b.tar.bz2
android-node-v8-ff8202c6f475973c8674e61c794b77e1b072144b.zip
test: fix undeclared variable access
Fixes all cases of undeclared variable access as uncovered by the no-undef rule of eslint. PR-URL: https://github.com/nodejs/io.js/pull/1794 Reviewed-By: Trevor Norris <trev.norris@gmail.com>
-rw-r--r--test/parallel/test-cluster-worker-exit.js16
-rw-r--r--test/parallel/test-cluster-worker-kill.js14
-rw-r--r--test/pummel/test-net-timeout.js2
3 files changed, 12 insertions, 20 deletions
diff --git a/test/parallel/test-cluster-worker-exit.js b/test/parallel/test-cluster-worker-exit.js
index 982b099cd1..3607a992a3 100644
--- a/test/parallel/test-cluster-worker-exit.js
+++ b/test/parallel/test-cluster-worker-exit.js
@@ -106,17 +106,13 @@ function checkResults(expected_results, results) {
var actual = results[k],
expected = expected_results[k];
- if (typeof expected === 'function') {
- expected(r[k]);
- } else {
- var msg = (expected[1] || '') +
- (' [expected: ' + expected[0] + ' / actual: ' + actual + ']');
+ var msg = (expected[1] || '') +
+ (' [expected: ' + expected[0] + ' / actual: ' + actual + ']');
- if (expected && expected.length) {
- assert.equal(actual, expected[0], msg);
- } else {
- assert.equal(actual, expected, msg);
- }
+ if (expected && expected.length) {
+ assert.equal(actual, expected[0], msg);
+ } else {
+ assert.equal(actual, expected, msg);
}
}
}
diff --git a/test/parallel/test-cluster-worker-kill.js b/test/parallel/test-cluster-worker-kill.js
index c73cf2dbd5..e12b466610 100644
--- a/test/parallel/test-cluster-worker-kill.js
+++ b/test/parallel/test-cluster-worker-kill.js
@@ -106,16 +106,12 @@ function checkResults(expected_results, results) {
var actual = results[k],
expected = expected_results[k];
- if (typeof expected === 'function') {
- expected(r[k]);
+ var msg = (expected[1] || '') +
+ (' [expected: ' + expected[0] + ' / actual: ' + actual + ']');
+ if (expected && expected.length) {
+ assert.equal(actual, expected[0], msg);
} else {
- var msg = (expected[1] || '') +
- (' [expected: ' + expected[0] + ' / actual: ' + actual + ']');
- if (expected && expected.length) {
- assert.equal(actual, expected[0], msg);
- } else {
- assert.equal(actual, expected, msg);
- }
+ assert.equal(actual, expected, msg);
}
}
}
diff --git a/test/pummel/test-net-timeout.js b/test/pummel/test-net-timeout.js
index 347e3a4344..25544a6d5f 100644
--- a/test/pummel/test-net-timeout.js
+++ b/test/pummel/test-net-timeout.js
@@ -79,7 +79,7 @@ process.on('exit', function() {
assert.ok(starttime != null);
assert.ok(timeouttime != null);
- diff = timeouttime - starttime;
+ var diff = timeouttime - starttime;
console.log('diff = ' + diff);
assert.ok(timeout < diff);