Skip to content

[PM-2678] Adding IDs for Settings Page elements#2584

Merged
ifernandezdiaz merged 3 commits intomasterfrom
PM-2678/automationIDs-for-Settings-page-elements
Jun 23, 2023
Merged

[PM-2678] Adding IDs for Settings Page elements#2584
ifernandezdiaz merged 3 commits intomasterfrom
PM-2678/automationIDs-for-Settings-page-elements

Conversation

@ifernandezdiaz
Copy link
Contributor

Type of change

  • Bug fix
  • New feature development
  • Tech debt (refactoring, code cleanup, dependency upgrades, etc)
  • Build/deploy pipeline (DevOps)
  • Other

Objective

This PR adds more AutomationIDs that will help us to improve the quality of our Mobile Automation tests

Code changes

  • SettingsPage.xaml: Adding IDs for all the setting options + statuses
  • SettingsPageListItem.cs: Adding new AutomationId variable

Before you submit

  • Please check for formatting errors (dotnet format --verify-no-changes) (required)
  • Please add unit tests where it makes sense to do so (encouraged but not required)
  • If this change requires a documentation update - notify the documentation team
  • If this change has particular deployment requirements - notify the DevOps team

@ifernandezdiaz ifernandezdiaz requested a review from mpbw2 June 23, 2023 15:54
@ifernandezdiaz ifernandezdiaz marked this pull request as ready for review June 23, 2023 16:02
@ifernandezdiaz ifernandezdiaz merged commit 1407aa5 into master Jun 23, 2023
@ifernandezdiaz ifernandezdiaz deleted the PM-2678/automationIDs-for-Settings-page-elements branch June 23, 2023 16:31
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.

2 participants