chore: adds ignores in c8 style

Author: owlstronaut

docs/bin/build.js

@@ -3,7 +3,9 @@ const { paths } = require('../lib/index')
 
 run(paths)
   .then((res) => console.error(`Wrote ${res.length} files`))
+  /* c8 ignore start - covered in istanbul not not in the c8 swap */
   .catch((err) => {
     process.exitCode = 1
     console.error(err)
   })
+  /* c8 ignore stop */

docs/lib/check-nav.js

@@ -17,9 +17,10 @@ function ensureNavigationComplete (nav, fsPaths, ext) {
     // otherwise its unmarked in the nav
     if (unmatchedFs[key]) {
       delete unmatchedFs[key]
-    } else {
+    } /* c8 ignore start - covered in istanbul not not in the c8 swap */ else {
       unmatchedNav[key] = true
     }
+    /* c8 ignore stop */
   }
 
   const toKeys = (v) => Object.keys(v).sort().map((p) => p.split(posix.sep).join(sep))
@@ -28,6 +29,7 @@ function ensureNavigationComplete (nav, fsPaths, ext) {
 
   const errors = []
 
+  /* c8 ignore start - covered in istanbul not not in the c8 swap */
   if (missingNav.length) {
     errors.push('The following path(s) exist on disk but are not present in /lib/content/nav.yml:')
     errors.push(...missingNav.map(n => `  ${n}`))
@@ -43,6 +45,7 @@ function ensureNavigationComplete (nav, fsPaths, ext) {
     errors.push('Update nav.yml to ensure that all files are listed in the appropriate place.')
     throw new Error(errors.join('\n'))
   }
+  /* c8 ignore stop */
 }
 
 function getNavigationPaths (entries) {

docs/lib/index.js

@@ -14,11 +14,13 @@ const TAGS = {
 }
 
 const assertPlaceholder = (src, path, placeholder) => {
+  /* c8 ignore start - covered in istanbul not not in the c8 swap */
   if (!src.includes(placeholder)) {
     throw new Error(
       `Cannot replace ${placeholder} in ${path} due to missing placeholder`
     )
   }
+  /* c8 ignore stop */
   return placeholder
 }
 

docs/lib/transform-html.js

@@ -40,8 +40,10 @@ function transformHTML (
       case 'version':
         return version
 
+      /* c8 ignore start - covered in istanbul not not in the c8 swap */
       default:
         throw new Error(`warning: unknown token '${token}' in ${path}`)
+      /* c8 ignore stop */
     }
   })
 
@@ -62,7 +64,7 @@ function transformHTML (
 
       if (url.startsWith('/')) {
         const childDepth = path.split('/').length - 1
-        const prefix = childDepth > 0 ? '../'.repeat(childDepth) : './'
+        const prefix = childDepth > 0 ? '../'.repeat(childDepth) : /* c8 ignore start - covered in istanbul not not in the c8 swap */ './' /* c8 ignore stop */
 
         url = url.replace(/^\//, prefix)
 

lib/commands/completion.js

@@ -163,9 +163,10 @@ class Completion extends BaseCommand {
         const comps = await completion(opts, this.npm)
         return this.wrap(opts, comps)
       }
-    } catch {
+    } /* c8 ignore start - covered in istanbul not not in the c8 swap */ catch {
       // it wasnt a valid command, so do nothing
     }
+    /* c8 ignore stop */
   }
 
   // The command should respond with an array.  Loop over that,
@@ -214,9 +215,10 @@ const dumpScript = async (p) => {
       // TODO Ignoring coverage, see 'non EPIPE errors cause failures' test.
       if (er.errno === 'EPIPE') /* c8 ignore next */ {
         res()
-      } else {
+      } /* c8 ignore start - covered in istanbul not not in the c8 swap */ else {
         rej(er)
       }
+      /* c8 ignore stop */
     })
 
     process.stdout.write(d, () => {

lib/commands/config.js

@@ -57,11 +57,13 @@ const isProtected = (k) => {
     }
     // //registry:_authToken or //registry:authToken
     for (const p of protected) {
-      if (k.endsWith(`:${p}`) || k.endsWith(`:_${p}`)) {
+      if (k.endsWith(`:${p}`) || /* c8 ignore start - covered in istanbul not not in the c8 swap */ k.endsWith(`:_${p}`) /* c8 ignore stop */) {
         return true
       }
     }
+  /* c8 ignore start - covered in istanbul not not in the c8 swap */
   }
+  /* c8 ignore stop */
   return false
 }
 

lib/commands/diff.js

@@ -177,7 +177,9 @@ class Diff extends BaseCommand {
         for (const edge of node.edgesIn) {
           return edge.spec
         }
+      /* c8 ignore start - covered in istanbul not not in the c8 swap */
       }
+      /* c8 ignore stop */
 
       const aSpec = `file:${node.realpath}`
 

lib/commands/fund.js

@@ -167,7 +167,9 @@ class Fund extends ArboristWorkspaceCmd {
               return item.package
             }
           }
+        /* c8 ignore start - covered in istanbul not not in the c8 swap */
         }
+        /* c8 ignore stop */
       } else {
         // tries to retrieve a package from arborist inventory
         // by matching resulted package name from the provided spec

lib/commands/link.js

@@ -165,9 +165,10 @@ class Link extends ArboristWorkspaceCmd {
         // TODO: write tests for unmatching version specs, this is hard to test
         // atm but should be simple once we have a mocked registry again
         if (arg.name !== node.name /* c8 ignore next */ || (
+          /* c8 ignore start - covered in istanbul not not in the c8 swap */
           arg.version &&
-          /* c8 ignore next */
           !semver.satisfies(node.version, arg.version)
+          /* c8 ignore stop */
         )) {
           foundNodes.push(node)
           return true

lib/commands/star.js

@@ -66,7 +66,9 @@ class Star extends BaseCommand {
       log.verbose('star', data)
       return data
     }
+  /* c8 ignore start - covered in istanbul not not in the c8 swap */
   }
+  /* c8 ignore stop */
 }
 
 module.exports = Star