Skip to content

Commit 0e02eb0

Browse files
TrottMyles Borins
authored and
Myles Borins
committed
test: only include http module once
A few tests in test/gc include the http module twice. Remove duplicate require(). PR-URL: #4606 Reviewed-By: Brian White <[email protected]> Reviewed-By: Michaël Zasso <[email protected]> Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: James M Snell <[email protected]>
1 parent 5990ba2 commit 0e02eb0

4 files changed

+0
-4
lines changed

test/gc/test-http-client-connaborted.js

-1
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,6 @@ let countGC = 0;
1818

1919
console.log('We should do ' + todo + ' requests');
2020

21-
var http = require('http');
2221
var server = http.createServer(serverHandler);
2322
server.listen(PORT, getall);
2423

test/gc/test-http-client-onerror.js

-1
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,6 @@ let countGC = 0;
2020

2121
console.log('We should do ' + todo + ' requests');
2222

23-
var http = require('http');
2423
var server = http.createServer(serverHandler);
2524
server.listen(PORT, runTest);
2625

test/gc/test-http-client-timeout.js

-1
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,6 @@ let countGC = 0;
2222

2323
console.log('We should do ' + todo + ' requests');
2424

25-
var http = require('http');
2625
var server = http.createServer(serverHandler);
2726
server.listen(PORT, getall);
2827

test/gc/test-http-client.js

-1
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,6 @@ let countGC = 0;
1818

1919
console.log('We should do ' + todo + ' requests');
2020

21-
var http = require('http');
2221
var server = http.createServer(serverHandler);
2322
server.listen(PORT, getall);
2423

0 commit comments

Comments
 (0)