src: make the npm source comply with `standard`

This is a huge set of mostly mechanical changes. Going forward, all
changes to the npm source base are expected to comply with `standard`,
and it's been integrated into the test suite to enforce that. There are
a few notes below about specific classes of changes that need to be
handled specially for npm's code base.

standard: "Expected error to be handled."

`standard` only expects errors spelled "err" to be handled.
`npm-registry-mock` never actually invokes its callback with an error,
so in some cases I just changed it to be spelled "er" and called it
good.

standard: "Expected a "break" statement before 'case'."

This behavior is actually on purpose, and I don't feel like rewriting
the affected code right now (or, you know, ever). So I added code
comments disabling the checks in the three applicable changes.

standard: "x is a function."

Rebinding functions created via declarations (as opposed to expressions)
is a no-no?

PR-URL: https://github.com/npm/npm/pull/8668

1 files changed, 8 insertions, 9 deletions

diff --git a/lib/whoami.js b/lib/whoami.js
index a3f83cf45..77d52eef9 100644
--- a/lib/whoami.js
+++ b/lib/whoami.js
@@ -1,31 +1,30 @@
-var npm = require("./npm.js")
+var npm = require('./npm.js')
 
 module.exports = whoami
 
-whoami.usage = "npm whoami [--registry <registry>]\n(just prints username according to given registry)"
+whoami.usage = 'npm whoami [--registry <registry>]\n(just prints username according to given registry)'
 
 function whoami (args, silent, cb) {
   // FIXME: need tighter checking on this, but is a breaking change
-  if (typeof cb !== "function") {
+  if (typeof cb !== 'function') {
     cb = silent
     silent = false
   }
 
-  var registry = npm.config.get("registry")
-  if (!registry) return cb(new Error("no default registry set"))
+  var registry = npm.config.get('registry')
+  if (!registry) return cb(new Error('no default registry set'))
 
   var auth = npm.config.getCredentialsByURI(registry)
   if (auth) {
     if (auth.username) {
       if (!silent) console.log(auth.username)
       return process.nextTick(cb.bind(this, null, auth.username))
-    }
-    else if (auth.token) {
-      return npm.registry.whoami(registry, { auth : auth }, function (er, username) {
+    } else if (auth.token) {
+      return npm.registry.whoami(registry, { auth: auth }, function (er, username) {
         if (er) return cb(er)
         if (!username) {
           var needNewSession = new Error(
-            "Your auth token is no longer valid. Please log in again."
+            'Your auth token is no longer valid. Please log in again.'
           )
           needNewSession.code = 'ENEEDAUTH'
           return cb(needNewSession)