Skip to content

test: use dynamic port in test-dgram-bind-shared-ports

Rodrigo Muino Tomonari requested to merge github/fork/orafaelfragoso/test-dgram into master

Use of common.PORT in parallel tests is not completely safe (because the same port can be previously assigned to another test running in parallel if that test uses port 0 to get an arbitrary available port).

Remove common.PORT from test-dgram-bind-shared-ports.

Refs: https://github.com/nodejs/node/issues/12376

Checklist
  • make -j4 test (UNIX), or vcbuild test (Windows) passes
  • tests and/or benchmarks are included
  • commit message follows commit guidelines
Affected core subsystem(s)

test

Merge request reports