8000 Replace @BeforeEach guard with @TestSetup in ClientApiV2Authorization… by Pepo48 · Pull Request #48364 · keycloak/keycloak · GitHub
[go: up one dir, main page]
Skip to content

Replace @BeforeEach guard with @TestSetup in ClientApiV2Authorization… - #48364

Merged
vmuzikar merged 1 commit into
keycloak:mainfrom
Pepo48:issue-48362
Apr 22, 2026
Merged

Replace @BeforeEach guard with @TestSetup in ClientApiV2Authorization…#48364
vmuzikar merged 1 commit into
keycloak:mainfrom
Pepo48:issue-48362

Conversation

@Pepo48
@Pepo48 Pepo48 commented Apr 22, 2026
Copy link
Copy Markdown
Contributor

…Test

Closes: #48362

@Pepo48
Replace @beforeeach guard with @testsetup in ClientApiV2Authorization…
8bf1bbd 
…Test

Closes: keycloak#48362

Signed-off-by: Peter Zaoral <pzaoral@redhat.com>
@Pepo48
Pepo48 requested review from a team as code owners April 22, 2026 11:41
@vmuzikar
vmuzikar enabled auto-merge (squash) April 22, 2026 12:30
@vmuzikar
vmuzikar merged commit 86e4f54 into keycloak:main Apr 22, 2026
86 checks passed
@keycloak-github-bot keycloak-github-bot Bot mentioned this pull request Apr 22, 2026
Closed
keycloak-github-bot[bot]
keycloak-github-bot Bot reviewed Apr 22, 2026
View reviewed changes
@keycloak-github-bot keycloak-github-bot Bot left a comment
Copy link
Copy Markdown

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

@keycloak-github-bot
keycloak-github-bot Bot commented Apr 22, 2026
Copy link
Copy Markdown

Unreported flaky test detected

If the 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.MultipleTabsLoginTest#testEmptyBaseUrl

Keycloak CI - Forms IT (chrome)

org.opentest4j.AssertionFailedError: Expected AppPage but was localhost (https://localhost:8543/auth/realms/test/login-actions/required-action?session_code=JmHSMcqlVwM68HtQ4N9xprBUF_vGKk3aK4FLEUm3n_s&execution=UPDATE_PASSWORD&client_id=empty-baseurl-client&tab_id=CrwUoK6sDrw&client_data=eyJydSI6Imh0dHBzOi8vbG9jYWxob3N0Ojg1NDMvYXV0aC9yZWFsbXMvbWFzdGVyL2FwcC9hdXRoIiwicnQiOiJjb2RlIn0) ==> expected: <true> but was: <false>
	at org.junit.jupiter.api.AssertionFailureBuilder.build(AssertionFailureBuilder.java:151)
	at org.junit.jupiter.api.AssertionFailureBuilder.buildAndThrow(AssertionFailureBuilder.java:132)
	at org.junit.jupiter.api.AssertTrue.failNotTrue(AssertTrue.java:63)
	at org.junit.jupiter.api.AssertTrue.assertTrue(AssertTrue.java:36)
...

Report flaky test

@stianst stianst mentioned this pull request Jun 1, 2026
Closed
sashyo pushed a commit to tide-foundation/keycloak-IGA that referenced this pull request Jul 20, 2026
@Pepo48 @sashyo
Replace @beforeeach guard with @testsetup in ClientApiV2Authorization…
b8c1051 
…Test (keycloak#48364)

Closes: keycloak#48362

Signed-off-by: Peter Zaoral <pzaoral@redhat.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@stianst stianst stianst approved these changes
@vmuzikar vmuzikar vmuzikar approved these changes
+1 more reviewer
@keycloak-github-bot keycloak-github-bot[bot] keycloak-github-bot[bot] left review comments
Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

flaky-test team/cloud-native

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Replace @BeforeEach guard with @TestSetup in ClientApiV2AuthorizationTest

3 participants

@Pepo48 @stianst @vmuzikar
0