npm / cli / #2733 / 1

97%

latest: 97%

push

fix(tests): move more tests to use real npm

This moves a handful of the smaller tests to using the new npm mock that
uses the real actual npm object.  It also extends the testing surface
area of a few tests back down into the actual `process.spawn` that
results, instead of anything internal to the code.

Some dead code in `lib/test.js` was found during this, as well as an
instance of a module throwing a string instead of an error object.

3180 of 3269 branches covered (97.28%)

Branch coverage included in aggregate %.

5420 of 5563 relevant lines covered (97.43%)

27.96 hits per line