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

[8.x] 10973 migrate flaky e2e tests to jest 2 (#211363) #211587

Merged
merged 2 commits into from
Feb 18, 2025

Conversation

alexreal1314
Copy link
Contributor

Backport

This will backport the following commits from main to 8.x:

Questions ?

Please refer to the Backport tool documentation

## Summary

This PR tries to fix the following issues - which are flaky FTR tests:

- #209529
- #201686
- #203680
- #178413 - test is still
skipped, added unit tests for rules table, header and counter
components.
- #193073
- #193616
- #191604
- #191593
- #191511
- #191474
- #191322
- #191128
- #191144
- #191027
- #190831
- #190779

There will be an RFC document which is going to be released to help us
better understand and decide which tests are more suitable to make as
E2E tests and which as unit tests.

### Checklist
- [x] [Unit or functional
tests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)
were updated or added to match the most common scenarios
- [x] [Flaky Test
Runner](https://ci-stats.kibana.dev/trigger_flaky_test_runner/1) was
used on any tests changed

### Closes
this PR closes the above mentioned issues in relation for this ticket -
elastic/security-team#10973

(cherry picked from commit dac3690)
@elasticmachine
Copy link
Contributor

💚 Build Succeeded

Metrics [docs]

Async chunks

Total size of all lazy-loaded chunks that will be downloaded as the user navigates the app

id before after diff
cloudSecurityPosture 485.5KB 485.7KB +216.0B

History

@alexreal1314 alexreal1314 merged commit 9d4ade1 into 8.x Feb 18, 2025
8 checks passed
@alexreal1314 alexreal1314 deleted the backport/8.x/pr-211363 branch February 18, 2025 19:56
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.

3 participants