[release/11.0.1xx-preview7] Wait for Entry keyboard before snapshot by kubaflo · Pull Request #37053 · dotnet/maui

kubaflo · 2026-08-03T18:52:22Z

Note

Are you waiting for the changes in this PR to be merged?
It would be very helpful if you could test the resulting artifacts from this PR and let us know in a comment if this change resolves your issue. Thank you!

Description

The preview7 iOS UI retry still failed VerifyTextWhenClearButtonVisibleSetNever because the screenshot could be captured while the focused Entry and software keyboard were still transitioning. The test intentionally keeps focus on iOS and crops the keyboard, so dismissing it would change coverage.

Wait for the keyboard to be visible before taking the screenshot. This uses the existing Appium condition rather than an arbitrary delay and preserves the focused-state baseline.

The targeted UI-test pipeline is requested below.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: 9be49656-7117-4235-9d96-404779ab6b16

kubaflo · 2026-08-03T18:52:30Z

/azp run maui-pr-uitests

github-actions · 2026-08-03T18:52:31Z

🚀 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 -- 37053

Or

Run remotely in PowerShell:

iex "& { $(irm https://raw.githubusercontent.com/dotnet/maui/main/eng/scripts/get-maui-pr.ps1) } 37053"

azure-pipelines · 2026-08-03T18:52:40Z

Azure Pipelines: Successfully started running 1 pipeline(s).

azure-pipelines · 2026-08-03T18:52:41Z

Azure Pipelines: Successfully started running 1 pipeline(s). There may be pipelines that require an authorized user to comment /azp run to run.

Copilot

Pull request overview

This PR updates the EntryFeatureTests UI screenshot test flow to better synchronize with the on-screen keyboard before capturing a snapshot, aiming to reduce flakiness caused by transient keyboard/Entry focus transitions.

Changes:

Add a keyboard-visibility wait step before taking the screenshot in VerifyTextWhenClearButtonVisibleSetNever.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: 9be49656-7117-4235-9d96-404779ab6b16

kubaflo · 2026-08-03T19:03:07Z

/azp run maui-pr-uitests

azure-pipelines · 2026-08-03T19:03:13Z

Azure Pipelines will not run the associated pipelines, because the pull request was updated after the run command was issued. Review the pull request again and issue a new run command.

kubaflo · 2026-08-03T19:08:51Z

/azp run

azure-pipelines · 2026-08-03T19:09:09Z

Azure Pipelines: Successfully started running 3 pipeline(s).

Copilot

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.

kubaflo · 2026-08-03T20:37:28Z

@PureWeen this fix is now proven at the current head 2f04b31f: UI build 1537979 contains that commit, the iOS vlatest Entry task succeeded, and VerifyTextWhenClearButtonVisibleSetNever passed in 4 seconds on both executions. Please merge this preview7 fix.

Assert the synchronization wait succeeds so the test cannot continue to the snapshot without reaching its expected keyboard-visible state. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: 9be49656-7117-4235-9d96-404779ab6b16

kubaflo · 2026-08-03T21:31:36Z

/azp run

azure-pipelines · 2026-08-03T21:31:57Z

Azure Pipelines: Successfully started running 3 pipeline(s).

Copilot

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.

kubaflo · 2026-08-03T22:55:05Z

Current head e8a61a4 is now proven in UI build 1538211: the iOS vlatest Entry job succeeded and VerifyTextWhenClearButtonVisibleSetNever passed in 4 seconds. This confirms the follow-up revision preserves the intended fix.

kubaflo · 2026-08-04T02:05:08Z

@PureWeen current head e8a61a4636 now has coverage for both iOS lanes. UI build 1538211 contains that exact head and passes VerifyTextWhenClearButtonVisibleSetNever on iOS vlatest. I also ran the exact head on the CI-matching iPhone 11 Pro / iOS 18.5 simulator: 1/1 passed in 3 seconds. The current Preview 7 branch build 1538409 still reproduces the legacy-lane failure at 4.38%, so this PR addresses an active branch failure. Please merge this Preview 7 fix.

kubaflo · 2026-08-04T18:25:15Z

Consolidated into #37081, which is now the single rebased test-only stabilization PR for Preview 7.

> [!NOTE] > Are you waiting for the changes in this PR to be merged? > It would be very helpful if you could [test the resulting artifacts](https://github.com/dotnet/maui/wiki/Testing-PR-Builds) from this PR and let us know in a comment if this change resolves your issue. Thank you! ### Description of Change Bundles independently valid Preview 7 test and test-infrastructure stabilizations into one merge path. Changes are cherry-picked or extracted from their source PRs without modifying MAUI runtime or product behavior. The aggregate currently covers: - exact DragEvents label polling and shared Appium text-wait behavior; - waiting for the iOS Entry keyboard before its snapshot; - adaptive ListView and CollectionView screenshot stabilization; - RefreshView gesture retry and SwipeView callback-result waiting; - stable initial SafeArea layout measurement; - deterministic iOS Picker-to-Entry keyboard transition handling; - a bounded Android screenshot tolerance for stable post-rotation antialiasing variance; - independent iOS and MacCatalyst Graphics image-scaling device coverage; - a bounded UIKit cleanup wait for the CarouselView leak assertion; - Android-only scoping for the Android modal-animation leak regression test; - an exact detached-page transition with a synchronous alert-request return signal; and - resilient Windows test-machine resolution setup, with focused Pester coverage for each fallback and failure path. Eligible UI tests, device tests, unit tests, test infrastructure, and screenshot baselines can all be included here when they are independently valid without a corresponding product-code change. ### Source PRs The standalone source PRs below are closed in favor of this aggregate. For #37057, only the independently valid test subset is included; its product-code rewrite remains excluded. | PR | Included test-side fix | |---|---| | #37044 | Windows screen-resolution test infrastructure and Pester coverage | | #37045 | DragEvents exact-text waits and Appium helper | | #37053 | iOS Entry keyboard readiness | | #37055 | ListView screenshot retry window | | #37057 | Graphics image-scaling device tests that pass without the runtime rewrite | | #37058 | RefreshView pull-to-refresh retry | | #37059 | SwipeView result-label wait | | #37066 | SafeArea initial-layout retry | | #37069 | Picker keyboard transition stabilization | | #37074 | Empty CollectionView footer screenshot retry/tolerance | | #37086 | iOS/MacCatalyst CarouselView leak-test cleanup wait | | #37088 | Detached-page alert request synchronization | | #37091 | Android modal-animation leak-test platform scope | | #37092 | Android Issue22306 post-rotation screenshot tolerance | ### Scope exclusions Mixed runtime/test fixes are included only when their test-side changes pass independently against the unmodified product code. The current device tests in #37052, the four nonpositive-size cases in #37057, and the device test plus screenshots in #37062 remain excluded because they expose or describe behavior that requires those PRs' functional fixes. #37070 has no test-side changes. Closed ineffective or unsafe fixes are also excluded. ### Validation - All 25 directly reusable source commits were cherry-picked in their original order. - The #37057 device-test subset was extracted into one additional test-only commit after proving it against the unmodified product implementation. - Every directly cherry-picked aggregate file matches the corresponding included source PR head. - The aggregate diff contains no MAUI runtime or product files. - Targeted Release builds pass for `UITest.Appium`, `Controls.TestCases.iOS.Tests`, `Controls.TestCases.Mac.Tests`, and `Controls.TestCases.Android.Tests`. - The focused screen-resolution Pester suite passes 14/14. - Exact local xUnit XML reports 46/46 Graphics device tests passing on both iOS 26 and MacCatalyst, including all 13 extracted cases. - The Graphics deadlock regression now asserts the bounded five-second completion result before awaiting the scaling task. - The CarouselView category passes 6 consecutive MacCatalyst runs at 4/4 each and passes 4/4 on iOS 26 with the final 100 ms cleanup wait. - The Android modal-animation leak test passes in both discovered Android variants, while the iOS device-test assembly excludes that Android-specific regression test. - The Issue22306 Android failure's three retries were byte-for-byte identical at a 1.85% visual difference, below the new Android-only 2% tolerance. - The unloaded-page alert probe now signals immediately after `DisplayAlertAsync` returns, avoiding a teardown-order dependency on the detached page task. - Rebased the aggregate onto release head `dee83edd121`; the resulting diff remains limited to tests and test infrastructure. - The Pester workflow now triggers when `eng/scripts/Set-ScreenResolution.ps1` changes. - A standalone `/azp run` was posted for aggregate head `83da465424b`; exact merge `bad9e4e9a57` builds `1539860`, `1539861`, and `1539862` are required before merge. ### Issues Fixed Contributes to stabilizing the .NET 11 Preview 7 test branch. --------- Co-authored-by: Vally Fixture <vally-fixture@example.invalid> Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Co-authored-by: kubaflo <34349119+kubaflo@users.noreply.github.com> Copilot-Session: 9be49656-7117-4235-9d96-404779ab6b16 Copilot-Session: d00747b7-96f3-4e7a-8dfb-e3a48db04b2d

Wait for iOS Entry keyboard before snapshot

a0ed673

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: 9be49656-7117-4235-9d96-404779ab6b16

Copilot AI review requested due to automatic review settings August 3, 2026 18:52

kubaflo temporarily deployed to copilot-pat-pool August 3, 2026 18:52 — with GitHub Actions Inactive

Copilot started reviewing on behalf of kubaflo August 3, 2026 18:52 View session

kubaflo temporarily deployed to copilot-pat-pool August 3, 2026 18:54 — with GitHub Actions Inactive

Copilot AI reviewed Aug 3, 2026

View reviewed changes

Comment thread src/Controls/tests/TestCases.Shared.Tests/Tests/FeatureMatrix/EntryFeatureTests.cs

kubaflo temporarily deployed to copilot-pat-pool August 3, 2026 18:56 — with GitHub Actions Inactive

kubaflo temporarily deployed to copilot-pat-pool August 3, 2026 18:57 — with GitHub Actions Inactive

github-actions Bot added area-controls-entry Entry platform/ios labels Aug 3, 2026

kubaflo temporarily deployed to copilot-pat-pool August 3, 2026 18:57 — with GitHub Actions Inactive

Limit Entry keyboard wait to iOS

2f04b31

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: 9be49656-7117-4235-9d96-404779ab6b16

Copilot AI review requested due to automatic review settings August 3, 2026 19:03

Copilot started reviewing on behalf of kubaflo August 3, 2026 19:03 View session

Copilot AI reviewed Aug 3, 2026

View reviewed changes

Copilot AI review requested due to automatic review settings August 3, 2026 21:31

Copilot started reviewing on behalf of kubaflo August 3, 2026 21:31 View session

Copilot AI reviewed Aug 3, 2026

View reviewed changes

github-actions Bot mentioned this pull request Aug 4, 2026

[PR Review Queue] 2026-08-04 #37079

Closed

kubaflo mentioned this pull request Aug 4, 2026

[release/11.0.1xx-preview7] Bundle test-only stabilizations #37081

Merged

kubaflo closed this Aug 4, 2026

Uh oh!

Conversation

kubaflo commented Aug 3, 2026

Description

Uh oh!

kubaflo commented Aug 3, 2026

Uh oh!

github-actions Bot commented Aug 3, 2026

Uh oh!

azure-pipelines Bot commented Aug 3, 2026

Uh oh!

azure-pipelines Bot commented Aug 3, 2026

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

kubaflo commented Aug 3, 2026

Uh oh!

azure-pipelines Bot commented Aug 3, 2026

Uh oh!

kubaflo commented Aug 3, 2026

Uh oh!

azure-pipelines Bot commented Aug 3, 2026

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

kubaflo commented Aug 3, 2026

Uh oh!

kubaflo commented Aug 3, 2026

Uh oh!

azure-pipelines Bot commented Aug 3, 2026

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

kubaflo commented Aug 3, 2026

Uh oh!

kubaflo commented Aug 4, 2026

Uh oh!

kubaflo commented Aug 4, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants