fix: sonar issues (#30)

* fix: sonar issues * fix: sonar issues
onecx · Jul 19, 2024 · 9647a5f · 9647a5f
1 parent ffaa24b
commit 9647a5f
Show file tree

Hide file tree

Showing 6 changed files with 114 additions and 140 deletions.
diff --git a/...dules/onecx-iam-kc-client-operator/pages/onecx-iam-kc-client-operator-docs.adoc b/...dules/onecx-iam-kc-client-operator/pages/onecx-iam-kc-client-operator-docs.adoc
@@ -13,6 +13,7 @@ quarkus.keycloak.admin-client.server-url=http://keycloak:8080
 quarkus.keycloak.admin-client.realm=master
 quarkus.keycloak.admin-client.username=admin
 quarkus.keycloak.admin-client.password=admin
+quarkus.operator-sdk.crd.validate=false
 quarkus.operator-sdk.helm.enabled=true
 onecx.iam.kc.client.realm=onecx
 onecx.iam.kc.client.config.ui.enabled=true

diff --git a/...onecx-iam-kc-client-operator/pages/onecx-iam-kc-client-operator-extensions.adoc b/...onecx-iam-kc-client-operator/pages/onecx-iam-kc-client-operator-extensions.adoc
@@ -11,81 +11,93 @@ h| Version
 | tkit-quarkus-log-cdi
 
 | https://1000kit.github.io/tkit-quarkus/current/tkit-quarkus/tkit-quarkus-log-cdi.html[Link]
-| https://github.com/1000kit/tkit-quarkus/blob/2.22.0/docs/modules/tkit-quarkus/pages/includes/tkit-quarkus-log-cdi.adoc[Link]
-| 2.22.0
+| https://github.com/1000kit/tkit-quarkus/blob/2.28.0/docs/modules/tkit-quarkus/pages/includes/tkit-quarkus-log-cdi.adoc[Link]
+| 2.28.0
 
 | tkit-quarkus-log-rs
 
 | https://1000kit.github.io/tkit-quarkus/current/tkit-quarkus/tkit-quarkus-log-rs.html[Link]
-| https://github.com/1000kit/tkit-quarkus/blob/2.22.0/docs/modules/tkit-quarkus/pages/includes/tkit-quarkus-log-rs.adoc[Link]
-| 2.22.0
+| https://github.com/1000kit/tkit-quarkus/blob/2.28.0/docs/modules/tkit-quarkus/pages/includes/tkit-quarkus-log-rs.adoc[Link]
+| 2.28.0
 
 | tkit-quarkus-log-json
 
 | https://1000kit.github.io/tkit-quarkus/current/tkit-quarkus/tkit-quarkus-log-json.html[Link]
-| https://github.com/1000kit/tkit-quarkus/blob/2.22.0/docs/modules/tkit-quarkus/pages/includes/tkit-quarkus-log-json.adoc[Link]
-| 2.22.0
+| https://github.com/1000kit/tkit-quarkus/blob/2.28.0/docs/modules/tkit-quarkus/pages/includes/tkit-quarkus-log-json.adoc[Link]
+| 2.28.0
 
 | quarkus-arc
 
 | https://quarkus.io/guides/cdi-reference[Link]
 | https://github.com/quarkusio/quarkusio.github.io/blob/develop/_generated-doc/latest/config/quarkus-arc.adoc[Link]
-| 3.9.3
+| 3.12.2
 
 | quarkus-micrometer-registry-prometheus
 
 | https://quarkus.io/guides/telemetry-micrometer[Link]
 | https://github.com/quarkusio/quarkusio.github.io/blob/develop/_generated-doc/latest/config/quarkus-micrometer-registry-prometheus.adoc[Link]
-| 3.9.3
+| 3.12.2
 
 | quarkus-opentelemetry
 
 | https://quarkus.io/guides/opentelemetry[Link]
 | https://github.com/quarkusio/quarkusio.github.io/blob/develop/_generated-doc/latest/config/quarkus-opentelemetry.adoc[Link]
-| 3.9.3
+| 3.12.2
 
 | quarkus-rest-client
 
 | https://quarkus.io/guides/rest-client[Link]
 | https://github.com/quarkusio/quarkusio.github.io/blob/develop/_generated-doc/latest/config/quarkus-rest-client.adoc[Link]
-| 3.9.3
+| 3.12.2
 
 | quarkus-rest-client-jackson
 
 | https://quarkus.io/guides/rest-client[Link]
 | 
-| 3.9.3
+| 3.12.2
+
+| tkit-quarkus-security
+
+| https://1000kit.github.io/tkit-quarkus/current/tkit-quarkus/tkit-quarkus-security.html[Link]
+| https://github.com/1000kit/tkit-quarkus/blob/2.28.0/docs/modules/tkit-quarkus/pages/includes/tkit-quarkus-security.adoc[Link]
+| 2.28.0
 
 | onecx-core
 
 | https://onecx.github.io/docs/onecx-quarkus/current/onecx-quarkus/onecx-core.html[Link]
 | 
-| 0.17.0
+| 0.24.0
+
+| quarkus-smallrye-health
+
+| https://quarkus.io/guides/smallrye-health[Link]
+| https://github.com/quarkusio/quarkusio.github.io/blob/develop/_generated-doc/latest/config/quarkus-smallrye-health.adoc[Link]
+| 3.12.2
 
 | quarkus-container-image-docker
 
 | https://quarkus.io/guides/container-image[Link]
 | https://github.com/quarkusio/quarkusio.github.io/blob/develop/_generated-doc/latest/config/quarkus-container-image-docker.adoc[Link]
-| 3.9.3
+| 3.12.2
 
 
 | quarkus-operator-sdk-bundle-generator
 
 | 
 | 
-| 6.6.7
+| 6.7.1
 
 | quarkus-operator-sdk
 
 | 
 | 
-| 6.6.7
+| 6.7.1
 
 | quarkus-keycloak-admin-rest-client
 
 | 
 | 
-| 3.9.3
+| 3.12.2
 
 
-|===|===
+|===
diff --git a/docs/modules/onecx-iam-kc-client-operator/pages/onecx-iam-kc-client-operator.adoc b/docs/modules/onecx-iam-kc-client-operator/pages/onecx-iam-kc-client-operator.adoc
@@ -231,20 +231,21 @@ endif::add-copy-button-to-env-var[]
 |`openid-connect`
 
 
-a| [[onecx-iam-kc-client-operator_onecx-iam-kc-client-config-config-attributes-attributes]]`link:#onecx-iam-kc-client-operator_onecx-iam-kc-client-config-config-attributes-attributes[onecx.iam.kc.client.config."config".attributes]`
+a| [[onecx-iam-kc-client-operator_onecx-iam-kc-client-config-config-attributes-attributes]]`link:#onecx-iam-kc-client-operator_onecx-iam-kc-client-config-config-attributes-attributes[onecx.iam.kc.client.config."config".attributes."attributes"]`
 
 
 [.description]
 --
 Attributes map for the client.
 
 ifdef::add-copy-button-to-env-var[]
-Environment variable: env_var_with_copy_button:+++ONECX_IAM_KC_CLIENT_CONFIG__CONFIG__ATTRIBUTES+++[]
+Environment variable: env_var_with_copy_button:+++ONECX_IAM_KC_CLIENT_CONFIG__CONFIG__ATTRIBUTES__ATTRIBUTES_+++[]
 endif::add-copy-button-to-env-var[]
 ifndef::add-copy-button-to-env-var[]
-Environment variable: `+++ONECX_IAM_KC_CLIENT_CONFIG__CONFIG__ATTRIBUTES+++`
+Environment variable: `+++ONECX_IAM_KC_CLIENT_CONFIG__CONFIG__ATTRIBUTES__ATTRIBUTES_+++`
 endif::add-copy-button-to-env-var[]
---|`Map<String,String>` 
+--|link:https://docs.oracle.com/javase/8/docs/api/java/lang/String.html[String]
+
 |
 
 

diff --git a/src/test/java/org/tkit/onecx/iam/kc/client/operator/KeycloakClientControllerMockTest.java b/src/test/java/org/tkit/onecx/iam/kc/client/operator/KeycloakClientControllerMockTest.java
@@ -49,7 +49,7 @@ void testThrowException() throws NoSuchMethodException {
 
     }
 
-    public class MockClientResource implements ClientResource {
+    public static class MockClientResource implements ClientResource {
 
         @Override
         public ManagementPermissionReference setPermissions(
@@ -74,12 +74,12 @@ public ClientRepresentation toRepresentation() {
 
         @Override
         public void update(ClientRepresentation clientRepresentation) {
-
+            throw new UnsupportedOperationException();
         }
 
         @Override
         public void remove() {
-
+            throw new UnsupportedOperationException();
         }
 
         @Override
@@ -129,7 +129,7 @@ public List<UserSessionRepresentation> getOfflineUserSessions(Integer integer, I
 
         @Override
         public void pushRevocation() {
-
+            throw new UnsupportedOperationException();
         }
 
         @Override
@@ -179,12 +179,12 @@ public UserRepresentation getServiceAccountUser() {
 
         @Override
         public void registerNode(Map<String, String> map) {
-
+            throw new UnsupportedOperationException();
         }
 
         @Override
         public void unregisterNode(String s) {
-
+            throw new UnsupportedOperationException();
         }
 
         @Override
@@ -204,7 +204,7 @@ public CredentialRepresentation getClientRotatedSecret() {
 
         @Override
         public void invalidateRotatedSecret() {
-
+            throw new UnsupportedOperationException();
         }
     }