Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Dec 8, 2025

Description of Change

Replicates all changes from PR #33056 which was closed as draft. This includes UI test infrastructure improvements and critical documentation to prevent common troubleshooting mistakes.

Test Infrastructure Enhancements:

  • Added ScrollToBottom() method with position validation to ensure true bottom is reached
  • Added CloseFlyout() helper that auto-detects screen dimensions
  • Fixed namespace ambiguity for PointerInputDevice in MacCatalyst swipe actions

Test Case Updates:

  • Added comprehensive test for Shell flyout footer padding bug ([Android] ShellFlyout Footer area not cleared after removing footer view #32883) that validates Y-position changes across three states: baseline, with footer, and after footer removal
  • Reduced flyout item count from 50 to 20 for faster execution
  • Added AutomationId to bottom flyout item for testability
  • Added SafeAreaEdges configuration to Bugzilla44338

Critical Documentation:

  • Added explicit warning section prohibiting deletion of build artifacts during troubleshooting
  • Prevents incorrect assumption of stale binaries when test results seem unexpected
// New ScrollToBottom validates true bottom position
App.ScrollToBottom(FlyoutItem, "Flyout Item Bottom", 
    strategy: ScrollStrategy.Gesture, swipeSpeed: 50, maxScrolls: 100);

// New CloseFlyout helper
App.CloseFlyout();

Issues Fixed

Fixes #32883

Original prompt

can you create a new pr with all the changes that are in this pr #33056

And then create a second PR that only has the instruction markdown files from that PR in it


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Copilot AI changed the title [WIP] Create a new PR with all changes from #33056 Replicate PR #33056: Fix ShellFlyoutContent UI test and improve Appium scroll actions Dec 8, 2025
Copilot AI requested a review from PureWeen December 8, 2025 20:35
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.

[Android] ShellFlyout Footer area not cleared after removing footer view

2 participants