af55641a69
Currently there are many instances where assert.fail is directly passed to a callback for error handling. Unfortunately this will swallow the error as it is the third argument of assert.fail that sets the message not the first. This commit adds a new function to test/common.js that simply wraps assert.fail and calls it with the provided message. Tip of the hat to @trott for pointing me in the direction of this. PR-URL: https://github.com/nodejs/node/pull/3453 Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Rich Trott <rtrott@gmail.com>
43 lines
1.2 KiB
JavaScript
43 lines
1.2 KiB
JavaScript
'use strict';
|
|
// Check that having a worker bind to a port that's already taken doesn't
|
|
// leave the master process in a confused state. Releasing the port and
|
|
// trying again should Just Work[TM].
|
|
|
|
var common = require('../common');
|
|
var assert = require('assert');
|
|
var cluster = require('cluster');
|
|
var fork = require('child_process').fork;
|
|
var net = require('net');
|
|
|
|
var id = '' + process.argv[2];
|
|
|
|
if (id === 'undefined') {
|
|
var server = net.createServer(common.fail);
|
|
server.listen(common.PORT, function() {
|
|
var worker = fork(__filename, ['worker']);
|
|
worker.on('message', function(msg) {
|
|
if (msg !== 'stop-listening') return;
|
|
server.close(function() {
|
|
worker.send('stopped-listening');
|
|
});
|
|
});
|
|
});
|
|
}
|
|
else if (id === 'worker') {
|
|
var server = net.createServer(common.fail);
|
|
server.listen(common.PORT, common.fail);
|
|
server.on('error', common.mustCall(function(e) {
|
|
assert(e.code, 'EADDRINUSE');
|
|
process.send('stop-listening');
|
|
process.once('message', function(msg) {
|
|
if (msg !== 'stopped-listening') return;
|
|
server = net.createServer(common.fail);
|
|
server.listen(common.PORT, common.mustCall(function() {
|
|
server.close();
|
|
}));
|
|
});
|
|
}));
|
|
}
|
|
else {
|
|
assert(0); // Bad argument.
|
|
}
|