[Discover][Metrics] Add Scout E2E tests for metrics insights flyout and breakdown dimensions by lucaslopezf · Pull Request #254753 · elastic/kibana

lucaslopezf · 2026-02-24T15:16:40Z

Summary

This PR adds Scout UI (Playwright) e2e coverage for Metrics in Discover, focusing on the grid’s key interactions and the Insights flyout experience. It also adds minimal, stable data-test-subj hooks in the flyout components to make selectors reliable.

What’s included

Insights flyout e2e
- Open flyout from a metric card via View details
- Verify Overview tab content renders
- Verify ES|QL Query tab renders the query code block
- Validate Dimensions pagination (jump to last page, next/prev)
Metrics grid e2e
- Grid activation compatibility across ES|QL commands (TS, WHERE, LIMIT, SORT) and non-supported commands (FROM, STATS)
- Pagination across metric cards
- Search filtering and empty state
- Breakdown selector dimension changes keep grid functional
- Metric card actions menu exposes View details and Copy to dashboard; hover actions expose Explore

Test plan

Run Scout UI tests locally (examples):

# Start a server (choose one)
node scripts/scout start-server --arch stateful --domain classic
node scripts/scout start-server --arch serverless --domain observability_complete

# Run the suite
npx playwright test \
  src/platform/plugins/shared/discover/test/scout/ui/parallel_tests/metrics_experience/ \
  --config src/platform/plugins/shared/discover/test/scout/ui/parallel.playwright.config.ts \
  --project local --trace on

Checklist

Check the PR satisfies following conditions.

Reviewers should verify this PR satisfies this list as well.

Any text added follows EUI's writing guidelines, uses sentence case text and includes i18n support
Documentation was added for features that require explanation or tutorials
Unit or functional tests were updated or added to match the most common scenarios
If a plugin configuration key changed, check if it needs to be allowlisted in the cloud and added to the docker list
This was checked for breaking HTTP API changes, and any breaking changes have been approved by the breaking-change committee. The release_note:breaking label should be applied in these situations.
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.

Identify risks

Does this PR introduce any risks? For example, consider risks like hard to test bugs, performance regression, potential of data loss.

Describe the risk, its severity, and mitigation for each identified risk. Invite stakeholders and evaluate how to proceed before merging.

See some risk examples
...

lucaslopezf · 2026-02-25T11:45:19Z

/flaky scoutConfig:src/platform/plugins/shared/discover/test/scout/ui/metrics_experience_parallel.playwright.config.ts:30

kibanamachine · 2026-02-25T11:45:36Z

Flaky Test Runner

✅ Build triggered - kibana-flaky-test-suite-runner#10854

src/platform/plugins/shared/discover/test/scout/ui/metrics_experience_parallel.playwright.config.ts x30

kibanamachine · 2026-02-25T12:53:08Z

Flaky Test Runner Stats

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

[✅] src/platform/plugins/shared/discover/test/scout/ui/metrics_experience_parallel.playwright.config.ts: 30/30 tests passed.

see run history

…and-security-tag

lucaslopezf · 2026-03-04T09:16:30Z

/ci

lucaslopezf · 2026-03-04T09:17:02Z

/flaky scoutConfig:src/platform/plugins/shared/discover/test/scout/ui/parallel.playwright.config.ts:1

kibanamachine · 2026-03-04T09:17:19Z

Flaky Test Runner

✅ Build triggered - kibana-flaky-test-suite-runner#10944

src/platform/plugins/shared/discover/test/scout/ui/parallel.playwright.config.ts x1

kibanamachine · 2026-03-04T10:04:08Z

Flaky Test Runner Stats

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

[❌] src/platform/plugins/shared/discover/test/scout/ui/parallel.playwright.config.ts: 0/1 tests passed.

see run history

lucaslopezf · 2026-03-04T10:13:01Z

/flaky scoutConfig:src/platform/plugins/shared/discover/test/scout/ui/parallel.playwright.config.ts:1

kibanamachine · 2026-03-04T10:13:20Z

Flaky Test Runner

✅ Build triggered - kibana-flaky-test-suite-runner#10945

src/platform/plugins/shared/discover/test/scout/ui/parallel.playwright.config.ts x1

kibanamachine · 2026-03-04T10:59:57Z

Flaky Test Runner Stats

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

[✅] src/platform/plugins/shared/discover/test/scout/ui/parallel.playwright.config.ts: 1/1 tests passed.

see run history

lucaslopezf · 2026-03-04T11:59:26Z

