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] Added lifecycle methods to UITest #19102

Merged
merged 2 commits into from Dec 11, 2023
Merged

Conversation

jsuarezruiz
Copy link
Contributor

Description of Change

We have gaps in our UITests with appium compared to Xamarin.UITest. When carrying legacy test we have certain needs such as rotating device, suspending and resuming apps, etc.
This PR include methods to manange the App lifecycle.

Added methods to:

  • Launch and App or move to foreground.
  • Move to background an App.
  • Close a running App.

@jsuarezruiz jsuarezruiz requested a review from a team as a code owner November 29, 2023 11:31
@jsuarezruiz jsuarezruiz requested review from rmarinho, PureWeen and jfversluis and removed request for PureWeen November 29, 2023 11:31
@jsuarezruiz
Copy link
Contributor Author

@rmarinho I realize that the problem with the Build are tests not related to this PR that are running on a device with a different resolution than the one used in the reference. Should we update the snapshots? Can we force the tests to run on a specific device?

Snapshot different than baseline: ContentPageBackgroundImageSourceWorks.png (size differs - baseline is 1125x2346 pixels, actual is 1178x2466 pixels)

@jsuarezruiz
Copy link
Contributor Author

@rmarinho I realize that the problem with the Build are tests not related to this PR that are running on a device with a different resolution than the one used in the reference. Should we update the snapshots? Can we force the tests to run on a specific device?

Snapshot different than baseline: ContentPageBackgroundImageSourceWorks.png (size differs - baseline is 1125x2346 pixels, actual is 1178x2466 pixels)

Is this related? #19133

@jsuarezruiz jsuarezruiz added the testing-legacy-ui-tests Issues/PRs that are associated with transitioning our legacy UI Tests to Appium label Dec 1, 2023
@jsuarezruiz
Copy link
Contributor Author

@rmarinho I realize that the problem with the Build are tests not related to this PR that are running on a device with a different resolution than the one used in the reference. Should we update the snapshots? Can we force the tests to run on a specific device?
Snapshot different than baseline: ContentPageBackgroundImageSourceWorks.png (size differs - baseline is 1125x2346 pixels, actual is 1178x2466 pixels)

Is this related? #19133

Nothing, already fixed after merged that PR!.

@rmarinho
Copy link
Member

rmarinho commented Dec 4, 2023

/rebase

jfversluis
jfversluis previously approved these changes Dec 4, 2023
@jfversluis jfversluis enabled auto-merge (squash) December 4, 2023 19:31
@jfversluis jfversluis merged commit 7c54927 into main Dec 11, 2023
47 checks passed
@jfversluis jfversluis deleted the uitest-lifecycle-actions branch December 11, 2023 19:00
@jsuarezruiz jsuarezruiz self-assigned this Dec 12, 2023
@github-actions github-actions bot locked and limited conversation to collaborators Jan 12, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
area/testing Unit tests, device tests t/housekeeping ♻︎ testing-legacy-ui-tests Issues/PRs that are associated with transitioning our legacy UI Tests to Appium
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants