npm / cli / #1743

97%

latest: 97%

push

fix: empty newline printed to stderr

Starting in v7.7.0 running `npm` (no args) is printing an empty newline
to stderr.

This fixes that by correctly exiting via errorHandler and avoiding
hitting the cb() never called error and adds a test to make sure we
avoid that regression moving forward.

Fixes: https://github.com/nodejs/node/pull/37678#issuecomment-808734374

Co-authored-by: Gar <gar+gh@danger.computer>

2998 of 3097 branches covered (96.8%)

Branch coverage included in aggregate %.

5110 of 5253 relevant lines covered (97.28%)

22.06 hits per line