-
Notifications
You must be signed in to change notification settings - Fork 7.9k
Testsuite PoC - distribution server run mode. #31186
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
Conversation
...amework/src/main/java/org/keycloak/test/framework/server/DistributionKeycloakTestServer.java
Outdated
Show resolved
Hide resolved
...amework/src/main/java/org/keycloak/test/framework/server/DistributionKeycloakTestServer.java
Outdated
Show resolved
Hide resolved
jonkoops
left a comment
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
Signed-off-by: Lukas Hanusovsky <[email protected]>
pedroigor
left a comment
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.
@lhanusov LGTM as a start and very nice we are able to reuse the implementation from keycloak-junit5 module.
It should be quite trivial to add support for running using a container too.
| keycloak.setEnvVar("KC_BOOTSTRAP_ADMIN_PASSWORD", serverConfig.adminUserPassword().get()); | ||
|
|
||
| List<String> rawOptions = new LinkedList<>(); | ||
| rawOptions.add("start-dev"); |
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.
Do we want to add options from org.keycloak.test.framework.server.KeycloakTestServerConfig#options now?
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.
@pedroigor It is copied from the EmbeddedServer setup, so not necessary to have it right now, but will be used in the future.
Signed-off-by: Lukas Hanusovsky <[email protected]>
Closes #29702