fix(outdated): add dependent location in long output

Author: Milan Meva

lib/commands/outdated.js

@@ -195,6 +195,9 @@ class Outdated extends ArboristWorkspaceCmd {
           wanted: wanted.version,
           latest: latest.version,
           workspaceDependent: edge.from?.isWorkspace ? edge.from.pkgid : null,
+          dependentLocation: edge.from?.name
+            ? edge.from?.location
+            : 'global',
           dependent: edge.from?.name ?? 'global',
           homepage: packument.homepage,
         })
@@ -226,7 +229,7 @@ class Outdated extends ArboristWorkspaceCmd {
         'Latest',
         'Location',
         'Depended by',
-        ...long ? ['Package Type', 'Homepage'] : [],
+        ...long ? ['Package Type', 'Homepage', 'Dependent Location'] : [],
       ].map(h => bold.underline(h)),
       ...list.map((d) => [
         d.current === d.wanted ? yellow(d.name) : red(d.name),
@@ -235,7 +238,7 @@ class Outdated extends ArboristWorkspaceCmd {
         blue(d.latest),
         d.location ?? '-',
         d.workspaceDependent ? blue(d.workspaceDependent) : d.dependent,
-        ...long ? [d.type, blue(d.homepage ?? '')] : [],
+        ...long ? [d.type, blue(d.homepage ?? ''), d.dependentLocation] : [],
       ]),
     ], {
       align: ['l', 'r', 'r', 'r', 'l'],
@@ -252,7 +255,7 @@ class Outdated extends ArboristWorkspaceCmd {
       d.current ? `${d.name}@${d.current}` : 'MISSING',
       `${d.name}@${d.latest}`,
       d.dependent,
-      ...this.npm.config.get('long') ? [d.type, d.homepage] : [],
+      ...this.npm.config.get('long') ? [d.type, d.homepage, d.dependentLocation] : [],
     ].join(':')).join('\n')
   }
 
@@ -268,7 +271,10 @@ class Outdated extends ArboristWorkspaceCmd {
         latest: d.latest,
         dependent: d.dependent,
         location: d.path,
-        ...this.npm.config.get('long') ? { type: d.type, homepage: d.homepage } : {},
+        ...this.npm.config.get('long') ? {
+          type: d.type,
+          homepage: d.homepage,
+          dependentLocation: d.dependentLocation } : {},
       }
       acc[d.name] = acc[d.name]
         // If this item alread has an outdated dep then we turn it into an array

tap-snapshots/test/lib/commands/outdated.js.test.cjs

@@ -15,6 +15,37 @@ Package         Current  Wanted  Latest  Location          Depended by
 cat:dog@^1.0.0    1.0.0   1.0.1   2.0.0  node_modules/cat  prefix
 `
 
+exports[`test/lib/commands/outdated.js TAP dependent location --long --json > should display dependent location when using --long and --json 1`] = `
+{
+  "dog": [
+    {
+      "current": "1.0.0",
+      "wanted": "1.0.1",
+      "latest": "2.0.0",
+      "dependent": "a",
+      "location": "{CWD}/prefix/node_modules/dog",
+      "type": "dependencies",
+      "dependentLocation": "a"
+    },
+    {
+      "current": "1.0.0",
+      "wanted": "1.0.1",
+      "latest": "2.0.0",
+      "dependent": "a",
+      "location": "{CWD}/prefix/node_modules/dog",
+      "type": "dependencies",
+      "dependentLocation": "nest/a"
+    }
+  ]
+}
+`
+
+exports[`test/lib/commands/outdated.js TAP dependent location --long > should display dependent location when using --long 1`] = `
+Package  Current  Wanted  Latest  Location          Depended by         Package Type  Homepage  Dependent Location
+dog        1.0.0   1.0.1   2.0.0  node_modules/dog  [email protected]             dependencies            a
+dog        1.0.0   1.0.1   2.0.0  node_modules/dog  a@npm:[email protected]  dependencies            nest/a
+`
+
 exports[`test/lib/commands/outdated.js TAP should display outdated deps outdated --all > must match snapshot 1`] = `
 Package  Current  Wanted  Latest  Location           Depended by
 cat        1.0.0   1.0.1   1.0.1  node_modules/cat   prefix
@@ -31,29 +62,33 @@ exports[`test/lib/commands/outdated.js TAP should display outdated deps outdated
     "latest": "1.0.1",
     "dependent": "prefix",
     "location": "{CWD}/prefix/node_modules/cat",
-    "type": "dependencies"
+    "type": "dependencies",
+    "dependentLocation": ""
   },
   "chai": {
     "current": "1.0.0",
     "wanted": "1.0.1",
     "latest": "1.0.1",
     "dependent": "prefix",
     "location": "{CWD}/prefix/node_modules/chai",
-    "type": "peerDependencies"
+    "type": "peerDependencies",
+    "dependentLocation": ""
   },
   "dog": {
     "current": "1.0.1",
     "wanted": "1.0.1",
     "latest": "2.0.0",
     "dependent": "prefix",
     "location": "{CWD}/prefix/node_modules/dog",
-    "type": "dependencies"
+    "type": "dependencies",
+    "dependentLocation": ""
   },
   "theta": {
     "wanted": "1.0.1",
     "latest": "1.0.1",
     "dependent": "prefix",
-    "type": "dependencies"
+    "type": "dependencies",
+    "dependentLocation": ""
   }
 }
 `
@@ -90,7 +125,7 @@ exports[`test/lib/commands/outdated.js TAP should display outdated deps outdated
 `
 
 exports[`test/lib/commands/outdated.js TAP should display outdated deps outdated --long > must match snapshot 1`] = `
-Package  Current  Wanted  Latest  Location           Depended by  Package Type      Homepage
+Package  Current  Wanted  Latest  Location           Depended by  Package Type      Homepage  Dependent Location
 cat        1.0.0   1.0.1   1.0.1  node_modules/cat   prefix       dependencies
 chai       1.0.0   1.0.1   1.0.1  node_modules/chai  prefix       peerDependencies
 dog        1.0.1   1.0.1   2.0.0  node_modules/dog   prefix       dependencies
@@ -120,10 +155,10 @@ exports[`test/lib/commands/outdated.js TAP should display outdated deps outdated
 `
 
 exports[`test/lib/commands/outdated.js TAP should display outdated deps outdated --parseable --long > must match snapshot 1`] = `
-{CWD}/prefix/node_modules/cat:[email protected]:[email protected]:[email protected]:prefix:dependencies:
-{CWD}/prefix/node_modules/chai:[email protected]:[email protected]:[email protected]:prefix:peerDependencies:
-{CWD}/prefix/node_modules/dog:[email protected]:[email protected]:[email protected]:prefix:dependencies:
-:[email protected]:MISSING:[email protected]:prefix:dependencies:
+{CWD}/prefix/node_modules/cat:[email protected]:[email protected]:[email protected]:prefix:dependencies::
+{CWD}/prefix/node_modules/chai:[email protected]:[email protected]:[email protected]:prefix:peerDependencies::
+{CWD}/prefix/node_modules/dog:[email protected]:[email protected]:[email protected]:prefix:dependencies::
+:[email protected]:MISSING:[email protected]:prefix:dependencies::
 `
 
 exports[`test/lib/commands/outdated.js TAP should display outdated deps outdated --parseable > must match snapshot 1`] = `

test/lib/commands/outdated.js

@@ -662,3 +662,72 @@ t.test('aliases with version range', async t => {
   )
   t.equal(process.exitCode, 1)
 })
+
+t.test('dependent location', async t => {
+  const testDir = {
+    'package.json': JSON.stringify({
+      name: 'similer-name',
+      version: '1.0.0',
+      workspaces: ['a', 'nest/a'],
+    }),
+    a: {
+      'package.json': JSON.stringify({
+        name: 'a',
+        version: '1.0.0',
+        dependencies: {
+          dog: '^1.0.0',
+        },
+      }),
+    },
+    nest: {
+      a: {
+        'package.json': JSON.stringify({
+          name: 'nest-a',
+          version: '1.0.0',
+          dependencies: {
+            dog: '^1.0.0',
+          },
+        }),
+      },
+    },
+    node_modules: {
+      dog: {
+        'package.json': JSON.stringify({
+          name: 'dog',
+          version: '1.0.0',
+        }),
+      },
+      a: t.fixture('symlink', '../a'),
+      'nest-a': t.fixture('symlink', '../nest/a'),
+    },
+
+  }
+  t.test(`--long`, async t => {
+    const { outdated, joinedOutput } = await mockNpm(t, {
+      prefixDir: testDir,
+      config: {
+        long: true,
+      },
+    })
+    await outdated.exec([])
+    t.matchSnapshot(
+      joinedOutput(),
+      'should display dependent location when using --long'
+    )
+  })
+
+  t.test('--long --json', async t => {
+    const { outdated, joinedOutput } = await mockNpm(t, {
+      prefixDir: testDir,
+      config: {
+        long: true,
+        json: true,
+      },
+    })
+    await outdated.exec([])
+    t.matchSnapshot(
+      joinedOutput(),
+      'should display dependent location when using --long and --json'
+    )
+  })
+})