test: dynamic port in cluster eaddrinuse

Removed common.PORT from test-cluster-eaddrinuse to eliminate the possibility that a dynamic port used in another test will collide with common.PORT. PR-URL: #12547 Ref: #12376 Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: James M Snell <[email protected]> Reviewed-By: Santiago Gimeno <[email protected]> Reviewed-By: Richard Lau <[email protected]> Reviewed-By: Luigi Pinca <[email protected]>
nodejs · May 2, 2017 · b82e076 · b82e076
1 parent 8ae5afe
commit b82e076
Showing 1 changed file with 5 additions and 4 deletions.
diff --git a/test/parallel/test-cluster-eaddrinuse.js b/test/parallel/test-cluster-eaddrinuse.js
@@ -9,11 +9,12 @@ const fork = require('child_process').fork;
 const net = require('net');
 
 const id = '' + process.argv[2];
+const port = '' + process.argv[3];
 
 if (id === 'undefined') {
   const server = net.createServer(common.mustNotCall());
-  server.listen(common.PORT, function() {
-    const worker = fork(__filename, ['worker']);
+  server.listen(0, function() {
+    const worker = fork(__filename, ['worker', server.address().port]);
     worker.on('message', function(msg) {
       if (msg !== 'stop-listening') return;
       server.close(function() {
@@ -23,14 +24,14 @@ if (id === 'undefined') {
   });
 } else if (id === 'worker') {
   let server = net.createServer(common.mustNotCall());
-  server.listen(common.PORT, common.mustNotCall());
+  server.listen(port, common.mustNotCall());
   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.mustNotCall());
-      server.listen(common.PORT, common.mustCall(function() {
+      server.listen(port, common.mustCall(function() {
         server.close();
       }));
     });