node/test/gc/test-http-client-onerror.js
Devin Nakamura 2413a4e99d test: fix race condition in test-http-client-onerror
Occasionally test-http-client-onerror will fail with a refused connection.
This patch fixes the possibility that connections will be attempted before
server is listening.

PR-URL: https://github.com/nodejs/node/pull/4346
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: Brian White <mscdex@mscdex.net>
Reviewed-By: Johan Bergström <bugs@bergstroem.nu>
Reviewed-By: James M Snell <jasnell@gmail.com>
2015-12-30 11:47:51 -08:00

81 lines
1.5 KiB
JavaScript

'use strict';
// just like test/gc/http-client.js,
// but with an on('error') handler that does nothing.
function serverHandler(req, res) {
req.resume();
res.writeHead(200, {'Content-Type': 'text/plain'});
res.end('Hello World\n');
}
var http = require('http'),
weak = require('weak'),
done = 0,
count = 0,
countGC = 0,
todo = 500,
common = require('../common'),
assert = require('assert'),
PORT = common.PORT;
console.log('We should do ' + todo + ' requests');
var http = require('http');
var server = http.createServer(serverHandler);
server.listen(PORT, runTest);
function getall() {
if (count >= todo)
return;
(function() {
function cb(res) {
res.resume();
done += 1;
statusLater();
}
function onerror(er) {
throw er;
}
var req = http.get({
hostname: 'localhost',
pathname: '/',
port: PORT
}, cb).on('error', onerror);
count++;
weak(req, afterGC);
})();
setImmediate(getall);
}
function runTest() {
for (var i = 0; i < 10; i++)
getall();
}
function afterGC() {
countGC ++;
}
var timer;
function statusLater() {
gc();
if (timer) clearTimeout(timer);
timer = setTimeout(status, 1);
}
function status() {
gc();
console.log('Done: %d/%d', done, todo);
console.log('Collected: %d/%d', countGC, count);
if (done === todo) {
console.log('All should be collected now.');
assert.strictEqual(count, countGC);
process.exit(0);
}
}