-
Notifications
You must be signed in to change notification settings - Fork 8k
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
[Entity Analytics] Simplify telemetry tests, add empty index check and logging #183915
Conversation
Flaky Test Runner Stats🟠 Some tests failed. - kibana-flaky-test-suite-runner#6059[❌] x-pack/test/security_solution_api_integration/test_suites/entity_analytics/risk_engine/trial_license_complete_tier/configs/ess.config.ts: 173/200 tests passed. |
@elasticmachine merge upstream |
Pinging @elastic/security-entity-analytics (Team:Entity Analytics) |
...tion/test_suites/entity_analytics/risk_engine/trial_license_complete_tier/telemetry_usage.ts
Show resolved
Hide resolved
💚 Build Succeeded
Metrics [docs]
History
To update your PR or re-run it, just comment with: cc @hop-dev |
Flaky Test Runner Stats🎉 All tests passed! - kibana-flaky-test-suite-runner#6066[✅] x-pack/test/security_solution_api_integration/test_suites/entity_analytics/risk_engine/trial_license_complete_tier/configs/serverless.config.ts: 100/100 tests passed. |
…d logging (elastic#183915) ## Summary Closes elastic#182881 Not sure what went wrong with this test btu I did find the structure slightly more complicated than needed, I have moved the beforeEach data loading into the test body and removed the describe blocks. I have also removed the afterEach, we really shouldnt need to delete the data twice 😓 I have done [a 200 flaky test run](https://buildkite.com/elastic/kibana-flaky-test-suite-runner/builds/6059#018f9b55-2231-4655-ab35-c6f26aa49720) 🟠 173 succeeded and 2 failed due to a bootstrap issue which caused the others not to run. --------- Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>
Summary
Closes #182881
Not sure what went wrong with this test btu I did find the structure slightly more complicated than needed, I have moved the beforeEach data loading into the test body and removed the describe blocks.
I have also removed the afterEach, we really shouldnt need to delete the data twice 😓
I have done a 200 flaky test run 🟠 173 succeeded and 2 failed due to a bootstrap issue which caused the others not to run.