Skip to content

[Testing] Disabled the flaky issues#33441

Merged
rmarinho merged 4 commits intodotnet:mainfrom
TamilarasanSF4853:fix-flaky-tests-1
Jan 14, 2026
Merged

[Testing] Disabled the flaky issues#33441
rmarinho merged 4 commits intodotnet:mainfrom
TamilarasanSF4853:fix-flaky-tests-1

Conversation

@TamilarasanSF4853
Copy link
Contributor

@TamilarasanSF4853 TamilarasanSF4853 commented Jan 9, 2026

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:

  • In Issue18751.cs, the test is now excluded on both iOS and Catalyst with an updated related issue reference.
  • In Issue18896.cs, the test exclusion is expanded to include Android, alongside iOS, Catalyst, and Windows, with updated related issue references.

Issue: #33507

@dotnet-policy-service dotnet-policy-service bot added community ✨ Community Contribution partner/syncfusion Issues / PR's with Syncfusion collaboration labels Jan 9, 2026
@Ahamed-Ali Ahamed-Ali added testing-flakiness area-testing Unit tests, device tests labels Jan 9, 2026
@sheiksyedm sheiksyedm marked this pull request as ready for review January 9, 2026 11:35
Copilot AI review requested due to automatic review settings January 9, 2026 11:35
Copy link
Contributor

Copilot AI left a 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 Issue18751Test to allow more time for network images to load

@rmarinho
Copy link
Member

rmarinho commented Jan 9, 2026

/azp run maui-pr-uitests

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@rmarinho
Copy link
Member

rmarinho commented Jan 9, 2026

Still failing

Screenshot 2026-01-09 at 19 17 04

@sheiksyedm
Copy link
Contributor

/azp run maui-pr-uitests 

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@rmarinho
Copy link
Member

/rebase

@rmarinho
Copy link
Member

/azp run maui-pr-uitests

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@sheiksyedm sheiksyedm changed the title [Testing] Fix for flaky UITests in CI - 1 [Testing] Disabled the flaky issues Jan 13, 2026
@sheiksyedm
Copy link
Contributor

/azp run maui-pr-uitests

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@rmarinho rmarinho merged commit 20a558f into dotnet:main Jan 14, 2026
28 checks passed
kubaflo pushed a commit to kubaflo/maui that referenced this pull request Jan 16, 2026
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:

* In `Issue18751.cs`, the test is now excluded on both iOS and Catalyst
with an updated related issue reference.
* In `Issue18896.cs`, the test exclusion is expanded to include Android,
alongside iOS, Catalyst, and Windows, with updated related issue
references.

Issue: dotnet#33507
simonrozsival pushed a commit that referenced this pull request Jan 16, 2026
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:

* In `Issue18751.cs`, the test is now excluded on both iOS and Catalyst
with an updated related issue reference.
* In `Issue18896.cs`, the test exclusion is expanded to include Android,
alongside iOS, Catalyst, and Windows, with updated related issue
references.

Issue: #33507
@github-actions github-actions bot locked and limited conversation to collaborators Feb 14, 2026
@TamilarasanSF4853 TamilarasanSF4853 deleted the fix-flaky-tests-1 branch February 25, 2026 12:28
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

area-testing Unit tests, device tests community ✨ Community Contribution partner/syncfusion Issues / PR's with Syncfusion collaboration testing-flakiness

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants