Skip to content
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

GH Actions -> archive surefire reports (#21913). #21915

Merged
merged 1 commit into from
Aug 31, 2023

Conversation

lhanusov
Copy link
Contributor

Closes #21913

@lhanusov lhanusov requested a review from a team as a code owner July 24, 2023 14:53
@lhanusov lhanusov self-assigned this Jul 24, 2023
@lhanusov lhanusov added this to the 22.0.2 milestone Jul 24, 2023
@lhanusov lhanusov added area/testsuite Indicates an issue on the Testsuite area team/continuous-testing labels Jul 24, 2023
@cypress
Copy link

cypress bot commented Jul 24, 2023

1 flaky test on run #8752 ↗︎

0 527 48 0 Flakiness 1

Details:

Merge 790aa9f into 4cd34f8...
Project: Keycloak Admin UI Commit: cac6ddfaf0 ℹ️
Status: Passed Duration: 50:13 💡
Started: Aug 31, 2023 10:16 AM Ended: Aug 31, 2023 11:06 AM
Flakiness  cypress/e2e/authentication_test.spec.ts • 1 flaky test • chrome

View Output Video

Test Artifacts
Authentication test > should add a condition Test Replay Output Screenshots

This comment has been generated by cypress-bot as a result of this project's GitHub integration settings.

.github/workflows/ci.yml Outdated Show resolved Hide resolved
.github/workflows/ci.yml Outdated Show resolved Hide resolved
.github/workflows/ci.yml Outdated Show resolved Hide resolved
@lhanusov lhanusov force-pushed the lhanusov/main/21913 branch 2 times, most recently from 459aa73 to ffc8d00 Compare August 1, 2023 13:29
@lhanusov
Copy link
Contributor Author

lhanusov commented Aug 2, 2023

@stianst all comments addressed, condition fixed, @miquelsi can you also review it? thanks

Copy link

@ghost ghost left a 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

@ghost
Copy link

ghost commented Aug 28, 2023

Unreported flaky test detected

If 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.RegisterWithUserProfileTest#testRegisterUserSuccess_attributeNotRequiredAndSelectedByScopeCanBeSet

Keycloak CI - Forms IT (chrome)

java.lang.IllegalArgumentException: No enum constant org.keycloak.testsuite.pages.AppPage.RequestType.
	at java.base/java.lang.Enum.valueOf(Enum.java:273)
	at org.keycloak.testsuite.pages.AppPage$RequestType.valueOf(AppPage.java:56)
	at org.keycloak.testsuite.pages.AppPage.getRequestType(AppPage.java:49)
	at jdk.internal.reflect.GeneratedMethodAccessor552.invoke(Unknown Source)
...

Report flaky test

@lhanusov lhanusov force-pushed the lhanusov/main/21913 branch 8 times, most recently from b545ed4 to faf05ef Compare August 29, 2023 14:46
@stianst stianst enabled auto-merge (squash) August 30, 2023 11:18
Copy link

@ghost ghost left a 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

@ghost
Copy link

ghost commented Aug 30, 2023

Unreported flaky test detected

If 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.ui.account2.MyResourcesTest#shouldAllowRequestToShare

Keycloak CI - Account Console IT (firefox)

org.openqa.selenium.UnhandledAlertException: 
Dismissed user prompt dialog: failed to initialize keycloak: 
Build info: version: '3.14.0', revision: 'aacccce0', time: '2018-08-02T20:19:58.91Z'
System info: host: 'fv-az344-249', ip: '10.1.0.69', os.name: 'Linux', os.arch: 'amd64', os.version: '5.15.0-1041-azure', java.version: '17.0.8'
Driver info: org.openqa.selenium.firefox.FirefoxDriver
...

Report flaky test

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/testsuite Indicates an issue on the Testsuite area flaky-test team/continuous-testing
Projects
None yet
Development

Successfully merging this pull request may close these issues.

GH Actions - Surefire reports artifact upload
3 participants