Skip to content

feat: add code coverage reporting via Codecov for backend and contracts#398

Open
binayyub4211 wants to merge 4 commits intoLabsCrypt:mainfrom
binayyub4211:feature/issue-373-codecov
Open

feat: add code coverage reporting via Codecov for backend and contracts#398
binayyub4211 wants to merge 4 commits intoLabsCrypt:mainfrom
binayyub4211:feature/issue-373-codecov

Conversation

@binayyub4211
Copy link
Copy Markdown

Closes #373

Implemented comprehensive code coverage tracking for both backend (Vitest) and smart contracts (Tarpaulin). Coverage reports are now automatically uploaded to Codecov during CI runs, with enforced thresholds to prevent coverage erosion.

Changes:

Backend: Enabled lcov reporting, set 60% thresholds, and added @vitest/coverage-v8.
Contracts: Integrated cargo-tarpaulin with 70% failure threshold.
CI: Added Codecov upload actions for all coverage reports.
UI/Docs: Added Codecov badge to README and created codecov.yml for PR comments.
Verification:

Verified Vitest configuration structure.
Validated GitHub Actions YAML syntax.
Ensured thresholds match the requirements.

@ogazboiz
Copy link
Copy Markdown
Contributor

hey, main was failing CI from broken auth imports + frontend parse errors. fixed and pushed to main now. please rebase to pick up the fixes:

git fetch upstream
git rebase upstream/main
git push --force-with-lease

if there's a conflict, resolve it locally and we'll review once CI is green.

@ogazboiz
Copy link
Copy Markdown
Contributor

ogazboiz commented May 1, 2026

hey, main CI is fully green now. there were several real TypeScript and test issues from #444's merge that have all been fixed. please rebase one more time:

git fetch upstream
git rebase upstream/main
git push --force-with-lease

your PR's diff should be much cleaner after this rebase since main now has the canonical versions of cancel.ts, sorobanService.ts, and the withdraw test.

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.

Add code coverage reporting via Codecov

2 participants