npm / cli / #2215

97%

latest: 97%

push

fix(libnpmexec): don't detach output from npm

The npm output function refers to this.log.  libnpmexec has that passed
in, which decoupled the function from the npm object.  This fixes it,
and sets the tests up in a way where if the output function ever becomes
detached from this.npm in the same way, tests will fail.

PR-URL: https://github.com/npm/cli/pull/3329
Credit: @wraithgar
Close: #3329
Reviewed-by: @ruyadorno

3104 of 3211 branches covered (96.67%)

Branch coverage included in aggregate %.

1 of 1 new or added line in 1 file covered. (100.0%)

5292 of 5456 relevant lines covered (96.99%)

24.33 hits per line