-
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] Unskip Lens smokescreen test #71986
[Lens] Unskip Lens smokescreen test #71986
Conversation
Pinging @elastic/kibana-app (Team:KibanaApp) |
@@ -36,22 +36,19 @@ export default function ({ getPageObjects, getService }) { | |||
await PageObjects.visualize.clickLensWidget(); | |||
await PageObjects.lens.goToTimeRange(); | |||
await PageObjects.lens.configureDimension({ | |||
dimension: | |||
'[data-test-subj="lnsXY_xDimensionPanel"] [data-test-subj="lns-empty-dimension"]', | |||
dimension: 'lnsXY_xDimensionPanel > lns-empty-dimension', |
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.
I recently read through the source code for our testSubjects
implementation, and found out that these are equivalent.
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.
Code LGTM, did not test locally.
This fixes the flakiness in the Lens smokescreen test in two places:
By adding a retry to work around this issue. #71256
I ran 42 successful tests in the flaky test runner: https://kibana-ci.elastic.co/job/kibana+flaky-test-suite-runner/592/
Fixes #71304