redescribed assertion, removed dupe assertion, added TODO

npm · Oct 14, 2021 · cec00ca · cec00ca
1 parent 1741bed
commit cec00ca
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 13 deletions.
diff --git a/test/lib/init.js b/test/lib/init.js
@@ -532,7 +532,7 @@ t.test('npm init workspces with root', t => {
     const pkg = require(resolve(npm.localPrefix, 'package.json'))
     t.equal(pkg.version, '1.0.0')
     t.equal(pkg.license, 'ISC')
-    t.matchSnapshot(npm._mockOutputs, 'should print helper info')
+    t.matchSnapshot(npm._mockOutputs, 'does not print helper info')
     t.end()
   })
 })
diff --git a/test/lib/outdated.js b/test/lib/outdated.js
@@ -572,18 +572,9 @@ t.test('workspaces', async t => {
       if (err)
         rej(err)
 
-      t.matchSnapshot(logs, 'should display only root outdated when ws disabled')
-      flatOptions.workspacesEnabled = true
-      res()
-    })
-  })
-
-  await new Promise((res, rej) => {
-    flatOptions.workspacesEnabled = false
-    outdated(testDir, {}).exec([], err => {
-      if (err)
-        rej(err)
-
+      // TODO: This should display dog, but doesn't because arborist filters
+      // workspace deps even if they're also root deps
+      // This will be fixed in a future arborist version
       t.matchSnapshot(logs, 'should display only root outdated when ws disabled')
       flatOptions.workspacesEnabled = true
       res()