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 ported UITests from Xamarin.UITests to Appium - 40 #26491

Merged
merged 6 commits into from
Dec 16, 2024

Conversation

anandhan-rajagopal
Copy link
Contributor

Description of Change

This PR focuses on re-enabling and updating following tests from the Xamarin.UITest framework to be compatible with Appium. The tests, previously commented out, are reviewed, and modified to ensure they are functional with the Appium framework. We are going to enable tests in blocks in different PRs. This is the 40th group of tests enabled.

TestCases

  • ShellFlyoutSizing
  • ShellSearchHandlerItemSizing
  • ShellTitleView
  • ShellItemIsVisible

* Migrated Issues from XamarinUITest into Appium

* Migrated Issue ShellSearchHandlerItemSizing

* Added comments

* Modified Issue ShellFlyoutSizing

* Added issue link

* Updated migrated test cases

* Removed unwanted usings

---------

Co-authored-by: NafeelaNazhir <nafeela.nazhirhussain@syncfusion.com>
@anandhan-rajagopal anandhan-rajagopal requested a review from a team as a code owner December 10, 2024 08:21
@anandhan-rajagopal anandhan-rajagopal changed the title En[Testing] Enabling ported UITests from Xamarin.UITests to Appium - 40 [Testing] Enabling ported UITests from Xamarin.UITests to Appium - 40 Dec 10, 2024
@dotnet-policy-service dotnet-policy-service bot added the community ✨ Community Contribution label Dec 10, 2024
@anandhan-rajagopal anandhan-rajagopal added partner/syncfusion Issues / PR's with Syncfusion collaboration area-testing Unit tests, device tests labels Dec 10, 2024
Copy link
Contributor

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

jfversluis
jfversluis previously approved these changes Dec 10, 2024
@rmarinho
Copy link
Member

The failing iOS tests are not related

@rmarinho
Copy link
Member

/rebase

@dotnet dotnet deleted a comment from azure-pipelines bot Dec 11, 2024
@dotnet dotnet deleted a comment from jfversluis Dec 11, 2024
anandhan-rajagopal and others added 2 commits December 11, 2024 15:25
* Migrated Issues from XamarinUITest into Appium

* Migrated Issue ShellSearchHandlerItemSizing

* Added comments

* Modified Issue ShellFlyoutSizing

* Added issue link

* Updated migrated test cases

* Removed unwanted usings

---------

Co-authored-by: NafeelaNazhir <nafeela.nazhirhussain@syncfusion.com>
@github-actions github-actions bot force-pushed the enable-xamarinuitest-40 branch from fc5edc7 to daca96d Compare December 11, 2024 15:25
@rmarinho
Copy link
Member

/azp run

Copy link

Azure Pipelines successfully started running 3 pipeline(s).

@jfversluis
Copy link
Member

/azp run

Copy link

Azure Pipelines successfully started running 3 pipeline(s).

@jfversluis jfversluis merged commit 72de3c9 into dotnet:main Dec 16, 2024
104 checks passed
@github-actions github-actions bot locked and limited conversation to collaborators Jan 16, 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