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 jasnell committed Oct 29, 2015
1 parent 39ff44e commit 6b75f10
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 6b75f10

Please sign in to comment.