test: remove unused vars from parallel tests
authorRich Trott <rtrott@gmail.com>
Sat, 2 Jan 2016 01:37:10 +0000 (17:37 -0800)
committerMyles Borins <mborins@us.ibm.com>
Tue, 19 Jan 2016 19:52:40 +0000 (11:52 -0800)
Remove all remaining unused variables from tests in test/parallel.

PR-URL: https://github.com/nodejs/node/pull/4511
Reviewed-By: James M Snell<jasnell@gmail.com>
Reviewed-By: Johan Bergström <bugs@bergstroem.nu>
46 files changed:
test/parallel/test-assert-typedarray-deepequal.js
test/parallel/test-buffer-fakes.js
test/parallel/test-child-process-exec-buffer.js
test/parallel/test-cluster-http-pipe.js
test/parallel/test-crypto-binary-default.js
test/parallel/test-crypto-dh.js
test/parallel/test-debug-signal-cluster.js
test/parallel/test-dgram-error-message-address.js
test/parallel/test-dgram-pingpong.js
test/parallel/test-domain-http-server.js
test/parallel/test-domain-implicit-fs.js
test/parallel/test-domain-uncaught-exception.js
test/parallel/test-domain-with-abort-on-uncaught-exception.js
test/parallel/test-dsa-fips-invalid-key.js
test/parallel/test-eval.js
test/parallel/test-event-emitter-remove-listeners.js
test/parallel/test-file-write-stream3.js
test/parallel/test-fs-open-flags.js
test/parallel/test-fs-open.js
test/parallel/test-fs-utimes.js
test/parallel/test-fs-write-file-sync.js
test/parallel/test-preload.js
test/parallel/test-process-release.js
test/parallel/test-promises-unhandled-rejections.js
test/parallel/test-readline-undefined-columns.js
test/parallel/test-regress-GH-4256.js
test/parallel/test-repl-.save.load.js
test/parallel/test-repl-autolibs.js
test/parallel/test-repl-domain.js
test/parallel/test-repl-syntax-error-stack.js
test/parallel/test-repl-tab-complete-crash.js
test/parallel/test-repl-tab-complete.js
test/parallel/test-stream-pipe-error-handling.js
test/parallel/test-stream-push-order.js
test/parallel/test-stream-unshift-read-race.js
test/parallel/test-stream-writev.js
test/parallel/test-stream2-compatibility.js
test/parallel/test-stream2-large-read-stall.js
test/parallel/test-stream2-objects.js
test/parallel/test-stream2-readable-from-list.js
test/parallel/test-stream2-transform.js
test/parallel/test-timers-ordering.js
test/parallel/test-timers-zero-timeout.js
test/parallel/test-url.js
test/parallel/test-util-decorate-error-stack.js
test/parallel/test-util-internal.js

index 3274878..68edefd 100644 (file)
@@ -1,6 +1,6 @@
 'use strict';
 
-const common = require('../common');
+require('../common');
 const assert = require('assert');
 const a = require('assert');
 
index 5804cbb..1a1d9c0 100644 (file)
@@ -3,7 +3,6 @@
 require('../common');
 const assert = require('assert');
 const Buffer = require('buffer').Buffer;
-const Bp = Buffer.prototype;
 
 function FakeBuffer() { }
 FakeBuffer.__proto__ = Buffer;
index b9727f0..a381af9 100644 (file)
@@ -18,7 +18,7 @@ exec('echo ' + str, function(err, stdout, stderr) {
 });
 
 // no encoding (Buffers expected)
-var child = exec('echo ' + str, {
+exec('echo ' + str, {
   encoding: null
 }, function(err, stdout, stderr) {
   assert.ok(stdout instanceof Buffer, 'Expected stdout to be a Buffer');
index 4c6dee3..cb9b090 100644 (file)
@@ -34,7 +34,6 @@ http.createServer(function(req, res) {
   res.writeHead(200);
   res.end('OK');
 }).listen(common.PIPE, function() {
-  var self = this;
   http.get({ socketPath: common.PIPE, path: '/' }, function(res) {
     res.resume();
     res.on('end', function(err) {
index 806f5de..c4b8990 100644 (file)
@@ -20,7 +20,6 @@ var fs = require('fs');
 var path = require('path');
 
 // Test Certificates
-var caPem = fs.readFileSync(common.fixturesDir + '/test_ca.pem', 'ascii');
 var certPem = fs.readFileSync(common.fixturesDir + '/test_cert.pem', 'ascii');
 var certPfx = fs.readFileSync(common.fixturesDir + '/test_cert.pfx');
 var keyPem = fs.readFileSync(common.fixturesDir + '/test_key.pem', 'ascii');
index f3e54c4..9322441 100644 (file)
@@ -171,7 +171,7 @@ var ecdh3 = crypto.createECDH('secp256k1');
 var key3 = ecdh3.generateKeys();
 
 assert.throws(function() {
-  var secret3 = ecdh2.computeSecret(key3, 'binary', 'buffer');
+  ecdh2.computeSecret(key3, 'binary', 'buffer');
 });
 
 // ECDH should allow .setPrivateKey()/.setPublicKey()
index 6572ef5..7a95045 100644 (file)
@@ -10,7 +10,6 @@ var options = { stdio: ['inherit', 'inherit', 'pipe', 'ipc'] };
 var child = spawn(process.execPath, args, options);
 
 var outputLines = [];
-var outputTimerId;
 var waitingForDebuggers = false;
 
 var pids = null;
index e307a23..0056956 100644 (file)
@@ -20,7 +20,6 @@ socket_ipv4.bind(common.PORT, '1.1.1.1');
 
 // IPv6 Test
 var socket_ipv6 = dgram.createSocket('udp6');
-var family_ipv6 = 'IPv6';
 
 socket_ipv6.on('listening', common.fail);
 
index 29ab927..883085a 100644 (file)
@@ -11,7 +11,6 @@ function pingPongTest(port, host) {
   var callbacks = 0;
   var N = 500;
   var count = 0;
-  var sent_final_ping = false;
 
   var server = dgram.createSocket('udp4', function(msg, rinfo) {
     if (debug) console.log('server got: ' + msg +
@@ -48,7 +47,6 @@ function pingPongTest(port, host) {
       if (count < N) {
         client.send(buf, 0, buf.length, port, 'localhost');
       } else {
-        sent_final_ping = true;
         client.send(buf, 0, buf.length, port, 'localhost', function() {
           client.close();
         });
index 99dbf41..37dbc30 100644 (file)
@@ -9,7 +9,6 @@ objects.baz.asdf = objects;
 
 var serverCaught = 0;
 var clientCaught = 0;
-var disposeEmit = 0;
 
 var server = http.createServer(function(req, res) {
   var dom = domain.create();
index a5bbb83..d0a4684 100644 (file)
@@ -8,7 +8,6 @@ var caught = 0;
 var expectCaught = 1;
 
 var d = new domain.Domain();
-var e = new events.EventEmitter();
 
 d.on('error', function(er) {
   console.error('caught', er);
index 8792eb1..35632db 100644 (file)
@@ -13,8 +13,6 @@ const assert = require('assert');
 const domain = require('domain');
 const child_process = require('child_process');
 
-const uncaughtExceptions = {};
-
 const tests = [];
 
 function test1() {
index 1307780..3158274 100644 (file)
@@ -31,7 +31,6 @@ const domainErrHandlerExMessage = 'exception from domain error handler';
 if (process.argv[2] === 'child') {
   var domain = require('domain');
   var d = domain.create();
-  var triggeredProcessUncaughtException = false;
 
   process.on('uncaughtException', function onUncaughtException() {
     // The process' uncaughtException event must not be emitted when
@@ -116,13 +115,6 @@ if (process.argv[2] === 'child') {
     var child = exec(cmdToExec);
 
     if (child) {
-      var childTriggeredOnUncaughtExceptionHandler = false;
-      child.on('message', function onChildMsg(msg) {
-        if (msg === 'triggeredProcessUncaughtEx') {
-          childTriggeredOnUncaughtExceptionHandler = true;
-        }
-      });
-
       child.on('exit', function onChildExited(exitCode, signal) {
         // When throwing errors from the top-level domain error handler
         // outside of a try/catch block, the process should not exit gracefully
index 0b57734..2d30ef6 100644 (file)
@@ -12,8 +12,6 @@ var fs = require('fs');
 
 var input = 'hello';
 
-var dsapub = fs.readFileSync(common.fixturesDir +
-                             '/keys/dsa_public_1025.pem');
 var dsapri = fs.readFileSync(common.fixturesDir +
                              '/keys/dsa_private_1025.pem');
 var sign = crypto.createSign('DSS1');
index d3a83fb..59b5d1a 100644 (file)
@@ -10,7 +10,7 @@ var error_count = 0;
 var cmd = ['"' + process.execPath + '"', '-e', '"console.error(process.argv)"',
            'foo', 'bar'].join(' ');
 var expected = util.format([process.execPath, 'foo', 'bar']) + '\n';
-var child = exec(cmd, function(err, stdout, stderr) {
+exec(cmd, function(err, stdout, stderr) {
   if (err) {
     console.log(err.toString());
     ++error_count;
index 8993aad..396979a 100644 (file)
@@ -15,11 +15,6 @@ function listener2() {
   count++;
 }
 
-function listener3() {
-  console.log('listener3');
-  count++;
-}
-
 function remove1() {
   assert(0);
 }
index 6ed852e..a2552d9 100644 (file)
@@ -124,7 +124,7 @@ function run_test_2() {
 
 
 function run_test_3() {
-  var file, buffer, options;
+  var file, options;
 
   var data = '\u2026\u2026',    // 3 bytes * 2 = 6 bytes in UTF-8
       fileData;
@@ -167,7 +167,7 @@ function run_test_3() {
 
 
 function run_test_4() {
-  var file, options;
+  var options;
 
   options = { start: -5,
               flags: 'r+' };
index b53a823..667abb8 100644 (file)
@@ -7,14 +7,9 @@ var fs = require('fs');
 
 var O_APPEND = constants.O_APPEND || 0;
 var O_CREAT = constants.O_CREAT || 0;
-var O_DIRECTORY = constants.O_DIRECTORY || 0;
 var O_EXCL = constants.O_EXCL || 0;
-var O_NOCTTY = constants.O_NOCTTY || 0;
-var O_NOFOLLOW = constants.O_NOFOLLOW || 0;
 var O_RDONLY = constants.O_RDONLY || 0;
 var O_RDWR = constants.O_RDWR || 0;
-var O_SYMLINK = constants.O_SYMLINK || 0;
-var O_SYNC = constants.O_SYNC || 0;
 var O_TRUNC = constants.O_TRUNC || 0;
 var O_WRONLY = constants.O_WRONLY || 0;
 
index 0a069b9..0818f2b 100644 (file)
@@ -1,6 +1,5 @@
 'use strict';
 require('../common');
-var constants = require('constants');
 var assert = require('assert');
 var fs = require('fs');
 
index 2ad6cea..5d5cb40 100644 (file)
@@ -48,7 +48,7 @@ function expect_ok(syscall, resource, err, atime, mtime) {
 // would be even better though (node doesn't have such functionality yet)
 function runTest(atime, mtime, callback) {
 
-  var fd, err;
+  var fd;
   //
   // test synchronized code paths, these functions throw on failure
   //
index 3737340..5f51878 100644 (file)
@@ -15,7 +15,7 @@ fs._closeSync = fs.closeSync;
 fs.closeSync = closeSync;
 
 // Reset the umask for testing
-var mask = process.umask(0o000);
+process.umask(0o000);
 
 // On Windows chmod is only able to manipulate read-only bit. Test if creating
 // the file in read-only mode works.
index c1d92c7..f30ad1d 100644 (file)
@@ -82,7 +82,6 @@ child_process.exec(nodeBinary + ' '
   });
 
 // https://github.com/nodejs/node/issues/1691
-var originalCwd = process.cwd();
 process.chdir(path.join(__dirname, '../fixtures/'));
 child_process.exec(nodeBinary + ' '
   + '--expose_debug_as=v8debug '
index af21daf..aac7009 100644 (file)
@@ -1,5 +1,5 @@
 'use strict';
-const common = require('../common');
+require('../common');
 const assert = require('assert');
 const versionParts = process.versions.node.split('.');
 
index e229bad..5e7739d 100644 (file)
@@ -185,12 +185,11 @@ asyncTest('When re-throwing new errors in a promise catch, only the' +
 asyncTest('Test params of unhandledRejection for a synchronously-rejected' +
           'promise', function(done) {
   var e = new Error();
-  var e2 = new Error();
   onUnhandledSucceed(done, function(reason, promise) {
     assert.strictEqual(e, reason);
     assert.strictEqual(promise, promise);
   });
-  var promise = Promise.reject(e);
+  Promise.reject(e);
 });
 
 asyncTest('When re-throwing new errors in a promise catch, only the ' +
@@ -629,7 +628,7 @@ asyncTest('Promise unhandledRejection handler does not interfere with domain' +
       assert.strictEqual(domainReceivedError, domainError);
       d.dispose();
     });
-    var a = Promise.reject(e);
+    Promise.reject(e);
     process.nextTick(function() {
       throw domainError;
     });
index 73945af..24f138f 100644 (file)
@@ -11,7 +11,7 @@ const readline = require('readline');
 const iStream = new PassThrough();
 const oStream = new PassThrough();
 
-const rli = readline.createInterface({
+readline.createInterface({
   terminal: true,
   input: iStream,
   output: oStream,
index 1a4a78b..a312fb2 100644 (file)
@@ -1,6 +1,6 @@
 'use strict';
 require('../common');
 process.domain = null;
-var timer = setTimeout(function() {
+setTimeout(function() {
   console.log('this console.log statement should not make node crash');
 }, 1);
index c71b383..6d28015 100644 (file)
@@ -1,6 +1,5 @@
 'use strict';
 var assert = require('assert');
-var util = require('util');
 var join = require('path').join;
 var fs = require('fs');
 var common = require('../common');
index 05cc299..15f779d 100644 (file)
@@ -8,7 +8,7 @@ var repl = require('repl');
 common.globalCheck = false;
 
 const putIn = new common.ArrayStream();
-var testMe = repl.start('', putIn, null, true);
+repl.start('', putIn, null, true);
 
 test1();
 
index 1de2405..adc8478 100644 (file)
@@ -1,7 +1,6 @@
 'use strict';
 var common = require('../common');
 
-var util   = require('util');
 var repl   = require('repl');
 
 const putIn = new common.ArrayStream();
index 647d3e3..ebf0b2e 100644 (file)
@@ -4,7 +4,6 @@ const common = require('../common');
 const assert = require('assert');
 const path = require('path');
 const repl = require('repl');
-const util = require('util');
 let found = false;
 
 process.on('exit', () => {
@@ -17,7 +16,7 @@ common.ArrayStream.prototype.write = function(output) {
 };
 
 const putIn = new common.ArrayStream();
-const testMe = repl.start('', putIn);
+repl.start('', putIn);
 let file = path.resolve(__dirname, '../fixtures/syntax/bad_syntax');
 
 if (common.isWindows)
index 484580f..ce28375 100644 (file)
@@ -2,7 +2,6 @@
 
 const common = require('../common');
 const assert = require('assert');
-const util = require('util');
 const repl = require('repl');
 
 var referenceErrorCount = 0;
index 6bf7d50..86a0444 100644 (file)
@@ -4,7 +4,6 @@
 
 var common = require('../common');
 var assert = require('assert');
-var util = require('util');
 var repl = require('repl');
 var referenceErrors = 0;
 var expectedReferenceErrors = 0;
index 8b674bf..88a70fb 100644 (file)
@@ -38,7 +38,6 @@ var Stream = require('stream').Stream;
 })();
 
 (function testErrorWithRemovedListenerThrows() {
-  var EE = require('events').EventEmitter;
   var R = Stream.Readable;
   var W = Stream.Writable;
 
@@ -73,7 +72,6 @@ var Stream = require('stream').Stream;
 })();
 
 (function testErrorWithRemovedListenerThrows() {
-  var EE = require('events').EventEmitter;
   var R = Stream.Readable;
   var W = Stream.Writable;
 
index 0bbcdf5..f09cf95 100644 (file)
@@ -21,7 +21,7 @@ s._read = function(n) {
   }
 };
 
-var v = s.read(0);
+s.read(0);
 
 // ACTUALLY [1, 3, 5, 6, 4, 2]
 
index 253427e..8239a38 100644 (file)
@@ -13,7 +13,6 @@ var stream = require('stream');
 var hwm = 10;
 var r = stream.Readable({ highWaterMark: hwm });
 var chunks = 10;
-var t = (chunks * 5);
 
 var data = new Buffer(chunks * hwm + Math.ceil(hwm / 2));
 for (var i = 0; i < data.length; i++) {
index 8893350..5b07dd1 100644 (file)
@@ -30,11 +30,9 @@ function test(decode, uncork, multi, next) {
   function cnt(msg) {
     expectCount++;
     var expect = expectCount;
-    var called = false;
     return function(er) {
       if (er)
         throw er;
-      called = true;
       counter++;
       assert.equal(counter, expect);
     };
index 3dbee62..1ee47c0 100644 (file)
@@ -4,7 +4,6 @@ var R = require('_stream_readable');
 var assert = require('assert');
 
 var util = require('util');
-var EE = require('events').EventEmitter;
 
 var ondataCalled = 0;
 
@@ -25,7 +24,7 @@ TestReader.prototype._read = function(n) {
   this._buffer = new Buffer(0);
 };
 
-var reader = new TestReader();
+new TestReader();
 setImmediate(function() {
   assert.equal(ondataCalled, 1);
   console.log('ok');
index 84457c7..6a38baa 100644 (file)
@@ -53,9 +53,6 @@ function push() {
     setTimeout(push);
 }
 
-// start the flow
-var ret = r.read(0);
-
 process.on('exit', function() {
   assert.equal(pushes, PUSHCOUNT + 1);
   assert(endEmitted);
index e79b8f9..e67359e 100644 (file)
@@ -167,8 +167,6 @@ test('read(0) for object streams', function(t) {
   r.push('foobar');
   r.push(null);
 
-  var v = r.read(0);
-
   r.pipe(toArray(function(array) {
     assert.deepEqual(array, ['foobar']);
 
index d4b42b6..ab9d9c5 100644 (file)
@@ -39,8 +39,6 @@ process.nextTick(run);
 
 
 test('buffers', function(t) {
-  // have a length
-  var len = 16;
   var list = [ new Buffer('foog'),
                new Buffer('bark'),
                new Buffer('bazy'),
@@ -69,8 +67,6 @@ test('buffers', function(t) {
 });
 
 test('strings', function(t) {
-  // have a length
-  var len = 16;
   var list = [ 'foog',
                'bark',
                'bazy',
index 7dad573..11ee459 100644 (file)
@@ -303,13 +303,10 @@ test('passthrough event emission', function(t) {
   var pt = new PassThrough();
   var emits = 0;
   pt.on('readable', function() {
-    var state = pt._readableState;
     console.error('>>> emit readable %d', emits);
     emits++;
   });
 
-  var i = 0;
-
   pt.write(new Buffer('foog'));
 
   console.error('need emit 0');
index 5da8896..cef91e5 100644 (file)
@@ -3,13 +3,10 @@ require('../common');
 var assert = require('assert');
 var Timer = process.binding('timer_wrap').Timer;
 
-var i;
-
 var N = 30;
 
 var last_i = 0;
 var last_ts = 0;
-var start = Timer.now();
 
 var f = function(i) {
   if (i <= N) {
index 78445ac..ab0c38f 100644 (file)
@@ -7,7 +7,7 @@ var assert = require('assert');
   var ncalled = 0;
 
   setTimeout(f, 0, 'foo', 'bar', 'baz');
-  var timer = setTimeout(function() {}, 0);
+  setTimeout(function() {}, 0);
 
   function f(a, b, c) {
     assert.equal(a, 'foo');
index a912b43..cd097be 100644 (file)
@@ -1514,9 +1514,6 @@ relativeTests2.forEach(function(relativeTest) {
 //if format and parse are inverse operations then
 //resolveObject(parse(x), y) == parse(resolve(x, y))
 
-//host and hostname are special, in this case a '' value is important
-var emptyIsImportant = {'host': true, 'hostname': ''};
-
 //format: [from, path, expected]
 relativeTests.forEach(function(relativeTest) {
   var actual = url.resolveObject(url.parse(relativeTest[0]), relativeTest[1]),
index 24fee56..017a5ab 100644 (file)
@@ -1,6 +1,6 @@
 // Flags: --expose_internals
 'use strict';
-const common = require('../common');
+require('../common');
 const assert = require('assert');
 const internalUtil = require('internal/util');
 
index 9ab883e..b557356 100644 (file)
@@ -1,7 +1,7 @@
 'use strict';
 // Flags: --expose_internals
 
-const common = require('../common');
+require('../common');
 const assert = require('assert');
 const internalUtil = require('internal/util');