npm / cli / #1299

98%

latest: 97%

push

Do not print error banner for shell proxy commands

There are a few commands (exec, run-script, and the run-script proxies)
where essentially npm is acting like a very fancy shell.  It is peculiar
and noisy for npm to print a verbose error banner at the end of these
commands, since presumably the command itself already did whatever it
had to do to report the error appropriately.

For example, `npm test` runs a test script, usually outputting test
results.  Having npm then tell me that my tests failed with exit status
1 and print a debug log, is unnecessary and unwanted.

When the error encountered for these commands does not have a non-zero
numeric 'code', then we still print the standard npm error reporting
messages, because presumably something went wrong OTHER than a process
exiting with a non-zero status code.

2883 of 2960 branches covered (97.4%)

Branch coverage included in aggregate %.

9 of 9 new or added lines in 3 files covered. (100.0%)

5035 of 5140 relevant lines covered (97.96%)

20.3 hits per line