Closed antonis closed 1 month ago
Issues
0 New issues
0 Accepted issues
Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code
App Name | WordPress | |
Flavor | Jalapeno | |
Build Type | Debug | |
Version | pr20549-e46b23e | |
Commit | e46b23e231f5e96b4b3b8c988d23a434dd25da9d | |
Direct Download | wordpress-prototype-build-pr20549-e46b23e.apk |
App Name | Jetpack | |
Flavor | Jalapeno | |
Build Type | Debug | |
Version | pr20549-e46b23e | |
Commit | e46b23e231f5e96b4b3b8c988d23a434dd25da9d | |
Direct Download | jetpack-prototype-build-pr20549-e46b23e.apk |
Attention: Patch coverage is 75.00000%
with 1 lines
in your changes are missing coverage. Please review.
Project coverage is 40.25%. Comparing base (
084b2c5
) to head (e46b23e
). Report is 6 commits behind head on trunk.
Files | Patch % | Lines |
---|---|---|
...wordpress/android/ui/engagement/GetLikesUseCase.kt | 75.00% | 0 Missing and 1 partial :warning: |
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Fixes #20548
Desctiption
Uses a CancelableContinuation to be able to check if it is active before resuming. This should prevent the following exception causing the crash:
To Test:
I was not able to reproduce the crash thus I suggest a sanity check that the functionality is not broken
https://github.com/wordpress-mobile/WordPress-Android/assets/304044/7a5c86d4-51bb-4620-b748-8a738fbd4223
Regression Notes
Potential unintended areas of impact
What I did to test those areas of impact (or what existing automated tests I relied on)
What automated tests I added (or what prevented me from doing so)
GetLikesUseCaseTest
PR Submission Checklist:
RELEASE-NOTES.txt
if necessary.Testing Checklist (strike-out the not-applying and unnecessary ones):