fix(codecov): fail the run with the path when a coverage report cannot be written - #376

Merged

roxblnfk merged 1 commit into

1.xfrom

fix/coverage-report-write-errors

Sep 29, 2026

roxblnfk commented Sep 29, 2026

Member

🔍 What was changed

Clover, Cobertura and PHPUnit XML reports throw CoverageReportNotWritten when the target directory or file cannot be written. The message names the path and the filesystem reason instead of a bare mkdir(): Permission denied warning.
The run now exits non-zero on such a failure; before, it passed without the report.
One unwritable target no longer stops the other reports: they are written and announced, then the first failure is rethrown.
testo-coverage skill documents the exception and the Windows-path-in-Linux pitfall behind the report.

Checklist

Closes Coverage reports fail silently and without a path on write errors #375
How was this tested:
- Unit tests added
- Codecov/Unit suite run on Windows and WSL


          fix(codecov): fail the run with the path when a coverage report canno…

ba19ceb

…t be written

Reports wrote their files unchecked, so an unwritable target printed a bare `mkdir(): Permission denied` warning and the run still passed. The remaining reports are written before the first failure is rethrown, so one bad target does not cost the others.

Closes #375

Assisted-By: Claude Opus 5.5 <noreply@anthropic.com>

roxblnfk requested a review from a team as a code owner

September 29, 2026 11:40

codecov Bot commented Sep 29, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

roxblnfk merged commit 2ef9a5f into 1.x

27 of 29 checks passed

roxblnfk deleted the fix/coverage-report-write-errors branch

September 29, 2026 12:15

roxblnfk mentioned this pull request

chore: release 1.x #377

Merged

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet