Skip to content

Commit

Permalink
fix coverage
Browse files Browse the repository at this point in the history
  • Loading branch information
Pessimistress committed Apr 27, 2024
1 parent c5c6a63 commit c425d28
Show file tree
Hide file tree
Showing 3 changed files with 2 additions and 6 deletions.
1 change: 0 additions & 1 deletion .github/workflows/test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -31,7 +31,6 @@ jobs:
- name: Run tests
run: |
npm run lint
npm run test cover
npm run cover
- name: Coveralls
Expand Down
2 changes: 1 addition & 1 deletion modules/dev-tools/scripts/test.sh
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,7 @@ run_test_script_pretty() {
}

generate_coverage_report() {
(set -x; npx c8 report --reporter=lcov)
(set -x; npx c8 report --reporter=text --reporter=lcov)
}

run_full_test() {
Expand Down
5 changes: 1 addition & 4 deletions modules/dev-tools/src/test.ts
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,7 @@ console.log(`Running ${mode} tests...`); // eslint-disable-line
switch (mode) {
case 'cover':
if (ocularConfig.coverage.test === 'node') {
runNodeTest(resolveNodeEntry('test'), 'npx c8');
runNodeTest(resolveNodeEntry('test'), 'npx c8 --reporter=none');
} else {
await runBrowserTest({
server: {
Expand Down Expand Up @@ -186,9 +186,6 @@ function writeCoverage(coverage) {
'utf8'
);
}

// Print coverage to console
execShellCommand('npx c8 report --reporter=text');
}

function sourcemapFromDataUrl(url: string): string | null {
Expand Down

0 comments on commit c425d28

Please sign in to comment.