/flaky scoutConfig:src/platform/plugins/shared/discover/test/scout/ui/parallel.playwright.config.ts:100

lucaslopezf · 2026-03-04T11:59:31Z

/ci

kibanamachine · 2026-03-04T11:59:44Z

Flaky Test Runner

✅ Build triggered - kibana-flaky-test-suite-runner#10946

src/platform/plugins/shared/discover/test/scout/ui/parallel.playwright.config.ts x100

kibanamachine · 2026-03-04T13:50:43Z

Flaky Test Runner Stats

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

[✅] src/platform/plugins/shared/discover/test/scout/ui/parallel.playwright.config.ts: 100/100 tests passed.

see run history

…and-security-tag

elasticmachine · 2026-03-04T14:39:07Z

Pinging @elastic/obs-exploration-team (Team:obs-exploration)

…and-security-tag

lucaslopezf · 2026-03-05T07:44:12Z

/flaky scoutConfig:src/platform/plugins/shared/discover/test/scout/ui/parallel.playwright.config.ts:30

kibanamachine · 2026-03-05T07:44:40Z

Flaky Test Runner

✅ Build triggered - kibana-flaky-test-suite-runner#10964

src/platform/plugins/shared/discover/test/scout/ui/parallel.playwright.config.ts x30

kibanamachine · 2026-03-05T08:42:31Z

Flaky Test Runner Stats

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

[❌] src/platform/plugins/shared/discover/test/scout/ui/parallel.playwright.config.ts: 29/30 tests passed.

see run history

…and-security-tag

lucaslopezf · 2026-03-05T08:46:28Z

/flaky scoutConfig:src/platform/plugins/shared/discover/test/scout/ui/parallel.playwright.config.ts:50

kibanamachine · 2026-03-05T08:46:51Z

Flaky Test Runner

✅ Build triggered - kibana-flaky-test-suite-runner#10966

src/platform/plugins/shared/discover/test/scout/ui/parallel.playwright.config.ts x50

kibanamachine · 2026-03-05T09:50:46Z

Flaky Test Runner Stats

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

[✅] src/platform/plugins/shared/discover/test/scout/ui/parallel.playwright.config.ts: 50/50 tests passed.

see run history

justinkambic

LGTM!

...gins/shared/discover/test/scout/ui/parallel_tests/metrics_experience/insights_flyout.spec.ts

…and-security-tag

lucaslopezf · 2026-03-06T10:29:14Z

/flaky scoutConfig:src/platform/plugins/shared/discover/test/scout/ui/parallel.playwright.config.ts:30

kibanamachine · 2026-03-06T10:29:31Z

Flaky Test Runner

✅ Build triggered - kibana-flaky-test-suite-runner#10993

src/platform/plugins/shared/discover/test/scout/ui/parallel.playwright.config.ts x30

…and-security-tag

kibanamachine · 2026-03-06T11:29:32Z

Flaky Test Runner Stats

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

[✅] src/platform/plugins/shared/discover/test/scout/ui/parallel.playwright.config.ts: 30/30 tests passed.

see run history

…and-security-tag

lucaslopezf · 2026-03-06T15:07:03Z

/flaky scoutConfig:src/platform/plugins/shared/discover/test/scout/ui/parallel.playwright.config.ts:30

kibanamachine · 2026-03-06T15:07:18Z

Flaky Test Runner

✅ Build triggered - kibana-flaky-test-suite-runner#10996

src/platform/plugins/shared/discover/test/scout/ui/parallel.playwright.config.ts x30

kibanamachine · 2026-03-06T16:12:40Z

Flaky Test Runner Stats

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

[✅] src/platform/plugins/shared/discover/test/scout/ui/parallel.playwright.config.ts: 30/30 tests passed.

see run history

…and-security-tag

elasticmachine · 2026-03-06T22:20:07Z

💛 Build succeeded, but was flaky

Buildkite Build
Commit: 387cfda

Failed CI Steps

Scout: [ platform / lens ] plugin

Test Failures

[job] [logs] Scout: [ platform / lens ] plugin / local-stateful-classic - Lens Convert to ES|QL - should update and reflect the visualization configuration after the conversion

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
`discover`	1.6MB	1.6MB	+284.0B

History

💔 Build #406101 failed 75960d9
💛 Build #405899 was flaky 3eef4a6
💔 Build #405739 failed 1b602bc
💛 Build #405683 was flaky 0f750b5
💔 Build #404554 failed 512a98e
💚 Build #404474 succeeded e2558fe

