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

feat: add tests improvements to pipeline CI #2407

Merged
merged 4 commits into from
Jun 27, 2024
Merged

Conversation

Cristhianzl
Copy link
Member

@Cristhianzl Cristhianzl commented Jun 27, 2024

Overview

This pull request introduces several improvements and changes aimed at enhancing the test structure and codebase maintainability. The key changes include breaking tests into smaller files to distribute across more shards, replacing XPath locators with test ID locators, and performing a general code refactor.

Changes

  • Break Tests into Smaller Files:

Tests have been split into multiple files to better fit across more shards. This change aims to improve test execution efficiency and parallelization.

  • Replace XPath Locators with Test ID Locators:

All locators previously using XPath have been updated to use data-testid attributes. This change improves the reliability and readability of the locators.

  • General Code Refactor:

The codebase has undergone a general refactor to improve maintainability and readability. This includes:
Renaming variables for better clarity.
Extracting reusable functions and components.
Improving code comments and documentation.

@Cristhianzl Cristhianzl requested a review from ogabrielluiz June 27, 2024 18:18
@Cristhianzl Cristhianzl self-assigned this Jun 27, 2024
@Cristhianzl Cristhianzl marked this pull request as ready for review June 27, 2024 18:36
@dosubot dosubot bot added the size:XL This PR changes 500-999 lines, ignoring generated files. label Jun 27, 2024
Copy link
Contributor

@ogabrielluiz ogabrielluiz left a comment

Choose a reason for hiding this comment

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

LGTM

@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Jun 27, 2024
@ogabrielluiz ogabrielluiz enabled auto-merge June 27, 2024 18:46
@ogabrielluiz ogabrielluiz added this pull request to the merge queue Jun 27, 2024
Copy link
Contributor

Cannot auto-update because of conflicts.

Merged via the queue into dev with commit 02ffc81 Jun 27, 2024
27 checks passed
@ogabrielluiz ogabrielluiz deleted the cz/tests-improvements branch June 27, 2024 19:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
lgtm This PR has been approved by a maintainer size:XL This PR changes 500-999 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants