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

[META] Test e2e optimization #10525

Open
6 tasks
Maymanaf opened this issue Jan 24, 2025 · 0 comments
Open
6 tasks

[META] Test e2e optimization #10525

Maymanaf opened this issue Jan 24, 2025 · 0 comments
Assignees
Labels
kind:performance Reduction of computing time or memory use kind:tests Writing unit / e2e and integration tests

Comments

@Maymanaf
Copy link
Contributor

Maymanaf commented Jan 24, 2025

Description and goal

Topic 1: #10584
The current modals, such as STDCM, have grown too large, with some exceeding 800 lines of code. This makes them difficult to understand and maintain. These modals need to be split into simplified versions to improve readability and modularity.

Topic 2: #10584
Page models with large numbers of locators can be challenging to navigate. Locators are currently declared both in the class and the constructor, leading to redundancy. This process should be optimized so that locators are declared only once.

Topic 3: #10585
Currently, the STDCM environment is restored only after all tests have finished. If a test run is canceled or encounters an issue, developers face difficulties restoring their previous environment. To address this, the STDCM environment should be reset in the afterEach hook instead of waiting for the global teardown.

Topic 4: #10586

The CI execution time for tests is high, taking approximately 8–9 minutes after stability fixes (#10423). To improve performance, tests should run on a single browser (defaulting to Chromium due to its lack of GPU issues and ability to handle map tests). For the merge queue, tests should run on two browsers(Firefox and Chrome)

Acceptance criteria

  • Simplify Modals: Separate and split large modals, such as STDCM, into smaller, more maintainable components.
  • Optimize Locator Management: Consolidate locator declarations to reduce redundancy and improve readability.
  • Environment Reset in afterEach: Move STDCM environment restoration from the global teardown to the afterEach hook.
  • Improve CI Performance: Update CI configuration to:
    • Run tests on a single browser (Chromium) by default.
    • Run tests on two browsers (Firefox and Chrome) for the merge queue.
@Maymanaf Maymanaf added kind:performance Reduction of computing time or memory use kind:tests Writing unit / e2e and integration tests labels Jan 24, 2025
@Maymanaf Maymanaf self-assigned this Jan 24, 2025
@Yohh Yohh self-assigned this Jan 29, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind:performance Reduction of computing time or memory use kind:tests Writing unit / e2e and integration tests
Projects
None yet
Development

No branches or pull requests

2 participants