Skip to content

Conversation

@jfversluis
Copy link
Member

No description provided.

Copilot AI review requested due to automatic review settings December 8, 2025 13:24
@jfversluis jfversluis added the area-testing Unit tests, device tests label Dec 8, 2025
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR aims to fix failing iOS test stages by improving the reliability of simulator boot detection. The issue was that the simctl boot command can return successfully before the simulator is fully ready to accept app launches, causing subsequent operations to fail.

Key Changes

  • Added a WaitForSimulatorReady method that polls the simulator's boot status using simctl bootstatus
  • Modified the Launch method to wait for the simulator to be fully booted with a 60-second timeout
  • Implemented a polling loop with 2-second intervals to check boot readiness

Copy link
Contributor

Copilot AI commented Dec 8, 2025

@jfversluis I've opened a new pull request, #33049, to work on those changes. Once the pull request is ready, I'll request review from you.

Copilot AI and others added 2 commits December 8, 2025 14:50
* Initial plan

* Print simulator boot status output for diagnostics

Co-authored-by: jfversluis <[email protected]>

---------

Co-authored-by: copilot-swe-agent[bot] <[email protected]>
Co-authored-by: jfversluis <[email protected]>
@github-actions
Copy link
Contributor

github-actions bot commented Dec 9, 2025

🚀 Dogfood this PR with:

⚠️ WARNING: Do not do this without first carefully reviewing the code of this PR to satisfy yourself it is safe.

curl -fsSL https://raw.githubusercontent.com/dotnet/maui/main/eng/scripts/get-maui-pr.sh | bash -s -- 33047

Or

  • Run remotely in PowerShell:
iex "& { $(irm https://raw.githubusercontent.com/dotnet/maui/main/eng/scripts/get-maui-pr.ps1) } 33047"

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants