Skip to content

Conversation

@HarishKumarSF4517
Copy link
Contributor

Description

This pull request makes minor updates to the test cases in ListViewViewCellBinding.cs to improve the reliability of UI interactions by adding explicit waits for elements before performing actions.

Test case improvements:

@HarishKumarSF4517 HarishKumarSF4517 changed the title [Testing] Fix for flaky UITests in CI that occasionally fail - 10 [Testing] Fix for flaky UITests in CI that occasionally fail - 11 May 9, 2025
@dotnet-policy-service dotnet-policy-service bot added the community ✨ Community Contribution label May 9, 2025
@dotnet-policy-service
Copy link
Contributor

Hey there @@HarishKumarSF4517! Thank you so much for your PR! Someone from the team will get assigned to your PR shortly and we'll get it reviewed.

@dotnet-policy-service dotnet-policy-service bot added the partner/syncfusion Issues / PR's with Syncfusion collaboration label May 9, 2025
@HarishKumarSF4517 HarishKumarSF4517 changed the title [Testing] Fix for flaky UITests in CI that occasionally fail - 11 [Testing] Fix for flaky ListViewViewCellBinding in CI May 9, 2025
@HarishKumarSF4517 HarishKumarSF4517 changed the title [Testing] Fix for flaky ListViewViewCellBinding in CI [Testing] Fix for flaky test(ListViewViewCellBinding) in CI May 9, 2025
@HarishKumarSF4517 HarishKumarSF4517 marked this pull request as ready for review May 9, 2025 05:53
Copilot AI review requested due to automatic review settings May 9, 2025 05:53
@HarishKumarSF4517 HarishKumarSF4517 requested a review from a team as a code owner May 9, 2025 05:53
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 PR improves the reliability of the UI tests in ListViewViewCellBinding.cs by adding explicit wait calls before tapping the "Add" and "Remove" buttons to mitigate flakiness in CI.

  • Added App.WaitForElement("Add") to ensure visibility before tapping the "Add" button.
  • Added App.WaitForElement("Remove") to ensure visibility before tapping the "Remove" button.

@ghost ghost added the area-testing Unit tests, device tests label May 9, 2025
@jsuarezruiz
Copy link
Contributor

/azp run MAUI-UITests-public

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@rmarinho rmarinho merged commit 7d2fde6 into dotnet:main May 9, 2025
78 checks passed
@github-actions github-actions bot locked and limited conversation to collaborators Jun 9, 2025
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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants