From 1645e71670929f9a971cd6e674c09b814c6d2281 Mon Sep 17 00:00:00 2001
From: OZGCloud <ozgcloud@mgm-tp.com>
Date: Fri, 1 Jul 2022 17:28:50 +0200
Subject: [PATCH] OZG-2566 OZG-2684 tiny cleanup on kommentar; adjust
FIELD_PATH_PREFIX in client for validation messages
---
.../kommentar-form/kommentar.formservice.ts | 3 ++-
.../wiedervorlage-form/wiedervorlage.formservice.ts | 2 +-
.../goofy/kommentar/KommentarCommandController.java | 4 ++--
.../de/itvsh/goofy/kommentar/KommentarService.java | 8 ++++----
.../kommentar/KommentarCommandControllerTest.java | 4 ++--
.../goofy/kommentar/KommentarCommandITCase.java | 12 ++++--------
.../itvsh/goofy/kommentar/KommentarServiceTest.java | 4 ++--
7 files changed, 17 insertions(+), 20 deletions(-)
diff --git a/goofy-client/libs/kommentar/src/lib/kommentar-list-in-vorgang-container/kommentar-form/kommentar.formservice.ts b/goofy-client/libs/kommentar/src/lib/kommentar-list-in-vorgang-container/kommentar-form/kommentar.formservice.ts
index e348fc3066..da289d23b0 100644
--- a/goofy-client/libs/kommentar/src/lib/kommentar-list-in-vorgang-container/kommentar-form/kommentar.formservice.ts
+++ b/goofy-client/libs/kommentar/src/lib/kommentar-list-in-vorgang-container/kommentar-form/kommentar.formservice.ts
@@ -8,7 +8,8 @@ import { Observable } from "rxjs";
export class KommentarFormService extends AbstractFormService {
static readonly TEXT = 'text';
- static readonly FIELD_PATH_PREFIX = 'command.body';
+
+ static readonly FIELD_PATH_PREFIX = 'kommentar';
constructor(formBuilder: FormBuilder, private kommentarService: KommentarService) {
super(formBuilder);
diff --git a/goofy-client/libs/wiedervorlage/src/lib/wiedervorlage-page-container/wiedervorlage-page/wiedervorlage-form/wiedervorlage.formservice.ts b/goofy-client/libs/wiedervorlage/src/lib/wiedervorlage-page-container/wiedervorlage-page/wiedervorlage-form/wiedervorlage.formservice.ts
index 57ab3de2f3..5ab808bee6 100644
--- a/goofy-client/libs/wiedervorlage/src/lib/wiedervorlage-page-container/wiedervorlage-page/wiedervorlage-form/wiedervorlage.formservice.ts
+++ b/goofy-client/libs/wiedervorlage/src/lib/wiedervorlage-page-container/wiedervorlage-page/wiedervorlage-form/wiedervorlage.formservice.ts
@@ -12,7 +12,7 @@ export class WiedervorlageFormService extends AbstractFormService {
static readonly FIELD_FRIST = 'frist';
static readonly FIELD_ATTACHMENTS = 'attachments';
- static readonly FIELD_PATH_PREFIX = 'command.body';
+ static readonly FIELD_PATH_PREFIX = 'wiedervorlage';
constructor(
formBuilder: FormBuilder,
diff --git a/goofy-server/src/main/java/de/itvsh/goofy/kommentar/KommentarCommandController.java b/goofy-server/src/main/java/de/itvsh/goofy/kommentar/KommentarCommandController.java
index c9e93df22a..3942590413 100644
--- a/goofy-server/src/main/java/de/itvsh/goofy/kommentar/KommentarCommandController.java
+++ b/goofy-server/src/main/java/de/itvsh/goofy/kommentar/KommentarCommandController.java
@@ -25,7 +25,7 @@ public class KommentarCommandController {
@PostMapping
public ResponseEntity<Void> editKommentar(@RequestBody KommentarCommand kommentarCommand, @PathVariable String kommentarId,
@PathVariable long kommentarVersion) {
- var createdCommand = service.editKommentar(kommentarCommand, kommentarId, kommentarVersion);
+ var createdCommand = service.editKommentar(kommentarCommand.getKommentar(), kommentarId, kommentarVersion);
return buildResponseLink(createdCommand);
}
@@ -45,7 +45,7 @@ public class KommentarCommandController {
@PostMapping
public ResponseEntity<Void> createKommentar(@RequestBody KommentarCommand command, @PathVariable String vorgangId) {
- var createdCommand = service.createKommentar(command, vorgangId);
+ var createdCommand = service.createKommentar(command.getKommentar(), vorgangId);
return ResponseEntity.created(linkTo(CommandController.class).slash(createdCommand.getId()).toUri()).build();
}
diff --git a/goofy-server/src/main/java/de/itvsh/goofy/kommentar/KommentarService.java b/goofy-server/src/main/java/de/itvsh/goofy/kommentar/KommentarService.java
index ccd15c57ea..1782047c75 100644
--- a/goofy-server/src/main/java/de/itvsh/goofy/kommentar/KommentarService.java
+++ b/goofy-server/src/main/java/de/itvsh/goofy/kommentar/KommentarService.java
@@ -24,8 +24,8 @@ class KommentarService {
@Autowired
private VorgangAttachedItemService vorgangAttachedItemService;
- public Command createKommentar(@Valid KommentarCommand kommentarCommand, String vorgangId) {
- return vorgangAttachedItemService.createNewKommentar(addCreated(kommentarCommand.getKommentar()), vorgangId);
+ public Command createKommentar(@Valid Kommentar kommentar, String vorgangId) {
+ return vorgangAttachedItemService.createNewKommentar(addCreated(kommentar), vorgangId);
}
Kommentar addCreated(Kommentar kommentar) {
@@ -35,9 +35,9 @@ class KommentarService {
.build();
}
- public Command editKommentar(@Valid KommentarCommand kommentarCommand, String kommentarId, long kommentarVersion) {
+ public Command editKommentar(@Valid Kommentar kommentar, String kommentarId, long kommentarVersion) {
var loadedKommentar = getById(kommentarId);
- var preparedKommentar = loadedKommentar.toBuilder().text(kommentarCommand.getKommentar().getText()).build();
+ var preparedKommentar = loadedKommentar.toBuilder().text(kommentar.getText()).build();
return vorgangAttachedItemService.editKommentar(preparedKommentar, kommentarId, kommentarVersion);
}
diff --git a/goofy-server/src/test/java/de/itvsh/goofy/kommentar/KommentarCommandControllerTest.java b/goofy-server/src/test/java/de/itvsh/goofy/kommentar/KommentarCommandControllerTest.java
index 51b2074e86..ada88241b2 100644
--- a/goofy-server/src/test/java/de/itvsh/goofy/kommentar/KommentarCommandControllerTest.java
+++ b/goofy-server/src/test/java/de/itvsh/goofy/kommentar/KommentarCommandControllerTest.java
@@ -53,7 +53,7 @@ class KommentarCommandControllerTest {
void shouldCallService() throws Exception {
doRequest();
- verify(service).createKommentar(any(KommentarCommand.class), eq(VorgangHeaderTestFactory.ID));
+ verify(service).createKommentar(any(Kommentar.class), eq(VorgangHeaderTestFactory.ID));
}
@Test
@@ -93,7 +93,7 @@ class KommentarCommandControllerTest {
void shouldCallService() throws Exception {
doRequest();
- verify(service).editKommentar(any(KommentarCommand.class), eq(KommentarTestFactory.ID), eq(KommentarTestFactory.VERSION));
+ verify(service).editKommentar(any(Kommentar.class), eq(KommentarTestFactory.ID), eq(KommentarTestFactory.VERSION));
}
@Test
diff --git a/goofy-server/src/test/java/de/itvsh/goofy/kommentar/KommentarCommandITCase.java b/goofy-server/src/test/java/de/itvsh/goofy/kommentar/KommentarCommandITCase.java
index f6ca7e3774..ef360511de 100644
--- a/goofy-server/src/test/java/de/itvsh/goofy/kommentar/KommentarCommandITCase.java
+++ b/goofy-server/src/test/java/de/itvsh/goofy/kommentar/KommentarCommandITCase.java
@@ -61,8 +61,6 @@ class KommentarCommandITCase {
@Nested
class TestValidation {
- private static final String FIELD_PREFIX = "kommentarCommand.kommentar";
-
@DisplayName("for null Text")
@Test
void createCommandWithInvalidText() throws Exception {
@@ -70,7 +68,7 @@ class KommentarCommandITCase {
doRequestByKommentarId(content).andExpect(status().isUnprocessableEntity())
.andExpect(jsonPath("$.issues.length()").value(1))
- .andExpect(jsonPath("$.issues.[0].field").value(FIELD_PREFIX + ".text"))
+ .andExpect(jsonPath("$.issues.[0].field").value("kommentar.text"))
.andExpect(jsonPath("$.issues.[0].messageCode").value(ValidationMessageCodes.FIELD_IS_EMPTY));
}
@@ -80,7 +78,7 @@ class KommentarCommandITCase {
String content = buildContentWithText(StringUtils.EMPTY);
doRequestByKommentarId(content).andExpect(status().isUnprocessableEntity())
- .andExpect(jsonPath("$.issues.[0].field").value(FIELD_PREFIX + ".text"));
+ .andExpect(jsonPath("$.issues.[0].field").value("kommentar.text"));
}
@@ -132,8 +130,6 @@ class KommentarCommandITCase {
@Nested
class TestValidation {
- private static final String FIELD_PREFIX = "kommentarCommand.kommentar";
-
@DisplayName("for null Text")
@Test
void createCommandWithInvalidText() throws Exception {
@@ -141,7 +137,7 @@ class KommentarCommandITCase {
doRequestByVorgangId(content).andExpect(status().isUnprocessableEntity())
.andExpect(jsonPath("$.issues.length()").value(1))
- .andExpect(jsonPath("$.issues.[0].field").value(FIELD_PREFIX + ".text"))
+ .andExpect(jsonPath("$.issues.[0].field").value("kommentar.text"))
.andExpect(jsonPath("$.issues.[0].messageCode").value(ValidationMessageCodes.FIELD_IS_EMPTY));
}
@@ -151,7 +147,7 @@ class KommentarCommandITCase {
String content = buildContentWithText(StringUtils.EMPTY);
doRequestByVorgangId(content).andExpect(status().isUnprocessableEntity())
- .andExpect(jsonPath("$.issues.[0].field").value(FIELD_PREFIX + ".text"));
+ .andExpect(jsonPath("$.issues.[0].field").value("kommentar.text"));
}
diff --git a/goofy-server/src/test/java/de/itvsh/goofy/kommentar/KommentarServiceTest.java b/goofy-server/src/test/java/de/itvsh/goofy/kommentar/KommentarServiceTest.java
index 0dae370862..ed1fcd6c06 100644
--- a/goofy-server/src/test/java/de/itvsh/goofy/kommentar/KommentarServiceTest.java
+++ b/goofy-server/src/test/java/de/itvsh/goofy/kommentar/KommentarServiceTest.java
@@ -97,7 +97,7 @@ class KommentarServiceTest {
}
private Command callCreateKommentar() {
- return service.createKommentar(KommentarCommandTestFactory.create(), VorgangHeaderTestFactory.ID);
+ return service.createKommentar(KommentarTestFactory.create(), VorgangHeaderTestFactory.ID);
}
}
@@ -137,7 +137,7 @@ class KommentarServiceTest {
}
private Command callEditKommentar() {
- return service.editKommentar(KommentarCommandTestFactory.create(), KommentarTestFactory.ID, KommentarTestFactory.VERSION);
+ return service.editKommentar(KommentarTestFactory.create(), KommentarTestFactory.ID, KommentarTestFactory.VERSION);
}
}
--
GitLab