Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@lhanusov
Copy link
Contributor

@lhanusov lhanusov commented Jul 8, 2024

Closes #29700

Copy link
Contributor

@stianst stianst left a comment

Choose a reason for hiding this comment

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

Perhaps it would be an idea to add AbstractKeycloakTestServerSupplier with all the boiler plate and one abstract method like:

public abstract KeycloakTestServer getServer()

jonkoops
jonkoops previously approved these changes Jul 9, 2024
Copy link
Contributor

@jonkoops jonkoops left a comment

Choose a reason for hiding this comment

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

LGTM

@stianst stianst enabled auto-merge (squash) July 9, 2024 12:06
@stianst stianst merged commit 5fdb572 into keycloak:main Jul 9, 2024
Captain-P-Goldfish pushed a commit to Captain-P-Goldfish/keycloak that referenced this pull request Jul 17, 2024
stianst pushed a commit to stianst/keycloak that referenced this pull request Jul 25, 2024
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.

Testsuite PoC - Ability to configure server run-mode within the JUnit5 framework

3 participants