Skip to content

Add Codecov integration for unit and e2e coverage#359

Merged
tomasz-tomczyk merged 4 commits intomainfrom
codecov-integration
Apr 25, 2026
Merged

Add Codecov integration for unit and e2e coverage#359
tomasz-tomczyk merged 4 commits intomainfrom
codecov-integration

Conversation

@tomasz-tomczyk
Copy link
Copy Markdown
Owner

Summary

  • Uploads Go unit test coverage (coverage.out) to Codecov with flag=unit
  • Builds the binary with go build -cover for e2e, collects GOCOVERDIR data as Playwright drives the server, converts and uploads with flag=e2e
  • run.sh now skips building if CRIT_BIN already points to an existing binary (enables CI to inject the coverage-instrumented build)
  • Adds Codecov badge to README

Test plan

  • CI passes — unit and e2e jobs complete
  • Coverage uploads appear on codecov.io/gh/tomasz-tomczyk/crit
  • Both unit and e2e flags visible in Codecov dashboard
  • Local make e2e still works (run.sh builds normally when CRIT_BIN is unset)

🤖 Generated with Claude Code

Unit tests upload coverage.out directly. E2E tests build the binary
with Go's integration test coverage (-cover flag), collect GOCOVERDIR
data as Playwright drives the server, then convert and upload separately.
Both appear as flags (unit/e2e) in the Codecov dashboard.

run.sh now skips building if CRIT_BIN already points to an existing
binary, so CI can inject the coverage-instrumented build.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@codecov-commenter
Copy link
Copy Markdown

Welcome to Codecov 🎉

Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests.

Thanks for integrating Codecov - We've got you covered ☂️

tomasz-tomczyk and others added 3 commits April 25, 2026 11:30
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
- project: auto target with 1% threshold (prevents regressions)
- patch: 80% target (new code must be well-covered)
- Removes manual bash threshold check — Codecov handles enforcement
- Ignores frontend/, e2e/, node_modules/, integrations/

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@tomasz-tomczyk tomasz-tomczyk merged commit 5703e40 into main Apr 25, 2026
4 checks passed
@tomasz-tomczyk tomasz-tomczyk deleted the codecov-integration branch April 25, 2026 10:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants