Skip to content
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: MLS service UT - WPB-16280 #2590

Open
wants to merge 2 commits into
base: release/cycle-3.118
Choose a base branch
from

Conversation

jullianm
Copy link
Contributor

@jullianm jullianm commented Feb 25, 2025

WPB-16280

Issue

Fix UT failing in MLS service with error "stall on main thread". Not exactly sure what happened as I was able to get the test passed multiple times so I simplified the UT, removing the expectations, now the UT runs much quicker.

Checklist

  • Title contains a reference JIRA issue number like [WPB-XXX].
  • Description is filled and free of optional paragraphs.
  • Adds/updates automated tests.

@samwyndham samwyndham self-requested a review February 25, 2025 14:36
@jullianm jullianm changed the title fix: MLS service UT - no ticket fix: MLS service UT - WPB-16280 Feb 25, 2025
Copy link
Contributor

@samwyndham samwyndham left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good. Is this still a WIP?

@jullianm jullianm marked this pull request as ready for review February 26, 2025 10:02
@jullianm jullianm requested a review from netbe February 26, 2025 10:03
Copy link
Contributor

github-actions bot commented Feb 26, 2025

Test Results

    1 files      1 suites   4m 38s ⏱️
2 433 tests 2 432 ✅ 0 💤 1 ❌
2 433 runs  2 433 ✅ 0 💤 0 ❌

For more details on these failures, see this check.

Results for commit dee036b.

♻️ This comment has been updated with latest results.

@datadog-wireapp
Copy link

datadog-wireapp bot commented Feb 26, 2025

Datadog Report

Branch report: fix/mls-service-ut
Commit report: 5e85217
Test service: wire-ios-mono

✅ 0 Failed, 2363 Passed, 0 Skipped, 4m 38.54s Total Time

@jullianm jullianm changed the base branch from release/cycle-3.118 to release/cycle-3.119 March 3, 2025 08:31
@jullianm jullianm changed the base branch from release/cycle-3.119 to release/cycle-3.118 March 3, 2025 08:35
Copy link
Collaborator

@netbe netbe left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good can be ported to 3.119.0 instead

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants