Skip to content

test(expo): add an Android emulator smoke test - #563

Merged
gre merged 4 commits into
masterfrom
test/expo-android-smoke
Nov 5, 2026
Merged

gre merged 4 commits into
masterfrom
test/expo-android-smoke

Conversation

@gre

@gre gre commented Nov 4, 2026

Copy link
Copy Markdown
Owner

Work in progress: the full description will follow once CI has run.

🤖 Generated with Claude Code

CI only bundled the Expo apps and never ran them, so startup crashes stayed
invisible. Build each app's release APK, open its screens with Maestro on an
emulator, and fail on any JS error or crash found in logcat.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
gre and others added 3 commits November 4, 2026 17:51
Hide system "isn't responding" dialogs, which covered the app in CI, and open
the camera examples without leaving them: expo-gl crashes when they unmount.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Maestro's screenshot call failed on demodesert in CI while the shader kept the
software GPU busy, with no error from the app. Take them with adb instead, and
leave each example in its own flow. Failure detection doesn't change.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Addresses CodeQL actions/missing-workflow-permissions. Without a permissions
block, these jobs inherit the default GITHUB_TOKEN permissions, and they only
need to read the repo.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@gre
gre marked this pull request as ready for review November 5, 2026 08:59
Copilot AI balanced review requested due to automatic review settings November 5, 2026 08:59

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has insufficient remaining quota. Try requesting a review at a lower effort level.

@gre
gre merged commit 8e9555d into master Nov 5, 2026
9 checks passed
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.

3 participants