Skip to content

Commit c6a99dd

Browse files
TrottFishrock123
authored andcommitted
test: remove unused variables from net tests
Remove a handful of variables that are declared but never used in the tests for the net module. PR-URL: #4430 Reviewed-By: Michaël Zasso <[email protected]> Reviewed-By: Colin Ihrig <[email protected]>
1 parent 54004f0 commit c6a99dd

7 files changed

+3
-7
lines changed

test/parallel/test-net-create-connection.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,7 @@ server.listen(tcpPort, 'localhost', function() {
2222

2323
function fail(opts, errtype, msg) {
2424
assert.throws(function() {
25-
var client = net.createConnection(opts, cb);
25+
net.createConnection(opts, cb);
2626
}, function(err) {
2727
return err instanceof errtype && msg === err.message;
2828
});

test/parallel/test-net-dns-custom-lookup.js

-1
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,6 @@
22
var common = require('../common');
33
var assert = require('assert');
44
var net = require('net');
5-
var dns = require('dns');
65
var ok = false;
76

87
function check(addressType, cb) {

test/parallel/test-net-dns-lookup-skip.js

-1
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,5 @@
11
'use strict';
22
var common = require('../common');
3-
var assert = require('assert');
43
var net = require('net');
54

65
function check(addressType) {

test/parallel/test-net-listen-close-server.js

-1
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,6 @@
22
var common = require('../common');
33
var assert = require('assert');
44
var net = require('net');
5-
var gotError = false;
65

76
var server = net.createServer(function(socket) {
87
});

test/parallel/test-net-local-address-port.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@ var common = require('../common');
33
var assert = require('assert');
44
var net = require('net');
55

6-
var conns = 0, conns_closed = 0;
6+
var conns = 0;
77

88
var server = net.createServer(function(socket) {
99
conns++;

test/parallel/test-net-localerror.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -17,6 +17,6 @@ connect({
1717

1818
function connect(opts, msg) {
1919
assert.throws(function() {
20-
var client = net.connect(opts);
20+
net.connect(opts);
2121
}, msg);
2222
}

test/parallel/test-net-reconnect.js

-1
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,6 @@ var assert = require('assert');
55
var net = require('net');
66

77
var N = 50;
8-
var c = 0;
98
var client_recv_count = 0;
109
var client_end_count = 0;
1110
var disconnect_count = 0;

0 commit comments

Comments
 (0)