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, 8 deletions

diff --git a/lib/config/find-prefix.js b/lib/config/find-prefix.js
index bb00cd6b1..58f5cc804 100644
--- a/lib/config/find-prefix.js
+++ b/lib/config/find-prefix.js
@@ -2,8 +2,8 @@
 
 module.exports = findPrefix
 
-var fs = require("fs")
-var path = require("path")
+var fs = require('fs')
+var path = require('path')
 
 function findPrefix (p, cb_) {
   function cb (er, p) {
@@ -17,7 +17,7 @@ function findPrefix (p, cb_) {
   // walk up until we hopefully find one.
   // if none anywhere, then use cwd.
   var walkedUp = false
-  while (path.basename(p) === "node_modules") {
+  while (path.basename(p) === 'node_modules') {
     p = path.dirname(p)
     walkedUp = true
   }
@@ -27,8 +27,8 @@ function findPrefix (p, cb_) {
 }
 
 function findPrefix_ (p, original, cb) {
-  if (p === "/"
-      || (process.platform === "win32" && p.match(/^[a-zA-Z]:(\\|\/)?$/))) {
+  if (p === '/' ||
+      (process.platform === 'win32' && p.match(/^[a-zA-Z]:(\\|\/)?$/))) {
     return cb(null, original)
   }
   fs.readdir(p, function (er, files) {
@@ -36,15 +36,15 @@ function findPrefix_ (p, original, cb) {
     // unless the prefix was simply a non
     // existent directory.
     if (er && p === original) {
-      if (er.code === "ENOENT") return cb(null, original);
+      if (er.code === 'ENOENT') return cb(null, original)
       return cb(er)
     }
 
     // walked up too high or something.
     if (er) return cb(null, original)
 
-    if (files.indexOf("node_modules") !== -1
-        || files.indexOf("package.json") !== -1) {
+    if (files.indexOf('node_modules') !== -1 ||
+        files.indexOf('package.json') !== -1) {
       return cb(null, p)
     }