diff --git a/alfa-client/pom.xml b/alfa-client/pom.xml index cc55992013e725a490a10c2b07ffeed9ad544ec7..65fcfa287789b0f4ef1bba1d07561b72974607ec 100644 --- a/alfa-client/pom.xml +++ b/alfa-client/pom.xml @@ -24,13 +24,12 @@ unter der Lizenz sind dem Lizenztext zu entnehmen. --> -<project xmlns="http://maven.apache.org/POM/4.0.0" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" - xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/xsd/maven-4.0.0.xsd"> +<project xmlns="http://maven.apache.org/POM/4.0.0" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/xsd/maven-4.0.0.xsd"> <parent> <groupId>de.ozgcloud.alfa</groupId> <artifactId>alfa</artifactId> - <version>2.17.0</version> + <version>2.17.1-SNAPSHOT</version> </parent> <modelVersion>4.0.0</modelVersion> diff --git a/alfa-server/pom.xml b/alfa-server/pom.xml index c1f9487c22a7397d78235ab907d18f997fa643c5..b310933cdb101e68c2394908b05ecd21e608097c 100644 --- a/alfa-server/pom.xml +++ b/alfa-server/pom.xml @@ -30,7 +30,7 @@ <parent> <groupId>de.ozgcloud.alfa</groupId> <artifactId>alfa</artifactId> - <version>2.17.0</version> + <version>2.17.1-SNAPSHOT</version> </parent> <artifactId>alfa-server</artifactId> diff --git a/alfa-service/pom.xml b/alfa-service/pom.xml index 5987635d5cbe66a21abd01551186e881a482fd7c..e357bb2ad222155333915cdde915d132f6c045a8 100644 --- a/alfa-service/pom.xml +++ b/alfa-service/pom.xml @@ -24,15 +24,14 @@ unter der Lizenz sind dem Lizenztext zu entnehmen. --> -<project xmlns="http://maven.apache.org/POM/4.0.0" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" - xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/xsd/maven-4.0.0.xsd"> +<project xmlns="http://maven.apache.org/POM/4.0.0" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/xsd/maven-4.0.0.xsd"> <modelVersion>4.0.0</modelVersion> <parent> <groupId>de.ozgcloud.alfa</groupId> <artifactId>alfa</artifactId> - <version>2.17.0</version> + <version>2.17.1-SNAPSHOT</version> </parent> <artifactId>alfa-service</artifactId> 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(); + } } } diff --git a/pom.xml b/pom.xml index 000974e00fffc4a4b0d4b413a9b185256b442121..2c536af25aa83f7e6f6bfc2b841466a53c3f306e 100644 --- a/pom.xml +++ b/pom.xml @@ -35,7 +35,7 @@ <groupId>de.ozgcloud.alfa</groupId> <artifactId>alfa</artifactId> - <version>2.17.0</version> + <version>2.17.1-SNAPSHOT</version> <name>Alfa Parent</name> <packaging>pom</packaging> <inceptionYear>2020</inceptionYear>