chore(refactor): promisify completion scripts

We also removed the "none" script because we handle a missing
script just fine. There is no need to put an empty one in

PR-URL: https://github.com/npm/cli/pull/2759
Credit: @wraithgar
Close: #2759
Reviewed-by: @nlf

1 files changed, 6 insertions, 15 deletions

diff --git a/test/lib/token.js b/test/lib/token.js
index f98881072..6ab841f49 100644
--- a/test/lib/token.js
+++ b/test/lib/token.js
@@ -48,27 +48,18 @@ test('completion', (t) => {
   t.plan(5)
 
   const testComp = (argv, expect) => {
-    tokenMock.completion({ conf: { argv: { remain: argv } } }, (err, res) => {
-      if (err)
-        throw err
-
-      t.strictSame(res, expect, argv.join(' '))
-    })
+    t.resolveMatch(tokenMock.completion({ conf: { argv: { remain: argv } } }), expect, argv.join(' '))
   }
 
-  testComp(['npm', 'token'], [
-    'list',
-    'revoke',
-    'create',
-  ])
-
+  testComp(['npm', 'token'], ['list', 'revoke', 'create'])
   testComp(['npm', 'token', 'list'], [])
   testComp(['npm', 'token', 'revoke'], [])
   testComp(['npm', 'token', 'create'], [])
 
-  tokenMock.completion({ conf: { argv: { remain: ['npm', 'token', 'foobar'] } } }, (err) => {
-    t.match(err, { message: 'foobar not recognized' })
-  })
+  t.rejects(
+    tokenMock.completion({ conf: { argv: { remain: ['npm', 'token', 'foobar'] } } }),
+    { message: 'foobar not recognize' }
+  )
 })
 
 test('token foobar', (t) => {