…nd breakdown dimensions (elastic#254753) ## Summary This PR adds Scout UI (Playwright) e2e coverage for **Metrics in Discover**, focusing on the grid’s key interactions and the Insights flyout experience. It also adds minimal, stable `data-test-subj` hooks in the flyout components to make selectors reliable. ## What’s included - **Insights flyout e2e** - Open flyout from a metric card via **View details** - Verify **Overview** tab content renders - Verify **ES|QL Query** tab renders the query code block - Validate **Dimensions pagination** (jump to last page, next/prev) - **Metrics grid e2e** - Grid activation compatibility across ES|QL commands (`TS`, `WHERE`, `LIMIT`, `SORT`) and non-supported commands (`FROM`, `STATS`) - **Pagination** across metric cards - **Search** filtering and empty state - **Breakdown selector** dimension changes keep grid functional - Metric card actions menu exposes **View details** and **Copy to dashboard**; hover actions expose **Explore** ## Test plan - Run Scout UI tests locally (examples): ```bash # Start a server (choose one) node scripts/scout start-server --arch stateful --domain classic node scripts/scout start-server --arch serverless --domain observability_complete # Run the suite npx playwright test \ src/platform/plugins/shared/discover/test/scout/ui/parallel_tests/metrics_experience/ \ --config src/platform/plugins/shared/discover/test/scout/ui/parallel.playwright.config.ts \ --project local --trace on ``` ### Checklist Check the PR satisfies following conditions. Reviewers should verify this PR satisfies this list as well. - [ ] Any text added follows [EUI's writing guidelines](https://elastic.github.io/eui/#/guidelines/writing), uses sentence case text and includes [i18n support](https://github.com/elastic/kibana/blob/main/src/platform/packages/shared/kbn-i18n/README.md) - [ ] [Documentation](https://www.elastic.co/guide/en/kibana/master/development-documentation.html) was added for features that require explanation or tutorials - [ ] [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 - [ ] If a plugin configuration key changed, check if it needs to be allowlisted in the cloud and added to the [docker list](https://github.com/elastic/kibana/blob/main/src/dev/build/tasks/os_packages/docker_generator/resources/base/bin/kibana-docker) - [ ] This was checked for breaking HTTP API changes, and any breaking changes have been approved by the breaking-change committee. The `release_note:breaking` label should be applied in these situations. - [ ] [Flaky Test Runner](https://ci-stats.kibana.dev/trigger_flaky_test_runner/1) 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](https://www.elastic.co/guide/en/kibana/master/contributing.html#kibana-release-notes-process) - [ ] Review the [backport guidelines](https://docs.google.com/document/d/1VyN5k91e5OVumlc0Gb9RPa3h1ewuPE705nRtioPiTvY/edit?usp=sharing) and apply applicable `backport:*` labels. ### Identify risks Does this PR introduce any risks? For example, consider risks like hard to test bugs, performance regression, potential of data loss. Describe the risk, its severity, and mitigation for each identified risk. Invite stakeholders and evaluate how to proceed before merging. - [ ] [See some risk examples](https://github.com/elastic/kibana/blob/main/RISK_MATRIX.mdx) - [ ] ...

lucaslopezf mentioned this pull request Feb 24, 2026

[Discover][Metrics] Add Scout E2E tests for metrics insights flyout and breakdown dimensions #254379

Closed

10 tasks

This was referenced Feb 26, 2026

[Discover] [Metrics] Add tier 2 journeys for Metrics in Discover E2E #255036

Merged

[Discover|Traces] E2E journeys in Discover: Traces experience enablement #254988

Merged

lucaslopezf force-pushed the 5194-metrics-discover-e2e-flyout-dimensions-and-security-tag branch from f4b6402 to 965813b Compare March 3, 2026 10:43

lucaslopezf added 3 commits March 3, 2026 14:53

Add flyout tests and security tag

691bd71

Use new testSubj for flyouts

d128aee

refactor and enhance metrics page object

9fc03b7

lucaslopezf force-pushed the 5194-metrics-discover-e2e-flyout-dimensions-and-security-tag branch from 965813b to 9fc03b7 Compare March 3, 2026 13:54

lucaslopezf and others added 3 commits March 4, 2026 10:01

Enhance code

1c67ed4

Remove security tag logic

e7f3705

Merge branch 'main' into 5194-metrics-discover-e2e-flyout-dimensions-…

55a8dc7

…and-security-tag

linting issues

ce1053c

Merge branch 'main' into 5194-metrics-discover-e2e-flyout-dimensions-…

e2558fe

…and-security-tag

lucaslopezf added release_note:skip Skip the PR/issue when compiling release notes backport:skip This PR does not require backporting Feature:Metrics in Discover labels Mar 4, 2026

lucaslopezf marked this pull request as ready for review March 4, 2026 14:39

lucaslopezf requested a review from a team as a code owner March 4, 2026 14:39

lucaslopezf added 2 commits March 4, 2026 17:02

Merge branch 'main' into 5194-metrics-discover-e2e-flyout-dimensions-…

512a98e

…and-security-tag

Merge branch 'main' into 5194-metrics-discover-e2e-flyout-dimensions-…

e21d605

…and-security-tag

Merge branch 'main' into 5194-metrics-discover-e2e-flyout-dimensions-…

d64ab6e

…and-security-tag

justinkambic approved these changes Mar 5, 2026

View reviewed changes

...gins/shared/discover/test/scout/ui/parallel_tests/metrics_experience/insights_flyout.spec.ts Show resolved Hide resolved

lucaslopezf and others added 2 commits March 6, 2026 09:48

Assert static content on flyout tabs

07c2566

Merge branch 'main' into 5194-metrics-discover-e2e-flyout-dimensions-…

0f750b5

…and-security-tag

Merge branch 'main' into 5194-metrics-discover-e2e-flyout-dimensions-…

1b602bc

…and-security-tag

lucaslopezf added 2 commits March 6, 2026 14:05

Merge branch 'main' into 5194-metrics-discover-e2e-flyout-dimensions-…

3eef4a6

…and-security-tag

Merge branch 'main' into 5194-metrics-discover-e2e-flyout-dimensions-…

85dcde9

…and-security-tag

lucaslopezf added 2 commits March 6, 2026 18:15

Merge branch 'main' into 5194-metrics-discover-e2e-flyout-dimensions-…

75960d9

…and-security-tag

Merge branch 'main' into 5194-metrics-discover-e2e-flyout-dimensions-…

387cfda

…and-security-tag

lucaslopezf merged commit fc0b7ce into elastic:main Mar 6, 2026
18 checks passed

Conversation

lucaslopezf commented Feb 24, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary

What’s included

Test plan

Checklist

Identify risks

Uh oh!

lucaslopezf commented Feb 25, 2026

Uh oh!

kibanamachine commented Feb 25, 2026

Flaky Test Runner

✅ Build triggered - kibana-flaky-test-suite-runner#10854

Uh oh!

kibanamachine commented Feb 25, 2026

Flaky Test Runner Stats

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

Uh oh!

lucaslopezf commented Mar 4, 2026

Uh oh!

lucaslopezf commented Mar 4, 2026

Uh oh!

kibanamachine commented Mar 4, 2026

Flaky Test Runner

✅ Build triggered - kibana-flaky-test-suite-runner#10944

Uh oh!

kibanamachine commented Mar 4, 2026

Flaky Test Runner Stats

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

Uh oh!

lucaslopezf commented Mar 4, 2026

Uh oh!

kibanamachine commented Mar 4, 2026

Flaky Test Runner

✅ Build triggered - kibana-flaky-test-suite-runner#10945

Uh oh!

kibanamachine commented Mar 4, 2026

Flaky Test Runner Stats

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

Uh oh!

lucaslopezf commented Mar 4, 2026

Uh oh!

lucaslopezf commented Mar 4, 2026

Uh oh!

kibanamachine commented Mar 4, 2026

Flaky Test Runner

✅ Build triggered - kibana-flaky-test-suite-runner#10946

Uh oh!

kibanamachine commented Mar 4, 2026

Flaky Test Runner Stats

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

Uh oh!

elasticmachine commented Mar 4, 2026

Uh oh!

lucaslopezf commented Mar 5, 2026

Uh oh!

kibanamachine commented Mar 5, 2026

Flaky Test Runner

✅ Build triggered - kibana-flaky-test-suite-runner#10964

Uh oh!

kibanamachine commented Mar 5, 2026

Flaky Test Runner Stats

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

Uh oh!

lucaslopezf commented Mar 5, 2026

Uh oh!

kibanamachine commented Mar 5, 2026

Flaky Test Runner

✅ Build triggered - kibana-flaky-test-suite-runner#10966

Uh oh!

kibanamachine commented Mar 5, 2026

Flaky Test Runner Stats

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

Uh oh!

justinkambic left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

lucaslopezf commented Mar 6, 2026

lucaslopezf commented Feb 24, 2026 •

edited

Loading