diff --git a/goofy-server/src/main/java/de/itvsh/goofy/wiedervorlage/WiedervorlageCommandController.java b/goofy-server/src/main/java/de/itvsh/goofy/wiedervorlage/WiedervorlageCommandController.java index 63b9498a8242280124eccb5ef883d728346e29d1..1ec7cafa46f5428db2d341f2471d15405d88f5eb 100644 --- a/goofy-server/src/main/java/de/itvsh/goofy/wiedervorlage/WiedervorlageCommandController.java +++ b/goofy-server/src/main/java/de/itvsh/goofy/wiedervorlage/WiedervorlageCommandController.java @@ -29,7 +29,7 @@ public class WiedervorlageCommandController { var wiedervorlage = service.getById(wiedervorlageId); var createdCommand = createCommand(wiedervorlage, command); - service.updateNextFrist(command.getWiedervorlage(), wiedervorlage.getVorgangId()); + service.updateNextFrist(wiedervorlage.getVorgangId()); return ResponseEntity.created(linkTo(CommandController.class).slash(createdCommand.getId()).toUri()).build(); } @@ -71,7 +71,7 @@ public class WiedervorlageCommandController { public ResponseEntity<Void> createWiedervorlage(@RequestBody WiedervorlageCommand command, @PathVariable String vorgangId) { var createdCommand = service.createWiedervorlage(command.getWiedervorlage(), vorgangId); - service.updateNextFrist(command.getWiedervorlage(), vorgangId); + service.updateNextFrist(vorgangId); return ResponseEntity.created(linkTo(CommandController.class).slash(createdCommand.getId()).toUri()).build(); } diff --git a/goofy-server/src/main/java/de/itvsh/goofy/wiedervorlage/WiedervorlageService.java b/goofy-server/src/main/java/de/itvsh/goofy/wiedervorlage/WiedervorlageService.java index 375a9d4e263d889100829ff51c8005c412b31294..5eaae1e2961bcb395cc3adc1727c40eba8d35bdb 100644 --- a/goofy-server/src/main/java/de/itvsh/goofy/wiedervorlage/WiedervorlageService.java +++ b/goofy-server/src/main/java/de/itvsh/goofy/wiedervorlage/WiedervorlageService.java @@ -52,10 +52,8 @@ class WiedervorlageService { } @Async - public void updateNextFrist(Wiedervorlage newWiedervorlage, String vorgangId) { - var allWiedervorlagen = Stream.concat(Stream.of(newWiedervorlage), findByVorgangId(vorgangId)); - - remoteService.updateNextFrist(vorgangId, calculateNextFrist(allWiedervorlagen)); + public void updateNextFrist(String vorgangId) { + remoteService.updateNextFrist(vorgangId, calculateNextFrist(findByVorgangId(vorgangId))); } Optional<LocalDate> calculateNextFrist(Stream<Wiedervorlage> wiedervorlagen) { diff --git a/goofy-server/src/test/java/de/itvsh/goofy/wiedervorlage/WiedervorlageCommandByVorgangControllerTest.java b/goofy-server/src/test/java/de/itvsh/goofy/wiedervorlage/WiedervorlageCommandByVorgangControllerTest.java index a07591594fc0a0f50001706deffcef3a7f19a320..b7f274ded3b8cb8e7affbc43da5914b63815307e 100644 --- a/goofy-server/src/test/java/de/itvsh/goofy/wiedervorlage/WiedervorlageCommandByVorgangControllerTest.java +++ b/goofy-server/src/test/java/de/itvsh/goofy/wiedervorlage/WiedervorlageCommandByVorgangControllerTest.java @@ -1,5 +1,6 @@ package de.itvsh.goofy.wiedervorlage; +import static org.assertj.core.api.Assertions.*; import static org.mockito.ArgumentMatchers.*; import static org.mockito.Mockito.*; import static org.springframework.test.web.servlet.request.MockMvcRequestBuilders.*; @@ -19,8 +20,6 @@ import org.springframework.test.web.servlet.MockMvc; import org.springframework.test.web.servlet.ResultActions; import org.springframework.test.web.servlet.setup.MockMvcBuilders; -import static org.assertj.core.api.Assertions.*; - import de.itvsh.goofy.common.binaryfile.BinaryFileTestFactory; import de.itvsh.goofy.common.command.CommandController.CommandByRelationController; import de.itvsh.goofy.common.command.CommandTestFactory; @@ -77,7 +76,7 @@ class WiedervorlageCommandByVorgangControllerTest { void shouldCallServiceToUpdateNextFrist() throws Exception { doRequest(); - verify(service).updateNextFrist(any(Wiedervorlage.class), eq(VorgangHeaderTestFactory.ID)); + verify(service).updateNextFrist(VorgangHeaderTestFactory.ID); } @Test diff --git a/goofy-server/src/test/java/de/itvsh/goofy/wiedervorlage/WiedervorlageCommandControllerTest.java b/goofy-server/src/test/java/de/itvsh/goofy/wiedervorlage/WiedervorlageCommandControllerTest.java index c9d8c0be62bdb488a2ffce153c943c3edf496f4a..61ed296c5225bc556c4aea48209823f7cf1bf673 100644 --- a/goofy-server/src/test/java/de/itvsh/goofy/wiedervorlage/WiedervorlageCommandControllerTest.java +++ b/goofy-server/src/test/java/de/itvsh/goofy/wiedervorlage/WiedervorlageCommandControllerTest.java @@ -1,5 +1,6 @@ package de.itvsh.goofy.wiedervorlage; +import static org.assertj.core.api.Assertions.*; import static org.mockito.ArgumentMatchers.*; import static org.mockito.Mockito.*; import static org.springframework.test.web.servlet.request.MockMvcRequestBuilders.*; @@ -21,8 +22,6 @@ import org.springframework.test.web.servlet.MockMvc; import org.springframework.test.web.servlet.ResultActions; import org.springframework.test.web.servlet.setup.MockMvcBuilders; -import static org.assertj.core.api.Assertions.*; - import de.itvsh.goofy.common.binaryfile.FileId; import de.itvsh.goofy.common.command.Command; import de.itvsh.goofy.common.command.CommandOrder; @@ -81,7 +80,7 @@ class WiedervorlageCommandControllerTest { void shouldCallServiceUpdateNextFrist() throws Exception { doRequest(); - verify(service).updateNextFrist(any(Wiedervorlage.class), eq(VorgangHeaderTestFactory.ID)); + verify(service).updateNextFrist(VorgangHeaderTestFactory.ID); } @Test diff --git a/goofy-server/src/test/java/de/itvsh/goofy/wiedervorlage/WiedervorlageServiceTest.java b/goofy-server/src/test/java/de/itvsh/goofy/wiedervorlage/WiedervorlageServiceTest.java index 101f5026b28be57557199638011f0e8aab0d642f..163839710f27a10a80e3639840bc985686e84ac9 100644 --- a/goofy-server/src/test/java/de/itvsh/goofy/wiedervorlage/WiedervorlageServiceTest.java +++ b/goofy-server/src/test/java/de/itvsh/goofy/wiedervorlage/WiedervorlageServiceTest.java @@ -183,7 +183,7 @@ class WiedervorlageServiceTest { } private void callUpdateNextFrist() { - service.updateNextFrist(WiedervorlageTestFactory.create(), VorgangHeaderTestFactory.ID); + service.updateNextFrist(VorgangHeaderTestFactory.ID); } }