Commit 0c62bd1
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: #6648
PR-URL: #6658
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Jeremiah Senkpiel <fishrock123@rocketmail.com>
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>1 parent bddf413 commit 0c62bd1
1 file changed
Lines changed: 4 additions & 1 deletion
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -33,7 +33,10 @@ const pkgPath = path.join(common.tmpDir, 'package.json'); | |||
| 33 | 33 | fs.writeFileSync(pkgPath, pkgContent); | |
| 34 | 34 | ||
| 35 | 35 | const proc = spawn(process.execPath, args, { | |
| 36 | - cwd: common.tmpDir | ||
| 36 | + cwd: common.tmpDir, | ||
| 37 | + env: { | ||
| 38 | + PATH: path.dirname(process.execPath) | ||
| 39 | + } | ||
| 37 | 40 | }); | |
| 38 | 41 | ||
| 39 | 42 | function handleExit(code, signalCode) { | |
0 commit comments