Skip to content

Commit

Permalink
test: use port number from env in tls socket test
Browse files Browse the repository at this point in the history
Tests normally use common.PORT to allow the user to select which port
number to listen on. Hardcoding the port number will cause parallel
instances of the test to fail.

PR-URL: #3557
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: Fedor Indutny <fedor@indutny.com>
Reviewed-By: Johan Bergström <bugs@bergstroem.nu>
  • Loading branch information
stefanmb authored and rvagg committed Oct 29, 2015
1 parent 5e76587 commit 8c6c0f9
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions test/parallel/test-tls-async-cb-after-socket-end.js
Original file line number Diff line number Diff line change
Expand Up @@ -36,9 +36,9 @@ server.on('resumeSession', function(id, cb) {
next();
});

server.listen(1443, function() {
server.listen(common.PORT, function() {
var clientOpts = {
port: 1443,
port: common.PORT,
rejectUnauthorized: false,
session: false
};
Expand Down

0 comments on commit 8c6c0f9

Please sign in to comment.