chore: use c8 ignore statements where possible

Author: owlstronaut

lib/base-cmd.js

@@ -31,10 +31,11 @@ class BaseCommand {
       let results = ''
       let line = ''
       for (const param of params) {
-        /* istanbul ignore next */
+        /* c8 ignore start */
         if (seenExclusive.has(param)) {
           continue
         }
+        /* c8 ignore stop */
         const { exclusive } = definitions[param]
         let paramUsage = `${definitions[param].usage}`
         if (exclusive) {

lib/cli.js

@@ -1,10 +1,11 @@
 try {
   const { enableCompileCache } = require('node:module')
-  /* istanbul ignore next */
+  /* c8 ignore start */
   if (enableCompileCache) {
     enableCompileCache()
   }
-} catch (e) { /* istanbul ignore next */ }
+  /* c8 ignore stop */
+} catch (e) { /* c8 ignore next */ }
 
 const validateEngines = require('./cli/validate-engines.js')
 const cliEntry = require('node:path').resolve(__dirname, 'cli/entry.js')

lib/cli/validate-engines.js

@@ -17,7 +17,7 @@ module.exports = (process, getCli) => {
 
   // coverage ignored because this is only hit in very unsupported node versions
   // and it's a best effort attempt to show something nice in those cases
-  /* istanbul ignore next */
+  /* c8 ignore start */
   const syntaxErrorHandler = (err) => {
     if (err instanceof SyntaxError) {
       // eslint-disable-next-line no-console
@@ -28,6 +28,7 @@ module.exports = (process, getCli) => {
     }
     throw err
   }
+  /* c8 ignore stop */
 
   process.on('uncaughtException', syntaxErrorHandler)
   process.on('unhandledRejection', syntaxErrorHandler)

lib/commands/completion.js

@@ -212,8 +212,7 @@ const dumpScript = async (p) => {
       // errors, so casually.  But, without this, `. <(npm completion)`
       // can never ever work on OS X.
       // TODO Ignoring coverage, see 'non EPIPE errors cause failures' test.
-      /* istanbul ignore next */
-      if (er.errno === 'EPIPE') {
+      if (er.errno === 'EPIPE') /* c8 ignore next */ {
         res()
       } else {
         rej(er)

lib/commands/config.js

@@ -290,10 +290,11 @@ ${defData}
       return // if validate doesn't throw we have nothing to do
     } catch (err) {
       // coverage skipped because we don't need to test rethrowing errors
-      // istanbul ignore next
+      /* c8 ignore start */
       if (err.code !== 'ERR_INVALID_AUTH') {
         throw err
       }
+      /* c8 ignore stop */
 
       problems = err.problems
     }
@@ -310,12 +311,11 @@ ${defData}
     output.standard('The following configuration problems have been repaired:\n')
     const summary = problems.map(({ action, from, to, key, where }) => {
       // coverage disabled for else branch because it is intentionally omitted
-      // istanbul ignore else
       if (action === 'rename') {
         // we keep track of which configs were modified here so we know what to save later
         locations.push(where)
         return `~ \`${from}\` renamed to \`${to}\` in ${where} config`
-      } else if (action === 'delete') {
+      } /* c8 ignore next */ else if (action === 'delete') {
         locations.push(where)
         return `- \`${key}\` deleted from ${where} config`
       }

lib/commands/edit.js

@@ -34,10 +34,11 @@ class Edit extends BaseCommand {
   static ignoreImplicitWorkspace = false
 
   // TODO
-  /* istanbul ignore next */
+  /* c8 ignore start */
   static async completion (opts, npm) {
     return completion(npm, opts)
   }
+  /* c8 ignore stop */
 
   async exec (args) {
     if (args.length !== 1) {

lib/commands/explain.js

@@ -18,11 +18,12 @@ class Explain extends ArboristWorkspaceCmd {
   static ignoreImplicitWorkspace = false
 
   // TODO
-  /* istanbul ignore next */
+  /* c8 ignore start */
   static async completion (opts, npm) {
     const completion = require('../utils/installed-deep.js')
     return completion(npm, opts)
   }
+  /* c8 ignore stop */
 
   async exec (args) {
     if (!args.length) {

lib/commands/explore.js

@@ -15,10 +15,11 @@ class Explore extends BaseCommand {
   static ignoreImplicitWorkspace = false
 
   // TODO
-  /* istanbul ignore next */
+  /* c8 ignore start */
   static async completion (opts, npm) {
     return completion(npm, opts)
   }
+  /* c8 ignore stop */
 
   async exec (args) {
     if (args.length < 1 || !args[0]) {

lib/commands/fund.js

@@ -35,11 +35,12 @@ class Fund extends ArboristWorkspaceCmd {
   }
 
   // TODO
-  /* istanbul ignore next */
+  /* c8 ignore start */
   static async completion (opts, npm) {
     const completion = require('../utils/installed-deep.js')
     return completion(npm, opts)
   }
+  /* c8 ignore stop */
 
   async exec (args) {
     const spec = args[0]

lib/commands/get.js

@@ -9,11 +9,12 @@ class Get extends BaseCommand {
   static ignoreImplicitWorkspace = false
 
   // TODO
-  /* istanbul ignore next */
+  /* c8 ignore start */
   static async completion (opts) {
     const Config = Npm.cmd('config')
     return Config.completion(opts)
   }
+  /* c8 ignore stop */
 
   async exec (args) {
     return this.npm.exec('config', ['get'].concat(args))