build: apply Spotless plugin (#293)

2 files changed, 186 insertions, 172 deletions

diff --git a/src/test/java/dev/dnpm/oshelper/services/therapieplan/DefaultTherapieplanServiceTest.java b/src/test/java/dev/dnpm/oshelper/services/therapieplan/DefaultTherapieplanServiceTest.java
index 18865a8..1d68e08 100644
--- a/src/test/java/dev/dnpm/oshelper/services/therapieplan/DefaultTherapieplanServiceTest.java
+++ b/src/test/java/dev/dnpm/oshelper/services/therapieplan/DefaultTherapieplanServiceTest.java
@@ -19,11 +19,17 @@
 
 package dev.dnpm.oshelper.services.therapieplan;
 
+import static org.assertj.core.api.Assertions.assertThat;
+import static org.mockito.Mockito.*;
+
 import de.itc.onkostar.api.IOnkostarApi;
 import de.itc.onkostar.api.Item;
 import de.itc.onkostar.api.Procedure;
 import de.itc.onkostar.api.constants.JaNeinUnbekannt;
 import dev.dnpm.oshelper.services.FormService;
+import java.time.Instant;
+import java.util.Date;
+import java.util.List;
 import org.junit.jupiter.api.BeforeEach;
 import org.junit.jupiter.api.Test;
 import org.junit.jupiter.api.extension.ExtendWith;
@@ -31,158 +37,166 @@ import org.mockito.ArgumentCaptor;
 import org.mockito.Mock;
 import org.mockito.junit.jupiter.MockitoExtension;
 
-import java.time.Instant;
-import java.util.Date;
-import java.util.List;
-
-import static org.assertj.core.api.Assertions.assertThat;
-import static org.mockito.Mockito.*;
-
 @ExtendWith(MockitoExtension.class)
 class DefaultTherapieplanServiceTest {
 
-    private IOnkostarApi onkostarApi;
-
-    private TherapieplanService service;
-
-    @BeforeEach
-    void setUp(
-            @Mock IOnkostarApi onkostarApi,
-            @Mock FormService formService
-    ) {
-        this.onkostarApi = onkostarApi;
-        this.service = new DefaultTherapieplanService(onkostarApi, formService);
-    }
-
-    @Test
-    void shouldNotUpdateSubformsOrSectionsIfMultipleMtbConfiguration() throws Exception {
-        this.service.updateRequiredMtbEntries(new Procedure(onkostarApi));
-        verify(onkostarApi, never()).saveProcedure(any(Procedure.class), anyBoolean());
-    }
-
-    @Test
-    void shouldNotUpdateSectionsIfSectionsNotEnabled() throws Exception {
-        var testProcedure = baseProcedure(onkostarApi);
-
-        // Keine humangenetische Beratung und keine Reevaluation empfohlen
-        testProcedure.setValue("humangenberatung", new Item("humangen_beratung", JaNeinUnbekannt.NEIN.getCode()));
-        testProcedure.setValue("reevaluation", new Item("reevaluation", JaNeinUnbekannt.NEIN.getCode()));
-
-        this.service.updateRequiredMtbEntries(testProcedure);
-
-        verify(onkostarApi, never()).saveProcedure(any(Procedure.class), anyBoolean());
-    }
-
-    @Test
-    void shouldUpdateSectionsIfNoReevaluation() throws Exception {
-        var testProcedure = baseProcedure(onkostarApi);
-
-        // Humangenetische Beratung aber keine Reevaluation
-        testProcedure.setValue("humangenberatung", new Item("humangen_beratung", JaNeinUnbekannt.JA.getCode()));
-        testProcedure.setValue("humangenberbegruendung", new Item("humangen_ber_begruendung", "Das ist die Begründung"));
-        testProcedure.setValue("reevaluation", new Item("reevaluation", JaNeinUnbekannt.NEIN.getCode()));
-
-        this.service.updateRequiredMtbEntries(testProcedure);
-
-        var captor = ArgumentCaptor.forClass(Procedure.class);
-        verify(onkostarApi, times(1)).saveProcedure(captor.capture(), anyBoolean());
-
-        var capturedProcedure = captor.getValue();
-
-        assertThat(capturedProcedure.getValue("reftkreevaluation")).isNull();
-        assertThat(capturedProcedure.getValue("datumtkreevaluation")).isNull();
-
-        assertThat(capturedProcedure.getValue("reftkhumangenber")).isNotNull();
-        assertThat(capturedProcedure.getValue("reftkhumangenber").getInt()).isEqualTo(procedureId);
-        assertThat(capturedProcedure.getValue("datumtkhumangenber")).isNotNull();
-        assertThat(capturedProcedure.getValue("datumtkhumangenber").getDate()).isEqualTo(testDate);
-    }
-
-    @Test
-    void shouldUpdateSectionsIfNoHumanGenConsultation() throws Exception {
-        var testProcedure = baseProcedure(onkostarApi);
-
-        // Humangenetische Beratung aber keine Reevaluation
-        testProcedure.setValue("humangenberatung", new Item("humangen_beratung", JaNeinUnbekannt.NEIN.getCode()));
-        testProcedure.setValue("reevaluation", new Item("reevaluation", JaNeinUnbekannt.JA.getCode()));
-
-        this.service.updateRequiredMtbEntries(testProcedure);
-
-        var captor = ArgumentCaptor.forClass(Procedure.class);
-        verify(onkostarApi, times(1)).saveProcedure(captor.capture(), anyBoolean());
-
-        var capturedProcedure = captor.getValue();
-
-        assertThat(capturedProcedure.getValue("reftkhumangenber")).isNull();
-        assertThat(capturedProcedure.getValue("datumtkhumangenber")).isNull();
-
-        assertThat(capturedProcedure.getValue("reftkreevaluation")).isNotNull();
-        assertThat(capturedProcedure.getValue("reftkreevaluation").getInt()).isEqualTo(procedureId);
-        assertThat(capturedProcedure.getValue("datumtkreevaluation")).isNotNull();
-        assertThat(capturedProcedure.getValue("datumtkreevaluation").getDate()).isEqualTo(testDate);
-    }
-
-    @Test
-    void shouldFindFollowUps() {
-        doAnswer(invocationOnMock -> {
-            var testProcedure = baseProcedure(onkostarApi);
-            testProcedure.setId(procedureId);
-            testProcedure.setFormName("DNPM UF Einzelempfehlung");
-            testProcedure.addDiseaseId(123);
-            return testProcedure;
-        }).when(onkostarApi).getProcedure(anyInt());
-
-        doAnswer(invocationOnMock -> {
-            var diseaseId = invocationOnMock.getArgument(0, Integer.class);
-            var formName = invocationOnMock.getArgument(1, String.class);
-            var procedure = baseProcedure(onkostarApi);
-            procedure.addDiseaseId(diseaseId);
-            procedure.setFormName(formName);
-            procedure.setValue("LinkTherapieempfehlung", new Item("LinkTherapieempfehlung", procedureId));
-            return List.of(procedure);
-        }).when(onkostarApi).getProceduresForDiseaseByForm(anyInt(), anyString());
-
-        var followUps = this.service.findReferencedFollowUpsForSubform(1);
-
-        assertThat(followUps).hasSize(1)
-                .allSatisfy(procedure -> assertThat(procedure.getFormName()).isEqualTo("DNPM FollowUp"));
-    }
-
-    @Test
-    void shouldFindFollowUpsById() {
-        var testProcedure = baseProcedure(onkostarApi);
-        testProcedure.setId(procedureId);
-        testProcedure.setFormName("DNPM UF Einzelempfehlung");
-        testProcedure.addDiseaseId(123);
-
-        doAnswer(invocationOnMock -> {
-            var diseaseId = invocationOnMock.getArgument(0, Integer.class);
-            var formName = invocationOnMock.getArgument(1, String.class);
-            var procedure = baseProcedure(onkostarApi);
-            procedure.addDiseaseId(diseaseId);
-            procedure.setFormName(formName);
-            procedure.setValue("LinkTherapieempfehlung", new Item("LinkTherapieempfehlung", procedureId));
-            return List.of(procedure);
-        }).when(onkostarApi).getProceduresForDiseaseByForm(anyInt(), anyString());
-
-        var followUps = this.service.findReferencedFollowUpsForSubform(testProcedure);
-
-        assertThat(followUps).hasSize(1)
-                .allSatisfy(procedure -> assertThat(procedure.getFormName()).isEqualTo("DNPM FollowUp"));
-    }
-
-    private static final int procedureId = 1234;
-    private static final Date testDate = Date.from(Instant.parse("2023-03-15T09:43:00Z"));
-
-    private Procedure baseProcedure(final IOnkostarApi onkostarApi) {
-        var testProcedure = new Procedure(onkostarApi);
-        testProcedure.setId(1000);
-
-        // Setzen MTB Referenz und Datum MTB
-        testProcedure.setValue("referstemtb", new Item("ref_tumorkonferenz", procedureId));
-        testProcedure.setValue("datum", new Item("datum", testDate));
-
-        return testProcedure;
-    }
-
+  private IOnkostarApi onkostarApi;
+
+  private TherapieplanService service;
+
+  @BeforeEach
+  void setUp(@Mock IOnkostarApi onkostarApi, @Mock FormService formService) {
+    this.onkostarApi = onkostarApi;
+    this.service = new DefaultTherapieplanService(onkostarApi, formService);
+  }
+
+  @Test
+  void shouldNotUpdateSubformsOrSectionsIfMultipleMtbConfiguration() throws Exception {
+    this.service.updateRequiredMtbEntries(new Procedure(onkostarApi));
+    verify(onkostarApi, never()).saveProcedure(any(Procedure.class), anyBoolean());
+  }
+
+  @Test
+  void shouldNotUpdateSectionsIfSectionsNotEnabled() throws Exception {
+    var testProcedure = baseProcedure(onkostarApi);
+
+    // Keine humangenetische Beratung und keine Reevaluation empfohlen
+    testProcedure.setValue(
+        "humangenberatung", new Item("humangen_beratung", JaNeinUnbekannt.NEIN.getCode()));
+    testProcedure.setValue(
+        "reevaluation", new Item("reevaluation", JaNeinUnbekannt.NEIN.getCode()));
+
+    this.service.updateRequiredMtbEntries(testProcedure);
+
+    verify(onkostarApi, never()).saveProcedure(any(Procedure.class), anyBoolean());
+  }
+
+  @Test
+  void shouldUpdateSectionsIfNoReevaluation() throws Exception {
+    var testProcedure = baseProcedure(onkostarApi);
+
+    // Humangenetische Beratung aber keine Reevaluation
+    testProcedure.setValue(
+        "humangenberatung", new Item("humangen_beratung", JaNeinUnbekannt.JA.getCode()));
+    testProcedure.setValue(
+        "humangenberbegruendung", new Item("humangen_ber_begruendung", "Das ist die Begründung"));
+    testProcedure.setValue(
+        "reevaluation", new Item("reevaluation", JaNeinUnbekannt.NEIN.getCode()));
+
+    this.service.updateRequiredMtbEntries(testProcedure);
+
+    var captor = ArgumentCaptor.forClass(Procedure.class);
+    verify(onkostarApi, times(1)).saveProcedure(captor.capture(), anyBoolean());
+
+    var capturedProcedure = captor.getValue();
+
+    assertThat(capturedProcedure.getValue("reftkreevaluation")).isNull();
+    assertThat(capturedProcedure.getValue("datumtkreevaluation")).isNull();
+
+    assertThat(capturedProcedure.getValue("reftkhumangenber")).isNotNull();
+    assertThat(capturedProcedure.getValue("reftkhumangenber").getInt()).isEqualTo(procedureId);
+    assertThat(capturedProcedure.getValue("datumtkhumangenber")).isNotNull();
+    assertThat(capturedProcedure.getValue("datumtkhumangenber").getDate()).isEqualTo(testDate);
+  }
+
+  @Test
+  void shouldUpdateSectionsIfNoHumanGenConsultation() throws Exception {
+    var testProcedure = baseProcedure(onkostarApi);
+
+    // Humangenetische Beratung aber keine Reevaluation
+    testProcedure.setValue(
+        "humangenberatung", new Item("humangen_beratung", JaNeinUnbekannt.NEIN.getCode()));
+    testProcedure.setValue("reevaluation", new Item("reevaluation", JaNeinUnbekannt.JA.getCode()));
+
+    this.service.updateRequiredMtbEntries(testProcedure);
+
+    var captor = ArgumentCaptor.forClass(Procedure.class);
+    verify(onkostarApi, times(1)).saveProcedure(captor.capture(), anyBoolean());
+
+    var capturedProcedure = captor.getValue();
+
+    assertThat(capturedProcedure.getValue("reftkhumangenber")).isNull();
+    assertThat(capturedProcedure.getValue("datumtkhumangenber")).isNull();
+
+    assertThat(capturedProcedure.getValue("reftkreevaluation")).isNotNull();
+    assertThat(capturedProcedure.getValue("reftkreevaluation").getInt()).isEqualTo(procedureId);
+    assertThat(capturedProcedure.getValue("datumtkreevaluation")).isNotNull();
+    assertThat(capturedProcedure.getValue("datumtkreevaluation").getDate()).isEqualTo(testDate);
+  }
+
+  @Test
+  void shouldFindFollowUps() {
+    doAnswer(
+            invocationOnMock -> {
+              var testProcedure = baseProcedure(onkostarApi);
+              testProcedure.setId(procedureId);
+              testProcedure.setFormName("DNPM UF Einzelempfehlung");
+              testProcedure.addDiseaseId(123);
+              return testProcedure;
+            })
+        .when(onkostarApi)
+        .getProcedure(anyInt());
+
+    doAnswer(
+            invocationOnMock -> {
+              var diseaseId = invocationOnMock.getArgument(0, Integer.class);
+              var formName = invocationOnMock.getArgument(1, String.class);
+              var procedure = baseProcedure(onkostarApi);
+              procedure.addDiseaseId(diseaseId);
+              procedure.setFormName(formName);
+              procedure.setValue(
+                  "LinkTherapieempfehlung", new Item("LinkTherapieempfehlung", procedureId));
+              return List.of(procedure);
+            })
+        .when(onkostarApi)
+        .getProceduresForDiseaseByForm(anyInt(), anyString());
+
+    var followUps = this.service.findReferencedFollowUpsForSubform(1);
+
+    assertThat(followUps)
+        .hasSize(1)
+        .allSatisfy(procedure -> assertThat(procedure.getFormName()).isEqualTo("DNPM FollowUp"));
+  }
+
+  @Test
+  void shouldFindFollowUpsById() {
+    var testProcedure = baseProcedure(onkostarApi);
+    testProcedure.setId(procedureId);
+    testProcedure.setFormName("DNPM UF Einzelempfehlung");
+    testProcedure.addDiseaseId(123);
+
+    doAnswer(
+            invocationOnMock -> {
+              var diseaseId = invocationOnMock.getArgument(0, Integer.class);
+              var formName = invocationOnMock.getArgument(1, String.class);
+              var procedure = baseProcedure(onkostarApi);
+              procedure.addDiseaseId(diseaseId);
+              procedure.setFormName(formName);
+              procedure.setValue(
+                  "LinkTherapieempfehlung", new Item("LinkTherapieempfehlung", procedureId));
+              return List.of(procedure);
+            })
+        .when(onkostarApi)
+        .getProceduresForDiseaseByForm(anyInt(), anyString());
+
+    var followUps = this.service.findReferencedFollowUpsForSubform(testProcedure);
+
+    assertThat(followUps)
+        .hasSize(1)
+        .allSatisfy(procedure -> assertThat(procedure.getFormName()).isEqualTo("DNPM FollowUp"));
+  }
+
+  private static final int procedureId = 1234;
+  private static final Date testDate = Date.from(Instant.parse("2023-03-15T09:43:00Z"));
+
+  private Procedure baseProcedure(final IOnkostarApi onkostarApi) {
+    var testProcedure = new Procedure(onkostarApi);
+    testProcedure.setId(1000);
+
+    // Setzen MTB Referenz und Datum MTB
+    testProcedure.setValue("referstemtb", new Item("ref_tumorkonferenz", procedureId));
+    testProcedure.setValue("datum", new Item("datum", testDate));
+
+    return testProcedure;
+  }
 }
diff --git a/src/test/java/dev/dnpm/oshelper/services/therapieplan/TherapieplanServiceFactoryTest.java b/src/test/java/dev/dnpm/oshelper/services/therapieplan/TherapieplanServiceFactoryTest.java
index 7c176bb..c6db22a 100644
--- a/src/test/java/dev/dnpm/oshelper/services/therapieplan/TherapieplanServiceFactoryTest.java
+++ b/src/test/java/dev/dnpm/oshelper/services/therapieplan/TherapieplanServiceFactoryTest.java
@@ -19,6 +19,8 @@
 
 package dev.dnpm.oshelper.services.therapieplan;
 
+import static org.assertj.core.api.Assertions.assertThat;
+
 import de.itc.onkostar.api.IOnkostarApi;
 import dev.dnpm.oshelper.services.FormService;
 import dev.dnpm.oshelper.services.SettingsService;
@@ -28,28 +30,26 @@ import org.junit.jupiter.api.extension.ExtendWith;
 import org.mockito.Mock;
 import org.mockito.junit.jupiter.MockitoExtension;
 
-import static org.assertj.core.api.Assertions.assertThat;
-
 @ExtendWith(MockitoExtension.class)
 class TherapieplanServiceFactoryTest {
 
-    private SettingsService settingsService;
-
-    private TherapieplanServiceFactory therapieplanServiceFactory;
-
-    @BeforeEach
-    void setup(
-            @Mock IOnkostarApi onkostarApi,
-            @Mock SettingsService settingsService,
-            @Mock FormService formService
-    ) {
-        this.settingsService = settingsService;
-        this.therapieplanServiceFactory = new TherapieplanServiceFactory(onkostarApi, settingsService, formService);
-    }
-
-    @Test
-    void testShouldAlwaysReturnDefaultTherapieplanService() {
-        var actual = this.therapieplanServiceFactory.currentUsableInstance();
-        assertThat(actual).isInstanceOf(DefaultTherapieplanService.class);
-    }
+  private SettingsService settingsService;
+
+  private TherapieplanServiceFactory therapieplanServiceFactory;
+
+  @BeforeEach
+  void setup(
+      @Mock IOnkostarApi onkostarApi,
+      @Mock SettingsService settingsService,
+      @Mock FormService formService) {
+    this.settingsService = settingsService;
+    this.therapieplanServiceFactory =
+        new TherapieplanServiceFactory(onkostarApi, settingsService, formService);
+  }
+
+  @Test
+  void testShouldAlwaysReturnDefaultTherapieplanService() {
+    var actual = this.therapieplanServiceFactory.currentUsableInstance();
+    assertThat(actual).isInstanceOf(DefaultTherapieplanService.class);
+  }
 }