From 9f541810e43885f1b703152ed67f74ea86ef6e95 Mon Sep 17 00:00:00 2001 From: Peter Zaoral Date: Thu, 5 Dec 2024 17:59:23 +0100 Subject: [PATCH] [Windows] Stabilizing QuarkusPropertiesDistTest Closes: #35496 Signed-off-by: Peter Zaoral --- .../keycloak/it/cli/dist/QuarkusPropertiesDistTest.java | 9 +++++---- 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/quarkus/tests/integration/src/test/java/org/keycloak/it/cli/dist/QuarkusPropertiesDistTest.java b/quarkus/tests/integration/src/test/java/org/keycloak/it/cli/dist/QuarkusPropertiesDistTest.java index 530678e28c11..5ecc227a1a72 100644 --- a/quarkus/tests/integration/src/test/java/org/keycloak/it/cli/dist/QuarkusPropertiesDistTest.java +++ b/quarkus/tests/integration/src/test/java/org/keycloak/it/cli/dist/QuarkusPropertiesDistTest.java @@ -26,6 +26,7 @@ import org.junit.jupiter.api.Test; import org.junit.jupiter.api.TestMethodOrder; import org.junit.jupiter.api.condition.DisabledOnOs; +import org.junit.jupiter.api.condition.EnabledOnOs; import org.junit.jupiter.api.condition.OS; import org.keycloak.it.junit5.extension.BeforeStartDistribution; import org.keycloak.it.junit5.extension.CLIResult; @@ -78,7 +79,7 @@ void testIgnoreQuarkusSystemPropertiesAtStart(CLIResult cliResult) { @Test @Launch({"-Dquarkus.log.handler.console.\\\"console-2\\\".enable=false", "start", "--http-enabled=true", "--hostname-strict=false"}) - @DisabledOnOs(value = { OS.LINUX, OS.MAC }, disabledReason = "Different handling of quotes within arguments on Windows") + @EnabledOnOs(value = { OS.WINDOWS }, disabledReason = "Different handling of quotes within arguments on Windows") @Order(3) void testIgnoreQuarkusSystemPropertiesAtStartWin(CLIResult cliResult) { cliResult.assertMessage("Keycloak is the best"); @@ -95,7 +96,7 @@ void testIgnoreQuarkusSystemPropertyAtBuild(CLIResult cliResult) { @Test @Launch({"-Dquarkus.log.handler.console.\\\"console-2\\\".enable=false", "build"}) - @DisabledOnOs(value = { OS.LINUX, OS.MAC }, disabledReason = "Different handling of quotes within arguments on Windows") + @EnabledOnOs(value = { OS.WINDOWS }, disabledReason = "Different handling of quotes within arguments on Windows") @Order(4) void testIgnoreQuarkusSystemPropertyAtBuildWin(CLIResult cliResult) { cliResult.assertMessage("Keycloak is the best"); @@ -175,7 +176,7 @@ void testInvalidSmallRyeKeyStorePathProperty(CLIResult cliResult) { @Test @Launch({ "start", "--http-enabled=true", "--hostname-strict=false", "--config-keystore=C:\\invalid\\path", "--config-keystore-password=secret" }) - @DisabledOnOs(value = { OS.LINUX, OS.MAC }, disabledReason = "Windows uses a different path separator.") + @EnabledOnOs(value = { OS.WINDOWS }, disabledReason = "Windows uses a different path separator.") @Order(11) void testInvalidSmallRyeKeyStorePathPropertyWin(CLIResult cliResult) { cliResult.assertError("java.lang.IllegalArgumentException: config-keystore path does not exist: C:\\invalid\\path"); @@ -204,7 +205,7 @@ void testHttpCertsPathTransformer(CLIResult cliResult) { @Test @BeforeStartDistribution(ForceRebuild.class) - @DisabledOnOs(value = { OS.LINUX, OS.MAC }, disabledReason = "Windows uses a different path separator.") + @EnabledOnOs(value = { OS.WINDOWS }, disabledReason = "Windows uses a different path separator.") @Launch({ "start", "--http-enabled=true", "--hostname-strict=false", "--https-certificate-file=C:\\tmp\\kc\\bin\\..\\conf/server.crt.pem", "--https-certificate-key-file=C:\\tmp\\kc\\bin\\..\\conf/server.key.pem" })