tests for npm doctor

PR-URL: #2323 Credit: @nlf Close: #2323 Reviewed-by: @isaacs
npm · Dec 11, 2020 · 7a4f0c9 · 7a4f0c9
1 parent d825e90
commit 7a4f0c9
Show file tree

Hide file tree

Showing 2 changed files with 945 additions and 2 deletions.
diff --git a/lib/doctor.js b/lib/doctor.js
@@ -20,7 +20,7 @@ const checkPing = async () => {
     if (/^E\d{3}$/.test(er.code || ''))
       throw er.code.substr(1) + ' ' + er.message
     else
-      throw er
+      throw er.message
   } finally {
     tracker.finish()
   }
@@ -92,7 +92,7 @@ const lstat = promisify(fs.lstat)
 const readdir = promisify(fs.readdir)
 const access = promisify(fs.access)
 const isWindows = require('./utils/is-windows.js')
-const checkFilesPermission = async (root, shouldOwn = true, mask = null) => {
+const checkFilesPermission = async (root, shouldOwn, mask = null) => {
   if (mask === null)
     mask = shouldOwn ? R_OK | W_OK : R_OK