[Testing] Enabling more UI Tests by removing platform specific condition - 11#27764
[Testing] Enabling more UI Tests by removing platform specific condition - 11#27764jfversluis merged 7 commits intomainfrom unknown repository
Conversation
There was a problem hiding this comment.
Copilot reviewed 5 out of 8 changed files in this pull request and generated 2 comments.
Files not reviewed (3)
- src/Controls/tests/TestCases.Shared.Tests/Tests/Issues/Issue20903.cs: Evaluated as low risk
- src/Controls/tests/TestCases.Shared.Tests/Tests/Issues/Issue19338.cs: Evaluated as low risk
- src/Controls/tests/TestCases.Shared.Tests/Tests/Issues/Issue16910.cs: Evaluated as low risk
| @@ -1,4 +1,5 @@ | |||
| using NUnit.Framework; | |||
| #if TEST_FAILS_ON_WINDOWS //This test is failing, likely due to product issue, for mor information: https://github.com/dotnet/maui/issues/27059 | |||
There was a problem hiding this comment.
The comment contains a spelling mistake: 'for mor information' should be 'for more information'.
| #if TEST_FAILS_ON_WINDOWS //This test is failing, likely due to product issue, for mor information: https://github.com/dotnet/maui/issues/27059 | |
| #if TEST_FAILS_ON_WINDOWS //This test is failing, likely due to product issue, for more information: https://github.com/dotnet/maui/issues/27059 |
There was a problem hiding this comment.
Could you fix the small typo?
There was a problem hiding this comment.
I have updated these changes in the latest commit.
src/Controls/tests/TestCases.Shared.Tests/Tests/Issues/Issue22674.cs
Outdated
Show resolved
Hide resolved
|
/azp run |
|
Azure Pipelines successfully started running 3 pipeline(s). |
| @@ -1,4 +1,4 @@ | |||
| #if ANDROID | |||
| #if TEST_FAILS_ON_CATALYST //TapCoordinates is not working on Catalyst | |||
There was a problem hiding this comment.
Should be fixed by #27603
After merge it, can re-enable the test on Catalyst.
There was a problem hiding this comment.
I merged the linked PR, can we update it for this one please? :)
There was a problem hiding this comment.
I have updated these changes in the latest commit.
|
/azp run |
|
Azure Pipelines successfully started running 3 pipeline(s). |
|
/azp run |
|
Azure Pipelines successfully started running 3 pipeline(s). |
|
/azp run |
|
Azure Pipelines successfully started running 3 pipeline(s). |
|
/azp run |
|
Azure Pipelines successfully started running 3 pipeline(s). |
|
Sorry new conflict here. Can you rebase this one? |
@jfversluis I have resolved the conflicts and rebased the branch. |
|
/azp run |
|
Azure Pipelines successfully started running 3 pipeline(s). |
…ion - 11 (#27764) * enable test case for all platforms * update the changes * modified changes * replace snapshot * update the changes * modified changes * modified
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 11th group of tests enabled.
Test Cases:
Fixes #22902