Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Testing] Enabling ContextMenu UITests from Xamarin.UITests to Appium - 2 #27405

Merged

Conversation

nivetha-nagalingam
Copy link
Contributor

Description of Change
This PR focuses on re-enabling and updating following tests from the Xamarin.UITest framework to be compatible with Appium. The tests, previously commented out, are reviewed, and modified to ensure they are functional with the Appium framework.

Additionally introduced the ActivateContextMenu and DismissContextMenu methods in the HelperExtensions.cs file, aimed at improving the cross-platform testing approach by centralizing platform-specific logic into helper methods. The following changes have been made:

ActivateContextMenu:

A unified method to trigger platform-specific context actions on an element, simplifying context menu interactions across different platforms.

  • Android: Performs a long press gesture on the element.
  • Windows: Simulates a right-click (touch-and-hold) on the element.
  • iOS: Executes a swipe from right to left on the element.
  • Catalyst: Executes a scroll from right to left on the element.

DismissContextMenu:

A single method to dismiss the context menu, eliminating the need for platform-specific code in test cases.

  • Android: Taps the back arrow to dismiss the menu.
  • Other platforms: Taps at coordinates (150, 150) to dismiss the menu.

TestCases

  • Bugzilla31330
  • Bugzilla57317
  • Bugzilla58833
  • Bugzilla58875
  • Bugzilla59580
  • Issue3653
  • Issue1658
  • Issue2414
  • Issue6258
  • Github1331
  • Issue4314

@dotnet-policy-service dotnet-policy-service bot added the community ✨ Community Contribution label Jan 28, 2025
@nivetha-nagalingam nivetha-nagalingam added area-testing Unit tests, device tests partner/syncfusion Issues / PR's with Syncfusion collaboration labels Jan 28, 2025
@nivetha-nagalingam nivetha-nagalingam changed the title [Testing] Enabling ContextMenu UITests from Xamarin.UITests to Appium [Testing] Enabling ContextMenu UITests from Xamarin.UITests to Appium - 2 Jan 28, 2025
@nivetha-nagalingam nivetha-nagalingam marked this pull request as ready for review January 28, 2025 16:25
@Copilot Copilot bot review requested due to automatic review settings January 28, 2025 16:25
@nivetha-nagalingam nivetha-nagalingam requested a review from a team as a code owner January 28, 2025 16:25
@jsuarezruiz
Copy link
Contributor

jsuarezruiz commented Jan 29, 2025

/azp run

Copy link

Azure Pipelines successfully started running 3 pipeline(s).

Copy link

Azure Pipelines successfully started running 3 pipeline(s).

@jsuarezruiz
Copy link
Contributor

/azp run

Copy link

Azure Pipelines successfully started running 3 pipeline(s).

@jfversluis jfversluis self-assigned this Feb 23, 2025
@jfversluis jfversluis dismissed jsuarezruiz’s stale review February 24, 2025 17:17

Feedback addressed

@jfversluis jfversluis merged commit 39fd179 into dotnet:main Feb 24, 2025
123 checks passed
@jfversluis jfversluis added this to the .NET 9 SR5 milestone Feb 24, 2025
tj-devel709 pushed a commit that referenced this pull request Mar 3, 2025
… - 2 (#27405)

* Migrated the ContextMenu related test cases

* Addressed the feedback and added one more case

* Removed snapshots and updated the some test cases

* Added the comments
@github-actions github-actions bot locked and limited conversation to collaborators Mar 27, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
area-testing Unit tests, device tests community ✨ Community Contribution partner/syncfusion Issues / PR's with Syncfusion collaboration
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants