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
[APM] e2e tests fix (Comparison feature flag) #137209
Conversation
@elasticmachine merge upstream |
@@ -8,13 +8,14 @@ | |||
declare namespace Cypress { | |||
interface Chainable { | |||
loginAsViewerUser(): void; | |||
loginAsEditorUser(): void; | |||
loginAsEditorUser(): Promise<Cypress.Response<any>>; |
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 expect also the other loginAs*
functions to return a promise. Do you think it make sense to update them?
Pinging @elastic/apm-ui (Team:apm) |
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.
Thanks for fixing it. Just left a comment
}, | ||
}); | ||
} | ||
); |
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.
Much better to do it via the api 👍 It felt like we were writing e2e tests for Advanced settings before.
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.
lgtm
buildkite test this |
💛 Build succeeded, but was flaky
Failed CI StepsMetrics [docs]
History
To update your PR or re-run it, just comment with: |
Fixed the tests by removing the dependency on each other state. I also created a new
command
to update values in the Advanced settings, we no longer need to navigate there to update a value.