Skip to content

ci: open an issue when the Expo preview road test fails - #471

Merged
hurali97 merged 2 commits into
callstack:mainfrom
KisaneNeko:fix/expo-preview-road-test
Oct 5, 2026
Merged

hurali97 merged 2 commits into
callstack:mainfrom
KisaneNeko:fix/expo-preview-road-test

Conversation

@KisaneNeko

@KisaneNeko KisaneNeko commented Sep 24, 2026 •

Copy link
Copy Markdown
Contributor

Follow-up to #464. The Android road-test fixes for SDK 58 landed in #467, so this PR is now just the fix for the failure-issue job:

  • Its if had no status check, so GitHub silently added success() && and skipped the job whenever a road test failed. Added !cancelled().
  • The job doesn't check out the repo, so gh couldn't tell which repo to use. Set GH_REPO.

🤖 Generated with Claude Code

@KisaneNeko
KisaneNeko marked this pull request as ready for review September 24, 2026 09:12
@KisaneNeko

Copy link
Copy Markdown
Contributor Author

@hurali97 the AndroidApp - Vanilla E2E tests failed but this seems like a flaky test that happens in other PRs occasionally as well. Re-run should help.

Since I added diagnostics to the android there's a screenshot that shows button was there as expected. Logcat shows tests failed because app lost focus and couldn't press it. I wasn't able to figure out what exactly stole focus because logcat is truncated at the end, but after this PR lands we'll have the full trace and I'll see if I can fix it.

@KisaneNeko
KisaneNeko force-pushed the fix/expo-preview-road-test branch from 1cc6dd3 to bf2ac7d Compare September 28, 2026 12:13
@KisaneNeko KisaneNeko changed the title fix: Android Expo preview road test on SDK 58 ci: open an issue when the Expo preview road test fails Sep 28, 2026
Radoslaw Nowacki added 2 commits October 1, 2026 09:53
The job's `if` had no status function, so GitHub prepended `success() &&`
and skipped it whenever android or ios failed, which is the only case it
is meant to run in. Add `!cancelled()` so it runs on failure but not on
a manually cancelled run.
The job has no checkout, so `gh issue list/create` failed with "not a git
repository" before reaching the API. Point gh at the workflow's repository.
@KisaneNeko
KisaneNeko force-pushed the fix/expo-preview-road-test branch from bf2ac7d to bf09889 Compare October 1, 2026 07:53
@hurali97
hurali97 merged commit 8ee0160 into callstack:main Oct 5, 2026
13 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.

2 participants