feat(typing): send typing indicator event tests pt3. (WPB-4590)#2122
Merged
yamilmedina merged 7 commits intofeat/enqueue-send-typing-and-deletefrom Oct 10, 2023
Merged
Conversation
7 tasks
…send-typing-and-delete-tests
alexandreferris
approved these changes
Oct 9, 2023
Contributor
ohassine
approved these changes
Oct 9, 2023
vitorhugods
approved these changes
Oct 10, 2023
yamilmedina
added a commit
that referenced
this pull request
Oct 10, 2023
…pt2. (WPB-4590) (#2121) * feat: mapper and repo ops deletion * chore: ip * chore: some fun refactoring, just ideas * chore: some fun refactoring, just ideas * chore: some fun refactoring, just ideas * chore: working base * chore: clear indicators * chore: clear indicators * chore: clear indicators * chore: notes * chore: refactor into new classes splitted * chore: comments cleanup * chore: detekt fixes * Empty-Commit * feat(typing): send typing indicator event tests pt3. (WPB-4590) (#2122) * chore: test cov * chore: test cov * chore: test cov * chore: test cov * chore: test cov
yamilmedina
added a commit
that referenced
this pull request
Oct 10, 2023
…-4590) (#2118) * fix: exclude selfuser from typingevents * fix: exclude selfuser from typingevents * fix: coverage for flag check * chore: api configuration for sending typing status * chore: api configuration for sending typing status v0 * chore: api configuration for sending typing status v0 * chore: pr comments * feat: mapper and repo ops * feat: mapper and repo ops * feat: mapper and repo ops * chore: detekt * chore: detekt * chore: tests broken * feat(typing): send typing indicator events use case and cleanup code pt2. (WPB-4590) (#2121) * feat: mapper and repo ops deletion * chore: ip * chore: some fun refactoring, just ideas * chore: some fun refactoring, just ideas * chore: some fun refactoring, just ideas * chore: working base * chore: clear indicators * chore: clear indicators * chore: clear indicators * chore: notes * chore: refactor into new classes splitted * chore: comments cleanup * chore: detekt fixes * Empty-Commit * feat(typing): send typing indicator event tests pt3. (WPB-4590) (#2122) * chore: test cov * chore: test cov * chore: test cov * chore: test cov * chore: test cov
7 tasks
github-merge-queue bot
pushed a commit
that referenced
this pull request
Oct 11, 2023
* feat(typing): send typing indicator events networking layer pt1. (WPB-4590) (#2118) * fix: exclude selfuser from typingevents * fix: exclude selfuser from typingevents * fix: coverage for flag check * chore: api configuration for sending typing status * chore: api configuration for sending typing status v0 * chore: api configuration for sending typing status v0 * chore: pr comments * feat: mapper and repo ops * feat: mapper and repo ops * feat: mapper and repo ops * chore: detekt * chore: detekt * chore: tests broken * feat(typing): send typing indicator events use case and cleanup code pt2. (WPB-4590) (#2121) * feat: mapper and repo ops deletion * chore: ip * chore: some fun refactoring, just ideas * chore: some fun refactoring, just ideas * chore: some fun refactoring, just ideas * chore: working base * chore: clear indicators * chore: clear indicators * chore: clear indicators * chore: notes * chore: refactor into new classes splitted * chore: comments cleanup * chore: detekt fixes * Empty-Commit * feat(typing): send typing indicator event tests pt3. (WPB-4590) (#2122) * chore: test cov * chore: test cov * chore: test cov * chore: test cov * chore: test cov * fix: not block delay, to not block send stop * feat: add missing usecase to clear reusing repository * feat: add missing usecase to clear reusing repository
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## feat/enqueue-send-typing-and-delete #2122 +/- ##
=========================================================================
+ Coverage 57.90% 57.94% +0.04%
Complexity 24 24
=========================================================================
Files 1028 1028
Lines 38321 38321
Branches 3467 3467
=========================================================================
+ Hits 22188 22207 +19
+ Misses 14634 14615 -19
Partials 1499 1499
... and 6 files with indirect coverage changes Continue to review full report in Codecov by Sentry.
🚀 New features to boost your workflow:
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
PR Submission Checklist for internal contributors
The PR Title
SQPIT-764The PR Description
What's new in this PR?
Refactor and split repositories, and create a periodic cleanup of orphaned typing indicators, so we clean those after some time.
Needs releases with:
Testing
Test Coverage (Optional)
Coverage for PR
feat(typing): send typing indicator events use case and cleanup code pt2. (WPB-4590) #2121
I have added automated test to this contribution
PR 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.