auth0 / auth0-flutter

Auth0 SDK for Flutter
https://pub.dev/documentation/auth0_flutter/latest/
Apache License 2.0
57 stars 37 forks source link

Consolidate upload of coverage reports in a single CI job #342

Closed Widcket closed 8 months ago

Widcket commented 8 months ago

📋 Changes

📎 References

Currently, the coverage reports for auth0_flutter, auth0_flutter_platform_interface, and iOS/Android native code get uploaded at the end of the respective test job.

This causes Codecov to report a reduction in coverage (failing its check) until the iOS/Android unit test jobs finish, which take significantly longer than the test jobs of auth0_flutter and auth0_flutter_platform_interface. So the PR gives the impression of failed CI until all test jobs finish and Codecov updates the check status.

This PR consolidates the upload of all coverage reports in a single job that runs after all the test jobs have finished.

codecov[bot] commented 8 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (253209a) 96.08% compared to head (543daf6) 95.80%. Report is 24 commits behind head on main.

:exclamation: Current head 543daf6 differs from pull request most recent head f6b7c87. Consider uploading reports for the commit f6b7c87 to get more accurate results

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #342 +/- ## ============================================ - Coverage 96.08% 95.80% -0.29% ============================================ Files 97 76 -21 Lines 1611 1121 -490 Branches 331 281 -50 ============================================ - Hits 1548 1074 -474 + Misses 49 46 -3 + Partials 14 1 -13 ``` | [Flag](https://app.codecov.io/gh/auth0/auth0-flutter/pull/342/flags?src=pr&el=flags&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=auth0) | Coverage Δ | | |---|---|---| | [auth0_flutter](https://app.codecov.io/gh/auth0/auth0-flutter/pull/342/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=auth0) | `100.00% <100.00%> (ø)` | | | [auth0_flutter_android](https://app.codecov.io/gh/auth0/auth0-flutter/pull/342/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=auth0) | `0.00% <ø> (-96.56%)` | :arrow_down: | | [auth0_flutter_ios](https://app.codecov.io/gh/auth0/auth0-flutter/pull/342/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=auth0) | `99.84% <ø> (ø)` | | | [auth0_flutter_platform_interface](https://app.codecov.io/gh/auth0/auth0-flutter/pull/342/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=auth0) | `87.03% <100.00%> (ø)` | | Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=auth0#carryforward-flags-in-the-pull-request-comment) to find out more. | [Files](https://app.codecov.io/gh/auth0/auth0-flutter/pull/342?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=auth0) | Coverage Δ | | |---|---|---| | [...lib/src/web/extensions/user\_profile\_extension.dart](https://app.codecov.io/gh/auth0/auth0-flutter/pull/342?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=auth0#diff-YXV0aDBfZmx1dHRlci9saWIvc3JjL3dlYi9leHRlbnNpb25zL3VzZXJfcHJvZmlsZV9leHRlbnNpb24uZGFydA==) | `100.00% <100.00%> (ø)` | | | [...lutter\_platform\_interface/lib/src/credentials.dart](https://app.codecov.io/gh/auth0/auth0-flutter/pull/342?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=auth0#diff-YXV0aDBfZmx1dHRlcl9wbGF0Zm9ybV9pbnRlcmZhY2UvbGliL3NyYy9jcmVkZW50aWFscy5kYXJ0) | `100.00% <100.00%> (ø)` | | ... and [23 files with indirect coverage changes](https://app.codecov.io/gh/auth0/auth0-flutter/pull/342/indirect-changes?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=auth0)

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.