Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

tools: remove NODE_PATH from environment for tests #11612

Closed
wants to merge 1 commit into from

Commits on Feb 28, 2017

  1. tools: remove NODE_PATH from environment for tests

    Unset NODE_PATH environment variable when running tests.
    
    On Ubuntu 16.04, some users experience test failures due to internal
    libraries being installed in /usr/lib/nodejs/internal and NODE_PATH
    including /usr/lib/nodejs. Tests that expect internal libraries to be
    off limits without the --expose-internals flag will fail in this
    situation. Currently, those tests are test/parallel/test-repl.js and
    test/parallel/test-internal-modules.js.
    
    This situation seems to (probably) be caused by some
    not-entirely-uncommon package that gets installed.
    
    Regardless, tests should ignore the user's NODE_PATH. (NODE_PATH is
    tested in test/parallel/test-module-globalpaths-nodepath.js and
    test/parallel/test-require-dot.js.)
    
    Refs: https://twitter.com/trott/status/835729396900061184
    Trott committed Feb 28, 2017
    Configuration menu
    Copy the full SHA
    c99ba2d View commit details
    Browse the repository at this point in the history