Skip to content

ci(ios): cache agent-device's XCUITest runner - #1185

Merged
ArekChr merged 2 commits into
mainfrom
ci/cache-agent-device-runner
Oct 22, 2026
Merged

ArekChr merged 2 commits into
mainfrom
ci/cache-agent-device-runner

Conversation

@krozniata

Copy link
Copy Markdown
Member

Summary

Test Plan

What's required for testing (prerequisites)?

What are the steps to reproduce (after prerequisites)?

Compatibility

OS Implemented
iOS ✅❌
Android ✅❌

Checklist

  • I have tested this on a device and a simulator
  • I added the documentation in README.md
  • I updated the typed files (TS and Flow)

Copilot AI lite review requested due to automatic review settings October 15, 2026 09:49

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🔵 Needs a closer look

The cache may retain session data and become stale when package contents change without a version change.

0 open findings

What changed in this PR

Adds GitHub Actions caching for the agent-device iOS XCUITest runner to reduce CI setup time.

Changes:

  • Restores and saves the runner cache around iOS tests.
  • Keys caching by platform, architecture, package version, and Xcode version.
  • Attempts to exclude session-specific files.
File Summary
.github/​workflows/​ios.yml Adds runner caching; cache exclusion and key composition require changes.

🧠 Review effort: Lite


💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@github-actions

github-actions Bot commented Oct 15, 2026 •

Copy link
Copy Markdown

🟢 e2e: 13 passed

All 13 tests in 11 files
Test Time
🟢 tests/basic-pager.e2e.ts · 3 passed 1m 48s
🟢 basic pager › in LTR › pages horizontally 40.3s
🟢 basic pager › in LTR › pages vertically 23.8s
🟢 basic pager › in RTL › pages horizontally with the reversed gesture 44.0s
🟢 tests/issues/issue-1083-modal-set-page.e2e.ts · 1 passed 13.7s
🟢 setPage called behind a native-stack modal lands on the requested page 13.7s
🟢 tests/issues/issue-1096-keyboard-shrink.e2e.ts · 1 passed 19.1s
🟢 keyboard avoidance does not shrink the page under the keyboard 19.1s
🟢 tests/issues/issue-1098-nested-pager.e2e.ts · 1 passed 12.0s
🟢 outer pager stays responsive after its page count changes around a nested pager 12.0s
🟢 tests/issues/issue-1142-search-bar-inset.e2e.ts · 1 passed 14.6s
🟢 FlatList inside the pager gets the search bar safe-area insets 14.6s
🟢 tests/material-top-bar.e2e.ts · 1 passed 17.3s
🟢 material top tabs survive a native-stack push and pop 17.3s
🟢 tests/nested-pager-view.e2e.ts · 1 passed 14.2s
🟢 nested horizontal and vertical pagers scroll independently 14.2s
🟢 tests/on-page-selected.e2e.ts · 1 passed 15.6s
🟢 onPageSelected fires an alert for every page change 15.6s
🟢 tests/scrollable-pager-view.e2e.ts · 1 passed 22.8s
🟢 pager inside a ScrollView keeps its page across vertical scrolls 22.8s
🟢 tests/smoke.e2e.ts · 1 passed 10.8s
🟢 horizontal pager pages with the panel buttons 10.8s
🟢 tests/tab-view-inside-scroll-view.e2e.ts · 1 passed 15.1s
🟢 TabView inside a ScrollView swipes tabs and scrolls the page 15.1s

e2e 0.21.0 · 4m 25s · ios · run artifacts

@krozniata
krozniata marked this pull request as ready for review October 21, 2026 10:03
@krozniata
krozniata marked this pull request as draft October 21, 2026 10:03
@ArekChr

ArekChr commented Oct 22, 2026

Copy link
Copy Markdown
Collaborator

@krozniata Could you change the first cache path from Products to Products/*? Caching the whole directory causes tar to include the session files despite the exclusion pattern.
Otherwise, LGTM. IMO this PR is ready for review and can be taken out of draft.

@ArekChr
ArekChr marked this pull request as ready for review October 22, 2026 09:44
@ArekChr ArekChr self-assigned this Oct 22, 2026

@ArekChr ArekChr left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM, Both iOS jobs passed all 13 E2E tests, including a successful cache restore

@ArekChr
ArekChr merged commit a06349e into main Oct 22, 2026
14 of 16 checks passed
@ArekChr
ArekChr deleted the ci/cache-agent-device-runner branch October 22, 2026 11:17
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