-
Notifications
You must be signed in to change notification settings - Fork 8.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[Lens]Test color syncing #86906
[Lens]Test color syncing #86906
Conversation
Jenkins, test this |
Pinging @elastic/kibana-app (Team:KibanaApp) |
@elasticmachine merge upstream |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tests LGTM, but left a potential change
await dashboardAddPanel.clickCreateNewLink(); | ||
await dashboardAddPanel.clickVisType('lens'); | ||
await PageObjects.header.waitUntilLoadingHasFinished(); | ||
await PageObjects.lens.goToTimeRange(); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is the time range cleared in between the previous page load and this one? I would have expected it to persist.
await PageObjects.lens.save('vis2', true, true); | ||
await PageObjects.header.waitUntilLoadingHasFinished(); | ||
await elasticChart.setNewChartUiDebugFlag(true); | ||
await PageObjects.lens.goToTimeRange(); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Same question here
💚 Build SucceededMetrics [docs]
History
To update your PR or re-run it, just comment with: |
Good catch @wylieconlon - cleaned that up |
Adds rudimentary functional test for the color syncing setting on dashboards.