-
Notifications
You must be signed in to change notification settings - Fork 6.5k
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
Use new "Chrome for Testing" browser and driver. #22029
Conversation
1 flaky tests on run #8370 ↗︎
Details:
cypress/e2e/authentication_test.spec.ts • 1 flaky test • chrome
This comment has been generated by cypress-bot as a result of this project's GitHub integration settings. |
5862689
to
50890d0
Compare
f01d254
to
8d1f29b
Compare
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.
Looking good to me 👍
c5dcf5d
to
07e7913
Compare
0eadda6
to
01b709c
Compare
2123d4d
to
038afcb
Compare
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.
Unreported flaky test detected, please review
Unreported flaky test detectedIf the below flaky tests below are affected by the changes, please review and update the changes accordingly. Otherwise, a maintainer should report the flaky tests prior to merging the PR. org.keycloak.testsuite.forms.LoginTest#loginDifferentUserAfterDisabledUserThrownOutKeycloak CI - Forms IT (firefox)
|
038afcb
to
7309ee7
Compare
7309ee7
to
17d882d
Compare
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
17d882d
to
fdae4c2
Compare
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.
Approving as per @jonkoops's review
Closes #22028