test: remove uses of common.PORT in test-tls-client tests

Change common.PORT to '0' to avoid the possibility of getting EADDRINUSE error if another test in 'parallel' uses port '0' at the same time. PR-URL: #12461 Ref: #12376 Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: James M Snell <[email protected]> Reviewed-By: Rich Trott <[email protected]> Reviewed-By: Santiago Gimeno <[email protected]>

Author: z0al

test/parallel/test-tls-client-abort.js

@@ -14,7 +14,7 @@ const path = require('path');
 constcert=fs.readFileSync(path.join(common.fixturesDir,'test_cert.pem'));
 constkey=fs.readFileSync(path.join(common.fixturesDir,'test_key.pem'));
 
-constconn=tls.connect({cert, key,port: common.PORT},common.mustNotCall());
+constconn=tls.connect({cert, key,port: 0},common.mustNotCall());
 conn.on('error',function(){
 });
 assert.doesNotThrow(function(){

test/parallel/test-tls-client-abort2.js

@@ -8,7 +8,7 @@ if (!common.hasCrypto){
 }
 consttls=require('tls');
 
-constconn=tls.connect(common.PORT,common.mustNotCall());
+constconn=tls.connect(0,common.mustNotCall());
 conn.on('error',common.mustCall(function(){
 assert.doesNotThrow(function(){
 conn.destroy();