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, 2 insertions, 2 deletions

diff --git a/test/parallel/test-net-bytes-read.js b/test/parallel/test-net-bytes-read.js
index ba2bc160d0..52fb577b9e 100644
--- a/test/parallel/test-net-bytes-read.js
+++ b/test/parallel/test-net-bytes-read.js
@@ -9,7 +9,7 @@ const big = Buffer.alloc(1024 * 1024);
 const server = net.createServer((socket) => {
   socket.end(big);
   server.close();
-}).listen(common.PORT, () => {
+}).listen(0, () => {
   let prev = 0;
 
   function checkRaise(value) {
@@ -17,7 +17,7 @@ const server = net.createServer((socket) => {
     prev = value;
   }
 
-  const socket = net.connect(common.PORT, () => {
+  const socket = net.connect(server.address().port, () => {
     socket.on('data', (chunk) => {
       checkRaise(socket.bytesRead);
     });