wip: make coverage mapping a bit better

lib/access.js

@@ -28,7 +28,6 @@ class Access extends BaseCommand {
     return 'access'
   }
 
-  /* istanbul ignore next - see test/lib/load-all-commands.js */
   static get params () {
     return [
       'registry',

lib/audit.js

@@ -5,17 +5,14 @@ const auditError = require('./utils/audit-error.js')
 const ArboristWorkspaceCmd = require('./workspaces/arborist-cmd.js')
 
 class Audit extends ArboristWorkspaceCmd {
-  /* istanbul ignore next - see test/lib/load-all-commands.js */
   static get description () {
     return 'Run a security audit'
   }
 
-  /* istanbul ignore next - see test/lib/load-all-commands.js */
   static get name () {
     return 'audit'
   }
 
-  /* istanbul ignore next - see test/lib/load-all-commands.js */
   static get params () {
     return [
       'audit-level',
@@ -28,7 +25,6 @@ class Audit extends ArboristWorkspaceCmd {
     ]
   }
 
-  /* istanbul ignore next - see test/lib/load-all-commands.js */
   static get usage () {
     return ['[fix]']
   }

lib/bugs.js

@@ -17,7 +17,6 @@ class Bugs extends BaseCommand {
     return ['[<pkgname>]']
   }
 
-  /* istanbul ignore next - see test/lib/load-all-commands.js */
   static get params () {
     return ['browser', 'registry']
   }

lib/cache.js

@@ -11,17 +11,14 @@ class Cache extends BaseCommand {
     return 'Manipulates packages cache'
   }
 
-  /* istanbul ignore next - see test/lib/load-all-commands.js */
   static get name () {
     return 'cache'
   }
 
-  /* istanbul ignore next - see test/lib/load-all-commands.js */
   static get params () {
     return ['cache']
   }
 
-  /* istanbul ignore next - see test/lib/load-all-commands.js */
   static get usage () {
     return [
       'add <tarball file>',

lib/ci.js

@@ -20,17 +20,14 @@ const removeNodeModules = async where => {
 const ArboristWorkspaceCmd = require('./workspaces/arborist-cmd.js')
 
 class CI extends ArboristWorkspaceCmd {
-  /* istanbul ignore next - see test/lib/load-all-commands.js */
   static get description () {
     return 'Install a project with a clean slate'
   }
 
-  /* istanbul ignore next - see test/lib/load-all-commands.js */
   static get name () {
     return 'ci'
   }
 
-  /* istanbul ignore next - see test/lib/load-all-commands.js */
   static get params () {
     return [
       'ignore-scripts',

lib/completion.js

@@ -45,12 +45,10 @@ const { promisify } = require('util')
 const BaseCommand = require('./base-command.js')
 
 class Completion extends BaseCommand {
-  /* istanbul ignore next - see test/lib/load-all-commands.js */
   static get description () {
     return 'Tab Completion for npm'
   }
 
-  /* istanbul ignore next - see test/lib/load-all-commands.js */
   static get name () {
     return 'completion'
   }

lib/config.js

@@ -34,12 +34,10 @@ class Config extends BaseCommand {
     return 'Manage the npm configuration files'
   }
 
-  /* istanbul ignore next - see test/lib/load-all-commands.js */
   static get name () {
     return 'config'
   }
 
-  /* istanbul ignore next - see test/lib/load-all-commands.js */
   static get usage () {
     return [
       'set <key>=<value> [<key>=<value> ...]',
@@ -50,7 +48,6 @@ class Config extends BaseCommand {
     ]
   }
 
-  /* istanbul ignore next - see test/lib/load-all-commands.js */
   static get params () {
     return [
       'json',

lib/dedupe.js

@@ -5,17 +5,14 @@ const reifyFinish = require('./utils/reify-finish.js')
 const ArboristWorkspaceCmd = require('./workspaces/arborist-cmd.js')
 
 class Dedupe extends ArboristWorkspaceCmd {
-  /* istanbul ignore next - see test/lib/load-all-commands.js */
   static get description () {
     return 'Reduce duplication in the package tree'
   }
 
-  /* istanbul ignore next - see test/lib/load-all-commands.js */
   static get name () {
     return 'dedupe'
   }
 
-  /* istanbul ignore next - see test/lib/load-all-commands.js */
   static get params () {
     return [
       'global-style',

lib/deprecate.js

@@ -11,17 +11,14 @@ class Deprecate extends BaseCommand {
     return 'Deprecate a version of a package'
   }
 
-  /* istanbul ignore next - see test/lib/load-all-commands.js */
   static get name () {
     return 'deprecate'
   }
 
-  /* istanbul ignore next - see test/lib/load-all-commands.js */
   static get usage () {
     return ['<pkg>[@<version>] <message>']
   }
 
-  /* istanbul ignore next - see test/lib/load-all-commands.js */
   static get params () {
     return [
       'registry',

lib/diff.js

@@ -16,19 +16,16 @@ class Diff extends BaseCommand {
     return 'The registry diff command'
   }
 
-  /* istanbul ignore next - see test/lib/load-all-commands.js */
   static get name () {
     return 'diff'
   }
 
-  /* istanbul ignore next - see test/lib/load-all-commands.js */
   static get usage () {
     return [
       '[...<paths>]',
     ]
   }
 
-  /* istanbul ignore next - see test/lib/load-all-commands.js */
   static get params () {
     return [
       'diff',

lib/dist-tag.js

@@ -12,17 +12,14 @@ class DistTag extends BaseCommand {
     return 'Modify package distribution tags'
   }
 
-  /* istanbul ignore next - see test/lib/load-all-commands.js */
   static get params () {
     return ['workspace', 'workspaces']
   }
 
-  /* istanbul ignore next - see test/lib/load-all-commands.js */
   static get name () {
     return 'dist-tag'
   }
 
-  /* istanbul ignore next - see test/lib/load-all-commands.js */
   static get usage () {
     return [
       'add <pkg>@<version> [<tag>]',

lib/docs.js

@@ -5,22 +5,18 @@ const hostedFromMani = require('./utils/hosted-git-info-from-manifest.js')
 
 const BaseCommand = require('./base-command.js')
 class Docs extends BaseCommand {
-  /* istanbul ignore next - see test/lib/load-all-commands.js */
   static get description () {
     return 'Open documentation for a package in a web browser'
   }
 
-  /* istanbul ignore next - see test/lib/load-all-commands.js */
   static get name () {
     return 'docs'
   }
 
-  /* istanbul ignore next - see test/lib/load-all-commands.js */
   static get params () {
     return ['browser', 'registry', 'workspace', 'workspaces']
   }
 
-  /* istanbul ignore next - see test/lib/load-all-commands.js */
   static get usage () {
     return ['[<pkgname> [<pkgname> ...]]']
   }

lib/doctor.js

@@ -32,17 +32,14 @@ const maskLabel = mask => {
 
 const BaseCommand = require('./base-command.js')
 class Doctor extends BaseCommand {
-  /* istanbul ignore next - see test/lib/load-all-commands.js */
   static get description () {
     return 'Check your npm environment'
   }
 
-  /* istanbul ignore next - see test/lib/load-all-commands.js */
   static get name () {
     return 'doctor'
   }
 
-  /* istanbul ignore next - see test/lib/load-all-commands.js */
   static get params () {
     return ['registry']
   }

lib/edit.js

@@ -13,22 +13,18 @@ class Edit extends BaseCommand {
     return 'Edit an installed package'
   }
 
-  /* istanbul ignore next - see test/lib/load-all-commands.js */
   static get name () {
     return 'edit'
   }
 
-  /* istanbul ignore next - see test/lib/load-all-commands.js */
   static get usage () {
     return ['<pkg>[/<subpkg>...]']
   }
 
-  /* istanbul ignore next - see test/lib/load-all-commands.js */
   static get params () {
     return ['editor']
   }
 
-  /* istanbul ignore next - see test/lib/load-all-commands.js */
   async completion (opts) {
     return completion(this.npm, opts)
   }

lib/exec.js

@@ -28,22 +28,18 @@ const getLocationMsg = require('./exec/get-workspace-location-msg.js')
 // process.env.npm_lifecycle_event = 'npx'
 
 class Exec extends BaseCommand {
-  /* istanbul ignore next - see test/lib/load-all-commands.js */
   static get description () {
     return 'Run a command from a local or remote npm package'
   }
 
-  /* istanbul ignore next - see test/lib/load-all-commands.js */
   static get params () {
     return ['package', 'call', 'workspace', 'workspaces']
   }
 
-  /* istanbul ignore next - see test/lib/load-all-commands.js */
   static get name () {
     return 'exec'
   }
 
-  /* istanbul ignore next - see test/lib/load-all-commands.js */
   static get usage () {
     return [
       '-- <pkg>[@<version>] [args...]',

lib/explain.js

@@ -12,25 +12,21 @@ class Explain extends ArboristWorkspaceCmd {
     return 'Explain installed packages'
   }
 
-  /* istanbul ignore next - see test/lib/load-all-commands.js */
   static get name () {
     return 'explain'
   }
 
-  /* istanbul ignore next - see test/lib/load-all-commands.js */
   static get usage () {
     return ['<folder | specifier>']
   }
 
-  /* istanbul ignore next - see test/lib/load-all-commands.js */
   static get params () {
     return [
       'json',
       'workspace',
     ]
   }
 
-  /* istanbul ignore next - see test/lib/load-all-commands.js */
   async completion (opts) {
     return completion(this.npm, opts)
   }

lib/explore.js

@@ -12,22 +12,18 @@ class Explore extends BaseCommand {
     return 'Browse an installed package'
   }
 
-  /* istanbul ignore next - see test/lib/load-all-commands.js */
   static get name () {
     return 'explore'
   }
 
-  /* istanbul ignore next - see test/lib/load-all-commands.js */
   static get usage () {
     return ['<pkg> [ -- <command>]']
   }
 
-  /* istanbul ignore next - see test/lib/load-all-commands.js */
   static get params () {
     return ['shell']
   }
 
-  /* istanbul ignore next - see test/lib/load-all-commands.js */
   async completion (opts) {
     return completion(this.npm, opts)
   }

lib/find-dupes.js

@@ -2,17 +2,14 @@
 const ArboristWorkspaceCmd = require('./workspaces/arborist-cmd.js')
 
 class FindDupes extends ArboristWorkspaceCmd {
-  /* istanbul ignore next - see test/lib/load-all-commands.js */
   static get description () {
     return 'Find duplication in the package tree'
   }
 
-  /* istanbul ignore next - see test/lib/load-all-commands.js */
   static get name () {
     return 'find-dupes'
   }
 
-  /* istanbul ignore next - see test/lib/load-all-commands.js */
   static get params () {
     return [
       'global-style',

lib/fund.js

@@ -21,17 +21,14 @@ const getPrintableName = ({ name, version }) => {
 }
 
 class Fund extends ArboristWorkspaceCmd {
-  /* istanbul ignore next - see test/lib/load-all-commands.js */
   static get description () {
     return 'Retrieve funding information'
   }
 
-  /* istanbul ignore next - see test/lib/load-all-commands.js */
   static get name () {
     return 'fund'
   }
 
-  /* istanbul ignore next - see test/lib/load-all-commands.js */
   static get params () {
     return [
       'json',
@@ -42,12 +39,10 @@ class Fund extends ArboristWorkspaceCmd {
     ]
   }
 
-  /* istanbul ignore next - see test/lib/load-all-commands.js */
   static get usage () {
     return ['[[<@scope>/]<pkg>]']
   }
 
-  /* istanbul ignore next - see test/lib/load-all-commands.js */
   async completion (opts) {
     return completion(this.npm, opts)
   }

lib/get.js

@@ -1,22 +1,18 @@
 const BaseCommand = require('./base-command.js')
 
 class Get extends BaseCommand {
-  /* istanbul ignore next - see test/lib/load-all-commands.js */
   static get description () {
     return 'Get a value from the npm configuration'
   }
 
-  /* istanbul ignore next - see test/lib/load-all-commands.js */
   static get name () {
     return 'get'
   }
 
-  /* istanbul ignore next - see test/lib/load-all-commands.js */
   static get usage () {
     return ['[<key> ...] (See `npm config`)']
   }
 
-  /* istanbul ignore next - see test/lib/load-all-commands.js */
   async completion (opts) {
     return this.npm.commands.config.completion(opts)
   }