npm / cli / #1182

98%

latest: 97%

push

fix(publish): follow config for registry auth check

The "do you have auth configured" only takes into consideration a hard-coded
"registry" config, which means that if you don't have auth configured for
the npm registry, but you do for the one you have tied to a scope elsewhere
in your npmrc, we would erroneously tell you that to add a token.

This uses the same method that the rest of the publish chain uses to determine
which registry it would be publishing to, then sees if you have auth for THAT registry.

Because that other function does a hard fallback to the npm registry, there is no more
need for the exception we throw if you do not have one configured.  Also, the npm cli
already defaults that config item, so you can't even set it to a falsey value if you
wanted

2809 of 2985 branches covered (94.1%)

Branch coverage included in aggregate %.

4968 of 5199 relevant lines covered (95.56%)

17.3 hits per line