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

UI Tests: State Restoration #2570

Merged
merged 131 commits into from
Apr 9, 2024
Merged

UI Tests: State Restoration #2570

merged 131 commits into from
Apr 9, 2024

Conversation

Halle
Copy link
Contributor

@Halle Halle commented Apr 8, 2024

Task/Issue URL: https://app.asana.com/0/1199230911884351/1205717021705374/f
Tech Design URL:
CC:

Description: Adds a series of UI tests for State Restoration

Steps to test this PR:

  1. Open the scheme UI Tests
  2. Navigate to the test pane
  3. Run StateRestorationTests

Note: If this builds a review build that is treated as a first run on your system during every test (for instance, asking you where to put the application), please build and run the scheme once instead of running its tests, and go through the new app install first run steps once before running the tests, and answer any first install questions.

UI Tests general guidelines for everyone:

  • It (unfortunately!) isn't possible to multitask while running UI tests on your local machine, because your mousing/interface usage will change or intercept screen interactions that the tests depend on.
  • Much as we all want UI testing to work on multi-monitor setups, we have noticed specific focus issues related to waiting for focus across two monitors simultaneously, so we have decided to test the tests on a single monitor.
  • We are currently testing with English as the app destination language.
  • If you experience test failures in your local environment, please always send the xcresult bundle so it is possible to view what happened differently on your machine, even if it seems like the same failure as a previous failure (it may be subtly different). Please give its screenshots or screen recording a look first before sending, just to rule out one-off failure causes like an unrelated app unexpectedly throwing up an update helper (or something similar) in front of where XCUITest is waiting for a UI element to exist.
  • Since the xcresult bundle will include a screen recording or screenshots, for your privacy, please consider things like your chats, calls, and open documents that you don't want to send to a contractor when you are running the tests.

Thank you very much for your help!

Halle added 30 commits March 14, 2024 16:07
@Halle Halle requested a review from ayoy April 9, 2024 15:06
Copy link
Collaborator

@ayoy ayoy left a comment

Choose a reason for hiding this comment

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

LGTM 👏

@Halle Halle merged commit 532ec92 into main Apr 9, 2024
17 checks passed
@Halle Halle deleted the halle/state-restoration-tests branch April 9, 2024 16:13
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