Skip to content

[9.2] [Security Solution] Delete skipped enrichments.cy.ts Cypress test and orphaned code (#252494)#252584

Merged
kibanamachine merged 1 commit intoelastic:9.2from
kibanamachine:backport/9.2/pr-252494
Feb 10, 2026
Merged

[9.2] [Security Solution] Delete skipped enrichments.cy.ts Cypress test and orphaned code (#252494)#252584
kibanamachine merged 1 commit intoelastic:9.2from
kibanamachine:backport/9.2/pr-252494

Conversation

@kibanamachine
Copy link
Contributor

Backport

This will backport the following commits from main to 9.2:

Questions ?

Please refer to the Backport tool documentation

… orphaned code (elastic#252494)

## Summary

Deletes the `enrichments.cy.ts` Cypress test that has been skipped since
March 2024 (elastic#176965) and removes all orphaned code that was only used by
this test.

**Why delete instead of fix:**
- The test has been skipped for ~2 years with no progress toward
re-enabling
- The key selector (`EnrichedDataRow`) no longer exists in the
application code — the alert flyout enrichment UI was redesigned
- The legacy risk engine code was removed in elastic#201810, further
invalidating the test's assumptions
- Server-side alert enrichment with risk scores is already well-covered
by API integration tests across multiple rule types (custom query, ESQL,
EQL, threshold, new terms, ML)

**Deleted:**
- `cypress/e2e/entity_analytics/enrichments.cy.ts` — the skipped test
- `es_archives/risk_scores_new_updated/` — es_archive only used by this
test

**Cleaned up orphaned selectors:**
- `ENRICHED_DATA_ROW` from `cypress/screens/alerts_details.ts`
- `HOST_RISK_HEADER_COLUMN`, `USER_RISK_HEADER_COLUMN`,
`HOST_RISK_COLUMN`, `USER_RISK_COLUMN`, `ACTION_COLUMN` from
`cypress/screens/alerts.ts`

(cherry picked from commit 9a7adf3)
@kibanamachine kibanamachine added the backport This PR is a backport of another PR label Feb 10, 2026
@kibanamachine kibanamachine enabled auto-merge (squash) February 10, 2026 17:04
@kibanamachine kibanamachine merged commit a7631d0 into elastic:9.2 Feb 10, 2026
17 checks passed
@elasticmachine
Copy link
Contributor

💚 Build Succeeded

Metrics [docs]

✅ unchanged

cc @MadameSheema

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

Labels

backport This PR is a backport of another PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants