-
Notifications
You must be signed in to change notification settings - Fork 6
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
test: cover some missed classes of sync with unit test - PART 2 (WPB-3472) #1974
Conversation
Codecov Report
@@ Coverage Diff @@
## develop #1974 +/- ##
=============================================
+ Coverage 56.03% 56.26% +0.22%
Complexity 49 49
=============================================
Files 1021 1021
Lines 37948 37946 -2
Branches 3451 3451
=============================================
+ Hits 21266 21349 +83
+ Misses 15244 15156 -88
- Partials 1438 1441 +3
... and 5 files with indirect coverage changes Continue to review full report in Codecov by Sentry.
|
Datadog ReportAll test runs ✅ 2 Total Test Services: 0 Failed, 0 with New Flaky, 2 Passed Test Services
|
PR Submission Checklist for internal contributors
The PR Title
SQPIT-764
The PR Description
What's new in this PR?
Issues
Follow up PR of #1973 to increase sync unit test
Mainly covering
ObserveSyncStateUseCase
andConversationEventReceiver
with unit testsPR Post Submission Checklist for internal contributors (Optional)
PR Post Merge Checklist for internal contributors
References
feat(conversation-list): Sort conversations by most emojis in the title #SQPIT-764
.