diff options
author | Gar <gar+gh@danger.computer> | 2022-03-24 23:23:02 +0300 |
---|---|---|
committer | GitHub <noreply@github.com> | 2022-03-24 23:23:02 +0300 |
commit | 716a07fde7905bb69e4c6f1991bb7289589a6669 (patch) | |
tree | a415c2acb0c4a08cc4338dcf17a0cdc6fda18680 /package.json | |
parent | 6dd1139c9f302ac71f47a75e70bbe9cdf2e64960 (diff) |
fix: 100% coverage in tests (#4607)
* fix: 100% coverage in tests
* Removed dead code in `lib/utils/usage.js`.
* Removed dead code in `lib/base-command.js`.
* Removed "load-all" test, we currently have 100% coverage and new PRs
without tests will be rejected if they don't add coverage for new
files.
* Removed `check-coverage` script as a separate command.
* Removed separate coverage test in ci.yml.
* Removed `coverage` flag from tap config, the default is already to
enforce 100% coverage.
Removed a tiny bit of dead code resulting from this
* fix: clean up usage output
Removed usage lib, rolled logic into base-command.js
Cleaned up usage output to be less redundant
Diffstat (limited to 'package.json')
-rw-r--r-- | package.json | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/package.json b/package.json index ac5f5dd0b..05cf493d7 100644 --- a/package.json +++ b/package.json @@ -216,7 +216,6 @@ "licenses": "licensee --production --errors-only", "test": "tap", "test-all": "npm run test --if-present --workspaces --include-workspace-root", - "check-coverage": "tap", "snap": "tap", "postsnap": "make -s mandocs", "test:nocleanup": "NO_TEST_CLEANUP=1 npm run test --", @@ -237,7 +236,6 @@ "color": 1, "files": "test/{lib,bin,index.js}", "coverage-map": "test/coverage-map.js", - "check-coverage": true, "timeout": 600 }, "templateOSS": { |