Skip to content

test: consolidate and improve coverage reports#279

Merged
acuarica merged 18 commits intomainfrom
consolidate-coverage-reports
Mar 25, 2025
Merged

test: consolidate and improve coverage reports#279
acuarica merged 18 commits intomainfrom
consolidate-coverage-reports

Conversation

@acuarica
Copy link
Copy Markdown
Contributor

@acuarica acuarica commented Mar 21, 2025

Description:

This PR consolidates and improves coverage reports. More specifically

  • it adds c8 package to provide V8 native coverage for the JS library,
  • it uploads coverage reports from our test suites to Codecov in CI, and
  • it adds a Coverage section to README to display coverage overview and instructions how to get coverage reports locally.

Related issue(s):

Fixes #280.

Notes for reviewer:

Checklist

  • Documented (Code comments, README, etc.)
  • Tested (unit, integration, etc.)

Signed-off-by: Luis Mastrangelo <luis@swirldslabs.com>
Signed-off-by: Luis Mastrangelo <luis@swirldslabs.com>
@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Mar 21, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 88.74%. Comparing base (93da076) to head (369e0a6).

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #279      +/-   ##
==========================================
+ Coverage   81.25%   88.74%   +7.49%     
==========================================
  Files          12       20       +8     
  Lines        1083     2302    +1219     
  Branches      186      186              
==========================================
+ Hits          880     2043    +1163     
- Misses        148      205      +57     
+ Partials       55       54       -1     

☔ 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.

acuarica added 16 commits March 21, 2025 18:33
Signed-off-by: Luis Mastrangelo <luis@swirldslabs.com>
Signed-off-by: Luis Mastrangelo <luis@swirldslabs.com>
Signed-off-by: Luis Mastrangelo <luis@swirldslabs.com>
Signed-off-by: Luis Mastrangelo <luis@swirldslabs.com>
Signed-off-by: Luis Mastrangelo <luis@swirldslabs.com>
Signed-off-by: Luis Mastrangelo <luis@swirldslabs.com>
Signed-off-by: Luis Mastrangelo <luis@swirldslabs.com>
Signed-off-by: Luis Mastrangelo <luis@swirldslabs.com>
Signed-off-by: Luis Mastrangelo <luis@swirldslabs.com>
Signed-off-by: Luis Mastrangelo <luis@swirldslabs.com>
Signed-off-by: Luis Mastrangelo <luis@swirldslabs.com>
Signed-off-by: Luis Mastrangelo <luis@swirldslabs.com>
Signed-off-by: Luis Mastrangelo <luis@swirldslabs.com>
Signed-off-by: Luis Mastrangelo <luis@swirldslabs.com>
Signed-off-by: Luis Mastrangelo <luis@swirldslabs.com>
Signed-off-by: Luis Mastrangelo <luis@swirldslabs.com>
@acuarica acuarica changed the title test: consolidate coverage reports test: consolidate and improve coverage reports Mar 23, 2025
@acuarica acuarica self-assigned this Mar 23, 2025
@acuarica acuarica added github_actions Github Actions and Platform CI Work docs Improvements or additions to documentation labels Mar 23, 2025
@acuarica acuarica marked this pull request as ready for review March 23, 2025 19:17
@acuarica acuarica requested review from a team as code owners March 23, 2025 19:17
Copy link
Copy Markdown
Contributor

@andrewb1269 andrewb1269 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Review and approve:
test.yml
.gitignore
README.md

Copy link
Copy Markdown
Member

@quiet-node quiet-node left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@acuarica acuarica merged commit 2770af9 into main Mar 25, 2025
18 checks passed
@acuarica acuarica deleted the consolidate-coverage-reports branch March 25, 2025 14:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docs Improvements or additions to documentation github_actions Github Actions and Platform CI Work

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Improve code coverage report in CI

4 participants