Skip to content

Conversation

findms
Copy link
Contributor

@findms findms commented Sep 26, 2025

TaskWPB-20289 [iOS/QA] Store all locators to a common file with common target to be available and used in Main+ UI tests

Issue

Please describe the issue.

Optional: add details about technical approach, solutions etc.

Optional: reference dependencies to other pull requests etc.

Testing

Describe how to test.

Optional: attachments like images, videos, etc.


Checklist

  • Title contains a reference JIRA issue number like [WPB-XXX].
  • Description is filled and free of optional paragraphs.
  • Adds/updates automated tests.

UI accessibility checklist

If your PR includes UI changes, please utilize this checklist:

  • Make sure you use the API for UI elements that support large fonts.
  • All colors are taken from WireDesign.ColorTheme or constructed using WireDesign.BaseColorPalette.
  • New UI elements have Accessibility strings for VoiceOver.

@findms findms changed the title Robust Locator handling chore: Robust Locator handling - WPB-20289 Sep 26, 2025
@findms findms added the QA label Sep 26, 2025
@findms findms changed the title chore: Robust Locator handling - WPB-20289 chore: Robust Locator placement - WPB-20289 Sep 26, 2025
@findms findms changed the title chore: Robust Locator placement - WPB-20289 chore: Common Locator placement - WPB-20289 Sep 26, 2025
@findms findms changed the title chore: Common Locator placement - WPB-20289 chore: common locator placement - WPB-20289 Sep 26, 2025
@findms findms requested review from johnxnguyen and netbe September 26, 2025 14:43
Copy link
Collaborator

@netbe netbe left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good left a suggestion

"account.hint" = "Doppeltippen, um Konto zu öffnen";
import Foundation

public enum Locators {
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

suggestion: I would move the locators to WireUI and have a specific target : Locators for it that way CI does not trigger running all tests when something changed in Foundation

otherwise we might want to ignore this file in the CI

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Updated accordingly.

@findms findms requested a review from netbe October 7, 2025 11:05
@findms findms marked this pull request as ready for review October 7, 2025 13:16
Copy link
Contributor

github-actions bot commented Oct 7, 2025

Test Results

2 085 tests   2 057 ✅  3m 3s ⏱️
  360 suites     28 💤
    3 files        0 ❌

Results for commit bc9cb02.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants