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

Update to Maze Runner v8 #1566

Merged
merged 34 commits into from
Jul 7, 2023
Merged

Update to Maze Runner v8 #1566

merged 34 commits into from
Jul 7, 2023

Conversation

twometresteve
Copy link
Contributor

@twometresteve twometresteve commented Jun 22, 2023

Goal

Upgrade e2e tests to Maze Runner v8.

Design

Whilst preparing this change, BrowserStack removed support for Appium 1.8.0. I've worked round that by changing the test fixture to poll for Maze commands rather than react to button presses (which cause instability on iOS 10/11 with the version of Appium that we now have access to). I've refactored the code for polling for commands into a new Thread class.

Changeset

Scenarios also updated in line with the Maze Runner upgrade guide.

Testing

Covered by a full CI run.

@github-actions
Copy link

github-actions bot commented Jun 22, 2023

Bugsnag.framework binary size did not change - 713,304 bytes

    FILE SIZE        VM SIZE    
 --------------  -------------- 
  [ = ]       0  [ = ]       0    TOTAL

Generated by 🚫 Danger

@twometresteve twometresteve marked this pull request as ready for review June 30, 2023 09:08
@twometresteve twometresteve merged commit c8409b2 into next Jul 7, 2023
@twometresteve twometresteve deleted the tms/maze-v8 branch July 7, 2023 08:23
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