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-https-client-reject.js b/test/parallel/test-https-client-reject.js
index ee19de2acd..1bf42e5df5 100644
--- a/test/parallel/test-https-client-reject.js
+++ b/test/parallel/test-https-client-reject.js
@@ -23,13 +23,13 @@ var server = https.createServer(options, function(req, res) {
   res.writeHead(200);
   res.end();
   req.resume();
-}).listen(common.PORT, function() {
+}).listen(0, function() {
   unauthorized();
 });
 
 function unauthorized() {
   var req = https.request({
-    port: common.PORT,
+    port: server.address().port,
     rejectUnauthorized: false
   }, function(res) {
     assert(!req.socket.authorized);
@@ -44,7 +44,7 @@ function unauthorized() {
 
 function rejectUnauthorized() {
   var options = {
-    port: common.PORT
+    port: server.address().port
   };
   options.agent = new https.Agent(options);
   var req = https.request(options, function(res) {
@@ -58,7 +58,7 @@ function rejectUnauthorized() {
 
 function authorized() {
   var options = {
-    port: common.PORT,
+    port: server.address().port,
     ca: [fs.readFileSync(path.join(common.fixturesDir, 'test_cert.pem'))]
   };
   options.agent = new https.Agent(options);