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

diff --git a/test/parallel/test-tls-client-reject.js b/test/parallel/test-tls-client-reject.js
index db53eb4b12..ddfcafd208 100644
--- a/test/parallel/test-tls-client-reject.js
+++ b/test/parallel/test-tls-client-reject.js
@@ -24,13 +24,13 @@ var server = tls.createServer(options, function(socket) {
     console.error(data.toString());
     assert.equal(data, 'ok');
   });
-}).listen(common.PORT, function() {
+}).listen(0, function() {
   unauthorized();
 });
 
 function unauthorized() {
   var socket = tls.connect({
-    port: common.PORT,
+    port: server.address().port,
     servername: 'localhost',
     rejectUnauthorized: false
   }, function() {
@@ -45,7 +45,7 @@ function unauthorized() {
 }
 
 function rejectUnauthorized() {
-  var socket = tls.connect(common.PORT, {
+  var socket = tls.connect(server.address().port, {
     servername: 'localhost'
   }, function() {
     assert(false);
@@ -58,7 +58,7 @@ function rejectUnauthorized() {
 }
 
 function authorized() {
-  var socket = tls.connect(common.PORT, {
+  var socket = tls.connect(server.address().port, {
     ca: [fs.readFileSync(path.join(common.fixturesDir, 'test_cert.pem'))],
     servername: 'localhost'
   }, function() {