Skip to content

Conversation

@gnalh
Copy link
Collaborator

@gnalh gnalh commented Dec 23, 2025

Add support for reporting code coverage for cli integration tests. This required moving away from escargot and moving the cli-tests into the cli directory. You can see the coverage below when running cargo llvm-cov --html against the cli integration tests.
Before:
Screenshot 2025-12-23 at 1 35 30 PM

After:
Screenshot 2025-12-23 at 1 34 02 PM

@trunk-io
Copy link

trunk-io bot commented Dec 23, 2025

😎 Merged manually by Gabriel Al-Harbi (@gnalh) - details.

@codecov-commenter
Copy link

codecov-commenter commented Dec 23, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 81.09%. Comparing base (4d1f204) to head (36f08b5).
⚠️ Report is 3 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #953      +/-   ##
==========================================
+ Coverage   73.58%   81.09%   +7.50%     
==========================================
  Files          72       66       -6     
  Lines       16461    14081    -2380     
==========================================
- Hits        12113    11419     -694     
+ Misses       4348     2662    -1686     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@gnalh gnalh force-pushed the gabe/code-coverage branch from ae6cc2f to 36f08b5 Compare December 23, 2025 21:43
@gnalh
Copy link
Collaborator Author

gnalh commented Dec 23, 2025

/trunk skip-check

@gnalh gnalh marked this pull request as ready for review December 23, 2025 21:48
@trunk-staging-io
Copy link

trunk-staging-io bot commented Dec 23, 2025

Static BadgeStatic BadgeStatic BadgeStatic Badge

View Full Report ↗︎Docs

@gnalh gnalh merged commit 84fcb1f into main Dec 23, 2025
16 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

4 participants