-
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
fix: handle federation not enabled [WPB-5237] #2189
Conversation
Bad checkout, I will need to have only one commit from cherry-pick |
21edd8f
to
9045d6e
Compare
9045d6e
to
514105f
Compare
Codecov Report
@@ Coverage Diff @@
## release/candidate #2189 +/- ##
====================================================
Coverage ? 58.19%
Complexity ? 21
====================================================
Files ? 1010
Lines ? 37750
Branches ? 3437
====================================================
Hits ? 21967
Misses ? 14317
Partials ? 1466 Continue to review full report in Codecov by Sentry.
|
514105f
to
c8be798
Compare
verify(arrangement.userDetailsApi) | ||
.suspendFunction(arrangement.userDetailsApi::getMultipleUsers) | ||
.with(any()) | ||
.wasInvoked(exactly = twice) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It would be nice to check that it invoked getMultipleUsers
with the different inputs, instead of any()
tiwce.
…p/kalium into fix/handle-federation-not-enabled
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
One of client servers returns
probably when user tries to fetch other user data with different domain
Causes (Optional)
Because fetching users is during slow sync, when it returns error the entire slow sync is aborted and user is stuck on connecting
Solutions
Handle this error and try to fetch again only those users which have the same domain as self user