Skip to content

[Lens] Init scout and add should display ES|QL conversion modal test#249819

Merged
mariairiartef merged 21 commits intoelastic:mainfrom
mariairiartef:lens/init-scout
Feb 4, 2026
Merged

[Lens] Init scout and add should display ES|QL conversion modal test#249819
mariairiartef merged 21 commits intoelastic:mainfrom
mariairiartef:lens/init-scout

Conversation

@mariairiartef
Copy link
Copy Markdown
Contributor

@mariairiartef mariairiartef commented Jan 21, 2026

Summary

Close #250383
Close #250384

  • Init scout in Lens
  • Add switchToEditMode and openInLineEditor to the dashboard app
  • Create lens app and add the following methods:
    • getConvertToEsqlButton
    • getConvertToEsqModal
    • getConvertToEsqModalConfirmButton
  • Add should display ES|QL conversion modal test

For the test, we have used the LOGSTASH ES archive.

Note

The complete conversion flow will be tested in a separate PR, once #248078 is merged.

How to run the tests

Run server:

node scripts/scout.js start-server --stateful

In a separate terminal, run tests:

npx playwright test --project local --grep @ess --config x-pack/platform/plugins/shared/lens/test/scout/ui/  --ui

Checklist

@mariairiartef mariairiartef force-pushed the lens/init-scout branch 2 times, most recently from ddd0009 to d500ee0 Compare January 26, 2026 11:13
@mariairiartef mariairiartef changed the title [Lens] Init scout [Lens] Init scout and add should display ES|QL conversion modal test Jan 26, 2026
@mariairiartef mariairiartef self-assigned this Jan 26, 2026
@mariairiartef mariairiartef added Team:Visualizations Team label for Lens, elastic-charts, Graph, legacy editors (TSVB, Visualize, Timelion) t// release_note:skip Skip the PR/issue when compiling release notes Feature:Lens backport:skip This PR does not require backporting labels Jan 26, 2026
@mariairiartef mariairiartef marked this pull request as ready for review January 26, 2026 14:20
@mariairiartef mariairiartef requested review from a team as code owners January 26, 2026 14:20
@elasticmachine
Copy link
Copy Markdown
Contributor

Pinging @elastic/kibana-visualizations (Team:Visualizations)

Copy link
Copy Markdown
Contributor

@nickofthyme nickofthyme left a comment

Choose a reason for hiding this comment

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

Very nice! Super excited for this 🎉


import type { ScoutPage } from '..';

export class LensApp {
Copy link
Copy Markdown
Member

@csr csr Feb 3, 2026

Choose a reason for hiding this comment

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

Hey! Just a heads up, my PR #249233 is also adding a Lens object. Expect to see some changes in this page object if my PR is merged after yours (I'm hoping that's the case so you don't have to go through merge conflict resolution).

- Run `node ../scripts/functional_test_runner.js --config ./test/api_integration/config.ts --grep=Lens`
- Performance journeys:
- Run `node scripts/run_performance --journey-path x-pack/performance/journeys_e2e/data_stress_test_lens`
- Run `node scripts/run_performance --journey-path x-pack/performance/journeys_e2e/data_stress_test_lens_http2`
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I don't see a journey being added in this PR. Was it intentional?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I think it was added automatically, 🤔 removed here

Comment on lines +10 to +12
export default createPlaywrightConfig({
testDir: './tests',
});
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Since Lens support Spaces, is it possible to consider running tests in space isolation and in parallel? It will be ~30-40% faster based on current metrics.


// Unskip after: https://github.com/elastic/kibana-team/issues/2740
// eslint-disable-next-line playwright/no-skipped-test
test.describe.skip('Lens ES|QL', { tag: ['@ess'] }, () => {
Copy link
Copy Markdown
Contributor

@dmlemeshko dmlemeshko Feb 3, 2026

Choose a reason for hiding this comment

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

I guess there is no point in flaky-test-runner since test suite is skipped...

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I unskipped the test and run the flaky-test-runner. The stats are here, thanks!

Copy link
Copy Markdown
Contributor

@dmlemeshko dmlemeshko left a comment

Choose a reason for hiding this comment

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

LGTM, left few nits

@kibanamachine
Copy link
Copy Markdown
Contributor

Flaky Test Runner Stats

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

[✅] x-pack/platform/plugins/shared/lens/test/scout/ui/playwright.config.ts: 25/25 tests passed.

see run history

@kibanamachine
Copy link
Copy Markdown
Contributor

Flaky Test Runner Stats

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

[✅] x-pack/platform/plugins/shared/lens/test/scout/ui/playwright.config.ts: 15/15 tests passed.

see run history

@mariairiartef mariairiartef enabled auto-merge (squash) February 4, 2026 12:39
@elasticmachine
Copy link
Copy Markdown
Contributor

elasticmachine commented Feb 4, 2026

⏳ Build in-progress, with failures

Failed CI Steps

Test Failures

  • [job] [logs] FTR Configs #53 / Endpoint plugin @ess @serverless @skipInServerlessMKI When attempting to call an endpoint api "before all" hook in "@ess @serverless @skipInServerlessMKI When attempting to call an endpoint api"
  • [job] [logs] Scout: [ platform / streams_app-serverless-oblt ] plugin / serverless-oblt - Stream data processing - data sources management - should allow adding a new kql data source
  • [job] [logs] Scout: [ platform / streams_app-serverless-oblt ] plugin / serverless-oblt - Stream data processing - data sources management - should allow adding a new kql data source
  • [job] [logs] Scout: [ platform / streams_app-stateful ] plugin / should allow adding a new kql data source
  • [job] [logs] Scout: [ platform / streams_app-serverless-oblt ] plugin / should allow adding a new kql data source
  • [job] [logs] Scout: [ platform / streams_app-stateful ] plugin / should allow adding a new kql data source
  • [job] [logs] Scout: [ platform / streams_app-serverless-oblt ] plugin / should allow adding a new kql data source
  • [job] [logs] Scout: [ platform / streams_app-stateful ] plugin / stateful - Stream data processing - data sources management - should allow adding a new kql data source
  • [job] [logs] Scout: [ platform / streams_app-stateful ] plugin / stateful - Stream data processing - data sources management - should allow adding a new kql data source

History

cc @mariairiartef

@mariairiartef mariairiartef merged commit b870e77 into elastic:main Feb 4, 2026
16 checks passed
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 Feature:Lens release_note:skip Skip the PR/issue when compiling release notes Team:Visualizations Team label for Lens, elastic-charts, Graph, legacy editors (TSVB, Visualize, Timelion) t// test:scout v9.4.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Lens] [ES|QL] Add functional test for ES|QL conversion modal display Init Scout in the Lens plugin

7 participants