Skip to content

Commit

Permalink
test: dynamic port in cluster worker disconnect
Browse files Browse the repository at this point in the history
Remove common.PORT from test-cluster-worker-disconnect-on-error
possibility that a dynamic port used in another test will collide
with common.PORT.

PR-URL: #12457
Ref: #12376
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
Reviewed-By: Santiago Gimeno <santiago.gimeno@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Yuta Hiroto <hello@about-hiroppy.com>
  • Loading branch information
Sebastian Plesciuc authored and evanlucas committed May 2, 2017
1 parent ddc3528 commit a213320
Showing 1 changed file with 18 additions and 5 deletions.
23 changes: 18 additions & 5 deletions test/parallel/test-cluster-worker-disconnect-on-error.js
Original file line number Diff line number Diff line change
Expand Up @@ -2,19 +2,32 @@
const common = require('../common');
const http = require('http');
const cluster = require('cluster');
const assert = require('assert');

cluster.schedulingPolicy = cluster.SCHED_NONE;

const server = http.createServer();
if (cluster.isMaster) {
server.listen(common.PORT);
const worker = cluster.fork();
let worker;

server.listen(0, common.mustCall((error) => {
assert.ifError(error);
assert(worker);

worker.send({port: server.address().port});
}));

worker = cluster.fork();
worker.on('exit', common.mustCall(() => {
server.close();
}));
} else {
server.listen(common.PORT);
server.on('error', common.mustCall((e) => {
cluster.worker.disconnect();
process.on('message', common.mustCall((msg) => {
assert(msg.port);

server.listen(msg.port);
server.on('error', common.mustCall((e) => {
cluster.worker.disconnect();
}));
}));
}

0 comments on commit a213320

Please sign in to comment.