-
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
[FieldFormats] Unskip url field formatter test #107651
Conversation
await settings.setFieldFormat('url'); | ||
await settings.controlChangeSave(); | ||
}); | ||
|
||
it('applied on dashboard', async () => { | ||
await common.navigateToApp('dashboard'); | ||
await dashboard.loadSavedDashboard('dashboard with everything'); | ||
await dashboard.loadSavedDashboard('dashboard with table'); |
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.
Looks like the test was flaky because this dashboard was flaky to load (too many panels?)
as a fix, I am adding a single-purpose dashboard as this test focuses on URL field format. This dashboard should be more stable
💚 Build SucceededMetrics [docs]
To update your PR or re-run it, just comment with: |
Pinging @elastic/kibana-app-services (Team:AppServices) |
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.
Looks good, nice to see a flaky test fixed
💚 Backport successful
This backport PR will be merged automatically after passing CI. |
Summary
Close #79463
flaky test runner: https://kibana-ci.elastic.co/job/kibana+flaky-test-suite-runner/1800/ - success in 42 runs
Part of working on improving ff tests: #59567