From ec11005b56202398d4eebbc60c9d41d6a263d802 Mon Sep 17 00:00:00 2001
From: OZGCloud <ozgcloud@mgm-tp.com>
Date: Fri, 2 Sep 2022 20:23:27 +0200
Subject: [PATCH] OZG-2865 do not considering "currentEditedWiedervorlage" on
calculation
---
.../goofy/wiedervorlage/WiedervorlageCommandController.java | 4 ++--
.../de/itvsh/goofy/wiedervorlage/WiedervorlageService.java | 6 ++----
.../WiedervorlageCommandByVorgangControllerTest.java | 5 ++---
.../wiedervorlage/WiedervorlageCommandControllerTest.java | 5 ++---
.../itvsh/goofy/wiedervorlage/WiedervorlageServiceTest.java | 2 +-
5 files changed, 9 insertions(+), 13 deletions(-)
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 63b9498a82..1ec7cafa46 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 375a9d4e26..5eaae1e296 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 a07591594f..b7f274ded3 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 c9d8c0be62..61ed296c52 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 101f5026b2..163839710f 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);
}
}
--
GitLab