Skip to content

test: add test file name to common.PIPE

Rodrigo Muino Tomonari requested to merge github/fork/joyeecheung/pipe-name into master
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

There have been some flaky EADDRINUSE errors on the CI. The theory is, using pid to avoid conflicts might still not be enough. This PR adds the file name to the mix to reduce the conflict.

Refs: https://github.com/nodejs/node/issues/16290 Refs: https://github.com/nodejs/node/issues/16323

Merge request reports