Repository navigation
ci(e2e): fix the Android job on main - #2
Merged
Merged
Conversation
![cubic-dev-ai[bot]](https://avatars.githubusercontent.com/in/1082092?s=60&v=4){kind=small}
There was a problem hiding this comment.
All reported issues were addressed across 2 files
Reply with feedback, questions, or to request a fix.
View guided diff | Turn on auto-fix | Re-trigger cubic
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The Android e2e job on
mainfailed in two different ways, and no job ever uploaded its report.sdkmanagerfailed to downloadsystem-images;android-36;google_apis;x86_64insideandroid-emulator-runner, so no tests ran. A new step installs the image first and retries up to 3 times. The action then finds it already installed.settings > change the skip forward interval: on the PR run,getByLabel('Skip forward').getByRole('button', '45s')matched no node on Android. The chip exists whether or not it's selected, so the container scope was the part that failed.45sonly appears once on the Settings screen, so the test now usesgetByRole('button', '45s')..e2eis a hidden directory, andactions/upload-artifactskips hidden files by default. All three jobs warned "No files were found". They now setinclude-hidden-files: true.x86_64, the emulator's ABI. Building all four ABIs took about 25 of the job's 60 minutes.The other Android failure on the PR run (
playlists > create a playlist) was already removed on main in 4e83bef.Issue
ENG-1271
Verified
Ran it locally: no - this is a CI-only change, and the local emulator was in use by another process. The Android job on this PR is the check.