Skip to content

tests: fix test-require-deps-deprecation for already installed deps

Rodrigo Muino Tomonari requested to merge github/fork/Tiriel/test-missing-error into master

Merry christmas!

Test test-require-deps-deprecation.js was failing when user already had node installed with acorn in require.resolve range. Modified test to acknowledge the possibility and throw only if acorn is found in the deps directory.

Fixes: https://github.com/nodejs/node/issues/17148 Refs: https://github.com/nodejs/node/issues/17148#issuecomment-346167296

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