test: use dynamic port in test-tls-connect.js

[oantoro]

Usage of common.PORT in parallel tests is not completely safe

Change common.PORT to dynamic port to prevent collision
with other parallel test which use common.PORT

Refs: #12376

Checklist

• make -j4 test (UNIX), or vcbuild test (Windows) passes
• commit message follows commit guidelines

Affected core subsystem(s)

test

[Trott]

Hi, @okyantoro! Welcome.

Listening on port 0 assigns an arbitrary port. Connecting to port 0 does not make any sense and results in an error. The test with your changes fail because the test cases expect connection failure, but now they are failing for the wrong reason. (I'll update the tests so no one else falls into this trap.)

I'm going to close this out as this is not the path to solving this problem. Feel free to re-open or open a separate pull request if you want to try a different approach.

Thanks!