Skip to content

Migrate Organization Authentication tests into new testsuite#50723

Merged
ahus1 merged 1 commit into
keycloak:mainfrom
vramik:48179-migrate-org-authn-tests
Jul 16, 2026
Merged

Migrate Organization Authentication tests into new testsuite#50723
ahus1 merged 1 commit into
keycloak:mainfrom
vramik:48179-migrate-org-authn-tests

Conversation

@vramik

@vramik vramik commented Jul 9, 2026

Copy link
Copy Markdown
Contributor

No description provided.

@vramik
vramik force-pushed the 48179-migrate-org-authn-tests branch from 29027f5 to 5cd5cff Compare July 15, 2026 13:31
@vramik vramik changed the title 48179 migrate org authn tests Migrate Organization Authentication tests into new testsuite Jul 15, 2026
@vramik
vramik marked this pull request as ready for review July 15, 2026 13:32
@vramik
vramik requested review from a team as code owners July 15, 2026 13:32
Copilot AI review requested due to automatic review settings July 15, 2026 13:32

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Migrates organization authentication coverage from the legacy Arquillian suite to the new test framework.

Changes:

  • Removes the legacy authentication test.
  • Recreates its scenarios using JUnit 5 and new framework APIs.
  • Adds shared organization fixtures and login page helpers.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 7 comments.

File Description
testsuite/.../OrganizationAuthenticationTest.java Removes the legacy test.
tests/.../OrganizationAuthenticationTest.java Adds migrated authentication tests.
tests/.../AbstractOrganizationTest.java Adds UI fixtures, helper logic, and cleanup changes.
test-framework/.../LoginPage.java Adds organization-login page interactions.

Comment thread test-framework/ui/src/main/java/org/keycloak/testframework/ui/page/LoginPage.java Outdated
Closes keycloak#48179

Signed-off-by: vramik <vramik@redhat.com>
Copilot AI review requested due to automatic review settings July 15, 2026 18:57
@vramik
vramik force-pushed the 48179-migrate-org-authn-tests branch from 5cd5cff to e9b3f3f Compare July 15, 2026 18:57

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.

@vramik
vramik requested a review from sguilhen July 15, 2026 19:11
@sguilhen

Copy link
Copy Markdown
Contributor

Thanks @vramik , I've approved the PR. I think the CI failure is unrelated

@ahus1
ahus1 merged commit b8a99e0 into keycloak:main Jul 16, 2026
147 of 149 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants