KEMBAR78
[Testing] Fix Appium Swipe actions on Mac by jsuarezruiz · Pull Request #30436 · dotnet/maui · GitHub
Skip to content

Conversation

@jsuarezruiz
Copy link
Contributor

@jsuarezruiz jsuarezruiz commented Jul 4, 2025

Description of Change

Fix Appium Swipe actions on Mac. Enabled SwipeView Gallery tests on Mac.

Will allow to re-enable UITests using swipe gestures on macOS. For example: https://github.com/dotnet/maui/pull/30165/files#r2168920426

@Copilot Copilot AI review requested due to automatic review settings July 4, 2025 11:30
@jsuarezruiz jsuarezruiz added the area-testing Unit tests, device tests label Jul 4, 2025
@jsuarezruiz jsuarezruiz requested a review from a team as a code owner July 4, 2025 11:30
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

Fix Appium swipe actions on Mac and enable SwipeView gallery tests on MacCatalyst.

  • Replace deprecated macos: clickAndDrag and pressAndDrag scripts with W3C PointerInputDevice action sequences for swipe gestures.
  • Extend SwipeView UI tests to run on MacCatalyst by updating the compilation condition.

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
src/TestUtils/src/UITest.Appium/Actions/AppiumCatalystSwipeActions.cs Updated swipe implementations to use Selenium W3C Actions API
src/Controls/tests/TestCases.Shared.Tests/Tests/SwipeViewUITests.cs Enabled SwipeView UITests on MacCatalyst by adding MACCATALYST define
Comments suppressed due to low confidence (1)

src/Controls/tests/TestCases.Shared.Tests/Tests/SwipeViewUITests.cs:1

  • You enabled Shared.Tests for MacCatalyst, but have not added or updated the corresponding TestCases.HostApp UI page for MacCatalyst. Ensure that a matching UI test page (with unique AutomationIds) is present in TestCases.HostApp as required by the UI testing guidelines.
#if ANDROID || IOS || MACCATALYST

@rmarinho rmarinho merged commit d552bac into main Jul 8, 2025
129 checks passed
@rmarinho rmarinho deleted the fix-uitest-swipe-mac branch July 8, 2025 10:52
@github-actions github-actions bot locked and limited conversation to collaborators Aug 8, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants