[Testing] Enabling some UITests from Issues folder in Appium-11 #27255
Conversation
|
@dotnet-policy-service agree company="Syncfusion, Inc." |
NafeelaNazhir
added
area-testing
|
Hey there @NafeelaNazhir! Thank you so much for your PR! Someone from the team will get assigned to your PR shortly and we'll get it reviewed. |
|
/azp run |
|
Azure Pipelines successfully started running 3 pipeline(s). |
|
/azp run |
|
Azure Pipelines successfully started running 3 pipeline(s). |
There was a problem hiding this comment.
Copilot reviewed 18 out of 22 changed files in this pull request and generated no comments.
Files not reviewed (4)
- src/Controls/tests/TestCases.HostApp/Elements/ScrollViewCoreGalleryPage.cs: Evaluated as low risk
- src/Controls/tests/TestCases.HostApp/Issues/Issue5367.cs: Evaluated as low risk
- src/Controls/tests/TestCases.Shared.Tests/Tests/ScrollViewUITests.cs: Evaluated as low risk
- src/Controls/tests/TestCases.Shared.Tests/Tests/Issues/Issue10234.cs: Evaluated as low risk
Comments suppressed due to low confidence (1)
src/Controls/tests/TestCases.Shared.Tests/Tests/Issues/Issue5367.cs:10
- [nitpick] The new name 'StringButton' is less descriptive than the original name 'ForceBigStringButton'. Consider reverting to the original name or choosing a more descriptive name.
const string ForceBigStringButton = "StringButton";
|
/azp run |
|
Azure Pipelines successfully started running 3 pipeline(s). |
|
/azp run |
|
Azure Pipelines successfully started running 3 pipeline(s). |
| [Test] | ||
| [Category(UITestCategories.Animation)] | ||
| [Category(UITestCategories.Compatibility)] | ||
| public void Issue1937Test() |
There was a problem hiding this comment.
This test is failing on Catalyst:
Expected timer ticks are greater than 30. Actual: 13
Assert.That(timerTicks1, Is.GreaterThan(30))
Expected: greater than 30
But was: 13
There was a problem hiding this comment.
Hi @jsuarezruiz , At the sample level, the timer has been started and the number of ticks has been validated. I have added a Thread.sleep after starting the timer to resolve the flakiness in the CI. Please review the fix and let me know if you have any concerns.
|
/azp run |
|
Azure Pipelines successfully started running 3 pipeline(s). |
Description of Change
This PR focuses on enabling and updating 7 testcases 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 11th group of tests enabled.
TestCases: