test: use dynamic port in test-cluster-bind-twice

Remove common.PORT from test-cluster-bind-twice to eliminate possibility that a dynamic port used in another test will collide with common.PORT. PR-URL: #12418 Ref: #12376 Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Sakthipriyan Vairamani <[email protected]> Reviewed-By: Benjamin Gruenbaum <[email protected]> Reviewed-By: Gibson Fahnestock <[email protected]> Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: James M Snell <[email protected]>

Author: Trott

test/parallel/test-cluster-bind-twice.js

@@ -46,9 +46,8 @@ if (!id){
 
 
 a.on('message',common.mustCall((m)=>{
-if(typeofm==='object')return;
-assert.strictEqual(m,'READY');
-b.send('START');
+assert.strictEqual(m.msg,'READY');
+b.send({msg: 'START',port: m.port});
 }));
 
 b.on('message',common.mustCall((m)=>{
@@ -60,10 +59,10 @@ if (!id){
 }elseif(id==='one'){
 if(cluster.isMaster)returnstartWorker();
 
-http.createServer(common.mustNotCall())
-.listen(common.PORT,common.mustCall(()=>{
-process.send('READY');
-}));
+constserver=http.createServer(common.mustNotCall());
+server.listen(0,common.mustCall(()=>{
+process.send({msg: 'READY',port: server.address().port});
+}));
 
 process.on('message',common.mustCall((m)=>{
 if(m==='QUIT')process.exit();
@@ -74,8 +73,8 @@ if (!id){
 constserver=http.createServer(common.mustNotCall());
 process.on('message',common.mustCall((m)=>{
 if(m==='QUIT')process.exit();
-assert.strictEqual(m,'START');
-server.listen(common.PORT,common.mustNotCall());
+assert.strictEqual(m.msg,'START');
+server.listen(m.port,common.mustNotCall());
 server.on('error',common.mustCall((e)=>{
 assert.strictEqual(e.code,'EADDRINUSE');
 process.send(e.code);