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, 23 insertions, 23 deletions

diff --git a/test/tap/init-interrupt.js b/test/tap/init-interrupt.js
index 962ab7244..2e85a5d1b 100644
--- a/test/tap/init-interrupt.js
+++ b/test/tap/init-interrupt.js
@@ -1,52 +1,52 @@
-// if "npm init" is interrupted with ^C, don't report
-// "init written successfully"
-var test = require("tap").test
-var path = require("path")
-var osenv = require("osenv")
-var rimraf = require("rimraf")
-var npmlog = require("npmlog")
-var requireInject = require("require-inject")
+// if 'npm init' is interrupted with ^C, don't report
+// 'init written successfully'
+var test = require('tap').test
+var path = require('path')
+var osenv = require('osenv')
+var rimraf = require('rimraf')
+var npmlog = require('npmlog')
+var requireInject = require('require-inject')
 
-var npm = require("../../lib/npm.js")
+var npm = require('../../lib/npm.js')
 
-var PKG_DIR = path.resolve(__dirname, "init-interrupt")
+var PKG_DIR = path.resolve(__dirname, 'init-interrupt')
 
-test("setup", function (t) {
+test('setup', function (t) {
   cleanup()
 
   t.end()
 })
 
-test("issue #6684 remove confusing message", function (t) {
+test('issue #6684 remove confusing message', function (t) {
 
   var initJsonMock = function (dir, input, config, cb) {
     process.nextTick(function () {
-      cb({message : "canceled"})
+      cb({ message: 'canceled' })
     })
   }
   initJsonMock.yes = function () { return true }
 
-  npm.load({loglevel : "silent"}, function () {
-    var log = ""
-    var init = requireInject("../../lib/init", {
-      "init-package-json": initJsonMock
+  npm.load({ loglevel: 'silent' }, function () {
+    var log = ''
+    var init = requireInject('../../lib/init', {
+      'init-package-json': initJsonMock
     })
 
     // capture log messages
-    npmlog.on("log", function (chunk) { log += chunk.message + "\n" } )
+    npmlog.on('log', function (chunk) { log += chunk.message + '\n' })
 
     init([], function (err, code) {
-      t.ifError(err, "init ran successfully")
-      t.notOk(code, "exited without issue")
-      t.notSimilar(log, /written successfully/, "no success message written")
-      t.similar(log, /canceled/, "alerted that init was canceled")
+      t.ifError(err, 'init ran successfully')
+      t.notOk(code, 'exited without issue')
+      t.notSimilar(log, /written successfully/, 'no success message written')
+      t.similar(log, /canceled/, 'alerted that init was canceled')
 
       t.end()
     })
   })
 })
 
-test("cleanup", function (t) {
+test('cleanup', function (t) {
   cleanup()
 
   t.end()