test: move test-net-connect-handle-econnrefused to sequential (!27014) · Merge requests · Rodrigo Test / Test Group-nodejs / node · GitLab

Do not update/delete: Banner broadcast message test data

Do not update/delete: Notification broadcast message test data

Rodrigo Muino Tomonari requested to merge github/fork/lpinca/gh-26907 into master Mar 30, 2019

First commit:

test: move test-net-connect-handle-econnrefused
    
The port used in the test could be taken by another process before
the callback of `server.close()` is called. Move it to sequential.
    
Refs: https://github.com/nodejs/node/pull/18257#discussion_r162717096
Fixes: https://github.com/nodejs/node/issues/26907

Second commit:

test: refactor net-connect-handle-econnrefused
    
- Remove unneeded server
- Use `common.PORT`

Checklist

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