Anpassung wegen Änderung openapi-Beta zu openapi-0.1.0

wls-basisdaten-service/pom.xml

@@ -393,7 +393,7 @@
                             <goal>generate</goal>
                         </goals>
                         <configuration>
-                            <inputSpec>${project.basedir}/src/main/resources/openapis/openapi.eai.0.1.1-Beta1.json</inputSpec>
+                            <inputSpec>${project.basedir}/src/main/resources/openapis/openapi.eai.0.1.0.json</inputSpec>
 
                             <generatorName>java</generatorName>
                             <library>resttemplate</library>

wls-basisdaten-service/src/main/java/de/muenchen/oss/wahllokalsystem/basisdatenservice/clients/WahltageClientImpl.java

@@ -2,14 +2,15 @@
 
 import de.muenchen.oss.wahllokalsystem.basisdatenservice.configuration.Profiles;
 import de.muenchen.oss.wahllokalsystem.basisdatenservice.eai.aou.client.WahldatenControllerApi;
-import de.muenchen.oss.wahllokalsystem.basisdatenservice.eai.aou.model.WahltageDTO;
+import de.muenchen.oss.wahllokalsystem.basisdatenservice.eai.aou.model.WahltagDTO;
 import de.muenchen.oss.wahllokalsystem.basisdatenservice.exception.ExceptionConstants;
 import de.muenchen.oss.wahllokalsystem.basisdatenservice.services.wahltag.WahltagModel;
 import de.muenchen.oss.wahllokalsystem.basisdatenservice.services.wahltag.WahltageClient;
 import de.muenchen.oss.wahllokalsystem.wls.common.exception.WlsException;
 import de.muenchen.oss.wahllokalsystem.wls.common.exception.util.ExceptionFactory;
 import java.time.LocalDate;
 import java.util.List;
+import java.util.Set;
 import lombok.RequiredArgsConstructor;
 import lombok.extern.slf4j.Slf4j;
 import org.springframework.context.annotation.Profile;
@@ -29,7 +30,7 @@ public class WahltageClientImpl implements WahltageClient {
     @Override
     public List<WahltagModel> getWahltage(LocalDate tag) throws WlsException {
 
-        final WahltageDTO wahltageDTO;
+        final Set<WahltagDTO> wahltageDTO;
         try {
             wahltageDTO = wahldatenControllerApi.loadWahltageSinceIncluding(tag);
         } catch (final Exception exception) {
@@ -39,6 +40,6 @@ public List<WahltagModel> getWahltage(LocalDate tag) throws WlsException {
         if (wahltageDTO == null) {
             throw exceptionFactory.createFachlicheWlsException(ExceptionConstants.NULL_FROM_CLIENT);
         }
-        return wahltageClientMapper.fromRemoteClientWahltageDTOtoListOfWahltagModel(wahltageDTO);
+        return wahltageClientMapper.fromRemoteClientSetOfWahltagDTOtoListOfWahltagModel(wahltageDTO);
     }
 }

wls-basisdaten-service/src/main/java/de/muenchen/oss/wahllokalsystem/basisdatenservice/clients/WahltageClientMapper.java

@@ -1,9 +1,9 @@
 package de.muenchen.oss.wahllokalsystem.basisdatenservice.clients;
 
 import de.muenchen.oss.wahllokalsystem.basisdatenservice.eai.aou.model.WahltagDTO;
-import de.muenchen.oss.wahllokalsystem.basisdatenservice.eai.aou.model.WahltageDTO;
 import de.muenchen.oss.wahllokalsystem.basisdatenservice.services.wahltag.WahltagModel;
 import java.util.List;
+import java.util.Set;
 import org.mapstruct.Mapper;
 import org.mapstruct.Mapping;
 
@@ -14,7 +14,5 @@ public interface WahltageClientMapper {
     @Mapping(target = "wahltag", source = "tag")
     WahltagModel toWahltagModel(WahltagDTO wahltagDTO);
 
-    default List<WahltagModel> fromRemoteClientWahltageDTOtoListOfWahltagModel(final WahltageDTO wahltageDTO) {
-        return wahltageDTO.getWahltage().stream().map(this::toWahltagModel).toList();
-    }
+    List<WahltagModel> fromRemoteClientSetOfWahltagDTOtoListOfWahltagModel(Set<WahltagDTO> wahltageDTO);
 }

wls-basisdaten-service/src/test/java/de/muenchen/oss/wahllokalsystem/basisdatenservice/clients/WahltageClientImplTest.java

@@ -2,14 +2,15 @@
 
 import static org.mockito.ArgumentMatchers.any;
 import de.muenchen.oss.wahllokalsystem.basisdatenservice.eai.aou.client.WahldatenControllerApi;
-import de.muenchen.oss.wahllokalsystem.basisdatenservice.eai.aou.model.WahltageDTO;
+import de.muenchen.oss.wahllokalsystem.basisdatenservice.eai.aou.model.WahltagDTO;
 import de.muenchen.oss.wahllokalsystem.basisdatenservice.exception.ExceptionConstants;
 import de.muenchen.oss.wahllokalsystem.basisdatenservice.services.wahltag.WahltagModel;
 import de.muenchen.oss.wahllokalsystem.wls.common.exception.FachlicheWlsException;
 import de.muenchen.oss.wahllokalsystem.wls.common.exception.TechnischeWlsException;
 import de.muenchen.oss.wahllokalsystem.wls.common.exception.util.ExceptionFactory;
 import java.time.LocalDate;
 import java.util.List;
+import java.util.Set;
 import lombok.val;
 import org.assertj.core.api.Assertions;
 import org.junit.jupiter.api.Nested;
@@ -43,12 +44,12 @@ class GetWahltage {
         void clientResponseIsMapped() {
             val testDate = LocalDate.now().minusMonths(3);
 
-            val mockedClientResponse = new WahltageDTO();
+            val mockedClientResponse = createClientWahltageDTO();
             val mockedMappedClientResponse = List.of(WahltagModel.builder().build());
 
             Mockito.when(wahldatenControllerApi.loadWahltageSinceIncluding(testDate))
                     .thenReturn(mockedClientResponse);
-            Mockito.when(wahltageClientMapper.fromRemoteClientWahltageDTOtoListOfWahltagModel(mockedClientResponse))
+            Mockito.when(wahltageClientMapper.fromRemoteClientSetOfWahltagDTOtoListOfWahltagModel(mockedClientResponse))
                     .thenReturn(mockedMappedClientResponse);
 
             val result = unitUnderTest.getWahltage(testDate);
@@ -77,5 +78,28 @@ void controllerApiExceptionIsMapped() {
             Mockito.when(exceptionFactory.createTechnischeWlsException(ExceptionConstants.FAILED_COMMUNICATION_WITH_EAI)).thenThrow(mockedException);
             Assertions.assertThatException().isThrownBy(() -> unitUnderTest.getWahltage(testDate)).isSameAs(mockedException);
         }
+
+        private Set<de.muenchen.oss.wahllokalsystem.basisdatenservice.eai.aou.model.WahltagDTO> createClientWahltageDTO() {
+
+            val wahltag1 = new de.muenchen.oss.wahllokalsystem.basisdatenservice.eai.aou.model.WahltagDTO();
+            wahltag1.setIdentifikator("identifikatorWahltag1");
+            wahltag1.setBeschreibung("beschreibungWahltag1");
+            wahltag1.setNummer("nummerWahltag1");
+            wahltag1.setTag(LocalDate.now().minusMonths(2));
+
+            val wahltag2 = new de.muenchen.oss.wahllokalsystem.basisdatenservice.eai.aou.model.WahltagDTO();
+            wahltag2.setIdentifikator("identifikatorWahltag2");
+            wahltag2.setBeschreibung("beschreibungWahltag2");
+            wahltag2.setNummer("nummerWahltag2");
+            wahltag2.setTag(LocalDate.now().minusMonths(1));
+
+            val wahltag3 = new WahltagDTO();
+            wahltag3.setIdentifikator("identifikatorWahltag3");
+            wahltag3.setBeschreibung("beschreibungWahltag3");
+            wahltag3.setNummer("nummerWahltag3");
+            wahltag3.setTag(LocalDate.now().plusMonths(1));
+
+            return Set.of(wahltag1, wahltag2, wahltag3);
+        }
     }
 }

wls-basisdaten-service/src/test/java/de/muenchen/oss/wahllokalsystem/basisdatenservice/clients/WahltageClientMapperTest.java

@@ -1,7 +1,6 @@
 package de.muenchen.oss.wahllokalsystem.basisdatenservice.clients;
 
 import de.muenchen.oss.wahllokalsystem.basisdatenservice.eai.aou.model.WahltagDTO;
-import de.muenchen.oss.wahllokalsystem.basisdatenservice.eai.aou.model.WahltageDTO;
 import de.muenchen.oss.wahllokalsystem.basisdatenservice.services.wahltag.WahltagModel;
 import java.time.LocalDate;
 import java.util.List;
@@ -47,8 +46,6 @@ class FromRemoteClientWahltageDTOtoListOfWahltagModel {
 
         @Test
         void isMapped() {
-            val dtoToMap = new WahltageDTO();
-
             val wahltag1 = new WahltagDTO();
             wahltag1.setIdentifikator("identifikatorWahltag1");
             wahltag1.setBeschreibung("beschreibungWahltag1");
@@ -67,13 +64,12 @@ void isMapped() {
             wahltag3.setNummer("nummerWahltag3");
             wahltag3.setTag(LocalDate.now().plusMonths(1));
 
-            dtoToMap.setWahltage(Set.of(
-                    wahltag1, wahltag2, wahltag3));
+            val dtoToMap = Set.of(wahltag1, wahltag2, wahltag3);
 
-            Assertions.assertThat(dtoToMap).hasNoNullFieldsOrProperties();
-            Assertions.assertThat(dtoToMap.getWahltage()).size().isEqualTo(3);
+            Assertions.assertThat(dtoToMap).size().isEqualTo(3);
+            dtoToMap.forEach(wahltag -> Assertions.assertThat(wahltag).hasNoNullFieldsOrProperties());
 
-            val result = unitUnderTest.fromRemoteClientWahltageDTOtoListOfWahltagModel(dtoToMap);
+            val result = unitUnderTest.fromRemoteClientSetOfWahltagDTOtoListOfWahltagModel(dtoToMap);
 
             val expectedWahltage = List.of(
                     new WahltagModel("identifikatorWahltag1",

wls-basisdaten-service/src/test/java/de/muenchen/oss/wahllokalsystem/basisdatenservice/rest/wahltage/WahltageControllerIntegrationTest.java

@@ -103,7 +103,7 @@ void loadedFromExternal() throws Exception {
                     de.muenchen.oss.wahllokalsystem.basisdatenservice.rest.wahltage.WahltagDTO[].class);
 
             val expectedResponseBody = dtoMapper
-                    .fromListOfWahltagModelToListOfWahltagDTO(wahltageClientMapper.fromRemoteClientWahltageDTOtoListOfWahltagModel(eaiWahltage));
+                    .fromListOfWahltagModelToListOfWahltagDTO(wahltageClientMapper.fromRemoteClientSetOfWahltagDTOtoListOfWahltagModel(eaiWahltage));
 
             Assertions.assertThat(responseBodyAsDTO).containsExactlyInAnyOrderElementsOf(expectedResponseBody);
         }
@@ -125,7 +125,7 @@ void externalDataIsPersisted() throws Exception {
             val dataFromRepo = wahltagRepository.findAllByOrderByWahltagAsc();
 
             val expectedListOfData = modelMapper.fromWahltagModelToWahltagEntityList(
-                    wahltageClientMapper.fromRemoteClientWahltageDTOtoListOfWahltagModel(eaiWahltage));
+                    wahltageClientMapper.fromRemoteClientSetOfWahltagDTOtoListOfWahltagModel(eaiWahltage));
 
             Assertions.assertThat(dataFromRepo)
                     .usingRecursiveComparison().ignoringCollectionOrder()
@@ -136,7 +136,7 @@ void externalDataIsPersisted() throws Exception {
         void loadFromRemoteFirstAndThanUpdateRepository() throws Exception {
             val entitiesToFindInRepository = modelMapper
                     .fromWahltagModelToWahltagEntityList(wahltageClientMapper
-                            .fromRemoteClientWahltageDTOtoListOfWahltagModel(createClientWahltageDTO(true)));
+                            .fromRemoteClientSetOfWahltagDTOtoListOfWahltagModel(createClientWahltageDTO(true)));
             val savedEntitiesInRepository_1 = wahltagRepository.saveAll(entitiesToFindInRepository);
 
             val request = MockMvcRequestBuilders.get("/businessActions/wahltage");
@@ -184,9 +184,8 @@ void technischeWlsExceptionWhenNoExternalDataFound() throws Exception {
         }
     }
 
-    private de.muenchen.oss.wahllokalsystem.basisdatenservice.eai.aou.model.WahltageDTO createClientWahltageDTO(
+    private Set<de.muenchen.oss.wahllokalsystem.basisdatenservice.eai.aou.model.WahltagDTO> createClientWahltageDTO(
             boolean beschreibungPartPreNumber) {
-        val clientWahltageDTO = new de.muenchen.oss.wahllokalsystem.basisdatenservice.eai.aou.model.WahltageDTO();
 
         val wahltag1 = new de.muenchen.oss.wahllokalsystem.basisdatenservice.eai.aou.model.WahltagDTO();
         wahltag1.setIdentifikator("identifikatorWahltag1");
@@ -206,9 +205,6 @@ private de.muenchen.oss.wahllokalsystem.basisdatenservice.eai.aou.model.Wahltage
         wahltag3.setNummer("nummerWahltag3");
         wahltag3.setTag(LocalDate.now().plusMonths(1));
 
-        val wahltage = Set.of(wahltag1, wahltag2, wahltag3);
-        clientWahltageDTO.setWahltage(wahltage);
-
-        return clientWahltageDTO;
+        return Set.of(wahltag1, wahltag2, wahltag3);
     }
 }

wls-basisdaten-service/src/test/java/de/muenchen/oss/wahllokalsystem/basisdatenservice/services/wahltag/WahltageServiceSecurityTest.java

@@ -6,7 +6,6 @@
 import de.muenchen.oss.wahllokalsystem.basisdatenservice.TestConstants;
 import de.muenchen.oss.wahllokalsystem.basisdatenservice.domain.WahltagRepository;
 import de.muenchen.oss.wahllokalsystem.basisdatenservice.eai.aou.model.WahltagDTO;
-import de.muenchen.oss.wahllokalsystem.basisdatenservice.eai.aou.model.WahltageDTO;
 import de.muenchen.oss.wahllokalsystem.basisdatenservice.utils.Authorities;
 import de.muenchen.oss.wahllokalsystem.wls.common.testing.SecurityUtils;
 import java.time.LocalDate;
@@ -86,8 +85,7 @@ private static Stream<Arguments> getMissingAuthoritiesVariations() {
             return SecurityUtils.buildArgumentsForMissingAuthoritiesVariations(Authorities.ALL_AUTHORITIES_DELETE_WAHLTAGE);
         }
 
-        private WahltageDTO createClientWahltageDTO() {
-            val dto = new WahltageDTO();
+        private Set<WahltagDTO> createClientWahltageDTO() {
             val wahltag1 = new WahltagDTO();
             wahltag1.setIdentifikator("identifikatorWahltag1");
             wahltag1.setBeschreibung("beschreibungWahltag1");
@@ -106,9 +104,7 @@ private WahltageDTO createClientWahltageDTO() {
             wahltag3.setNummer("nummerWahltag3");
             wahltag3.setTag(LocalDate.now().plusMonths(1));
 
-            dto.setWahltage(Set.of(wahltag3, wahltag2, wahltag1));
-
-            return dto;
+            return Set.of(wahltag3, wahltag2, wahltag1);
         }
     }
 }