From 71d03eb45ce8c24981b2a24f01e64b90c424dab6 Mon Sep 17 00:00:00 2001 From: vjohnslhm <145964798+vjohnslhm@users.noreply.github.com> Date: Tue, 12 Nov 2024 17:15:00 +0100 Subject: [PATCH] #138 :art:, :bulb: handle todos --- .../wahlvorstandservice/clients/DummyClientImpl.java | 2 +- .../clients/aoueai/WahlvorstandClientMapper.java | 4 ++-- .../rest/wahlvorstand/WahlvorstandController.java | 4 ++-- .../rest/wahlvorstand/WahlvorstandControllerTest.java | 6 +++--- .../src/test/resources/http.request/wahlvorstand.http | 8 ++++++-- 5 files changed, 14 insertions(+), 10 deletions(-) diff --git a/wls-wahlvorstand-service/src/main/java/de/muenchen/oss/wahllokalsystem/wahlvorstandservice/clients/DummyClientImpl.java b/wls-wahlvorstand-service/src/main/java/de/muenchen/oss/wahllokalsystem/wahlvorstandservice/clients/DummyClientImpl.java index 27c9a640e..635f8c7e7 100644 --- a/wls-wahlvorstand-service/src/main/java/de/muenchen/oss/wahllokalsystem/wahlvorstandservice/clients/DummyClientImpl.java +++ b/wls-wahlvorstand-service/src/main/java/de/muenchen/oss/wahllokalsystem/wahlvorstandservice/clients/DummyClientImpl.java @@ -21,7 +21,7 @@ @Slf4j @Component -@Profile(Profiles.DUMMY_CLIENTS) // todo: das profil dürfte hier eigentlich nicht manuell hinzugefügt werden +@Profile(Profiles.DUMMY_CLIENTS) public class DummyClientImpl implements WahlvorstandEaiClient, WahlenClient, KonfigurierterWahltagClient { @Override public WahlvorstandModel getWahlvorstand(String wahlbezirkID, LocalDate wahltag) { diff --git a/wls-wahlvorstand-service/src/main/java/de/muenchen/oss/wahllokalsystem/wahlvorstandservice/clients/aoueai/WahlvorstandClientMapper.java b/wls-wahlvorstand-service/src/main/java/de/muenchen/oss/wahllokalsystem/wahlvorstandservice/clients/aoueai/WahlvorstandClientMapper.java index d6b3fb3d2..4ef27616a 100644 --- a/wls-wahlvorstand-service/src/main/java/de/muenchen/oss/wahllokalsystem/wahlvorstandservice/clients/aoueai/WahlvorstandClientMapper.java +++ b/wls-wahlvorstand-service/src/main/java/de/muenchen/oss/wahllokalsystem/wahlvorstandservice/clients/aoueai/WahlvorstandClientMapper.java @@ -16,12 +16,12 @@ public interface WahlvorstandClientMapper { ZoneId ZONE_ID = ZoneId.systemDefault(); - @Mapping(target = "anwesenheitBeginn", ignore = true) // todo: darf das ignoriert werden? + @Mapping(target = "anwesenheitBeginn", ignore = true) @Mapping(target = "wahlvorstandsmitglieder", source = "mitglieder") WahlvorstandModel toModel(WahlvorstandDTO wahlvorstandDTO); @Mapping(target = "familienname", source = "nachname") - @Mapping(target = "funktionsname", ignore = true) // todo: darf das ignoriert werden? + @Mapping(target = "funktionsname", ignore = true) WahlvorstandsmitgliedModel toModel(WahlvorstandsmitgliedDTO wahlvorstandsmitgliedDTO); @Mapping(target = "mitglieder", source = "wahlvorstandsmitglieder") diff --git a/wls-wahlvorstand-service/src/main/java/de/muenchen/oss/wahllokalsystem/wahlvorstandservice/rest/wahlvorstand/WahlvorstandController.java b/wls-wahlvorstand-service/src/main/java/de/muenchen/oss/wahllokalsystem/wahlvorstandservice/rest/wahlvorstand/WahlvorstandController.java index 158cf7edc..6a5b40a11 100644 --- a/wls-wahlvorstand-service/src/main/java/de/muenchen/oss/wahllokalsystem/wahlvorstandservice/rest/wahlvorstand/WahlvorstandController.java +++ b/wls-wahlvorstand-service/src/main/java/de/muenchen/oss/wahllokalsystem/wahlvorstandservice/rest/wahlvorstand/WahlvorstandController.java @@ -47,12 +47,12 @@ public class WahlvorstandController { ) @GetMapping("/{wahlbezirkID}") public ResponseEntity getWahlvorstand( - @RequestHeader(value = "forceupdate", required = false) String forceUpdate, + @RequestHeader(value = "forceupdate", required = false) Boolean forceUpdate, @PathVariable("wahlbezirkID") String wahlbezirkID) { Optional result; - if (forceUpdate != null && forceUpdate.equals("true")) { + if (Boolean.TRUE.equals(forceUpdate)) { result = wahlvorstandService.updateWahlvorstand(wahlbezirkID); } else { result = wahlvorstandService.getWahlvorstand(wahlbezirkID); diff --git a/wls-wahlvorstand-service/src/test/java/de/muenchen/oss/wahllokalsystem/wahlvorstandservice/rest/wahlvorstand/WahlvorstandControllerTest.java b/wls-wahlvorstand-service/src/test/java/de/muenchen/oss/wahllokalsystem/wahlvorstandservice/rest/wahlvorstand/WahlvorstandControllerTest.java index 53d224261..91de605a3 100644 --- a/wls-wahlvorstand-service/src/test/java/de/muenchen/oss/wahllokalsystem/wahlvorstandservice/rest/wahlvorstand/WahlvorstandControllerTest.java +++ b/wls-wahlvorstand-service/src/test/java/de/muenchen/oss/wahllokalsystem/wahlvorstandservice/rest/wahlvorstand/WahlvorstandControllerTest.java @@ -32,7 +32,7 @@ class GetWahlvorstand { @Test void should_returnWahlvorstandDTO_when_givenValidWahlbezirkIdAndForceUpdateIsFalse() { val wahlbezirkID = "wahlbezirkID"; - val forceUpdate = "false"; + val forceUpdate = false; val mockedWahlvorstandModel = TestDataFactory.CreateWahlvorstandModel.withData(); val expectedWahlvorstandDto = TestDataFactory.CreateWahlvorstandDto.fromModel(mockedWahlvorstandModel); @@ -47,7 +47,7 @@ void should_returnWahlvorstandDTO_when_givenValidWahlbezirkIdAndForceUpdateIsFal @Test void should_returnWahlvorstandDTO_when_givenValidWahlbezirkIdAndForceUpdateIsTrue() { val wahlbezirkID = "wahlbezirkID"; - val forceUpdate = "true"; + val forceUpdate = true; val mockedWahlvorstandModel = TestDataFactory.CreateWahlvorstandModel.withData(); val expectedWahlvorstandDto = TestDataFactory.CreateWahlvorstandDto.fromModel(mockedWahlvorstandModel); @@ -62,7 +62,7 @@ void should_returnWahlvorstandDTO_when_givenValidWahlbezirkIdAndForceUpdateIsTru @Test void should_returnFallbackWahlvorstandDTO_when_WahlvorstandDoesNotExist() { val wahlbezirkID = "wahlbezirkID"; - val forceUpdate = "false"; + val forceUpdate = false; val mockedFallbackWahlvorstandModel = TestDataFactory.CreateWahlvorstandModel.fallback(wahlbezirkID); val expectedWahlvorstandDto = TestDataFactory.CreateWahlvorstandDto.fromModel(mockedFallbackWahlvorstandModel); diff --git a/wls-wahlvorstand-service/src/test/resources/http.request/wahlvorstand.http b/wls-wahlvorstand-service/src/test/resources/http.request/wahlvorstand.http index 5e49de4b3..5795ee0c1 100644 --- a/wls-wahlvorstand-service/src/test/resources/http.request/wahlvorstand.http +++ b/wls-wahlvorstand-service/src/test/resources/http.request/wahlvorstand.http @@ -32,8 +32,8 @@ username = wls_all_uwb GET {{ SSO_URL }}/auth/realms/wls_realm/protocol/openid-connect/userinfo Authorization: {{ token_type }} {{ auth_token }} -### GET wahlvorstand - working with dummy.client -GET {{ WLS_WAHLVORSTAND_SERVICE_URL }}/businessActions/wahlvorstand/wahlbezirkID +### GET wahlvorstand fallback - working with dummy.client +GET {{ WLS_WAHLVORSTAND_SERVICE_URL }}/businessActions/wahlvorstand/wahlbezirkID1 Authorization: {{ token_type }} {{ auth_token }} ### POST wahlvorstand @@ -63,3 +63,7 @@ Content-Type: application/json } ] } + +### GET wahlvorstand - working with dummy.client +GET {{ WLS_WAHLVORSTAND_SERVICE_URL }}/businessActions/wahlvorstand/wahlbezirkID +Authorization: {{ token_type }} {{ auth_token }} \ No newline at end of file