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

diff --git a/test/parallel/test-http-set-timeout.js b/test/parallel/test-http-set-timeout.js
index ca238de431..08777d30d2 100644
--- a/test/parallel/test-http-set-timeout.js
+++ b/test/parallel/test-http-set-timeout.js
@@ -15,14 +15,14 @@ var server = http.createServer(function(req, res) {
   });
 });
 
-server.listen(common.PORT, function() {
-  console.log('Server running at http://127.0.0.1:' + common.PORT + '/');
+server.listen(0, function() {
+  console.log(`Server running at http://127.0.0.1:${this.address().port}/`);
 
   var errorTimer = setTimeout(function() {
     throw new Error('Timeout was not successful');
   }, common.platformTimeout(2000));
 
-  var x = http.get({port: common.PORT, path: '/'});
+  var x = http.get({port: this.address().port, path: '/'});
   x.on('error', function() {
     clearTimeout(errorTimer);
     console.log('HTTP REQUEST COMPLETE (this is good)');