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, 6 deletions

diff --git a/test/parallel/test-dgram-error-message-address.js b/test/parallel/test-dgram-error-message-address.js
index 005695654f..d27b004332 100644
--- a/test/parallel/test-dgram-error-message-address.js
+++ b/test/parallel/test-dgram-error-message-address.js
@@ -9,14 +9,14 @@ var socket_ipv4 = dgram.createSocket('udp4');
 socket_ipv4.on('listening', common.fail);
 
 socket_ipv4.on('error', common.mustCall(function(e) {
-  assert.equal(e.message, 'bind EADDRNOTAVAIL 1.1.1.1:' + common.PORT);
+  assert.strictEqual(e.port, undefined);
+  assert.equal(e.message, 'bind EADDRNOTAVAIL 1.1.1.1');
   assert.equal(e.address, '1.1.1.1');
-  assert.equal(e.port, common.PORT);
   assert.equal(e.code, 'EADDRNOTAVAIL');
   socket_ipv4.close();
 }));
 
-socket_ipv4.bind(common.PORT, '1.1.1.1');
+socket_ipv4.bind(0, '1.1.1.1');
 
 // IPv6 Test
 var socket_ipv6 = dgram.createSocket('udp6');
@@ -27,10 +27,10 @@ socket_ipv6.on('error', common.mustCall(function(e) {
   // EAFNOSUPPORT or EPROTONOSUPPORT means IPv6 is disabled on this system.
   var allowed = ['EADDRNOTAVAIL', 'EAFNOSUPPORT', 'EPROTONOSUPPORT'];
   assert.notEqual(allowed.indexOf(e.code), -1);
-  assert.equal(e.message, 'bind ' + e.code + ' 111::1:' + common.PORT);
+  assert.strictEqual(e.port, undefined);
+  assert.equal(e.message, 'bind ' + e.code + ' 111::1');
   assert.equal(e.address, '111::1');
-  assert.equal(e.port, common.PORT);
   socket_ipv6.close();
 }));
 
-socket_ipv6.bind(common.PORT, '111::1');
+socket_ipv6.bind(0, '111::1');