Skip to content

Commit

Permalink
test: remove NODE_DEBUG in global module loading test
Browse files Browse the repository at this point in the history
Otherwise the debug log output might be mixed up with
the expected errors and the assertion matching the error
message would fail.

PR-URL: nodejs#19177
Refs: nodejs#19112
Reviewed-By: Gus Caplan <me@gus.host>
Reviewed-By: Matteo Collina <matteo.collina@gmail.com>
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
  • Loading branch information
joyeecheung committed Mar 30, 2018
1 parent ae516be commit d65da3b
Showing 1 changed file with 0 additions and 2 deletions.
2 changes: 0 additions & 2 deletions test/parallel/test-module-loading-globalpaths.js
Original file line number Diff line number Diff line change
Expand Up @@ -41,8 +41,6 @@ if (process.argv[2] === 'child') {
const testFixturesDir = fixtures.path(path.basename(__filename, '.js'));

const env = Object.assign({}, process.env);
// Turn on module debug to aid diagnosing failures.
env.NODE_DEBUG = 'module';
// Unset NODE_PATH.
delete env.NODE_PATH;

Expand Down

0 comments on commit d65da3b

Please sign in to comment.