Skip to content

[Security Solution] unskip event_rendered_view Cypress tests#245312

Merged
PhilippeOberti merged 3 commits intoelastic:mainfrom
PhilippeOberti:unskip-event-rendered-cypress
Dec 5, 2025
Merged

[Security Solution] unskip event_rendered_view Cypress tests#245312
PhilippeOberti merged 3 commits intoelastic:mainfrom
PhilippeOberti:unskip-event-rendered-cypress

Conversation

@PhilippeOberti
Copy link
Contributor

@PhilippeOberti PhilippeOberti commented Dec 4, 2025

Summary

A recent PR fixed an issue with the alerts table that was not properly switching between grid view and event rendered view.

This PR unskips the Cypress tests related to testing the event rendered view in our alerts page alerts table.

Screenshot 2025-12-04 at 4 42 06 PM

Flaky test runner: https://buildkite.com/elastic/kibana-flaky-test-suite-runner/builds/10002

Checklist

  • Flaky Test Runner was used on any tests changed
  • The PR description includes the appropriate Release Notes section, and the correct release_note:* label is applied per the guidelines
  • Review the backport guidelines and apply applicable backport:* labels.

#236608

@PhilippeOberti PhilippeOberti requested a review from a team as a code owner December 4, 2025 21:57
@PhilippeOberti PhilippeOberti added release_note:skip Skip the PR/issue when compiling release notes backport:skip This PR does not require backporting Team:Threat Hunting:Investigations Security Solution Threat Hunting Investigations Team v9.3.0 labels Dec 4, 2025
@elasticmachine
Copy link
Contributor

Pinging @elastic/security-threat-hunting-investigations (Team:Threat Hunting:Investigations)

@PhilippeOberti PhilippeOberti force-pushed the unskip-event-rendered-cypress branch from da4da5e to 7911510 Compare December 4, 2025 22:51
Copy link
Contributor

@NicholasPeretti NicholasPeretti left a comment

Choose a reason for hiding this comment

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

Nice! 🚀

@PhilippeOberti PhilippeOberti force-pushed the unskip-event-rendered-cypress branch from 7911510 to da719db Compare December 5, 2025 20:22
@PhilippeOberti PhilippeOberti requested a review from a team as a code owner December 5, 2025 20:22
@kibanamachine
Copy link
Contributor

Flaky Test Runner Stats

🎉 All tests passed! - kibana-flaky-test-suite-runner#10002

[✅] Security Solution Investigations - Cypress: 25/25 tests passed.

see run history

@elastic elastic deleted a comment from kibanamachine Dec 5, 2025
@PhilippeOberti PhilippeOberti enabled auto-merge (squash) December 5, 2025 21:05
@PhilippeOberti PhilippeOberti merged commit 3d4938f into elastic:main Dec 5, 2025
12 checks passed
@PhilippeOberti PhilippeOberti deleted the unskip-event-rendered-cypress branch December 5, 2025 22:54
@elasticmachine
Copy link
Contributor

💚 Build Succeeded

Metrics [docs]

✅ unchanged

History

@kibanamachine
Copy link
Contributor

Flaky Test Runner Stats

🟠 Some tests failed. - kibana-flaky-test-suite-runner#9999

[❌] Security Solution Investigations - Cypress: 18/25 tests passed.

see run history

JordanSh pushed a commit to JordanSh/kibana that referenced this pull request Dec 9, 2025
…#245312)

## Summary

A [recent PR](elastic#245253) fixed an
issue with the alerts table that was not properly switching between grid
view and event rendered view.

This PR unskips the Cypress tests related to testing the event rendered
view in our alerts page alerts table.

<img width="639" height="273" alt="Screenshot 2025-12-04 at 4 42 06 PM"
src="https://github.com/user-attachments/assets/510a6de3-ccbc-4268-96fd-2ced9c6cb45d"
/>

Flaky test runner:
https://buildkite.com/elastic/kibana-flaky-test-suite-runner/builds/10002

### Checklist

- [ ] [Flaky Test
Runner](https://ci-stats.kibana.dev/trigger_flaky_test_runner/1) was
used on any tests changed
- [x] The PR description includes the appropriate Release Notes section,
and the correct `release_note:*` label is applied per the
[guidelines](https://www.elastic.co/guide/en/kibana/master/contributing.html#kibana-release-notes-process)
- [x] Review the [backport
guidelines](https://docs.google.com/document/d/1VyN5k91e5OVumlc0Gb9RPa3h1ewuPE705nRtioPiTvY/edit?usp=sharing)
and apply applicable `backport:*` labels.

elastic#236608
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport:skip This PR does not require backporting release_note:skip Skip the PR/issue when compiling release notes Team:Threat Hunting:Investigations Security Solution Threat Hunting Investigations Team v9.3.0

4 participants