Skip to content

Commit f96721f

Browse files
authored
Merge pull request #79 from outerbase/bwilmoth/test-updates
fix: Report test failures
2 parents b263f3f + 2e058fb commit f96721f

File tree

4 files changed

+12
-4
lines changed

4 files changed

+12
-4
lines changed

.github/workflows/test.yaml

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -39,7 +39,8 @@ jobs:
3939
run: pnpm install
4040

4141
- name: Run tests
42-
run: pnpm vitest --coverage.enabled true --coverage.reporter json-summary --coverage.reporter json
42+
run: pnpm vitest --coverage.enabled true --coverage.reportOnFailure --coverage.reportsDirectory ./coverage || true
43+
continue-on-error: true
4344

4445
- name: Report Coverage
4546
uses: davelosert/vitest-coverage-report-action@v2

package.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -37,7 +37,7 @@
3737
"lint-staged": "^15.2.11",
3838
"prettier": "3.4.2",
3939
"typescript": "^5.7.2",
40-
"vitest": "^2.1.8",
40+
"vitest": "2.1.8",
4141
"wrangler": "^3.96.0"
4242
},
4343
"dependencies": {

pnpm-lock.yaml

Lines changed: 1 addition & 1 deletion
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

vitest.config.ts

Lines changed: 8 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,9 +4,16 @@ export default defineConfig({
44
test: {
55
coverage: {
66
provider: 'istanbul',
7-
reporter: ['text', 'html', 'json', 'json-summary', 'lcov'],
7+
reporter: ['text', 'html', 'json', 'lcov', 'json-summary'], // Ensure json-summary is included
88
include: ['src/**/*.ts'],
99
exclude: ['**/node_modules/**'],
10+
reportOnFailure: true, // Ensures the report is generated even if tests fail
11+
thresholds: {
12+
lines: 75,
13+
branches: 75,
14+
functions: 75,
15+
statements: 75,
16+
},
1017
},
1118
},
1219
})

0 commit comments

Comments
 (0)