-
Notifications
You must be signed in to change notification settings - Fork 0
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
Maestro test reliability. #11
base: master
Are you sure you want to change the base?
Conversation
265e9af
to
311ae2d
Compare
2 out of 5 ios-test jobs failed despite setting MAESTRO_DRIVER_STARTUP_TIMEOUT=60000, with:
Others have had this issue too: mobile-dev-inc/Maestro#1585. I reduced the timeout from the default! Oops. |
Maestro will be increasing default to 180s here: mobile-dev-inc/Maestro#2389 |
No timeouts with five runs of 180s nor 5 runs of 240s. Instead, for iOS only, got 2 cases of this out of 10 runs. E.g.:
...despite it being visible in commands json and screenshot:
|
There is a lot of variability in time for Barnet assertion to complete. Up to 50s (oddly, that case passed). Often 15s. The App is connecting to a real API, and it can be slow to start up. It's spring boot hosted by lambda functions, which take >10s to start up. There's concurrency here while running 5 jobs in parallel, so maybe test is starting multiple lambda functions? Although, have seen same error twice on master branch, where don't have concurrency. Maybe it's from slow start of lambda function for first request? Maybe both things? Possible solutions:
|
… 50s when running test five times.
c7aaca9
to
ccdd3d7
Compare
Two cases out of twenty, android only: ...of: https://github.com/jg210/expo-experiments/actions/runs/14043603252/job/39322020881
I.e. app launch failed since device offline, but later the |
Temporarily run tests multiple times to assess and then improve reliability.