[Lens] Init scout and add should display ES|QL conversion modal test#249819
[Lens] Init scout and add should display ES|QL conversion modal test#249819mariairiartef merged 21 commits intoelastic:mainfrom
should display ES|QL conversion modal test#249819Conversation
d1fce81 to
b0d1874
Compare
ddd0009 to
d500ee0
Compare
d500ee0 to
a0a41c3
Compare
should display ES|QL conversion modal test
d905d03 to
9c505ea
Compare
x-pack/platform/plugins/shared/lens/test/scout/ui/fixtures/esql_conversion_dashboard.json
Show resolved
Hide resolved
|
Pinging @elastic/kibana-visualizations (Team:Visualizations) |
e07355b to
2b9c81e
Compare
2b9c81e to
e93e84f
Compare
src/platform/packages/shared/kbn-scout/src/playwright/page_objects/dashboard_app.ts
Outdated
Show resolved
Hide resolved
src/platform/packages/shared/kbn-scout/src/playwright/page_objects/lens_app.ts
Show resolved
Hide resolved
nickofthyme
left a comment
There was a problem hiding this comment.
Very nice! Super excited for this 🎉
src/platform/packages/shared/kbn-scout/src/playwright/page_objects/dashboard_app.ts
Outdated
Show resolved
Hide resolved
...tform/plugins/shared/lens/public/app_plugin/shared/edit_on_the_fly/convert_to_esql_modal.tsx
Outdated
Show resolved
Hide resolved
|
|
||
| import type { ScoutPage } from '..'; | ||
|
|
||
| export class LensApp { |
There was a problem hiding this comment.
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` |
There was a problem hiding this comment.
I don't see a journey being added in this PR. Was it intentional?
There was a problem hiding this comment.
I think it was added automatically, 🤔 removed here
| export default createPlaywrightConfig({ | ||
| testDir: './tests', | ||
| }); |
There was a problem hiding this comment.
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'] }, () => { |
There was a problem hiding this comment.
I guess there is no point in flaky-test-runner since test suite is skipped...
There was a problem hiding this comment.
I unskipped the test and run the flaky-test-runner. The stats are here, thanks!
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. |
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. |
⏳ Build in-progress, with failures
Failed CI Steps
Test Failures
History
|
Summary
Close #250383
Close #250384
switchToEditModeandopenInLineEditorto the dashboard appgetConvertToEsqlButtongetConvertToEsqModalgetConvertToEsqModalConfirmButtonshould display ES|QL conversion modaltestFor 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:
In a separate terminal, run tests:
Checklist
release_note:*label is applied per the guidelinesbackport:*labels.