Skip to content

Commit

Permalink
test: improve test-gc-http-client-connaborted
Browse files Browse the repository at this point in the history
* refactor out usage of 'function' for scoping
* wait till server is up to start firing requests

PR-URL: #23193
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Sakthipriyan Vairamani <thechargingvolcano@gmail.com>
  • Loading branch information
lundibundi authored and rvagg committed Nov 28, 2018
1 parent bf06d05 commit c40bf97
Showing 1 changed file with 19 additions and 21 deletions.
40 changes: 19 additions & 21 deletions test/sequential/test-gc-http-client-connaborted.js
Original file line number Diff line number Diff line change
Expand Up @@ -3,13 +3,9 @@
// just like test-gc-http-client.js,
// but aborting every connection that comes in.

require('../common');
const common = require('../common');
const onGC = require('../common/ongc');

function serverHandler(req, res) {
res.connection.destroy();
}

const http = require('http');
const todo = 500;
let done = 0;
Expand All @@ -18,33 +14,35 @@ let countGC = 0;

console.log(`We should do ${todo} requests`);

function serverHandler(req, res) {
res.connection.destroy();
}

const server = http.createServer(serverHandler);
server.listen(0, getall);
server.listen(0, common.mustCall(() => {
for (let i = 0; i < 10; i++)
getall();
}));

function getall() {
if (count >= todo)
return;

(function() {
function cb(res) {
done += 1;
}
const req = http.get({
hostname: 'localhost',
pathname: '/',
port: server.address().port
}, cb).on('error', cb);

const req = http.get({
hostname: 'localhost',
pathname: '/',
port: server.address().port
}, cb).on('error', cb);

count++;
onGC(req, { ongc });
})();
count++;
onGC(req, { ongc });

setImmediate(getall);
}

for (let i = 0; i < 10; i++)
getall();
function cb(res) {
done += 1;
}

function ongc() {
countGC++;
Expand Down

0 comments on commit c40bf97

Please sign in to comment.