Skip to content

test(e2e): stabilize Maestro mobile flows#958

Closed
IgorGanapolsky wants to merge 7 commits intodevelopfrom
fix/maestro-flow-repair
Closed

test(e2e): stabilize Maestro mobile flows#958
IgorGanapolsky wants to merge 7 commits intodevelopfrom
fix/maestro-flow-repair

Conversation

@IgorGanapolsky
Copy link
Copy Markdown
Owner

Summary

  • repair stale Maestro assertions and Android system-UI gestures
  • add Android debug-only -ui-test-state seeding so alarm flows are deterministic
  • add an Android Maestro bootstrap wrapper with driver install, port forwarding, and transport retries
  • wire the wrapper into existing device-test scripts and refresh the tracked iOS voice-focus evidence image

Verification

  • Android Maestro full matrix: 10/10 passed
    • summary: /tmp/maestro-android-final-matrix2-5501/summary.txt
  • iOS Maestro full matrix: 4/4 passed
    • summary: /tmp/maestro-ios-rerun-59399/summary.txt
  • Callstack agent-device smoke
    • iOS smoke log dir: /tmp/agent-device-ios-final2-86014
    • Android smoke log dir: /tmp/agent-device-android-final2-96905

IgorGanapolsky and others added 7 commits March 30, 2026 14:49
1. Voice callouts default to OFF on both platforms (was ON).
   Users must explicitly enable them.

2. Voice callouts now track used cues per session and cycle through
   all available callouts before repeating. Previously only prevented
   back-to-back duplicates, allowing the same callout multiple times
   in a single round.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
iOS: Added .animation(nil, value: isPaused) to prevent SwiftUI from
animating the layout change when button text toggles.

Android: Replaced dual AnimatedVisibility with Crossfade for status
text. AnimatedVisibility adds/removes views from layout, causing
parent reflow. Crossfade maintains fixed layout bounds.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@trunk-io
Copy link
Copy Markdown

trunk-io bot commented Mar 30, 2026

Merging to develop in this repository is managed by Trunk.

  • To merge this pull request, check the box to the left or comment /trunk merge below.

After your PR is submitted to the merge queue, this comment will be automatically updated with its status. If the PR fails, failure details will also be posted here

@gemini-code-assist
Copy link
Copy Markdown

Important

Installation incomplete: to start using Gemini Code Assist, please ask the organization owner(s) to visit the Gemini Code Assist Admin Console and sign the Terms of Services.

@github-actions github-actions bot added the pr-state:ci_running Required CI checks are still running label Mar 30, 2026
@sonarqubecloud
Copy link
Copy Markdown

@IgorGanapolsky
Copy link
Copy Markdown
Owner Author

Superseded by clean rebased replacement #995 (same stabilization commit on top of current develop).

@IgorGanapolsky IgorGanapolsky deleted the fix/maestro-flow-repair branch April 8, 2026 17:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr-state:ci_running Required CI checks are still running

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant