Make OK more consistent

PR-URL: #241 Credit: @gemal Close: #241 Reviewed-by: @isaacs
npm · Oct 8, 2019 · 70f54dc · 70f54dc
1 parent f46edae
commit 70f54dc
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/lib/doctor/check-ping.js b/lib/doctor/check-ping.js
@@ -8,7 +8,7 @@ function checkPing (cb) {
     if (err && err.code && err.code.match(/^E\d{3}$/)) {
       return cb(null, [err.code.substr(1)])
     } else {
-      cb(null, [200, 'OK'])
+      cb(null, [200, 'ok'])
     }
   })
 }

diff --git a/test/tap/doctor.js b/test/tap/doctor.js
@@ -92,7 +92,7 @@ t.test('npm doctor', function (t) {
   npm.commands.doctor({'node-url': node_url}, true, function (e, list) {
     t.ifError(e, 'npm loaded successfully')
     t.same(list.length, 9, 'list should have 9 prop')
-    t.same(list[0][1], 'OK', 'npm ping')
+    t.same(list[0][1], 'ok', 'npm ping')
     t.same(list[1][1], 'v' + npm.version, 'npm -v')
     t.same(list[2][1], process.version, 'node -v')
     t.same(list[3][1], common.registry + '/', 'npm config get registry')
@@ -113,7 +113,7 @@ t.test('npm doctor works without registry', function (t) {
   npm.commands.doctor({'node-url': node_url}, true, function (e, list) {
     t.ifError(e, 'npm loaded successfully')
     t.same(list.length, 9, 'list should have 9 prop')
-    t.same(list[0][1], 'OK', 'npm ping')
+    t.same(list[0][1], 'ok', 'npm ping')
     t.same(list[1][1], 'v' + npm.version, 'npm -v')
     t.same(list[2][1], process.version, 'node -v')
     t.same(list[3][1], '', 'no registry, but no crash')