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

[Testing] Enabling more UI Tests by removing platform specific condition - 13 #27904

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

HarishKumarSF4517
Copy link
Contributor

@HarishKumarSF4517 HarishKumarSF4517 commented Feb 19, 2025

Description of Change

The tests, previously added for specific platforms alone, now we are reviewed, and enabled the tests in all applicable platforms with the Appium framework. We are going to enable tests in blocks in different PRs. This is the 13th group of tests enabled.

Test Cases:

  • Issue417
  • Issue935
  • CarouselViewRemoveAt
  • CarouselViewUITests.UpdatePosition
  • CollectionViewUI.CollectionViewVisibility

Fixes #22902

@dotnet-policy-service dotnet-policy-service bot added the community ✨ Community Contribution label Feb 19, 2025
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 Feb 19, 2025
@anandhan-rajagopal anandhan-rajagopal added the area-testing Unit tests, device tests label Feb 19, 2025
@HarishKumarSF4517 HarishKumarSF4517 marked this pull request as ready for review February 20, 2025 11:02
@Copilot Copilot bot review requested due to automatic review settings February 20, 2025 11:02
@HarishKumarSF4517 HarishKumarSF4517 requested a review from a team as a code owner February 20, 2025 11:02

Choose a reason for hiding this comment

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

Copilot reviewed 5 out of 6 changed files in this pull request and generated no comments.

Files not reviewed (1)
  • src/Controls/tests/TestCases.Shared.Tests/Tests/Issues/CollectionViewUI.CollectionViewVisibility.cs: Evaluated as low risk
Comments suppressed due to low confidence (1)

src/Controls/tests/TestCases.Shared.Tests/Tests/Issues/Issue935.cs:24

  • [nitpick] The variable 'label' is declared twice. It should be renamed to 'label1' and 'label2' for clarity.
var label=App.WaitForElement("TestLabel").GetText();
@jsuarezruiz
Copy link
Contributor

/rebase

@github-actions github-actions bot force-pushed the enable-xamarinuitest-harish33 branch from 493f239 to 2380835 Compare February 21, 2025 11:37
@dotnet dotnet deleted a comment from azure-pipelines bot Feb 21, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
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.

[TEST] Review all (UI) Tests and remove platform-specific runs where possible
3 participants