fixup! address PR review comments

Author: wraithgar

lib/commands/completion.js

@@ -212,7 +212,7 @@ const dumpScript = async () => {
       // Really, one should not be tossing away EPIPE errors, or any
       // errors, so casually.  But, without this, `. <(npm completion)`
       // can never ever work on OS X.
-      // Ignoring coverage, see 'non EPIPE errors cause failures' test.
+      // TODO Ignoring coverage, see 'non EPIPE errors cause failures' test.
       /* istanbul ignore next */
       if (er.errno === 'EPIPE')
         res()

lib/npm.js

@@ -154,7 +154,7 @@ class Npm extends EventEmitter {
     }
   }
 
-  load () {
+  async load () {
     if (!this.loadPromise) {
       process.emit('time', 'npm:load')
       this.log.pause()

lib/utils/npm-usage.js

@@ -31,7 +31,7 @@ Configuration fields: npm help 7 config
 npm@${npm.version} ${dirname(dirname(__dirname))}`
 }
 
-const allCommands = (npm) => {
+const allCommands = async (npm) => {
   if (npm.config.get('long'))
     return usages(npm)
   return ('\n    ' + wrap(cmdList))

test/fixtures/sandbox.js

@@ -292,10 +292,6 @@ class Sandbox extends EventEmitter {
 
     const cmd = this[_npm].argv.shift()
     return this[_npm].exec(cmd, this[_npm].argv)
-    // const cmd = this[_npm].argv.shift()
-    // const impl = await this[_npm].cmd(cmd)
-
-    // return impl.exec(this[_npm].argv)
   }
 
   async complete (command, argv, partial) {