Skip to content
Snippets Groups Projects
Commit ec11005b authored by OZGCloud's avatar OZGCloud
Browse files

OZG-2865 do not considering "currentEditedWiedervorlage" on calculation

parent 221d4330
No related branches found
No related tags found
No related merge requests found
...@@ -29,7 +29,7 @@ public class WiedervorlageCommandController { ...@@ -29,7 +29,7 @@ public class WiedervorlageCommandController {
var wiedervorlage = service.getById(wiedervorlageId); var wiedervorlage = service.getById(wiedervorlageId);
var createdCommand = createCommand(wiedervorlage, command); 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(); return ResponseEntity.created(linkTo(CommandController.class).slash(createdCommand.getId()).toUri()).build();
} }
...@@ -71,7 +71,7 @@ public class WiedervorlageCommandController { ...@@ -71,7 +71,7 @@ public class WiedervorlageCommandController {
public ResponseEntity<Void> createWiedervorlage(@RequestBody WiedervorlageCommand command, @PathVariable String vorgangId) { public ResponseEntity<Void> createWiedervorlage(@RequestBody WiedervorlageCommand command, @PathVariable String vorgangId) {
var createdCommand = service.createWiedervorlage(command.getWiedervorlage(), 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(); return ResponseEntity.created(linkTo(CommandController.class).slash(createdCommand.getId()).toUri()).build();
} }
......
...@@ -52,10 +52,8 @@ class WiedervorlageService { ...@@ -52,10 +52,8 @@ class WiedervorlageService {
} }
@Async @Async
public void updateNextFrist(Wiedervorlage newWiedervorlage, String vorgangId) { public void updateNextFrist(String vorgangId) {
var allWiedervorlagen = Stream.concat(Stream.of(newWiedervorlage), findByVorgangId(vorgangId)); remoteService.updateNextFrist(vorgangId, calculateNextFrist(findByVorgangId(vorgangId)));
remoteService.updateNextFrist(vorgangId, calculateNextFrist(allWiedervorlagen));
} }
Optional<LocalDate> calculateNextFrist(Stream<Wiedervorlage> wiedervorlagen) { Optional<LocalDate> calculateNextFrist(Stream<Wiedervorlage> wiedervorlagen) {
......
package de.itvsh.goofy.wiedervorlage; package de.itvsh.goofy.wiedervorlage;
import static org.assertj.core.api.Assertions.*;
import static org.mockito.ArgumentMatchers.*; import static org.mockito.ArgumentMatchers.*;
import static org.mockito.Mockito.*; import static org.mockito.Mockito.*;
import static org.springframework.test.web.servlet.request.MockMvcRequestBuilders.*; import static org.springframework.test.web.servlet.request.MockMvcRequestBuilders.*;
...@@ -19,8 +20,6 @@ import org.springframework.test.web.servlet.MockMvc; ...@@ -19,8 +20,6 @@ import org.springframework.test.web.servlet.MockMvc;
import org.springframework.test.web.servlet.ResultActions; import org.springframework.test.web.servlet.ResultActions;
import org.springframework.test.web.servlet.setup.MockMvcBuilders; 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.binaryfile.BinaryFileTestFactory;
import de.itvsh.goofy.common.command.CommandController.CommandByRelationController; import de.itvsh.goofy.common.command.CommandController.CommandByRelationController;
import de.itvsh.goofy.common.command.CommandTestFactory; import de.itvsh.goofy.common.command.CommandTestFactory;
...@@ -77,7 +76,7 @@ class WiedervorlageCommandByVorgangControllerTest { ...@@ -77,7 +76,7 @@ class WiedervorlageCommandByVorgangControllerTest {
void shouldCallServiceToUpdateNextFrist() throws Exception { void shouldCallServiceToUpdateNextFrist() throws Exception {
doRequest(); doRequest();
verify(service).updateNextFrist(any(Wiedervorlage.class), eq(VorgangHeaderTestFactory.ID)); verify(service).updateNextFrist(VorgangHeaderTestFactory.ID);
} }
@Test @Test
......
package de.itvsh.goofy.wiedervorlage; package de.itvsh.goofy.wiedervorlage;
import static org.assertj.core.api.Assertions.*;
import static org.mockito.ArgumentMatchers.*; import static org.mockito.ArgumentMatchers.*;
import static org.mockito.Mockito.*; import static org.mockito.Mockito.*;
import static org.springframework.test.web.servlet.request.MockMvcRequestBuilders.*; import static org.springframework.test.web.servlet.request.MockMvcRequestBuilders.*;
...@@ -21,8 +22,6 @@ import org.springframework.test.web.servlet.MockMvc; ...@@ -21,8 +22,6 @@ import org.springframework.test.web.servlet.MockMvc;
import org.springframework.test.web.servlet.ResultActions; import org.springframework.test.web.servlet.ResultActions;
import org.springframework.test.web.servlet.setup.MockMvcBuilders; 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.binaryfile.FileId;
import de.itvsh.goofy.common.command.Command; import de.itvsh.goofy.common.command.Command;
import de.itvsh.goofy.common.command.CommandOrder; import de.itvsh.goofy.common.command.CommandOrder;
...@@ -81,7 +80,7 @@ class WiedervorlageCommandControllerTest { ...@@ -81,7 +80,7 @@ class WiedervorlageCommandControllerTest {
void shouldCallServiceUpdateNextFrist() throws Exception { void shouldCallServiceUpdateNextFrist() throws Exception {
doRequest(); doRequest();
verify(service).updateNextFrist(any(Wiedervorlage.class), eq(VorgangHeaderTestFactory.ID)); verify(service).updateNextFrist(VorgangHeaderTestFactory.ID);
} }
@Test @Test
......
...@@ -183,7 +183,7 @@ class WiedervorlageServiceTest { ...@@ -183,7 +183,7 @@ class WiedervorlageServiceTest {
} }
private void callUpdateNextFrist() { private void callUpdateNextFrist() {
service.updateNextFrist(WiedervorlageTestFactory.create(), VorgangHeaderTestFactory.ID); service.updateNextFrist(VorgangHeaderTestFactory.ID);
} }
} }
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment