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 some UITests from Issues folder in Appium-2 #27064

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

Conversation

HarishKumarSF4517
Copy link
Contributor

Description of Change

This PR focuses on enabling and updating 8 related testscases in Appium. The tests, previously ignored using Fails attribute, 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 2nd group of tests enabled.

Test cases

  • Issue11962
  • Issue11969
  • Issue1267
  • Issue1469
  • Issue1685
  • Issue1747
  • Issue1769
  • Issue1851

@HarishKumarSF4517 HarishKumarSF4517 marked this pull request as ready for review January 10, 2025 15:26
@HarishKumarSF4517 HarishKumarSF4517 requested a review from a team as a code owner January 10, 2025 15:26
@dotnet-policy-service dotnet-policy-service bot added the community ✨ Community Contribution label Jan 10, 2025
@HarishKumarSF4517
Copy link
Contributor Author

@dotnet-policy-service agree company="Syncfusion, Inc."

@HarishKumarSF4517 HarishKumarSF4517 added area-testing Unit tests, device tests partner/syncfusion Issues / PR's with Syncfusion collaboration labels Jan 10, 2025
@jfversluis
Copy link
Member

jfversluis commented Jan 11, 2025

/azp run

Copy link

Azure Pipelines successfully started running 3 pipeline(s).

@jfversluis
Copy link
Member

/rebase

@github-actions github-actions bot force-pushed the enable-xamarinuitest-harish25 branch from 173460b to 350e584 Compare January 11, 2025 19:56
Copy link

Azure Pipelines successfully started running 3 pipeline(s).

@jfversluis
Copy link
Member

@HarishKumarSF4517 this one has conflicts now with another PR that merged first. Could you fix those please? Thanks!

@anandhan-rajagopal
Copy link
Contributor

@HarishKumarSF4517 this one has conflicts now with another PR that merged first. Could you fix those please? Thanks!

@jfversluis Resolved the conflicts and committed the changes.

Copy link

Azure Pipelines successfully started running 3 pipeline(s).

@@ -19,13 +19,11 @@ public Issue1685(TestDevice testDevice) : base(testDevice)
[Test]
[Category(UITestCategories.Entry)]
[Category(UITestCategories.Compatibility)]
[FailsOnAndroidWhenRunningOnXamarinUITest]
public void EntryOneWayBindingShouldUpdate()
Copy link
Contributor

Choose a reason for hiding this comment

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

This test is failing on iOS and Catalyst:

   at UITest.Appium.HelperExtensions.Wait(Func`1 query, Func`2 satisfactory, String timeoutMessage, Nullable`1 timeout, Nullable`1 retryFrequency) in /_/src/TestUtils/src/UITest.Appium/HelperExtensions.cs:line 2163
   at UITest.Appium.HelperExtensions.WaitForAtLeastOne(Func`1 query, String timeoutMessage, Nullable`1 timeout, Nullable`1 retryFrequency) in /_/src/TestUtils/src/UITest.Appium/HelperExtensions.cs:line 2180
   at UITest.Appium.HelperExtensions.WaitForElement(IApp app, String marked, String timeoutMessage, Nullable`1 timeout, Nullable`1 retryFrequency, Nullable`1 postTimeout) in /_/src/TestUtils/src/UITest.Appium/HelperExtensions.cs:line 665
   at Microsoft.Maui.TestCases.Tests.Issues.Issue1685.EntryOneWayBindingShouldUpdate() in /_/src/Controls/tests/TestCases.Shared.Tests/Tests/Issues/Issue1685.cs:line 24
   at System.RuntimeMethodHandle.InvokeMethod(Object target, Void** arguments, Signature sig, Boolean isConstructor)
   at System.Reflection.MethodBaseInvoker.InvokeWithNoArgs(Object obj, BindingFlags invokeAttr)

Could you review it?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Hi @jsuarezruiz ,
These case failure encountered issues on Windows. I suspect there might have been a typo in the string that was autocorrected while entering the text. The problem has now been resolved, and I have committed the changes. Please review them and share any concerns you may have.

@rmarinho
Copy link
Member

/azp run

Copy link

Azure Pipelines successfully started running 3 pipeline(s).

@jsuarezruiz jsuarezruiz self-assigned this Jan 14, 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.

5 participants