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

diff --git a/test/parallel/test-http-head-request.js b/test/parallel/test-http-head-request.js
index faf2168e36..68aaace7c3 100644
--- a/test/parallel/test-http-head-request.js
+++ b/test/parallel/test-http-head-request.js
@@ -1,15 +1,11 @@
 'use strict';
-var common = require('../common');
+require('../common');
 var assert = require('assert');
 var http = require('http');
 
-
 var body = 'hello world\n';
-var id = 0;
 
 function test(headers) {
-  var port = common.PORT + id++;
-
   var server = http.createServer(function(req, res) {
     console.error('req: %s headers: %j', req.method, headers);
     res.writeHead(200, headers);
@@ -19,9 +15,9 @@ function test(headers) {
 
   var gotEnd = false;
 
-  server.listen(port, function() {
+  server.listen(0, function() {
     var request = http.request({
-      port: port,
+      port: this.address().port,
       method: 'HEAD',
       path: '/'
     }, function(response) {