test: use random ports where possible

This helps to prevent issues where a failed test can keep a bound
socket open long enough to cause other tests to fail with EADDRINUSE
because the same port number is used.

PR-URL: https://github.com/nodejs/node/pull/7045
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: Matteo Collina <matteo.collina@gmail.com>
Reviewed-By: Rod Vagg <rod@vagg.org>

1 files changed, 6 insertions, 5 deletions

diff --git a/test/parallel/test-dgram-udp6-send-default-host.js b/test/parallel/test-dgram-udp6-send-default-host.js
index 9ef080dc6e..e893f248ec 100644
--- a/test/parallel/test-dgram-udp6-send-default-host.js
+++ b/test/parallel/test-dgram-udp6-send-default-host.js
@@ -19,10 +19,11 @@ const toSend = [Buffer.alloc(256, 'x'),
 const received = [];
 
 client.on('listening', common.mustCall(() => {
-  client.send(toSend[0], 0, toSend[0].length, common.PORT);
-  client.send(toSend[1], common.PORT);
-  client.send([toSend[2]], common.PORT);
-  client.send(toSend[3], 0, toSend[3].length, common.PORT);
+  const port = client.address().port;
+  client.send(toSend[0], 0, toSend[0].length, port);
+  client.send(toSend[1], port);
+  client.send([toSend[2]], port);
+  client.send(toSend[3], 0, toSend[3].length, port);
 }));
 
 client.on('message', common.mustCall((buf, info) => {
@@ -38,4 +39,4 @@ client.on('message', common.mustCall((buf, info) => {
   }
 }, toSend.length));
 
-client.bind(common.PORT);
+client.bind(0);