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

OZG-5570: apply code review improvements

parent 87e0ca0f
Branches
Tags
No related merge requests found
...@@ -204,7 +204,7 @@ class BescheidModelAssemblerTest { ...@@ -204,7 +204,7 @@ class BescheidModelAssemblerTest {
@Test @Test
void shouldReturnFalse() { void shouldReturnFalse() {
var hasEnabled = callProcessor(VorgangWithEingangTestFactory.create()); var hasEnabled = callAssembler(VorgangWithEingangTestFactory.create());
assertThat(hasEnabled).isFalse(); assertThat(hasEnabled).isFalse();
} }
...@@ -226,14 +226,14 @@ class BescheidModelAssemblerTest { ...@@ -226,14 +226,14 @@ class BescheidModelAssemblerTest {
.formEngineName("different").build()) .formEngineName("different").build())
.build()); .build());
var hasEnabled = callProcessor(vorgang); var hasEnabled = callAssembler(vorgang);
assertThat(hasEnabled).isFalse(); assertThat(hasEnabled).isFalse();
} }
@Test @Test
void shouldReturnTrue() { void shouldReturnTrue() {
var hasEnabled = callProcessor(VorgangWithEingangTestFactory.create()); var hasEnabled = callAssembler(VorgangWithEingangTestFactory.create());
assertThat(hasEnabled).isTrue(); assertThat(hasEnabled).isTrue();
} }
...@@ -243,7 +243,7 @@ class BescheidModelAssemblerTest { ...@@ -243,7 +243,7 @@ class BescheidModelAssemblerTest {
void shouldReturnFalseOnEmptyEingang() { void shouldReturnFalseOnEmptyEingang() {
var vorgangWithEmptyEingang = createVorgang(null); var vorgangWithEmptyEingang = createVorgang(null);
var hasEnabled = callProcessor(vorgangWithEmptyEingang); var hasEnabled = callAssembler(vorgangWithEmptyEingang);
assertThat(hasEnabled).isFalse(); assertThat(hasEnabled).isFalse();
} }
...@@ -252,7 +252,7 @@ class BescheidModelAssemblerTest { ...@@ -252,7 +252,7 @@ class BescheidModelAssemblerTest {
void shouldReturnFalseOnEmptyEingangHeader() { void shouldReturnFalseOnEmptyEingangHeader() {
var vorgangWithEmptyEingangHeader = EingangTestFactory.createBuilder().header(null).build(); var vorgangWithEmptyEingangHeader = EingangTestFactory.createBuilder().header(null).build();
var hasEnabled = callProcessor(createVorgang(vorgangWithEmptyEingangHeader)); var hasEnabled = callAssembler(createVorgang(vorgangWithEmptyEingangHeader));
assertThat(hasEnabled).isFalse(); assertThat(hasEnabled).isFalse();
} }
...@@ -262,7 +262,7 @@ class BescheidModelAssemblerTest { ...@@ -262,7 +262,7 @@ class BescheidModelAssemblerTest {
var vorgangWithEmptyFormEngineName = createVorgang( var vorgangWithEmptyFormEngineName = createVorgang(
EingangTestFactory.createBuilder().header(EingangHeaderTestFactory.createBuilder().formEngineName(null).build()).build()); EingangTestFactory.createBuilder().header(EingangHeaderTestFactory.createBuilder().formEngineName(null).build()).build());
var hasEnabled = callProcessor(vorgangWithEmptyFormEngineName); var hasEnabled = callAssembler(vorgangWithEmptyFormEngineName);
assertThat(hasEnabled).isFalse(); assertThat(hasEnabled).isFalse();
} }
...@@ -272,12 +272,12 @@ class BescheidModelAssemblerTest { ...@@ -272,12 +272,12 @@ class BescheidModelAssemblerTest {
var vorgangWithEmptyFormId = createVorgang( var vorgangWithEmptyFormId = createVorgang(
EingangTestFactory.createBuilder().header(EingangHeaderTestFactory.createBuilder().formId(null).build()).build()); EingangTestFactory.createBuilder().header(EingangHeaderTestFactory.createBuilder().formId(null).build()).build());
var hasEnabled = callProcessor(vorgangWithEmptyFormId); var hasEnabled = callAssembler(vorgangWithEmptyFormId);
assertThat(hasEnabled).isFalse(); assertThat(hasEnabled).isFalse();
} }
private boolean callProcessor(VorgangWithEingang vorgang) { private boolean callAssembler(VorgangWithEingang vorgang) {
return assembler.hasVorgangCreateBescheidDocumentEnabled(vorgang); return assembler.hasVorgangCreateBescheidDocumentEnabled(vorgang);
} }
...@@ -293,7 +293,7 @@ class BescheidModelAssemblerTest { ...@@ -293,7 +293,7 @@ class BescheidModelAssemblerTest {
private final VorgangWithEingang vorgangWithEingang = VorgangWithEingangTestFactory.create(); private final VorgangWithEingang vorgangWithEingang = VorgangWithEingangTestFactory.create();
@Test @Test
void shoulReturnTrueIfServiceConfigured() { void shouldReturnTrueIfServiceConfigured() {
doReturn(true).when(assembler).hasVorgangCreateBescheidDocumentEnabled(vorgangWithEingang); doReturn(true).when(assembler).hasVorgangCreateBescheidDocumentEnabled(vorgangWithEingang);
when(bescheidService.canCreateBescheidDocumentAutomatically()).thenReturn(true); when(bescheidService.canCreateBescheidDocumentAutomatically()).thenReturn(true);
...@@ -311,16 +311,6 @@ class BescheidModelAssemblerTest { ...@@ -311,16 +311,6 @@ class BescheidModelAssemblerTest {
assertThat(canCreate).isFalse(); 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 @Test
void shouldReturnFalseIfVorgangNotConfigured() { void shouldReturnFalseIfVorgangNotConfigured() {
doReturn(false).when(assembler).hasVorgangCreateBescheidDocumentEnabled(vorgangWithEingang); doReturn(false).when(assembler).hasVorgangCreateBescheidDocumentEnabled(vorgangWithEingang);
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment