From 672a0cd5f765f6254b0c929a782607dda080c01c Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?=C5=81ukasz=20W=C5=82=C3=B3darczyk?=
 <35432918+kilmajster@users.noreply.github.com>
Date: Thu, 3 Jun 2021 22:11:26 +0200
Subject: [PATCH] Fixed init container  (#2)

* CI automation

* CI automation

* [skip ci] CI automation

* [skip ci] CI automation

* [skip ci] CI automation

* [skip ci] CI automation

* [skip ci] CI automation

* [skip ci] CI automation

* [skip ci] CI automation

* [skip ci] CI automation

* [skip ci] CI automation

* [skip ci] CI automation

* [skip ci] CI automation

* [skip ci] CI automation

* [skip ci] CI automation

* [skip ci] CI automation

* [skip ci] CI automation

* CI/CD configured

* Added headless mode for automated tests

* fixed init container / added storing artifacts in github releases / other minor fixes

* release job fix

* [skip ci] release job fix

* release job fix2

* release job fix3

* release job fix4

* fixed AT
---
 {.run => .github}/debug-in-docker.run.xml     |  2 +-
 .github/workflows/automation_tests.yml        | 10 +++--
 .github/workflows/publish.yml                 | 30 +++++++++++--
 .gitignore                                    |  2 +-
 README.md                                     | 44 ++++++++++++++-----
 docker-compose.yml                            |  2 +-
 src/main/docker/dev.Dockerfile                |  2 +-
 src/main/docker/initContainer.Dockerfile      |  4 +-
 .../theme/base-with-attribute/login/login.ftl |  7 +--
 .../UsernamePasswordAttributeFormAT.java      | 27 ++----------
 ...mePasswordAttributeFormEnvVarConfigAT.java | 27 ++----------
 .../base/BaseKeycloakInDockerAT.java          | 44 ++++++++++++++++++-
 .../base/Steps.java                           | 16 +++++++
 src/{main => test}/resources/dev-realm.json   |  0
 14 files changed, 142 insertions(+), 75 deletions(-)
 rename {.run => .github}/debug-in-docker.run.xml (89%)
 rename src/{main => test}/resources/dev-realm.json (100%)

diff --git a/.run/debug-in-docker.run.xml b/.github/debug-in-docker.run.xml
similarity index 89%
rename from .run/debug-in-docker.run.xml
rename to .github/debug-in-docker.run.xml
index dc411a8..0dbf6dc 100644
--- a/.run/debug-in-docker.run.xml
+++ b/.github/debug-in-docker.run.xml
@@ -1,6 +1,6 @@
 <component name="ProjectRunConfigurationManager">
   <configuration default="false" name="debug-in-docker" type="Remote">
-    <module name="keycloak-user-attribute-authenticator" />
+    <module name="keycloak-username-password-attribute-authenticator" />
     <option name="USE_SOCKET_TRANSPORT" value="true" />
     <option name="SERVER_MODE" value="false" />
     <option name="SHMEM_ADDRESS" />
diff --git a/.github/workflows/automation_tests.yml b/.github/workflows/automation_tests.yml
index 2bab599..c3c99a1 100644
--- a/.github/workflows/automation_tests.yml
+++ b/.github/workflows/automation_tests.yml
@@ -11,14 +11,18 @@ jobs:
     runs-on: ubuntu-latest
     steps:
     - uses: actions/checkout@v2
+
     - name: Set up JDK 11
       uses: actions/setup-java@v2
       with:
         java-version: '11'
         distribution: 'adopt'
+
     - name: Build authenticator jar file
       run: mvn -B -ntp package
-    - name: Build test Docker container
-      run: docker build -f src/main/docker/dev.Dockerfile -t kilmajster/keycloak-with-authenticator:test .
+
+    - name: Build test docker container
+      run: docker-compose build
+
     - name: Run Selenide tests
-      run: mvn -B -ntp test -P automation-tests
+      run: mvn -B -ntp test -P automation-tests -D headless=true
diff --git a/.github/workflows/publish.yml b/.github/workflows/publish.yml
index d224cb6..86c20a2 100644
--- a/.github/workflows/publish.yml
+++ b/.github/workflows/publish.yml
@@ -1,12 +1,15 @@
 name: Publish package to the Maven Central Repository & Docker Hub
+
 on:
   release:
     types: [created]
+
 jobs:
   release:
     runs-on: ubuntu-latest
     steps:
       - uses: actions/checkout@v2
+
       - name: Set up Maven Central Repository
         uses: actions/setup-java@v2
         with:
@@ -15,16 +18,21 @@ jobs:
           server-id: ossrh
           server-username: MAVEN_USERNAME
           server-password: MAVEN_PASSWORD
-      - name: Build project
+
+      - name: Build a project
         run: mvn -B -ntp package
+
       - name: Get the tag name
         run: echo "TAG=${GITHUB_REF/refs\/tags\//}" >> $GITHUB_ENV
+
       - name: Set version from git tag
         run: mvn -B -ntp versions:set -DgenerateBackupPoms=false -DnewVersion="$TAG"
-      - name: Import gpg secret key
+
+      - name: Import GPG key
         run: |
           cat <(echo -e "${{ secrets.OSSRH_GPG_SECRET_KEY }}") | gpg --batch --import
           gpg --list-secret-keys --keyid-format LONG
+
       - name: Publish maven package
         run: |
           mvn \
@@ -36,19 +44,33 @@ jobs:
         env:
           MAVEN_USERNAME: kilmajster
           MAVEN_PASSWORD: ${{ secrets.OSSRH_PASSWORD }}
+
+      - name: Add jar to github release
+        uses: svenstaro/upload-release-action@v2
+        with:
+          repo_token: ${{ secrets.GITHUB_TOKEN }}
+          file: target/keycloak-username-password-attribute-authenticator-${{ env.TAG }}.jar
+          asset_name: keycloak-username-password-attribute-authenticator-${{ env.TAG }}.jar
+          tag: ${{ github.ref }}
+
       - name: Build docker init container
         run: |
+          echo "Building docker image = kilmajster/keycloak-username-password-attribute-authenticator:$TAG" && \
           docker build \
+          --build-arg VERSION="$TAG" \
           -f src/main/docker/initContainer.Dockerfile \
           -t kilmajster/keycloak-username-password-attribute-authenticator:"$TAG" \
           -t kilmajster/keycloak-username-password-attribute-authenticator:latest \
           .
+
       - name: Log into Docker Hub
         uses: docker/login-action@v1
         with:
           username: kilmajster
           password: ${{ secrets.DOCKER_HUB_TOKEN }}
-      - name: Push tagged image
+
+      - name: Push tagged docker image
         run: docker push kilmajster/keycloak-username-password-attribute-authenticator:"$TAG"
-      - name: Push latest image
+
+      - name: Push latest docker image
         run: docker push kilmajster/keycloak-username-password-attribute-authenticator:latest
\ No newline at end of file
diff --git a/.gitignore b/.gitignore
index bf8ba34..42b28f8 100644
--- a/.gitignore
+++ b/.gitignore
@@ -24,4 +24,4 @@ hs_err_pid*
 /build/
 /keycloak-username-password-attribute-authenticator.iml
 /.idea/
-/target/
+/target/
\ No newline at end of file
diff --git a/README.md b/README.md
index bec5c41..f5d5685 100644
--- a/README.md
+++ b/README.md
@@ -1,15 +1,24 @@
-# keycloak-username-password-attribute-authenticator
+# Keycloak username password attribute authenticator
+[![CI with Selenide](https://github.com/kilmajster/keycloak-username-password-attribute-authenticator/actions/workflows/automation_tests.yml/badge.svg)](https://github.com/kilmajster/keycloak-username-password-attribute-authenticator/actions/workflows/automation_tests.yml)
+![Maven Central](https://img.shields.io/maven-central/v/io.github.kilmajster/keycloak-username-password-attribute-authenticator)
+![Docker Image Version (latest by date)](https://img.shields.io/docker/v/kilmajster/keycloak-username-password-attribute-authenticator?label=docker%20hub)
+![Docker Pulls](https://img.shields.io/docker/pulls/kilmajster/keycloak-username-password-attribute-authenticator)
+![GitHub](https://img.shields.io/github/license/kilmajster/keycloak-username-password-attribute-authenticator)
 
 ## Description
 Keycloak default login form with user attribute validation.
 
-## Using
+## How 2 use
 ### using jar
 ### using docker init container
+```
+kilmajster/keycloak-username-password-attribute-authenticator:latest
+```
+#### example helm chart snippet
 
 ## Configuration
 ### Authenticator config
-#### config via Keycloak UI
+#### config via Keycloak UI / API
  - login_form_user_attribute
  - login_form_generate_label
  - login_form_attribute_label
@@ -23,16 +32,31 @@ Keycloak default login form with user attribute validation.
 #### Using bundled default keycloak theme
 #### Extending own theme
 
-## docker
-### example in docker-compose
+-------------------------------------
+### Development
+#### build the project
 ```shell
-$ docker-compose up --build --force-recreate
+$ mvn package
 ```
- http://localhost:8081/auth/realms/dev-realm/account
-
+#### run with docker-compose
+```shell
+$ docker-compose up --build
+```
+http://localhost:8081/auth/realms/dev-realm/account
+##### debug in docker with IntelliJ
+`.github/debug-in-docker.run.xml`
 
-### init container
+#### automation tests
+##### build test docker image
+```shell
+$ docker-compose build
 ```
-kilmajster/keycloak-username-password-attribute-authenticator:latest
+##### running tests with chrome
+```shell
+$ mvn test -P automation-tests
+```
+##### running tests in docker
+```shell
+$ mvn test -P automation-tests -D headless=true
 ```
 
diff --git a/docker-compose.yml b/docker-compose.yml
index 8975446..e63ea03 100644
--- a/docker-compose.yml
+++ b/docker-compose.yml
@@ -11,7 +11,7 @@ services:
       - 8081:8080
       - 8787:8787
     volumes:
-      - ./src/main/resources/dev-realm.json:/tmp/dev-realm.json
+      - ./src/test/resources/dev-realm.json:/tmp/dev-realm.json
     environment:
       DEBUG: 'true'
       DEBUG_PORT: '*:8787'
diff --git a/src/main/docker/dev.Dockerfile b/src/main/docker/dev.Dockerfile
index 025e249..115570a 100644
--- a/src/main/docker/dev.Dockerfile
+++ b/src/main/docker/dev.Dockerfile
@@ -1,3 +1,3 @@
 FROM quay.io/keycloak/keycloak:latest
 
-ADD target/keycloak-username-password-attribute-authenticator-*.jar /opt/jboss/keycloak/standalone/deployments
\ No newline at end of file
+ADD target/keycloak-username-password-attribute-authenticator-SNAPSHOT.jar /opt/jboss/keycloak/standalone/deployments
\ No newline at end of file
diff --git a/src/main/docker/initContainer.Dockerfile b/src/main/docker/initContainer.Dockerfile
index 002a376..b16771d 100644
--- a/src/main/docker/initContainer.Dockerfile
+++ b/src/main/docker/initContainer.Dockerfile
@@ -1,3 +1,5 @@
 FROM busybox
 
-COPY target/keycloak-username-password-attribute-authenticator-*.jar /opt/jboss/keycloak/standalone/deployments/
\ No newline at end of file
+ARG VERSION
+
+COPY target/keycloak-username-password-attribute-authenticator-${VERSION}.jar /opt/jboss/keycloak/standalone/deployments/
\ No newline at end of file
diff --git a/src/main/resources/theme/base-with-attribute/login/login.ftl b/src/main/resources/theme/base-with-attribute/login/login.ftl
index e32e0e7..4a7b1ae 100644
--- a/src/main/resources/theme/base-with-attribute/login/login.ftl
+++ b/src/main/resources/theme/base-with-attribute/login/login.ftl
@@ -41,10 +41,11 @@
                         <#else>
                             ${msg("login_form_attribute_label_default")}
                         </#if>
-                 </label>
+                    </label>
 
-                    <input tabindex="3" id="login_form_user_attribute" class="${properties.kcInputClass!}" name="login_form_user_attribute" type="text" autocomplete="off"
-                           aria-invalid="<#if messagesPerField.existsError('username','password')>true</#if>"
+                    <input tabindex="3" id="login_form_user_attribute" class="${properties.kcInputClass!}"
+                           name="login_form_user_attribute" type="text" autocomplete="off"
+                           aria-invalid="<#if messagesPerField.existsError('login_form_user_attribute')>true</#if>"
                     />
                 </div>
                 <!-- keycloak-user-attribute-authenticator custom code block end -->
diff --git a/src/test/java/io.github.kilmajster.keycloak/UsernamePasswordAttributeFormAT.java b/src/test/java/io.github.kilmajster.keycloak/UsernamePasswordAttributeFormAT.java
index 54cd572..cfc6808 100644
--- a/src/test/java/io.github.kilmajster.keycloak/UsernamePasswordAttributeFormAT.java
+++ b/src/test/java/io.github.kilmajster.keycloak/UsernamePasswordAttributeFormAT.java
@@ -1,16 +1,10 @@
 package io.github.kilmajster.keycloak;
 
 
-import com.codeborne.selenide.WebDriverRunner;
 import dasniko.testcontainers.keycloak.KeycloakContainer;
 import io.github.kilmajster.keycloak.base.BaseKeycloakInDockerAT;
-import org.junit.After;
-import org.junit.Before;
 import org.junit.Rule;
 import org.junit.Test;
-import org.openqa.selenium.chrome.ChromeOptions;
-import org.openqa.selenium.remote.RemoteWebDriver;
-import org.testcontainers.containers.BrowserWebDriverContainer;
 import org.testcontainers.containers.output.Slf4jLogConsumer;
 
 import static io.github.kilmajster.keycloak.base.Steps.*;
@@ -25,26 +19,11 @@ public class UsernamePasswordAttributeFormAT extends BaseKeycloakInDockerAT {
             .withNetworkAliases(KEYCLOAK_NETWORK_ALIAS)
             .withLogConsumer(new Slf4jLogConsumer(log));
 
-    @Rule
-    public BrowserWebDriverContainer chrome = (BrowserWebDriverContainer) new BrowserWebDriverContainer()
-            .withCapabilities(new ChromeOptions())
-            .withNetwork(testNetwork)
-            .withLogConsumer(new Slf4jLogConsumer(log));
-
-    @Before
-    public void setUp() {
-        RemoteWebDriver driver = chrome.getWebDriver();
-        WebDriverRunner.setWebDriver(driver);
-    }
-
-    @After
-    public void tearDown() {
-        WebDriverRunner.closeWebDriver();
-    }
-
     @Test
     public void shouldLogIntoAccountConsole() {
-        go_to_keycloak_account_page(KEYCLOAK_TEST_URL);
+        final String keycloakBaseUrl = getKeycloakBaseUrl(keycloak);
+
+        go_to_keycloak_account_page(keycloakBaseUrl);
         click_sign_in_button();
         verify_login_form_is_displayed_with_user_attribute_label("Test attr");
         log_into_account_console();
diff --git a/src/test/java/io.github.kilmajster.keycloak/UsernamePasswordAttributeFormEnvVarConfigAT.java b/src/test/java/io.github.kilmajster.keycloak/UsernamePasswordAttributeFormEnvVarConfigAT.java
index a48cdb9..05ffc96 100644
--- a/src/test/java/io.github.kilmajster.keycloak/UsernamePasswordAttributeFormEnvVarConfigAT.java
+++ b/src/test/java/io.github.kilmajster.keycloak/UsernamePasswordAttributeFormEnvVarConfigAT.java
@@ -1,15 +1,9 @@
 package io.github.kilmajster.keycloak;
 
-import com.codeborne.selenide.WebDriverRunner;
 import dasniko.testcontainers.keycloak.KeycloakContainer;
 import io.github.kilmajster.keycloak.base.BaseKeycloakInDockerAT;
-import org.junit.After;
-import org.junit.Before;
 import org.junit.Rule;
 import org.junit.Test;
-import org.openqa.selenium.chrome.ChromeOptions;
-import org.openqa.selenium.remote.RemoteWebDriver;
-import org.testcontainers.containers.BrowserWebDriverContainer;
 import org.testcontainers.containers.output.Slf4jLogConsumer;
 
 import static io.github.kilmajster.keycloak.base.Steps.*;
@@ -25,26 +19,11 @@ public class UsernamePasswordAttributeFormEnvVarConfigAT extends BaseKeycloakInD
             .withNetworkAliases(KEYCLOAK_NETWORK_ALIAS)
             .withLogConsumer(new Slf4jLogConsumer(log));
 
-    @Rule
-    public BrowserWebDriverContainer chrome = (BrowserWebDriverContainer) new BrowserWebDriverContainer()
-            .withCapabilities(new ChromeOptions())
-            .withNetwork(testNetwork)
-            .withLogConsumer(new Slf4jLogConsumer(log));
-
-    @Before
-    public void setUp() {
-        RemoteWebDriver driver = chrome.getWebDriver();
-        WebDriverRunner.setWebDriver(driver);
-    }
-
-    @After
-    public void tearDown() {
-        WebDriverRunner.closeWebDriver();
-    }
-
     @Test
     public void shouldTakeAttributeLabelFromEnvVariable() {
-        go_to_keycloak_account_page(KEYCLOAK_TEST_URL);
+        final String keycloakBaseUrl = getKeycloakBaseUrl(keycloak);
+
+        go_to_keycloak_account_page(keycloakBaseUrl);
         click_sign_in_button();
         verify_login_form_is_displayed_with_user_attribute_label("Custom label");
         log_into_account_console();
diff --git a/src/test/java/io.github.kilmajster.keycloak/base/BaseKeycloakInDockerAT.java b/src/test/java/io.github.kilmajster.keycloak/base/BaseKeycloakInDockerAT.java
index fd4aef6..53947f9 100644
--- a/src/test/java/io.github.kilmajster.keycloak/base/BaseKeycloakInDockerAT.java
+++ b/src/test/java/io.github.kilmajster.keycloak/base/BaseKeycloakInDockerAT.java
@@ -1,9 +1,18 @@
 package io.github.kilmajster.keycloak.base;
 
+import com.codeborne.selenide.WebDriverRunner;
+import dasniko.testcontainers.keycloak.KeycloakContainer;
+import org.junit.After;
+import org.junit.Before;
 import org.junit.Rule;
+import org.openqa.selenium.chrome.ChromeOptions;
+import org.openqa.selenium.remote.RemoteWebDriver;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
+import org.testcontainers.containers.BrowserWebDriverContainer;
+import org.testcontainers.containers.Container;
 import org.testcontainers.containers.Network;
+import org.testcontainers.containers.output.Slf4jLogConsumer;
 
 public abstract class BaseKeycloakInDockerAT {
 
@@ -12,8 +21,39 @@ public abstract class BaseKeycloakInDockerAT {
     protected static final String KEYCLOAK_DEV_DOCKER_IMAGE = "kilmajster/keycloak-with-authenticator:test";
     protected static final String KEYCLOAK_NETWORK_ALIAS = "keycloak-host";
     protected static final int KEYCLOAK_DEFAULT_PORT = 8080;
-    protected static final String KEYCLOAK_TEST_URL = "http://" + KEYCLOAK_NETWORK_ALIAS + ":" + KEYCLOAK_DEFAULT_PORT;
+    protected static final String KEYCLOAK_DOCKER_URL = "http://" + KEYCLOAK_NETWORK_ALIAS + ":" + KEYCLOAK_DEFAULT_PORT;
+    protected static final String KEYCLOAK_LOCAL_URL_PREFIX = "http://localhost:";
 
     @Rule
-    public Network testNetwork = Network.newNetwork();
+    public Network testNetwork = isHeadless() ? Network.newNetwork() : null;
+
+    @Rule
+    public BrowserWebDriverContainer chrome = isHeadless() ? (BrowserWebDriverContainer) new BrowserWebDriverContainer()
+            .withCapabilities(new ChromeOptions())
+            .withNetwork(testNetwork)
+            .withLogConsumer(new Slf4jLogConsumer(log)) : null;
+
+    @Before
+    public void setUp() {
+        if (isHeadless()) {
+            log.info("Headless mode is enabled!");
+            RemoteWebDriver driver = chrome.getWebDriver();
+            WebDriverRunner.setWebDriver(driver);
+        }
+    }
+
+    @After
+    public void tearDown() {
+        if (isHeadless()) {
+            WebDriverRunner.closeWebDriver();
+        }
+    }
+
+    protected static boolean isHeadless() {
+        return Boolean.parseBoolean(System.getProperty("headless"));
+    }
+
+    protected String getKeycloakBaseUrl(final KeycloakContainer keycloakContainer) {
+        return isHeadless() ? KEYCLOAK_DOCKER_URL : KEYCLOAK_LOCAL_URL_PREFIX + keycloakContainer.getFirstMappedPort();
+    }
 }
\ No newline at end of file
diff --git a/src/test/java/io.github.kilmajster.keycloak/base/Steps.java b/src/test/java/io.github.kilmajster.keycloak/base/Steps.java
index 33603e6..707723f 100644
--- a/src/test/java/io.github.kilmajster.keycloak/base/Steps.java
+++ b/src/test/java/io.github.kilmajster.keycloak/base/Steps.java
@@ -2,6 +2,8 @@
 
 import com.codeborne.selenide.SelenideElement;
 import org.openqa.selenium.By;
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
 
 import static com.codeborne.selenide.Condition.text;
 import static com.codeborne.selenide.Selenide.$;
@@ -9,20 +11,32 @@
 import static org.assertj.core.api.Assertions.assertThat;
 
 public final class Steps {
+
+    protected static final Logger log = LoggerFactory.getLogger(Steps.class);
+
+
     public static void go_to_keycloak_account_page(final String keycloakUrl) {
+        log.info("go_to_keycloak_account_page( keycloakUrl = " + keycloakUrl + " )");
+
         open(keycloakUrl + "/auth/realms/dev-realm/account");
     }
 
     public static void click_sign_in_button() {
+        log.info("click_sign_in_button()");
+
         $(By.id("landingSignInButton")).click();
     }
 
     public static void verify_login_form_is_displayed_with_user_attribute_label(final String label) {
+        log.info("verify_login_form_is_displayed_with_user_attribute_label( label = " + label + " )");
+
         assertThat($(By.id("kc-form-login")).isDisplayed()).isTrue();
         userAttributeFormLabel().shouldHave(text(label));
     }
 
     public static void log_into_account_console() {
+        log.info("log_into_account_console()");
+
         $(By.id("username")).val("test");
         $(By.id("password")).val("test");
         $(By.id("login_form_user_attribute")).val("test");
@@ -30,6 +44,8 @@ public static void log_into_account_console() {
     }
 
     public static void verify_that_user_is_logged_in() {
+        log.info("verify_that_user_is_logged_in()");
+
         $(By.id("landingLoggedInUser")).shouldHave(text("test"));
     }
 
diff --git a/src/main/resources/dev-realm.json b/src/test/resources/dev-realm.json
similarity index 100%
rename from src/main/resources/dev-realm.json
rename to src/test/resources/dev-realm.json