From a27063ce870e8353d15b447d0b9fc0db52d9c1e7 Mon Sep 17 00:00:00 2001 From: OZGCloud <ozgcloud@mgm-tp.com> Date: Fri, 24 May 2024 13:23:44 +0200 Subject: [PATCH] OZG-5400 remove unused imports --- .../operator/keycloak/KeycloakGenericRemoteService.java | 2 -- .../operator/keycloak/realm/KeycloakRealmMapper.java | 9 +-------- .../keycloak/realm/KeycloakRealmRemoteService.java | 3 +-- .../operator/keycloak/user/KeycloakLivelTest.java | 1 + 4 files changed, 3 insertions(+), 12 deletions(-) diff --git a/ozgcloud-keycloak-operator/src/main/java/de/ozgcloud/operator/keycloak/KeycloakGenericRemoteService.java b/ozgcloud-keycloak-operator/src/main/java/de/ozgcloud/operator/keycloak/KeycloakGenericRemoteService.java index 3d1662e..3bcbe7f 100644 --- a/ozgcloud-keycloak-operator/src/main/java/de/ozgcloud/operator/keycloak/KeycloakGenericRemoteService.java +++ b/ozgcloud-keycloak-operator/src/main/java/de/ozgcloud/operator/keycloak/KeycloakGenericRemoteService.java @@ -4,7 +4,6 @@ import java.util.Objects; import java.util.Optional; import org.keycloak.admin.client.Keycloak; -import org.keycloak.admin.client.resource.RealmResource; import org.keycloak.representations.idm.ClientRepresentation; import org.keycloak.representations.idm.RealmRepresentation; import org.keycloak.representations.idm.RoleRepresentation; @@ -39,7 +38,6 @@ public class KeycloakGenericRemoteService { return Optional.of(keycloak.realm(realmName).toRepresentation()); } - public Optional<RoleRepresentation> getClientRole(String roleName, String realClientId, String realm) { return Optional.ofNullable(keycloak.realm(realm).clients().get(realClientId)) .orElseThrow(() -> new KeycloakException("Client with ID " + realClientId + " for realm " + realm + " not found.")) diff --git a/ozgcloud-keycloak-operator/src/main/java/de/ozgcloud/operator/keycloak/realm/KeycloakRealmMapper.java b/ozgcloud-keycloak-operator/src/main/java/de/ozgcloud/operator/keycloak/realm/KeycloakRealmMapper.java index 29d9aa9..d2696dc 100644 --- a/ozgcloud-keycloak-operator/src/main/java/de/ozgcloud/operator/keycloak/realm/KeycloakRealmMapper.java +++ b/ozgcloud-keycloak-operator/src/main/java/de/ozgcloud/operator/keycloak/realm/KeycloakRealmMapper.java @@ -25,22 +25,16 @@ package de.ozgcloud.operator.keycloak.realm; import java.util.HashMap; import java.util.Map; -import java.util.ArrayList; -import java.util.List; import java.util.Set; -import java.util.stream.Collectors; import org.keycloak.representations.idm.RealmRepresentation; import org.keycloak.representations.idm.RoleRepresentation; -import org.keycloak.representations.idm.RolesRepresentation; import org.mapstruct.Mapper; import org.mapstruct.Mapping; import org.mapstruct.MappingTarget; import org.mapstruct.Named; import org.mapstruct.ReportingPolicy; -import de.ozgcloud.operator.keycloak.realm.OzgCloudKeycloakRealmSpec.RealmRole; - @Mapper(unmappedTargetPolicy = ReportingPolicy.IGNORE, unmappedSourcePolicy = ReportingPolicy.IGNORE) interface KeycloakRealmMapper { @@ -56,7 +50,6 @@ interface KeycloakRealmMapper { @Mapping(target = "roles.realm", source = "realmRoles") public RealmRepresentation map(OzgCloudKeycloakRealmSpec realm); - @Mapping(target = "displayName", source = "displayName") @Mapping(target = "enabled", constant = "true") @Mapping(target = "resetPasswordAllowed", constant = "true") @@ -70,7 +63,7 @@ interface KeycloakRealmMapper { RealmRepresentation update(@MappingTarget RealmRepresentation existingRealm, OzgCloudKeycloakRealmSpec spec); @Mapping(target = "name", source = "name") - RoleRepresentation map(OzgCloudKeycloakRealmSpec.RealmRole role); + RoleRepresentation map(OzgCloudKeycloakRealmSpec.RealmRole role); @Named("supportedLocales") default Set<String> mapPassword(OzgCloudKeycloakRealmSpec spec) { diff --git a/ozgcloud-keycloak-operator/src/main/java/de/ozgcloud/operator/keycloak/realm/KeycloakRealmRemoteService.java b/ozgcloud-keycloak-operator/src/main/java/de/ozgcloud/operator/keycloak/realm/KeycloakRealmRemoteService.java index c995ae2..b58998e 100644 --- a/ozgcloud-keycloak-operator/src/main/java/de/ozgcloud/operator/keycloak/realm/KeycloakRealmRemoteService.java +++ b/ozgcloud-keycloak-operator/src/main/java/de/ozgcloud/operator/keycloak/realm/KeycloakRealmRemoteService.java @@ -24,7 +24,6 @@ package de.ozgcloud.operator.keycloak.realm; import org.keycloak.admin.client.Keycloak; -import org.keycloak.admin.client.resource.RealmResource; import org.keycloak.representations.idm.RealmRepresentation; import org.springframework.stereotype.Component; @@ -43,9 +42,9 @@ class KeycloakRealmRemoteService { public void deleteRealm(String realmName) { keycloak.realm(realmName).remove(); } + public void updateRealm(RealmRepresentation realm) { keycloak.realm(realm.getRealm()).update(realm); } - } diff --git a/ozgcloud-keycloak-operator/src/test/java/de/ozgcloud/operator/keycloak/user/KeycloakLivelTest.java b/ozgcloud-keycloak-operator/src/test/java/de/ozgcloud/operator/keycloak/user/KeycloakLivelTest.java index fb6cfcd..11b81fc 100644 --- a/ozgcloud-keycloak-operator/src/test/java/de/ozgcloud/operator/keycloak/user/KeycloakLivelTest.java +++ b/ozgcloud-keycloak-operator/src/test/java/de/ozgcloud/operator/keycloak/user/KeycloakLivelTest.java @@ -79,6 +79,7 @@ class KeycloakLivelTest { userRemoteService.updateUser(u, TESTNAMESPACE); } + @SuppressWarnings("unused") private UserRepresentation createUser() { UserRepresentation u = new UserRepresentation(); u.setUsername(TESTUSERNAME); -- GitLab