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

test: Remove testHelpers file #13044

Draft
wants to merge 4 commits into
base: main
Choose a base branch
from
Draft

Conversation

cortisiko
Copy link
Member

@cortisiko cortisiko commented Jan 17, 2025

Description

The helpers.js file was a bit of a dumping ground for all test actions: Selecting elements, asserting elements, and using utility methods like delays, deep linking, launching the app, and interacting with elements. We did some work last year to separate the various types of logic.

Locating elements - Matchers.js
Interacting with elements - Guestures.js
Asserting elements - Assertions.js

They all live in the e2e/utils folder.

The purpose of this PR is to remove all instances where the helper's file is used and to move the remaining utility methods: launch the app, open deep link, and delay to the main utility class. With this PR merged, we can avoid engineers using anti-patterns within the e2e framework. Now, the codebase will strictly abide by our test guidelines.

Because the helpers.js file existed in nearly all of our page objects, the PR might seem significant, but it is not. Many changes are either 2-3 lines in each file. I will do my self-review so that others can understand what changed.

Related issues

Fixes:

Manual testing steps

  1. Go to this page...

Screenshots/Recordings

Before

After

Pre-merge author checklist

Pre-merge reviewer checklist

  • I've manually tested the PR (e.g. pull and build branch, run the app, test code being changed).
  • I confirm that this PR addresses all acceptance criteria described in the ticket it closes and includes the necessary testing evidence such as recordings and or screenshots.

@cortisiko cortisiko requested review from a team as code owners January 17, 2025 00:23
@cortisiko cortisiko marked this pull request as draft January 17, 2025 00:23
@cortisiko cortisiko changed the title gut testHelpers file. test: Remove testHelpers file Jan 17, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants