Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Turns out there were three files that still had no test coverage because of the combination of the mocks in tests and the coverage map. Removing the map altogether exposed them. One of them was easy enough to cover with some differing tar sizes in the `npm view` tests. The others will be more complicated and will need a future effort. This PR removes the coverage map, and specifically ignores the one directory that doesn't have coverage right now. Fixing that still needs to happen, but we're no worse off now than we were before, and now we have a specific signal that those files have no coverage (the nyc-arg entry in package.json) rather than them being silently ignored. The `npm view` tests were also refactored to use the real npm object.
- Loading branch information