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.

Backport-PR-URL: #19374
PR-URL: #19177
Refs: #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 authored and targos committed Apr 4, 2018
1 parent ebdcf91 commit 83ebaf0
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 83ebaf0

Please sign in to comment.