-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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 - 16 #26029
base: main
Are you sure you want to change the base?
[Testing] Enabling ported UITests from Xamarin.UITests to Appium - 16 #26029
Conversation
Co-authored-by: nivetha-nagalingam <[email protected]>
/azp run |
Azure Pipelines successfully started running 3 pipeline(s). |
@@ -1,4 +1,5 @@ | |||
using NUnit.Framework; | |||
#if TEST_FAILS_ON_ANDROID && TEST_FAILS_ON_CATALYST && TEST_FAILS_ON_IOS && TEST_FAILS_ON_WINDOWS //Issue reproduced and logged: https://github.com/dotnet/maui/issues/26019. |
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.
The issue was a duplicated and has been closed, could you update the comment to replace it by #26026?
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.
Replaced the issue link and committed the changes, kindly review and share your feedback.
App.WaitForElementTillPageNavigationSettled(PushNewPage); | ||
App.Tap(PushNewPage); | ||
App.WaitForElementTillPageNavigationSettled(TabPage2); | ||
App.Tap(TabPage2); |
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.
Mnnn, maybe I could include a TapTab method that internally does the wait, and also in the case of Android the ToUpperCase.
@anandhan-rajagopal Could you rebase and fix the conflict? Thanks in advance. |
Resolved the conflicts and rebased the branch. |
/azp run |
Azure Pipelines successfully started running 3 pipeline(s). |
|
||
[Test] | ||
[Category(UITestCategories.Page)] | ||
public void Issue45926Test() |
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.
// } | ||
[Test] | ||
[Category(UITestCategories.TabbedPage)] | ||
public void Bugzilla52419Test() |
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.
@anandhan-rajagopal Could you rebase and fix the conflict? Thanks in advance. |
Resolved conflicts |
/azp run |
Azure Pipelines successfully started running 3 pipeline(s). |
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 16th group of tests enabled.
Add WaitForElementTillPageNavigationSettled Extension Method
Description
Also introduces a new extension method
WaitForElementTillPageNavigationSettled
to enhance the stability of UI tests, particularly for MacCatalyst applications. The method provides a more robust way to wait for elements during page transitions, helping to prevent null reference exceptions.TestCases