Skip to content

ci: Enable coverage and test results reporting#3618

Merged
buenaflor merged 5 commits intomainfrom
buenaflor/ci/enable-coverage-reporting
Apr 2, 2026
Merged

ci: Enable coverage and test results reporting#3618
buenaflor merged 5 commits intomainfrom
buenaflor/ci/enable-coverage-reporting

Conversation

@buenaflor
Copy link
Copy Markdown
Contributor

@buenaflor buenaflor commented Apr 1, 2026

📜 Description

Enable coverage upload and test results reporting for sentry_link, sentry_supabase, and sentry_firebase_remote_config. These were commented out during initial publishing and never re-enabled.

Also fixes incorrect directory paths in the coverage action inputs (e.g. linkpackages/link, firebase_remote_configpackages/firebase_remote_config).

💡 Motivation and Context

All other packages report coverage and test results to Codecov. These three were left behind with a TODO comment ("don't set coverage for now to finish publishing it"). The packages have been published for a while, so it's time to enable reporting.

Closes #3619

💚 How did you test it?

CI will validate — coverage and test-results steps will run on the next workflow execution for these packages.

📝 Checklist

  • I reviewed submitted code
  • I added tests to verify changes
  • No new PII added or SDK only sends newly added PII if sendDefaultPii is enabled
  • I updated the docs if needed
  • All tests passing
  • No breaking changes

🔮 Next steps

Monitor CI to verify coverage uploads succeed and adjust min-coverage thresholds if needed.

#skip-changelog

Enable coverage upload and test results reporting for
sentry_link, sentry_supabase, and sentry_firebase_remote_config.
These were commented out during initial publishing and never
re-enabled. Also fixes incorrect directory paths in the
coverage action inputs.

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

codecov bot commented Apr 1, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 57.79%. Comparing base (0ced0c1) to head (c590654).
⚠️ Report is 1 commits behind head on main.
✅ All tests successful. No failed tests found.

❗ There is a different number of reports uploaded between BASE (0ced0c1) and HEAD (c590654). Click for more details.

HEAD has 9 uploads less than BASE
Flag BASE (0ced0c1) HEAD (c590654)
sentry_file 1 0
sentry_drift 1 0
sentry_hive 1 0
sentry_logging 1 0
sentry_dio 1 0
sentry_isar 1 0
sentry_sqflite 1 0
sentry_flutter 1 0
sentry 1 0
Additional details and impacted files
@@             Coverage Diff             @@
##             main    #3618       +/-   ##
===========================================
- Coverage   88.02%   57.79%   -30.24%     
===========================================
  Files         304       16      -288     
  Lines       10372      417     -9955     
===========================================
- Hits         9130      241     -8889     
+ Misses       1242      176     -1066     
Flag Coverage Δ
sentry ?
sentry_dio ?
sentry_drift ?
sentry_file ?
sentry_firebase_remote_config 100.00% <ø> (?)
sentry_flutter ?
sentry_hive ?
sentry_isar ?
sentry_link 3.95% <ø> (?)
sentry_logging ?
sentry_sqflite ?
sentry_supabase 97.27% <ø> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

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

buenaflor and others added 2 commits April 1, 2026 15:43
sentry_link currently has ~4% test coverage (only 2 test files for
9 source files). Set min-coverage to 3% so CI passes while coverage
reporting is enabled. This can be raised as test coverage improves.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
The flutter-test action checks matrix.target (linux, macos, windows)
to decide when to run tests and collect coverage. The supabase and
firebase_remote_config workflows were using matrix.os instead, which
meant tests were silently skipped and no coverage was generated.

Switch to matrix.target to match isar.yml and other Flutter package
workflows. Also remove unused web input from flutter-test step.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@buenaflor buenaflor marked this pull request as ready for review April 2, 2026 10:00
@buenaflor buenaflor requested a review from denrase as a code owner April 2, 2026 10:00
Copilot AI review requested due to automatic review settings April 2, 2026 10:00
token: ${{ secrets.CODECOV_TOKEN }}
directory: packages/link
coverage: sentry_link
min-coverage: 3
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

yes, the package really has that low coverage

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

we adopted the package externally but we should in the future add some more coverage

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

3 ^^

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Enables Codecov coverage uploads and JUnit test-results reporting for the sentry_supabase and sentry_firebase_remote_config packages, and re-enables coverage for sentry_link, aligning these workflows with the rest of the repo’s package CI reporting.

Changes:

  • Re-enabled Codecov coverage uploads for sentry_link, sentry_supabase, and sentry_firebase_remote_config.
  • Added Codecov test-results reporting steps for sentry_supabase and sentry_firebase_remote_config.
  • Standardized Flutter-package workflow matrices to use matrix.target (macos/linux/windows) and fixed coverage action directory inputs to packages/....

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

File Description
.github/workflows/supabase.yml Switches to matrix.target and adds coverage + test-results uploads for the supabase package.
.github/workflows/link.yml Re-enables coverage upload for sentry_link and corrects the coverage directory path.
.github/workflows/firebase_remote_config.yml Switches to matrix.target and adds coverage + test-results uploads for firebase_remote_config.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link
Copy Markdown

@cursor cursor bot left a comment

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

@buenaflor buenaflor merged commit 5b3a9e9 into main Apr 2, 2026
39 checks passed
@buenaflor buenaflor deleted the buenaflor/ci/enable-coverage-reporting branch April 2, 2026 11:43
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.

Enable coverage and test results reporting for link, supabase, and firebase_remote_config

3 participants