From 07a70cbf536eeb2efcd4c264d3b5e9c1441526a5 Mon Sep 17 00:00:00 2001
From: OZGCloud <ozgcloud@mgm-tp.com>
Date: Mon, 29 Apr 2024 10:58:37 +0200
Subject: [PATCH] OZG-5570: apply code review improvements
---
.../bescheid/BescheidModelAssemblerTest.java | 28 ++++++-------------
1 file changed, 9 insertions(+), 19 deletions(-)
diff --git a/alfa-service/src/test/java/de/ozgcloud/alfa/bescheid/BescheidModelAssemblerTest.java b/alfa-service/src/test/java/de/ozgcloud/alfa/bescheid/BescheidModelAssemblerTest.java
index b89bf7cbd6..33ced87d6c 100644
--- a/alfa-service/src/test/java/de/ozgcloud/alfa/bescheid/BescheidModelAssemblerTest.java
+++ b/alfa-service/src/test/java/de/ozgcloud/alfa/bescheid/BescheidModelAssemblerTest.java
@@ -204,7 +204,7 @@ class BescheidModelAssemblerTest {
@Test
void shouldReturnFalse() {
- var hasEnabled = callProcessor(VorgangWithEingangTestFactory.create());
+ var hasEnabled = callAssembler(VorgangWithEingangTestFactory.create());
assertThat(hasEnabled).isFalse();
}
@@ -226,14 +226,14 @@ class BescheidModelAssemblerTest {
.formEngineName("different").build())
.build());
- var hasEnabled = callProcessor(vorgang);
+ var hasEnabled = callAssembler(vorgang);
assertThat(hasEnabled).isFalse();
}
@Test
void shouldReturnTrue() {
- var hasEnabled = callProcessor(VorgangWithEingangTestFactory.create());
+ var hasEnabled = callAssembler(VorgangWithEingangTestFactory.create());
assertThat(hasEnabled).isTrue();
}
@@ -243,7 +243,7 @@ class BescheidModelAssemblerTest {
void shouldReturnFalseOnEmptyEingang() {
var vorgangWithEmptyEingang = createVorgang(null);
- var hasEnabled = callProcessor(vorgangWithEmptyEingang);
+ var hasEnabled = callAssembler(vorgangWithEmptyEingang);
assertThat(hasEnabled).isFalse();
}
@@ -252,7 +252,7 @@ class BescheidModelAssemblerTest {
void shouldReturnFalseOnEmptyEingangHeader() {
var vorgangWithEmptyEingangHeader = EingangTestFactory.createBuilder().header(null).build();
- var hasEnabled = callProcessor(createVorgang(vorgangWithEmptyEingangHeader));
+ var hasEnabled = callAssembler(createVorgang(vorgangWithEmptyEingangHeader));
assertThat(hasEnabled).isFalse();
}
@@ -262,7 +262,7 @@ class BescheidModelAssemblerTest {
var vorgangWithEmptyFormEngineName = createVorgang(
EingangTestFactory.createBuilder().header(EingangHeaderTestFactory.createBuilder().formEngineName(null).build()).build());
- var hasEnabled = callProcessor(vorgangWithEmptyFormEngineName);
+ var hasEnabled = callAssembler(vorgangWithEmptyFormEngineName);
assertThat(hasEnabled).isFalse();
}
@@ -272,12 +272,12 @@ class BescheidModelAssemblerTest {
var vorgangWithEmptyFormId = createVorgang(
EingangTestFactory.createBuilder().header(EingangHeaderTestFactory.createBuilder().formId(null).build()).build());
- var hasEnabled = callProcessor(vorgangWithEmptyFormId);
+ var hasEnabled = callAssembler(vorgangWithEmptyFormId);
assertThat(hasEnabled).isFalse();
}
- private boolean callProcessor(VorgangWithEingang vorgang) {
+ private boolean callAssembler(VorgangWithEingang vorgang) {
return assembler.hasVorgangCreateBescheidDocumentEnabled(vorgang);
}
@@ -293,7 +293,7 @@ class BescheidModelAssemblerTest {
private final VorgangWithEingang vorgangWithEingang = VorgangWithEingangTestFactory.create();
@Test
- void shoulReturnTrueIfServiceConfigured() {
+ void shouldReturnTrueIfServiceConfigured() {
doReturn(true).when(assembler).hasVorgangCreateBescheidDocumentEnabled(vorgangWithEingang);
when(bescheidService.canCreateBescheidDocumentAutomatically()).thenReturn(true);
@@ -311,16 +311,6 @@ class BescheidModelAssemblerTest {
assertThat(canCreate).isFalse();
}
- @Test
- void shouldTrueIfVorgangConfigured() {
- doReturn(true).when(assembler).hasVorgangCreateBescheidDocumentEnabled(vorgangWithEingang);
- when(bescheidService.canCreateBescheidDocumentAutomatically()).thenReturn(true);
-
- var canCreate = assembler.canCreateBescheidDocumentAutomatically(vorgangWithEingang);
-
- assertThat(canCreate).isTrue();
- }
-
@Test
void shouldReturnFalseIfVorgangNotConfigured() {
doReturn(false).when(assembler).hasVorgangCreateBescheidDocumentEnabled(vorgangWithEingang);
--
GitLab