Skip to content

test(mobile): add maestro foundation#128

Merged
dhairyashiil merged 3 commits into
mainfrom
codex/mobile-maestro-foundation
Jun 29, 2026
Merged

test(mobile): add maestro foundation#128
dhairyashiil merged 3 commits into
mainfrom
codex/mobile-maestro-foundation

Conversation

@dhairyashiil

Copy link
Copy Markdown
Member

Summary

  • Add the first Maestro folder under apps/mobile/maestro.
  • Add an unauthenticated launch smoke flow for the login screen using the app IDs from apps/mobile/app.json.
  • Add root and mobile scripts for syntax checking and local iOS/Android Maestro runs.

Why

This starts the E2E lane without making it PR-blocking yet. The first flow covers installed-app launch and the logged-out entry screen, which does not require seeded server data or a test account.

Validation

  • bun run mobile:e2e:check
  • bun run --filter mobile e2e:check
  • maestro check-syntax apps/mobile/maestro/flows/smoke/launch-login.yaml
  • bun run mobile:test
  • bunx biome ci package.json apps/mobile/package.json apps/mobile/maestro/README.md
  • git diff --check HEAD~1..HEAD

Not validated yet

  • Full Maestro execution against a simulator/emulator. Maestro MCP listed available iOS simulators, but none were connected in this session, and the flow also needs the app installed/built on the target device.

Refs ENG-1926.

@vercel

vercel Bot commented Jun 27, 2026

Copy link
Copy Markdown

Deployment failed with the following error:

You don't have permission to create a Preview Deployment for this Vercel project: cal-companion-mcp.

View Documentation: https://vercel.com/docs/accounts/team-members-and-roles

@github-actions github-actions Bot added the config Changes to project configuration files label Jun 27, 2026
@vercel

vercel Bot commented Jun 28, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment
Project Deployment Actions Updated (UTC)
cal-companion-chat Ignored Ignored Jun 28, 2026 6:12am

Request Review

@dhairyashiil dhairyashiil marked this pull request as ready for review June 28, 2026 05:50
@dhairyashiil dhairyashiil requested a review from a team as a code owner June 28, 2026 05:50
@dhairyashiil dhairyashiil enabled auto-merge (squash) June 28, 2026 05:50

@devin-ai-integration devin-ai-integration Bot 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.

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no bugs or issues to report.

Open in Devin Review

@cubic-dev-ai cubic-dev-ai Bot 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.

1 issue found across 7 files

Reply with feedback, questions, or to request a fix.

Re-trigger cubic

Comment thread apps/mobile/scripts/check-maestro-flows.sh Outdated
@dhairyashiil dhairyashiil merged commit fa80a9e into main Jun 29, 2026
12 of 14 checks passed
@dhairyashiil dhairyashiil deleted the codex/mobile-maestro-foundation branch June 29, 2026 09:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

config Changes to project configuration files

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants