-
Notifications
You must be signed in to change notification settings - Fork 1.9k
[Testing] Disabled the flaky issues #33441
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
base: main
Are you sure you want to change the base?
[Testing] Disabled the flaky issues #33441
Conversation
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.
Pull request overview
This pull request addresses flakiness in the Issue18751Test UI test by increasing the image loading wait time from 1 second to 2 seconds. The test verifies that a CollectionView inside a RefreshView can properly display monkey images loaded from the internet, and the additional delay ensures images have sufficient time to load before screenshot verification.
Key Changes
- Increased delay from 1000ms to 2000ms in
Issue18751Testto allow more time for network images to load
|
/azp run maui-pr-uitests |
|
Azure Pipelines successfully started running 1 pipeline(s). |
|
/azp run maui-pr-uitests |
|
Azure Pipelines successfully started running 1 pipeline(s). |
|
/rebase |
dafad41 to
1c376c2
Compare
|
/azp run maui-pr-uitests |
|
Azure Pipelines successfully started running 1 pipeline(s). |
|
/azp run maui-pr-uitests |
|
Azure Pipelines successfully started running 1 pipeline(s). |

This pull request updates conditional compilation directives in two test files to reflect additional platforms where the tests are known to fail. These changes help ensure that problematic tests are properly excluded on all relevant platforms.
Test exclusion updates:
Issue18751.cs, the test is now excluded on both iOS and Catalyst with an updated related issue reference.Issue18896.cs, the test exclusion is expanded to include Android, alongside iOS, Catalyst, and Windows, with updated related issue references.Issue: #33507