Browse Source

test: ensure test-npm-install uses correct node

Currently it is possible that the shelled out instance of npm will use
the system copy of node. This PR changes the test to shim the build
directory into the path. This will ensure that npm will use the correct
version of node.

fixes: https://github.com/nodejs/node/issues/6648

PR-URL: https://github.com/nodejs/node/pull/6658
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Jeremiah Senkpiel <fishrock123@rocketmail.com>
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
v4.x
Myles Borins 9 years ago
parent
commit
cc4c5187ed
  1. 5
      test/parallel/test-npm-install.js

5
test/parallel/test-npm-install.js

@ -33,7 +33,10 @@ const pkgPath = path.join(common.tmpDir, 'package.json');
fs.writeFileSync(pkgPath, pkgContent); fs.writeFileSync(pkgPath, pkgContent);
const proc = spawn(process.execPath, args, { const proc = spawn(process.execPath, args, {
cwd: common.tmpDir cwd: common.tmpDir,
env: {
PATH: path.dirname(process.execPath)
}
}); });
function handleExit(code, signalCode) { function handleExit(code, signalCode) {

Loading…
Cancel
Save