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 5e5ca6852b860bfaa771cdf35f11d477d8c98932..71d81170a9d3f924be141c3cb85b37e54986d5ed 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 @@ -99,12 +99,18 @@ class PostfachSettingsService { .orElse(false); } - Optional<OrganisationsEinheitSettings> getOrganisationsEinheitSettings(final String organisationId) { - return Optional.ofNullable(postfachProperties.getOrganisationsEinheitSettings().get(organisationId)).map(this::mapOrganisationsEinheitSettings); + 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 82811fbf49b38006929e75692950a54185acb1ed..23af6bf5791e241304e8bb780d56ac35e90ca354 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 @@ -384,41 +384,57 @@ class PostfachSettingsServiceTest { @Nested class TestGetOrganisationsEinheitSettings { - private final OrganisationsEinheitSettings organisationsEinheitSettings = OrganisationsEinheitSettingsTestFactory.create(); - private final Map<String, Object> organisationsEinheitSettingsMap = Map.of(PostfachSettingsService.FIELD_SIGNATUR, OrganisationsEinheitSettingsTestFactory.TEST_SIGNATUR); + @Nested + class OnOrganisationIdNotNull { - @BeforeEach - void setUp() { - when(postfachProperties.getOrganisationsEinheitSettings()).thenReturn( - Map.of(OrganisationsEinheitTestFactory.ID, organisationsEinheitSettingsMap)); - } + private final OrganisationsEinheitSettings organisationsEinheitSettings = OrganisationsEinheitSettingsTestFactory.create(); + private final Map<String, Object> organisationsEinheitSettingsMap = Map.of(PostfachSettingsService.FIELD_SIGNATUR, + OrganisationsEinheitSettingsTestFactory.TEST_SIGNATUR); - @Test - void shouldGetOrganisationsEinheitSettings() { - service.getOrganisationsEinheitSettings(OrganisationsEinheitTestFactory.ID); + @BeforeEach + void setUp() { + when(postfachProperties.getOrganisationsEinheitSettings()).thenReturn( + Map.of(OrganisationsEinheitTestFactory.ID, organisationsEinheitSettingsMap)); + } - verify(postfachProperties).getOrganisationsEinheitSettings(); - } + @Test + void shouldGetOrganisationsEinheitSettings() { + service.getOrganisationsEinheitSettings(OrganisationsEinheitTestFactory.ID); - @Test - void shouldMapToOrganisationsEinheitSettings() { - service.getOrganisationsEinheitSettings(OrganisationsEinheitTestFactory.ID); + verify(postfachProperties).getOrganisationsEinheitSettings(); + } - verify(service).mapOrganisationsEinheitSettings(organisationsEinheitSettingsMap); - } + @Test + void shouldMapToOrganisationsEinheitSettings() { + service.getOrganisationsEinheitSettings(OrganisationsEinheitTestFactory.ID); - @Test - void shouldReturnSettings() { - var settings = 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).isPresent().get().usingRecursiveComparison().isEqualTo(organisationsEinheitSettings); + assertThat(settings).isEmpty(); + } } - @Test - void shouldReturnEmpty() { - var settings = service.getOrganisationsEinheitSettings(UUID.randomUUID().toString()); + @Nested + class OnOrganisationIdNull { - assertThat(settings).isEmpty(); + @Test + void shouldReturnEmpty() { + var settings = service.getOrganisationsEinheitSettings(null); + + assertThat(settings).isEmpty(); + } } }