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

Add UI tests for Emoji Search Android frontends #1697

Merged
merged 2 commits into from
Nov 23, 2023

Conversation

JakeWharton
Copy link
Collaborator

@JakeWharton JakeWharton commented Nov 23, 2023

The view-based sample is currently crashing, which is why it's ignored.

Refs #537. Refs #1696.

The view-based sample is currently crashing, which is why it's ignored.
@JakeWharton JakeWharton force-pushed the jw.emoji-tests.2023-11-22 branch from 392ec01 to 7b5506f Compare November 23, 2023 03:48
- uses: reactivecircus/android-emulator-runner@v2
with:
api-level: 29
emulator-boot-timeout: 20000
Copy link
Collaborator

Choose a reason for hiding this comment

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

Have we ever really figured out why the emulator boot would timeout in SQLDelight PR checks?
Hopefully this doesn't introduce flakiness.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

We have two other jobs that already have emulators. They don't fail that often. Maybe one out of every 30 PRs. Not great, but not terrible either.

@JakeWharton JakeWharton merged commit f0bdef9 into trunk Nov 23, 2023
8 checks passed
@JakeWharton JakeWharton deleted the jw.emoji-tests.2023-11-22 branch November 23, 2023 16:39
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