Skip to content
Snippets Groups Projects
Commit 49ed3318 authored by Krzysztof Witukiewicz's avatar Krzysztof Witukiewicz
Browse files

OZG-7855 OZG-8187 Remove unused method

parent 2add00a9
No related branches found
No related tags found
1 merge request!2Ozg 7855 remove saving ozgclouduserid
......@@ -31,7 +31,6 @@ import java.util.stream.StreamSupport;
import org.eclipse.microprofile.config.inject.ConfigProperty;
import org.keycloak.admin.client.resource.RealmResource;
import org.keycloak.admin.client.resource.UserResource;
import org.keycloak.representations.idm.UserRepresentation;
import de.ozgcloud.user.RemoteUserIterator;
......@@ -41,7 +40,6 @@ import de.ozgcloud.user.common.errorhandling.KeycloakUnavailableException;
import de.ozgcloud.user.common.logging.OzgCloudLogging;
import jakarta.enterprise.context.ApplicationScoped;
import jakarta.inject.Inject;
import jakarta.ws.rs.BadRequestException;
import jakarta.ws.rs.ClientErrorException;
import jakarta.ws.rs.NotFoundException;
import jakarta.ws.rs.ProcessingException;
......@@ -94,13 +92,4 @@ class KeycloakApiService {
throw new KeycloakUnavailableException(properties.user(), properties.realm(), keycloakUrl, e);
}
}
void tryUpdateUserResource(UserResource userResource, UserRepresentation userRepresentation, String attributeName) {
try {
userResource.update(userRepresentation);
} catch (BadRequestException e) {
LOG.warn("Could not update user attribute {} in Keycloak.", attributeName, e);
}
}
}
\ No newline at end of file
......@@ -39,8 +39,6 @@ import org.keycloak.admin.client.resource.RealmResource;
import org.keycloak.admin.client.resource.UserResource;
import org.keycloak.admin.client.resource.UsersResource;
import org.keycloak.representations.idm.UserRepresentation;
import org.mockito.ArgumentCaptor;
import org.mockito.Captor;
import org.mockito.InjectMocks;
import org.mockito.Mock;
import org.mockito.Spy;
......@@ -189,32 +187,4 @@ class KeycloakApiServiceTest {
return service.handlingKeycloakException(runnable);
}
}
@Nested
class TestTryUpdateUserResource {
private static final String ATTRIBUTE_NAME_USER_ID = "userId";
@Captor
private ArgumentCaptor<UserRepresentation> userRepresentationArgumentCaptor;
@Mock
private UserRepresentation userRepresentation;
@Test
void shouldUpdateUserResource() {
service.tryUpdateUserResource(userResource, userRepresentation, ATTRIBUTE_NAME_USER_ID);
verify(userResource).update(any(UserRepresentation.class));
}
@Test
void shouldCreateOlderUserRepresentation() {
service.tryUpdateUserResource(userResource, UserRepresentationTestFactory.create(), ATTRIBUTE_NAME_USER_ID);
verify(userResource).update(userRepresentationArgumentCaptor.capture());
assertThat(userRepresentationArgumentCaptor.getValue()).usingRecursiveComparison().isEqualTo(UserRepresentationTestFactory.create());
}
}
}
\ No newline at end of file
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment