Skip to content

Commit

Permalink
test: use dynamic port in 3 test-cluster-worker tests
Browse files Browse the repository at this point in the history
Remove common.PORT from test-cluster-worker-disconnect,
test-cluster-worker-exit and test-cluster-worker-kill to
eliminate the possibility that a dynamic port used in
another test will collide with common.PORT.

PR-URL: #12443
Ref: #12376
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
Reviewed-By: Gibson Fahnestock <gibfahn@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>
  • Loading branch information
Sebastian Plesciuc authored and jasnell committed Apr 18, 2017
1 parent def78e8 commit 2519757
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 3 deletions.
2 changes: 1 addition & 1 deletion test/parallel/test-cluster-worker-disconnect.js
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,7 @@ if (cluster.isWorker) {
const http = require('http');
http.Server(() => {

}).listen(common.PORT, '127.0.0.1');
}).listen(0, '127.0.0.1');
const worker = cluster.worker;
assert.strictEqual(worker.exitedAfterDisconnect, worker.suicide);

Expand Down
2 changes: 1 addition & 1 deletion test/parallel/test-cluster-worker-exit.js
Original file line number Diff line number Diff line change
Expand Up @@ -40,7 +40,7 @@ if (cluster.isWorker) {
server.once('listening', common.mustCall(() => {
process.exit(EXIT_CODE);
}));
server.listen(common.PORT, '127.0.0.1');
server.listen(0, '127.0.0.1');

} else if (cluster.isMaster) {

Expand Down
2 changes: 1 addition & 1 deletion test/parallel/test-cluster-worker-kill.js
Original file line number Diff line number Diff line change
Expand Up @@ -36,7 +36,7 @@ if (cluster.isWorker) {
const server = http.Server(() => { });

server.once('listening', common.mustCall(() => { }));
server.listen(common.PORT, '127.0.0.1');
server.listen(0, '127.0.0.1');

} else if (cluster.isMaster) {

Expand Down

0 comments on commit 2519757

Please sign in to comment.