Skip to content

test: remove modifcation to common.PORT

Rodrigo Muino Tomonari requested to merge github/fork/Trott/no-plus into master
Checklist
  • tests and code linting passes
  • the commit message follows commit guidelines
Affected core subsystem(s)

test debugger

Description of change

A possibly-buggy fixture server uses common.PORT+1000 for its port rather than common.PORT. That could result in it clashing with other ports if tests are run in parallel. The test runner increments common.PORT by 100 for each running instance for tests. Change to use common.PORT and have the tests that use the fixture start with common.PORT+1 for anything they need.

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

Merge request reports