fix: config should not error when workspaces are configured

PR-URL: #2945 Credit: @nlf Close: #2945 Reviewed-by: @wraithgar
npm · Mar 24, 2021 · e8d2adc · e8d2adc
1 parent 98efade
commit e8d2adc
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 0 deletions.
diff --git a/lib/config.js b/lib/config.js
@@ -88,6 +88,10 @@ class Config extends BaseCommand {
     this.config(args).then(() => cb()).catch(cb)
   }
 
+  execWorkspaces (args, filters, cb) {
+    this.exec(args, cb)
+  }
+
   async config ([action, ...args]) {
     this.npm.log.disableProgress()
     try {

diff --git a/test/lib/config.js b/test/lib/config.js
@@ -93,6 +93,13 @@ t.test('config no args', t => {
   })
 })
 
+t.test('config ignores workspaces', t => {
+  config.execWorkspaces([], [], (err) => {
+    t.match(err, /usage instructions/, 'should not error out when workspaces are defined')
+    t.end()
+  })
+})
+
 t.test('config list', t => {
   t.plan(2)