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

Move Maestro test to Robin #3630

Merged
merged 2 commits into from
Nov 28, 2024
Merged

Move Maestro test to Robin #3630

merged 2 commits into from
Nov 28, 2024

Conversation

loremattei
Copy link
Contributor

@loremattei loremattei commented Nov 27, 2024

Task/Issue URL: https://app.asana.com/0/1204165176092271/1208804974015522/f
Tech Design URL:
CC:

Description:
Move nightly Maestro test runs to Robin.
Maestro Cloud is being sunset and has proven to be very unreliable for us. Switching tests to its successor which seems to be more reliable.

Test runs of e2e tests and sync tests were successful:
https://github.com/duckduckgo/iOS/actions/runs/12065608659
https://github.com/duckduckgo/iOS/actions/runs/12049269866

This PR also updates Maestro to a new-ish version required to run on Robin.
I'm not updating to the latest 1.39.2 because it looked buggy and sluggish when testing locally.

Steps to test this PR:

  1. Check CI is green.
  2. Verify tests run locally.

Internal references:

Software Engineering Expectations
Technical Design Template

@loremattei loremattei marked this pull request as ready for review November 28, 2024 10:18
@loremattei loremattei requested a review from bwaresiak November 28, 2024 10:18
Copy link
Collaborator

@bwaresiak bwaresiak left a comment

Choose a reason for hiding this comment

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

LGTM

@loremattei loremattei merged commit 20a81a8 into main Nov 28, 2024
37 of 38 checks passed
@loremattei loremattei deleted the loremattei/test-robin branch November 28, 2024 12:14
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