Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Get correct npm prefix on all Windows unix shells
1. Set the shebang to /usr/bin/env bash instead of /bin/sh (which might be dash or some other shell) 2. Use Unix-style line endings, not Windows-style (Cygwin accepts either, but mingw bash sometimes objects, and WSL bash always does) 3. Test against paths using wslpath if available, but still pass win32 paths to node.exe, since it is a Windows binary that only knows how to handle Windows paths. The test is not quite as comprehensive as I'd like. It runs on the various Windows bash implementations if they are found in their expected locations, skipping any that are not installed. Short of shipping mingw, cygwin, and wsl as test fixtures, I'm not sure how we could do much better, however. At least, we can use this test to assist debug and catch issues on Windows machines (ours or users who report problems).
- Loading branch information
Showing
3 changed files
with
172 additions
and
40 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,119 @@ | ||
| const t = require('tap') | ||
|
|
||
| if (process.platform !== 'win32') { | ||
| t.plan(0, 'test only relevant on windows') | ||
| process.exit(0) | ||
| } | ||
|
|
||
| const has = path => { | ||
| try { | ||
| return statSync(path).isFile() | ||
| } catch (er) { | ||
| console.error(`does not have ${path}`, er) | ||
| return false | ||
| } | ||
| } | ||
|
|
||
| const { version } = require('../../package.json') | ||
| const spawn = require('@npmcli/promise-spawn') | ||
| const { resolve } = require('path') | ||
| const { ProgramFiles, SystemRoot } = process.env | ||
| const { readFileSync, statSync, chmodSync } = require('fs') | ||
| const gitBash = resolve(ProgramFiles, 'Git', 'bin', 'bash.exe') | ||
| const gitUsrBinBash = resolve(ProgramFiles, 'Git', 'usr', 'bin', 'bash.exe') | ||
| const wslBash = resolve(SystemRoot, 'System32', 'bash.exe') | ||
| const cygwinBash = resolve(SystemRoot, '/', 'cygwin64', 'bin', 'bash.exe') | ||
|
|
||
| const bashes = Object.entries({ | ||
| 'wsl bash': wslBash, | ||
| 'git bash': gitBash, | ||
| 'git internal bash': gitUsrBinBash, | ||
| 'cygwin bash': cygwinBash, | ||
| }) | ||
|
|
||
| const npmShim = resolve(__dirname, '../../bin/npm') | ||
| const npxShim = resolve(__dirname, '../../bin/npx') | ||
|
|
||
| const path = t.testdir({ | ||
| 'node.exe': readFileSync(process.execPath), | ||
| npm: readFileSync(npmShim), | ||
| npx: readFileSync(npxShim), | ||
| // simulate the state where one version of npm is installed | ||
| // with node, but we should load the globally installed one | ||
| 'global-prefix': { | ||
| node_modules: { | ||
| npm: t.fixture('symlink', resolve(__dirname, '../..')), | ||
| }, | ||
| }, | ||
| // put in a shim that ONLY prints the intended global prefix, | ||
| // and should not be used for anything else. | ||
| node_modules: { | ||
| npm: { | ||
| bin: { | ||
| 'npx-cli.js': ` | ||
| throw new Error('this should not be called') | ||
| `, | ||
| 'npm-cli.js': ` | ||
| const assert = require('assert') | ||
| const args = process.argv.slice(2) | ||
| assert.equal(args[0], 'prefix') | ||
| assert.equal(args[1], '-g') | ||
| const { resolve } = require('path') | ||
| console.log(resolve(__dirname, '../../../global-prefix')) | ||
| `, | ||
| }, | ||
| }, | ||
| }, | ||
| }) | ||
| chmodSync(resolve(path, 'npm'), 0o755) | ||
| chmodSync(resolve(path, 'npx'), 0o755) | ||
|
|
||
| for (const [name, bash] of bashes) { | ||
| if (!has(bash)) { | ||
| t.skip(`${name} not installed`, { bin: bash, diagnostic: true }) | ||
| continue | ||
| } | ||
|
|
||
| t.test(name, async t => { | ||
| t.plan(2) | ||
| t.test('npm', async t => { | ||
| // only cygwin *requires* the -l, but the others are ok with it | ||
| // don't hit the registry for the update check | ||
| const args = ['-l', 'npm', 'help'] | ||
|
|
||
| const result = await spawn(bash, args, { | ||
| env: { PATH: path, npm_config_update_notifier: 'false' }, | ||
| cwd: path, | ||
| stdioString: true, | ||
| }) | ||
| t.match(result, { | ||
| cmd: bash, | ||
| args: ['-l', 'npm', 'help'], | ||
| code: 0, | ||
| signal: null, | ||
| stderr: String, | ||
| // should have loaded this instance of npm we symlinked in | ||
| stdout: `npm@${version} ${resolve(__dirname, '../..')}`, | ||
| }) | ||
| }) | ||
|
|
||
| t.test('npx', async t => { | ||
| const args = ['-l', 'npx', '--version'] | ||
|
|
||
| const result = await spawn(bash, args, { | ||
| env: { PATH: path, npm_config_update_notifier: 'false' }, | ||
| cwd: path, | ||
| stdioString: true, | ||
| }) | ||
| t.match(result, { | ||
| cmd: bash, | ||
| args: ['-l', 'npx', '--version'], | ||
| code: 0, | ||
| signal: null, | ||
| stderr: String, | ||
| // should have loaded this instance of npm we symlinked in | ||
| stdout: version, | ||
| }) | ||
| }) | ||
| }) | ||
| } |