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

Skip to content

Conversation

@lhanusov
Copy link
Contributor

@lhanusov lhanusov commented Jul 10, 2024

Closes #29702

@lhanusov lhanusov requested a review from a team as a code owner July 10, 2024 11:28
@lhanusov lhanusov marked this pull request as draft July 10, 2024 11:28
@lhanusov lhanusov self-assigned this Jul 10, 2024
@lhanusov lhanusov changed the title Draft: Testsuite PoC - distribution server run mode. Testsuite PoC - distribution server run mode. Jul 11, 2024
@lhanusov lhanusov marked this pull request as ready for review July 11, 2024 10:31
stianst
stianst previously approved these changes Jul 11, 2024
stianst
stianst previously approved these changes Jul 11, 2024
jonkoops
jonkoops previously approved these changes Jul 11, 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

Copy link
Contributor

@pedroigor pedroigor left a 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");
Copy link
Contributor

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?

Copy link
Contributor Author

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.

@stianst stianst merged commit 72fb64f into keycloak:main Jul 15, 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 - Distribution server supplier

5 participants