diff options
author | nlf <quitlahok@gmail.com> | 2022-08-01 21:02:18 +0300 |
---|---|---|
committer | GitHub <noreply@github.com> | 2022-08-01 21:02:18 +0300 |
commit | eac1bf26f28e9e8fc07e1350c3d7b14a94005ebf (patch) | |
tree | 6bffad5b665a5830d2b3a1615972aab247c1fdf7 | |
parent | 58cc362ceec82ddeb16311d3fb574908294ac789 (diff) |
fix(ls): when filtering workspaces, make sure the edge has a to before checking if its a workspace (#5164)
-rw-r--r-- | lib/commands/ls.js | 2 | ||||
-rw-r--r-- | tap-snapshots/test/lib/commands/ls.js.test.cjs | 7 | ||||
-rw-r--r-- | test/lib/commands/ls.js | 38 |
3 files changed, 46 insertions, 1 deletions
diff --git a/lib/commands/ls.js b/lib/commands/ls.js index d3932072b..073ca0c69 100644 --- a/lib/commands/ls.js +++ b/lib/commands/ls.js @@ -92,7 +92,7 @@ class LS extends ArboristWorkspaceCmd { } if (this.npm.flatOptions.includeWorkspaceRoot - && !edge.to.isWorkspace) { + && edge.to && !edge.to.isWorkspace) { return true } diff --git a/tap-snapshots/test/lib/commands/ls.js.test.cjs b/tap-snapshots/test/lib/commands/ls.js.test.cjs index a10d37ab9..9b6749acf 100644 --- a/tap-snapshots/test/lib/commands/ls.js.test.cjs +++ b/tap-snapshots/test/lib/commands/ls.js.test.cjs @@ -678,6 +678,13 @@ dedupe-entries@1.0.0 {CWD}/tap-testdir-ls-ls-with-no-args-dedupe-entries-and-not ` +exports[`test/lib/commands/ls.js TAP ls workspace and missing optional dep > should omit missing optional dep 1`] = ` +root@ {CWD}/tap-testdir-ls-ls-workspace-and-missing-optional-dep ++-- baz@1.0.0 -> ./baz +\`-- foo@1.0.0 + +` + exports[`test/lib/commands/ls.js TAP show multiple invalid reasons > ls result 1`] = ` test-npm-ls@1.0.0 {cwd}/tap-testdir-ls-show-multiple-invalid-reasons +-- cat@1.0.0 invalid: "^2.0.0" from the root project diff --git a/test/lib/commands/ls.js b/test/lib/commands/ls.js index 412d5ce65..f4cd4ef33 100644 --- a/test/lib/commands/ls.js +++ b/test/lib/commands/ls.js @@ -178,6 +178,44 @@ t.test('ls', t => { ) }) + t.test('workspace and missing optional dep', async t => { + npm.prefix = npm.localPrefix = t.testdir({ + 'package.json': JSON.stringify({ + name: 'root', + dependencies: { + foo: '^1.0.0', + }, + optionalDependencies: { + bar: '^1.0.0', + }, + workspaces: ['./baz'], + }), + baz: { + 'package.json': JSON.stringify({ + name: 'baz', + version: '1.0.0', + }), + }, + node_modules: { + baz: t.fixture('symlink', '../baz'), + foo: { + 'package.json': JSON.stringify({ + name: 'foo', + version: '1.0.0', + }), + }, + }, + }) + + npm.flatOptions.includeWorkspaceRoot = true + t.teardown(() => { + delete npm.flatOptions.includeWorkspaceRoot + }) + + await ls.execWorkspaces([], ['baz']) + t.matchSnapshot(redactCwd(result), 'should omit missing optional dep') + }) + t.test('extraneous deps', async t => { npm.prefix = t.testdir({ 'package.json': JSON.stringify({ |