-
Notifications
You must be signed in to change notification settings - Fork 328
FIX: Attempt to fix UI test instabilities. [ISXB-1284] #2187
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
Conversation
…ired on macOS otherwise failing locally.
Codecov ReportAll modified and coverable lines are covered by tests ✅ @@ Coverage Diff @@
## develop #2187 +/- ##
===========================================
+ Coverage 65.45% 67.78% +2.32%
===========================================
Files 367 367
Lines 53505 53505
===========================================
+ Hits 35024 36268 +1244
+ Misses 18481 17237 -1244
Flags with carried forward coverage won't be shown. Click here to find out more. see 33 files with indirect coverage changes 🚀 New features to boost your workflow:
|
Since the iOS CI noise was resolved with #2190 its likely easier to assess stability of this tentative fix now. |
Seen CI instabilities on this branch since the fix but none seem related to UI tests. |
A few re-runs should be made on this before landing. |
Description
Attempt to fix automated UI test instabilities detected in CI.
Testing status & QA
Tests runs green locally on macOS after the proposed changes (consistently failed before the changes).
Overall Product Risks
Small risk, only affects test code and has positive impact on coverage.
Comments to reviewers
Nothing special to call out for reviewing this branch.
Checklist
Before review:
Changed
,Fixed
,Added
sections.Area_CanDoX
,Area_CanDoX_EvenIfYIsTheCase
,Area_WhenIDoX_AndYHappens_ThisIsTheResult
.During merge:
NEW: ___
.FIX: ___
.DOCS: ___
.CHANGE: ___
.RELEASE: 1.1.0-preview.3
.After merge: