Skip to content

Commit

Permalink
test: only include http module once
Browse files Browse the repository at this point in the history
A few tests in test/gc include the http module twice. Remove duplicate
require().

PR-URL: #4606
Reviewed-By: Brian White <mscdex@mscdex.net>
Reviewed-By: Michaël Zasso <mic.besace@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
  • Loading branch information
Trott authored and Myles Borins committed Jan 12, 2016
1 parent 102fb7d commit f28a640
Show file tree
Hide file tree
Showing 4 changed files with 0 additions and 4 deletions.
1 change: 0 additions & 1 deletion test/gc/test-http-client-connaborted.js
Expand Up @@ -18,7 +18,6 @@ var http = require('http'),

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

var http = require('http');
var server = http.createServer(serverHandler);
server.listen(PORT, getall);

Expand Down
1 change: 0 additions & 1 deletion test/gc/test-http-client-onerror.js
Expand Up @@ -20,7 +20,6 @@ var http = require('http'),

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

var http = require('http');
var server = http.createServer(serverHandler);
server.listen(PORT, runTest);

Expand Down
1 change: 0 additions & 1 deletion test/gc/test-http-client-timeout.js
Expand Up @@ -22,7 +22,6 @@ var http = require('http'),

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

var http = require('http');
var server = http.createServer(serverHandler);
server.listen(PORT, getall);

Expand Down
1 change: 0 additions & 1 deletion test/gc/test-http-client.js
Expand Up @@ -18,7 +18,6 @@ var http = require('http'),

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

var http = require('http');
var server = http.createServer(serverHandler);
server.listen(PORT, getall);

Expand Down

0 comments on commit f28a640

Please sign in to comment.