Skip to content

test: improve test-npm-install

Rodrigo Muino Tomonari requested to merge github/fork/santigimeno/npm_install into master

Pull Request check-list

Please make sure to review and check all of these items:

  • Does make -j8 test (UNIX) or vcbuild test nosign (Windows) pass with this change (including linting)?
  • Is the commit message formatted according to CONTRIBUTING.md?
  • If this change fixes a bug (or a performance problem), is a regression test (or a benchmark) included?
  • Is a documentation update included (if this change modifies existing APIs, or introduces new ones)?

    Affected core subsystem(s)

test

Description of change

Make npm install a dependency that is defined as a relative path, so it avoids any network interaction.

Ref: https://github.com/nodejs/node/pull/5166#issuecomment-192602004

Merge request reports