Skip to content
Snippets Groups Projects
Commit 95d25b52 authored by Felix Reichenbach's avatar Felix Reichenbach
Browse files

OZG-7038 apply code review comments

parent 124ca097
No related branches found
No related tags found
1 merge request!3Ozg 7038 evaluation dms quittung
...@@ -18,7 +18,6 @@ public class XtaImportConfirmationHandler { ...@@ -18,7 +18,6 @@ public class XtaImportConfirmationHandler {
private XtaImportConfirmationHandler(XtaFile file) { private XtaImportConfirmationHandler(XtaFile file) {
importConfirmation = XtaFileReader.createAbgabeImportBestaetigen0402Reader().read(file); importConfirmation = XtaFileReader.createAbgabeImportBestaetigen0402Reader().read(file);
} }
public Stream<XtaAbgabeAdapter> getAbgaben() { public Stream<XtaAbgabeAdapter> getAbgaben() {
...@@ -38,7 +37,7 @@ public class XtaImportConfirmationHandler { ...@@ -38,7 +37,7 @@ public class XtaImportConfirmationHandler {
return abgabeType.isErfolgreich(); return abgabeType.isErfolgreich();
} }
public String getFehlermeldung() { public String getErrorMessage() {
return abgabeType.getFehlermeldung().stream().collect(Collectors.joining(";")); return abgabeType.getFehlermeldung().stream().collect(Collectors.joining(";"));
} }
} }
......
...@@ -81,6 +81,6 @@ class QuittungService { ...@@ -81,6 +81,6 @@ class QuittungService {
} }
void publishCommandFailedEvent(XtaAbgabeAdapter contentHandler, Command pendingCommand) { void publishCommandFailedEvent(XtaAbgabeAdapter contentHandler, Command pendingCommand) {
eventPublisher.publishEvent(new CommandFailedEvent(pendingCommand.getId(), contentHandler.getFehlermeldung())); eventPublisher.publishEvent(new CommandFailedEvent(pendingCommand.getId(), contentHandler.getErrorMessage()));
} }
} }
...@@ -54,7 +54,7 @@ class XtaFileReaderTest { ...@@ -54,7 +54,7 @@ class XtaFileReaderTest {
@SneakyThrows @SneakyThrows
void shouldThrowTechnicalException() { void shouldThrowTechnicalException() {
var reader = XtaFileReader.createAbgabeImportBestaetigen0402Reader(); var reader = XtaFileReader.createAbgabeImportBestaetigen0402Reader();
XtaFile unknownFile = XtaFileTestFactory.create(); XtaFile unknownFile = XtaFileTestFactory.create(XtaFileTestFactory.DUMMY_XML_FILE_PATH);
assertThrows(TechnicalException.class, () -> reader.read(unknownFile)); assertThrows(TechnicalException.class, () -> reader.read(unknownFile));
} }
......
...@@ -102,7 +102,7 @@ class XtaImportConfirmationHandlerTest { ...@@ -102,7 +102,7 @@ class XtaImportConfirmationHandlerTest {
@Test @Test
void shouldReturnErrorMessage() { void shouldReturnErrorMessage() {
var errorMessage = initAbgabeHandler().getFehlermeldung(); var errorMessage = initAbgabeHandler().getErrorMessage();
assertThat(errorMessage).isEqualTo(AbgabeImportBestaetigen0402TestFactory.FEHLERMELDUNG); assertThat(errorMessage).isEqualTo(AbgabeImportBestaetigen0402TestFactory.FEHLERMELDUNG);
} }
......
...@@ -80,7 +80,7 @@ class QuittungServiceTest { ...@@ -80,7 +80,7 @@ class QuittungServiceTest {
} }
} }
@DisplayName("Consume message") @DisplayName("Consume Quittung")
@Nested @Nested
class TestConsumeQuittung { class TestConsumeQuittung {
...@@ -287,7 +287,7 @@ class QuittungServiceTest { ...@@ -287,7 +287,7 @@ class QuittungServiceTest {
@DisplayName("should publish command failed event") @DisplayName("should publish command failed event")
@Test @Test
void shouldPublishCommandFailedEvent() { void shouldPublishCommandFailedEvent() {
when(abgabeHandler.getFehlermeldung()).thenReturn(AbgabeImportBestaetigen0402TestFactory.FEHLERMELDUNG); when(abgabeHandler.getErrorMessage()).thenReturn(AbgabeImportBestaetigen0402TestFactory.FEHLERMELDUNG);
service.publishCommandFailedEvent(abgabeHandler, command); service.publishCommandFailedEvent(abgabeHandler, command);
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment