-
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
[Cloud Security] wait for action button to be present before clicking to fix flacky FTR tests #164054
Conversation
175fa4b
to
3b58d35
Compare
Pinging @elastic/kibana-cloud-security-posture (Team:Cloud Security) |
@@ -9,11 +9,10 @@ import expect from '@kbn/expect'; | |||
import { FtrProviderContext } from '../ftr_provider_context'; | |||
|
|||
// eslint-disable-next-line import/no-default-export | |||
export default ({ getPageObjects }: FtrProviderContext) => { | |||
export default ({ getPageObjects, getService }: FtrProviderContext) => { |
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.
getService
doesn't seem to be in use
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.
fixed
@elasticmachine merge upstream |
💚 Build Succeeded
Metrics [docs]
History
To update your PR or re-run it, just comment with: |
Summary
Fixing flaky FTR test where we need to navigate to the integrations page from our onboarding views. Confirmed in the flaky test runner that with the fix tests are not flaky anymore https://buildkite.com/elastic/kibana-flaky-test-suite-runner/builds/2901
fixes:
No integrations installed
prompt action button -install CNVM
: navigates to the CNVM integration installation page #163950