Added UI_Tests to check functionality for required action authenticator

conciso · Oct 2, 2024 · e0f0d25 · e0f0d25
1 parent 94c5e5c
commit e0f0d25
Show file tree

Hide file tree

Showing 2 changed files with 37 additions and 24 deletions.
diff --git a/...ava/de/conciso/keycloak/authentication/required_action/RequiredActionAuthenticatorIT.java b/...ava/de/conciso/keycloak/authentication/required_action/RequiredActionAuthenticatorIT.java
@@ -6,6 +6,7 @@
 import com.microsoft.playwright.Playwright;
 import com.microsoft.playwright.options.AriaRole;
 import dasniko.testcontainers.keycloak.KeycloakContainer;
+import org.junit.jupiter.api.Assertions;
 import org.junit.jupiter.params.ParameterizedTest;
 import org.junit.jupiter.params.provider.Arguments;
 import org.junit.jupiter.params.provider.MethodSource;
@@ -33,12 +34,20 @@ class RequiredActionAuthenticatorIT {
 
   @Container
   private static final KeycloakContainer keycloak =
-    new KeycloakContainer("quay.io/keycloak/keycloak:" + KEYCLOAK_VERSION)
-      .withEnv("KEYCLOAK_ADMIN", ADMIN_USER)
-      .withEnv("KEYCLOAK_ADMIN_PASSWORD", ADMIN_PASS)
-      .withLogConsumer(new Slf4jLogConsumer(LOGGER).withSeparateOutputStreams())
-      .withProviderClassesFrom("target/classes")
-      .withRealmImportFile("required-action-realm.json");
+      new KeycloakContainer("quay.io/keycloak/keycloak:" + KEYCLOAK_VERSION)
+          .withEnv("KEYCLOAK_ADMIN", ADMIN_USER)
+          .withEnv("KEYCLOAK_ADMIN_PASSWORD", ADMIN_PASS)
+          .withLogConsumer(new Slf4jLogConsumer(LOGGER).withSeparateOutputStreams())
+          .withProviderClassesFrom("target/classes")
+          .withRealmImportFile("required-action-realm.json");
+
+  private static Stream<Arguments> provideUserLogins() {
+    return Stream.of(
+        Arguments.of("dieterbohlen", "dietersPassword", "TERMS_AND_CONDITIONS"),
+        Arguments.of("mannimammut", "mannimammutsPassword", "UPDATE_PASSWORD"),
+        Arguments.of("peterpommes", "peterpommesPassword", "VERIFY_PROFILE")
+    );
+  }
 
   @ParameterizedTest
   @MethodSource("provideUserLogins")
@@ -54,29 +63,16 @@ void testThatUsersHaveCorrectRequiredActionAfterLogin(String userName, String pa
     var keycloakAdminClient = keycloak.getKeycloakAdminClient().realm(REALM_NAME);
     keycloakAdminClient.flows().updateAuthenticatorConfig(AUTHENTICATOR_CONFIG_ID, authenticatorConfig);
     assertThat(keycloakAdminClient.users().searchByUsername(userName, true).get(0).getRequiredActions())
-      .isEmpty();
-
-    executeBrowserFlow(userName, password);
-
-    assertThat(keycloakAdminClient.users().searchByUsername(userName, true).get(0).getRequiredActions())
-      .containsExactly(requiredAction);
+        .isEmpty();
 
+    executeBrowserFlow(userName, password, requiredAction);
   }
 
-
-  private static Stream<Arguments> provideUserLogins() {
-    return Stream.of(
-      Arguments.of("dieterbohlen", "dietersPassword", "TERMS_AND_CONDITIONS")
-//      Arguments.of("mannimammut", "mannimammutsPassword", "UPDATE_PASSWORD"),
-//      Arguments.of("peterpommes", "peterpommesPassword", "VERIFY_EMAIL")
-    );
-  }
-
-  void executeBrowserFlow(String userName, String password) {
+  void executeBrowserFlow(String userName, String password, String requiredAction) {
     try (Playwright playwright = Playwright.create()) {
       BrowserType chromium = playwright.chromium();
       // comment me in if you want to run in headless mode !
-      // Browser browser = chromium.launch(new BrowserType.LaunchOptions().setHeadless(false));
+      //Browser browser = chromium.launch(new BrowserType.LaunchOptions().setHeadless(false));
       Browser browser = chromium.launch();
       Page page = browser.newPage();
       page.navigate(String.format("http://localhost:%s/realms/required-action/account/", keycloak.getHttpPort()));
@@ -91,6 +87,23 @@ void executeBrowserFlow(String userName, String password) {
       page.getByLabel("Password").first().fill(password);
       page.getByLabel("Password").first().press("Enter");
 
+      // split here, for the specific required action
+      switch (requiredAction) {
+        case "TERMS_AND_CONDITIONS":
+          page.getByRole(AriaRole.BUTTON, new Page.GetByRoleOptions().setName("Accept")).click();
+          break;
+        case "UPDATE_PASSWORD":
+          page.getByLabel("New Password").fill("Test123!");
+          page.getByLabel("Confirm Password").fill("Test123!");
+          page.getByLabel("Confirm Password").press("Enter");
+          break;
+        case "VERIFY_PROFILE":
+          assertThat(keycloak.getKeycloakAdminClient().realm(REALM_NAME).users().searchByUsername(userName, true).get(0).getRequiredActions()).containsExactly(requiredAction);
+          break;
+        default:
+          Assertions.fail();
+      }
+
       page.waitForURL(String.format("http://localhost:%s/realms/required-action/account/#/security/signingin", keycloak.getHttpPort()));
 
       assertThat(page.getByRole(AriaRole.HEADING, new Page.GetByRoleOptions().setName("Signing in"))).isVisible();

diff --git a/authenticator-required-action/src/test/resources/required-action-realm.json b/authenticator-required-action/src/test/resources/required-action-realm.json
@@ -1728,7 +1728,7 @@
     "alias" : "TERMS_AND_CONDITIONS",
     "name" : "Terms and Conditions",
     "providerId" : "TERMS_AND_CONDITIONS",
-    "enabled" : false,
+    "enabled" : true,
     "defaultAction" : false,
     "priority" : 20,
     "config" : { }