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

ci: Add mobile UI Tests #2494

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

ci: Add mobile UI Tests #2494

wants to merge 43 commits into from

Conversation

eriklimakc
Copy link
Contributor

GitHub Issue (If applicable): closes #

PR Type

What kind of change does this PR introduce?

  • Build or CI related changes

What is the current behavior?

What is the new behavior?

PR Checklist

Please check if your PR fulfills the following requirements:

  • Tested code with current supported SDKs
  • Docs have been added/updated which fit documentation template. (for bug fixes / features)
  • Unit Tests and/or UI Tests for the changes have been added (for bug fixes / features) (if applicable)
  • Wasm UI Tests are not showing unexpected any differences. Validate PR Screenshots Compare Test Run results.
  • Contains NO breaking changes
  • Updated the Release Notes
  • Associated with an issue (GitHub or internal)

Other information

Internal Issue (If applicable):

@kazo0
Copy link
Contributor

kazo0 commented Aug 13, 2024

/azp run

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@kazo0
Copy link
Contributor

kazo0 commented Aug 14, 2024

/azp run

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@kazo0 kazo0 force-pushed the dev/erli/add-mobile-ui-tests branch from 3aa57fb to 33d9f18 Compare August 16, 2024 14:16
@eriklimakc eriklimakc force-pushed the dev/erli/add-mobile-ui-tests branch 4 times, most recently from a04f3a1 to d86d74c Compare September 2, 2024 16:44
@eriklimakc eriklimakc force-pushed the dev/erli/add-mobile-ui-tests branch 2 times, most recently from 81b188a to 620a653 Compare September 12, 2024 18:11
@eriklimakc
Copy link
Contributor Author

/azp run

Copy link

Azure Pipelines failed to run 1 pipeline(s).

@eriklimakc
Copy link
Contributor Author

Tests failing locally on Android

Given_ContentDialog

When_SimpleContentDialog: The following tests fail because the Dialog closes right after it is open, not allowing enough time for the next actions to be performed.

[TestCase("SimpleDialogCodebehindWithCancelButton", 0, false)]
[TestCase("SimpleDialogCodebehindWithCancelButton", 3, true)]

Given_FlyoutDialog

When_FlyoutsButton: The tests fail because the Flyout closes right after it is open, not allowing enough time for the next actions to be performed.
[TestCase("FlyoutDialogCodebehindWithCancelButton", 0, false)]
[TestCase("FlyoutDialogCodebehindWithCancelButton", 3, true)]

Given_MessageDialog

When_MessageDialogFromXAML: The test fail because the MessageDialog is not closed. Apparently this code (I can confirm the code is being executed) doesn't work on Android.

Given_Responsive

When_Navigation_Responsive: When navigating to another page we check that the ListView in the previous page is not visible, but it actually is visible, even if not showing.

@eriklimakc eriklimakc force-pushed the dev/erli/add-mobile-ui-tests branch 2 times, most recently from 17ab3b9 to faaabb6 Compare September 27, 2024 10:53
@eriklimakc
Copy link
Contributor Author

Even though we're setting [ActivePlatforms(Platform.Browser)] here:

[Test]
[ActivePlatforms(Platform.Browser)]
public void When_PageNavigationNavigateRootUpdateUrl()
this test is running on Android.

cc @kazo0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants