diff --git a/alfa-service/src/main/java/de/ozgcloud/alfa/postfach/PostfachSettingsService.java b/alfa-service/src/main/java/de/ozgcloud/alfa/postfach/PostfachSettingsService.java index 71d81170a9d3f924be141c3cb85b37e54986d5ed..5d384eb0c9e4b4ccb538ea92ac0c51dc532bfdbf 100644 --- a/alfa-service/src/main/java/de/ozgcloud/alfa/postfach/PostfachSettingsService.java +++ b/alfa-service/src/main/java/de/ozgcloud/alfa/postfach/PostfachSettingsService.java @@ -23,7 +23,6 @@ */ package de.ozgcloud.alfa.postfach; -import java.util.Map; import java.util.Objects; import java.util.Optional; @@ -57,19 +56,14 @@ class PostfachSettingsService { public PostfachSettings getPostfachSettings(VorgangWithEingang vorgang) { return PostfachSettings.builder() - .features(Features.builder() - .reply(isReplyToMessageAllowed(vorgang)) - .build()) - .settings(Settings.builder() - .signatur(getSignatur(vorgang.getOrganisationseinheitenID())) - .build()) + .features(Features.builder().reply(isReplyToMessageAllowed(vorgang)).build()) + .settings(Settings.builder().signatur(getSignatur()).build()) .build(); } - String getSignatur(String organisationseinheitenID) { + String getSignatur() { refreshPostfachProperties(); - var settings = getOrganisationsEinheitSettings(organisationseinheitenID); - return settings.map(OrganisationsEinheitSettings::getSignatur).orElseGet(postfachProperties::getSignatur); + return postfachProperties.getSignatur(); } void refreshPostfachProperties() { @@ -98,19 +92,4 @@ class PostfachSettingsService { .findFirst() .orElse(false); } - - Optional<OrganisationsEinheitSettings> getOrganisationsEinheitSettings(String organisationId) { - return Optional.ofNullable(organisationId) - .map(this::getOrganisationsEinheitPostfachSettings) - .map(this::mapOrganisationsEinheitSettings); - } - - OrganisationsEinheitSettings mapOrganisationsEinheitSettings(Map<String, Object> organisationsEinheitSettings) { - var signatur = Optional.ofNullable(organisationsEinheitSettings.get(FIELD_SIGNATUR)).map(Object::toString).orElse(null); - return OrganisationsEinheitSettings.builder().signatur(signatur).build(); - } - - private Map<String, Object> getOrganisationsEinheitPostfachSettings(String organisationId) { - return postfachProperties.getOrganisationsEinheitSettings().get(organisationId); - } } diff --git a/alfa-service/src/test/java/de/ozgcloud/alfa/postfach/PostfachSettingsServiceTest.java b/alfa-service/src/test/java/de/ozgcloud/alfa/postfach/PostfachSettingsServiceTest.java index 23af6bf5791e241304e8bb780d56ac35e90ca354..c35cfec8c5a0157df5be23b59fbb18d05a34766a 100644 --- a/alfa-service/src/test/java/de/ozgcloud/alfa/postfach/PostfachSettingsServiceTest.java +++ b/alfa-service/src/test/java/de/ozgcloud/alfa/postfach/PostfachSettingsServiceTest.java @@ -26,11 +26,7 @@ package de.ozgcloud.alfa.postfach; import static org.assertj.core.api.Assertions.*; import static org.mockito.Mockito.*; -import java.util.Collections; import java.util.List; -import java.util.Map; -import java.util.Optional; -import java.util.UUID; import org.junit.jupiter.api.BeforeEach; import org.junit.jupiter.api.DisplayName; @@ -46,7 +42,6 @@ import org.springframework.test.util.ReflectionTestUtils; import com.thedeanda.lorem.LoremIpsum; -import de.ozgcloud.alfa.collaboration.OrganisationsEinheitTestFactory; import de.ozgcloud.alfa.common.FeatureToggleProperties; import de.ozgcloud.alfa.vorgang.ServiceKontoTestFactory; import de.ozgcloud.alfa.vorgang.VorgangHeadTestFactory; @@ -156,12 +151,12 @@ class PostfachSettingsServiceTest { void shouldGetSignatur() { callService(); - verify(service).getSignatur(vorgang.getOrganisationseinheitenID()); + verify(service).getSignatur(); } @Test void shouldSetSignatur() { - doReturn(SettingsTestFactory.SIGNATUR).when(service).getSignatur(vorgang.getOrganisationseinheitenID()); + doReturn(SettingsTestFactory.SIGNATUR).when(service).getSignatur(); var postfach = callService(); @@ -184,60 +179,24 @@ class PostfachSettingsServiceTest { } @Test - void shouldGetOrganisationEinheitSettings() { + void shouldGetSignatur() { callService(); - verify(service).getOrganisationsEinheitSettings(OrganisationsEinheitTestFactory.ID); - } - - @Nested - class OnExistingOrganisationsEinheitSignatur { - - @BeforeEach - void setUp() { - doReturn(Optional.of(OrganisationsEinheitSettingsTestFactory.create())).when(service) - .getOrganisationsEinheitSettings(OrganisationsEinheitTestFactory.ID); - } - - @Test - void shouldReturnOrganisationsEinheitSignatur() { - var signatur = callService(); - - assertThat(signatur).isEqualTo(OrganisationsEinheitSettingsTestFactory.TEST_SIGNATUR); - } + verify(postfachProperties).getSignatur(); } - @Nested - class OnNoOrganisationsEinheitSignatur { - - @BeforeEach - void setUp() { - doReturn(Optional.empty()).when(service) - .getOrganisationsEinheitSettings(OrganisationsEinheitTestFactory.ID); - } - - @Test - void shouldGetSignatur() { - callService(); - - verify(postfachProperties).getSignatur(); - - } - - @Test - void shouldReturnSignatur() { - when(postfachProperties.getSignatur()).thenReturn(SettingsTestFactory.SIGNATUR); - - var signatur = callService(); + @Test + void shouldReturnSignatur() { + when(postfachProperties.getSignatur()).thenReturn(SettingsTestFactory.SIGNATUR); - assertThat(signatur).isEqualTo(SettingsTestFactory.SIGNATUR); - } + var signatur = callService(); + assertThat(signatur).isEqualTo(SettingsTestFactory.SIGNATUR); } private String callService() { - return service.getSignatur(OrganisationsEinheitTestFactory.ID); + return service.getSignatur(); } } @@ -380,82 +339,4 @@ class PostfachSettingsServiceTest { private void setPostfachConfigGroup(PostfachConfigGroup postfachConfigGroup) { ReflectionTestUtils.setField(service, "postfachConfigGroup", postfachConfigGroup); } - - @Nested - class TestGetOrganisationsEinheitSettings { - - @Nested - class OnOrganisationIdNotNull { - - private final OrganisationsEinheitSettings organisationsEinheitSettings = OrganisationsEinheitSettingsTestFactory.create(); - private final Map<String, Object> organisationsEinheitSettingsMap = Map.of(PostfachSettingsService.FIELD_SIGNATUR, - OrganisationsEinheitSettingsTestFactory.TEST_SIGNATUR); - - @BeforeEach - void setUp() { - when(postfachProperties.getOrganisationsEinheitSettings()).thenReturn( - Map.of(OrganisationsEinheitTestFactory.ID, organisationsEinheitSettingsMap)); - } - - @Test - void shouldGetOrganisationsEinheitSettings() { - service.getOrganisationsEinheitSettings(OrganisationsEinheitTestFactory.ID); - - verify(postfachProperties).getOrganisationsEinheitSettings(); - } - - @Test - void shouldMapToOrganisationsEinheitSettings() { - service.getOrganisationsEinheitSettings(OrganisationsEinheitTestFactory.ID); - - verify(service).mapOrganisationsEinheitSettings(organisationsEinheitSettingsMap); - } - - @Test - void shouldReturnSettings() { - var settings = service.getOrganisationsEinheitSettings(OrganisationsEinheitTestFactory.ID); - - assertThat(settings).isPresent().get().usingRecursiveComparison().isEqualTo(organisationsEinheitSettings); - } - - @Test - void shouldReturnEmpty() { - var settings = service.getOrganisationsEinheitSettings(UUID.randomUUID().toString()); - - assertThat(settings).isEmpty(); - } - } - - @Nested - class OnOrganisationIdNull { - - @Test - void shouldReturnEmpty() { - var settings = service.getOrganisationsEinheitSettings(null); - - assertThat(settings).isEmpty(); - } - } - } - - @Nested - class TestMapOrganisationsEinheitSettings { - - private final String signatur = LoremIpsum.getInstance().getWords(2); - private final Map<String, Object> organisationsEinheitSettings = Map.of(PostfachSettingsService.FIELD_SIGNATUR, signatur); - - @Test - void shouldMap() { - var settings = service.mapOrganisationsEinheitSettings(organisationsEinheitSettings); - - assertThat(settings.getSignatur()).isEqualTo(signatur); - } - - @Test - void shouldReturnSettingsWithNullSignatur() { - var settings = service.mapOrganisationsEinheitSettings(Collections.emptyMap()); - - assertThat(settings.getSignatur()).isNull(); - } - } }