Merge pull request #21 from CCC-MF/issue_20

Anpassung des ConsentManagements an verschiedene Standorte

5 files changed, 278 insertions, 31 deletions

diff --git a/src/test/java/DNPM/ConsentManagerTest.java b/src/test/java/DNPM/ConsentManagerTest.java
index 6755ed6..41c9a02 100644
--- a/src/test/java/DNPM/ConsentManagerTest.java
+++ b/src/test/java/DNPM/ConsentManagerTest.java
@@ -1,20 +1,15 @@
 package DNPM;
 
+import DNPM.services.consent.ConsentManagerServiceFactory;
+import DNPM.services.consent.MrConsentManagerService;
 import de.itc.onkostar.api.IOnkostarApi;
 import de.itc.onkostar.api.Procedure;
-import org.hibernate.SQLQuery;
-import org.hibernate.Session;
-import org.hibernate.SessionFactory;
-import org.hibernate.type.Type;
 import org.junit.jupiter.api.BeforeEach;
 import org.junit.jupiter.api.Test;
 import org.junit.jupiter.api.extension.ExtendWith;
-import org.mockito.ArgumentCaptor;
 import org.mockito.Mock;
 import org.mockito.junit.jupiter.MockitoExtension;
 
-import static org.assertj.core.api.Assertions.assertThat;
-import static org.mockito.ArgumentMatchers.anyString;
 import static org.mockito.Mockito.*;
 
 @ExtendWith(MockitoExtension.class)
@@ -22,39 +17,30 @@ class ConsentManagerTest {
 
     private IOnkostarApi onkostarApi;
 
+    private ConsentManagerServiceFactory consentManagerServiceFactory;
+
     private ConsentManager consentManager;
 
     @BeforeEach
     void setup(
-            @Mock IOnkostarApi onkostarApi
+            @Mock IOnkostarApi onkostarApi,
+            @Mock ConsentManagerServiceFactory consentManagerServiceFactory
     ) {
         this.onkostarApi = onkostarApi;
-        this.consentManager = new ConsentManager(onkostarApi);
+        this.consentManagerServiceFactory = consentManagerServiceFactory;
+        this.consentManager = new ConsentManager(onkostarApi, consentManagerServiceFactory);
     }
 
     @Test
-    void testShouldCreateSqlQueriesWithRelatedEntityIds() {
-        var sessionFactory = mock(SessionFactory.class);
-        var session = mock(Session.class);
-        var query = mock(SQLQuery.class);
-
-        when(onkostarApi.getSessionFactory()).thenReturn(sessionFactory);
-        when(sessionFactory.getCurrentSession()).thenReturn(session);
-        when(session.createSQLQuery(anyString())).thenReturn(query);
-        when(query.addScalar(anyString(), any(Type.class))).thenReturn(query);
-        when(query.uniqueResult()).thenReturn("");
-
-        var dummyProzedur = new Procedure(this.onkostarApi);
-        dummyProzedur.setId(111);
-        dummyProzedur.setPatientId(123);
-
-        consentManager.analyze(dummyProzedur, null);
-
-        var argumentCaptor = ArgumentCaptor.forClass(String.class);
-        verify(session, times(2)).createSQLQuery(argumentCaptor.capture());
-        assertThat(argumentCaptor.getAllValues()).hasSize(2);
-        assertThat(argumentCaptor.getAllValues().get(0)).contains("where entity_id = '111'");
-        assertThat(argumentCaptor.getAllValues().get(1)).contains("WHERE patient_id = 123 AND geloescht = 0");
+    void shouldRunServiceMethodsOnAnalyzeCalled() {
+        var consentManagerServiceMock = mock(MrConsentManagerService.class);
+
+        when(this.consentManagerServiceFactory.currentUsableInstance())
+                .thenReturn(consentManagerServiceMock);
+
+        this.consentManager.analyze(new Procedure(onkostarApi), null);
+
+        verify(consentManagerServiceMock, times(1)).applyConsent(any(Procedure.class));
     }
 
 }
diff --git a/src/test/java/DNPM/config/ConsentManagerServiceFactoryTest.java b/src/test/java/DNPM/config/ConsentManagerServiceFactoryTest.java
new file mode 100644
index 0000000..6e32e7f
--- /dev/null
+++ b/src/test/java/DNPM/config/ConsentManagerServiceFactoryTest.java
@@ -0,0 +1,54 @@
+package DNPM.config;
+
+import DNPM.services.consent.ConsentManagerService;
+import DNPM.services.consent.ConsentManagerServiceFactory;
+import DNPM.services.consent.MrConsentManagerService;
+import DNPM.services.consent.UkwConsentManagerService;
+import de.itc.onkostar.api.IOnkostarApi;
+import org.junit.jupiter.api.BeforeEach;
+import org.junit.jupiter.api.extension.ExtendWith;
+import org.junit.jupiter.params.ParameterizedTest;
+import org.junit.jupiter.params.provider.MethodSource;
+import org.mockito.Mock;
+import org.mockito.junit.jupiter.MockitoExtension;
+
+import java.util.Map;
+import java.util.Set;
+
+import static org.assertj.core.api.Assertions.assertThat;
+import static org.mockito.ArgumentMatchers.anyString;
+import static org.mockito.Mockito.when;
+
+@ExtendWith(MockitoExtension.class)
+class ConsentManagerServiceFactoryTest {
+
+    private IOnkostarApi onkostarApi;
+
+    private ConsentManagerServiceFactory consentManagerServiceFactory;
+
+    @BeforeEach
+    void setup(
+            @Mock IOnkostarApi onkostarApi
+    ) {
+        this.onkostarApi = onkostarApi;
+        this.consentManagerServiceFactory = new ConsentManagerServiceFactory(onkostarApi);
+    }
+
+    private static Set<Map.Entry<String, Class<? extends ConsentManagerService>>> expectedMappings() {
+        return Map.ofEntries(
+                Map.entry("MR.Consent", MrConsentManagerService.class),
+                Map.entry("Excel-Formular", UkwConsentManagerService.class)
+        ).entrySet();
+    }
+
+    @ParameterizedTest
+    @MethodSource("expectedMappings")
+    void testShouldMapFormNameToService(Map.Entry<String, Class<?>> expectedMapping) {
+        when(onkostarApi.getGlobalSetting(anyString())).thenReturn(expectedMapping.getKey());
+
+        var actual = consentManagerServiceFactory.currentUsableInstance();
+
+        assertThat(actual).isExactlyInstanceOf(expectedMapping.getValue());
+    }
+
+}
diff --git a/src/test/java/DNPM/config/PluginConfigurationTest.java b/src/test/java/DNPM/config/PluginConfigurationTest.java
index 32ac53b..606817b 100644
--- a/src/test/java/DNPM/config/PluginConfigurationTest.java
+++ b/src/test/java/DNPM/config/PluginConfigurationTest.java
@@ -3,6 +3,7 @@ package DNPM.config;
 import DNPM.services.FormService;
 import DNPM.services.SettingsService;
 import DNPM.services.TherapieplanServiceFactory;
+import DNPM.services.consent.ConsentManagerServiceFactory;
 import de.itc.onkostar.api.IOnkostarApi;
 import org.junit.jupiter.api.BeforeEach;
 import org.junit.jupiter.api.Test;
@@ -32,6 +33,12 @@ class PluginConfigurationTest {
     }
 
     @Test
+    void testShouldReturnConsentManagerServiceFactory() {
+        var actual = this.configuration.consentManagerServiceFactory(onkostarApi);
+        assertThat(actual).isInstanceOf(ConsentManagerServiceFactory.class);
+    }
+
+    @Test
     void testShouldReturnTherapieplanServiceFactory() {
         var actual = this.configuration.therapieplanServiceFactory(onkostarApi, settingsService, formService);
         assertThat(actual).isInstanceOf(TherapieplanServiceFactory.class);
diff --git a/src/test/java/DNPM/services/consent/MrConsentManagerServiceTest.java b/src/test/java/DNPM/services/consent/MrConsentManagerServiceTest.java
new file mode 100644
index 0000000..2c8448e
--- /dev/null
+++ b/src/test/java/DNPM/services/consent/MrConsentManagerServiceTest.java
@@ -0,0 +1,61 @@
+package DNPM.services.consent;
+
+import de.itc.onkostar.api.IOnkostarApi;
+import de.itc.onkostar.api.Procedure;
+import org.hibernate.SQLQuery;
+import org.hibernate.Session;
+import org.hibernate.SessionFactory;
+import org.hibernate.type.Type;
+import org.junit.jupiter.api.BeforeEach;
+import org.junit.jupiter.api.Test;
+import org.junit.jupiter.api.extension.ExtendWith;
+import org.mockito.ArgumentCaptor;
+import org.mockito.Mock;
+import org.mockito.junit.jupiter.MockitoExtension;
+
+import static org.assertj.core.api.Assertions.assertThat;
+import static org.mockito.ArgumentMatchers.any;
+import static org.mockito.ArgumentMatchers.anyString;
+import static org.mockito.Mockito.*;
+
+@ExtendWith(MockitoExtension.class)
+public class MrConsentManagerServiceTest {
+
+    private IOnkostarApi onkostarApi;
+
+    private MrConsentManagerService service;
+
+    @BeforeEach
+    void setup(
+            @Mock IOnkostarApi onkostarApi
+    ) {
+        this.onkostarApi = onkostarApi;
+        this.service = new MrConsentManagerService(onkostarApi);
+    }
+
+    @Test
+    void testShouldCreateSqlQueriesWithRelatedEntityIds() {
+        var sessionFactory = mock(SessionFactory.class);
+        var session = mock(Session.class);
+        var query = mock(SQLQuery.class);
+
+        when(onkostarApi.getSessionFactory()).thenReturn(sessionFactory);
+        when(sessionFactory.getCurrentSession()).thenReturn(session);
+        when(session.createSQLQuery(anyString())).thenReturn(query);
+        when(query.addScalar(anyString(), any(Type.class))).thenReturn(query);
+        when(query.uniqueResult()).thenReturn("");
+
+        var dummyProzedur = new Procedure(this.onkostarApi);
+        dummyProzedur.setId(111);
+        dummyProzedur.setPatientId(123);
+
+        this.service.applyConsent(dummyProzedur);
+
+        var argumentCaptor = ArgumentCaptor.forClass(String.class);
+        verify(session, times(2)).createSQLQuery(argumentCaptor.capture());
+        assertThat(argumentCaptor.getAllValues()).hasSize(2);
+        assertThat(argumentCaptor.getAllValues().get(0)).contains("where entity_id = '111'");
+        assertThat(argumentCaptor.getAllValues().get(1)).contains("WHERE patient_id = 123 AND geloescht = 0");
+    }
+
+}
diff --git a/src/test/java/DNPM/services/consent/UkwConsentManagerServiceTest.java b/src/test/java/DNPM/services/consent/UkwConsentManagerServiceTest.java
new file mode 100644
index 0000000..fdcd6e3
--- /dev/null
+++ b/src/test/java/DNPM/services/consent/UkwConsentManagerServiceTest.java
@@ -0,0 +1,139 @@
+package DNPM.services.consent;
+
+import de.itc.onkostar.api.IOnkostarApi;
+import de.itc.onkostar.api.Item;
+import de.itc.onkostar.api.Procedure;
+import org.junit.jupiter.api.BeforeEach;
+import org.junit.jupiter.api.Test;
+import org.junit.jupiter.api.extension.ExtendWith;
+import org.mockito.ArgumentCaptor;
+import org.mockito.Mock;
+import org.mockito.junit.jupiter.MockitoExtension;
+
+import java.sql.Date;
+import java.time.Instant;
+
+import static org.assertj.core.api.Assertions.assertThat;
+import static org.mockito.ArgumentMatchers.any;
+import static org.mockito.Mockito.*;
+
+@ExtendWith(MockitoExtension.class)
+public class UkwConsentManagerServiceTest {
+
+    private IOnkostarApi onkostarApi;
+
+    private UkwConsentManagerService service;
+
+    @BeforeEach
+    void setup(
+            @Mock IOnkostarApi onkostarApi
+    ) {
+        this.onkostarApi = onkostarApi;
+        this.service = new UkwConsentManagerService(onkostarApi);
+    }
+
+    @Test
+    void testShouldSkipUpdateRelatedDnpmKlinikAnamneseFormIfNoConsentAvailable() throws Exception {
+
+        var excelForm = new Procedure(this.onkostarApi);
+        excelForm.setId(111);
+        excelForm.setPatientId(123);
+        excelForm.setValue("refdnpmklinikanamnese", new Item("refdnpmklinikanamnese", 2));
+
+        var dnpmKlinikAnamneseForm = new Procedure(this.onkostarApi);
+        dnpmKlinikAnamneseForm.setId(2);
+        dnpmKlinikAnamneseForm.setPatientId(123);
+
+        when(onkostarApi.getProcedure(anyInt())).thenReturn(dnpmKlinikAnamneseForm);
+
+        this.service.applyConsent(excelForm);
+
+        verify(onkostarApi, times(0)).saveProcedure(any(Procedure.class), anyBoolean());
+    }
+
+    @Test
+    void testShouldSkipUpdateRelatedDnpmKlinikAnamneseFormIfNoConsentDateAvailable() throws Exception {
+
+        var consentSubForm = new Procedure(this.onkostarApi);
+        consentSubForm.setId(1);
+        consentSubForm.setPatientId(123);
+        consentSubForm.setValue("status", new Item("status", "accepted"));
+
+
+        var excelForm = new Procedure(this.onkostarApi);
+        excelForm.setId(111);
+        excelForm.setPatientId(123);
+        excelForm.setValue("refdnpmklinikanamnese", new Item("refdnpmklinikanamnese", 2));
+        excelForm.addSubProcedure("ufdnpmconsent", consentSubForm);
+
+        var dnpmKlinikAnamneseForm = new Procedure(this.onkostarApi);
+        dnpmKlinikAnamneseForm.setId(2);
+        dnpmKlinikAnamneseForm.setPatientId(123);
+
+        when(onkostarApi.getProcedure(anyInt())).thenReturn(dnpmKlinikAnamneseForm);
+
+        this.service.applyConsent(excelForm);
+
+        verify(onkostarApi, times(0)).saveProcedure(any(Procedure.class), anyBoolean());
+    }
+
+    @Test
+    void testShouldSkipUpdateRelatedDnpmKlinikAnamneseFormIfNoConsentValueAvailable() throws Exception {
+
+        var consentSubForm = new Procedure(this.onkostarApi);
+        consentSubForm.setId(1);
+        consentSubForm.setPatientId(123);
+        consentSubForm.setStartDate(Date.from(Instant.parse("2023-04-03T12:00:00Z")));
+        consentSubForm.setValue("datum", new Item("datum", Date.from(Instant.parse("2023-04-03T12:00:00Z"))));
+
+        var excelForm = new Procedure(this.onkostarApi);
+        excelForm.setId(111);
+        excelForm.setPatientId(123);
+        excelForm.setValue("refdnpmklinikanamnese", new Item("refdnpmklinikanamnese", 2));
+        excelForm.addSubProcedure("ufdnpmconsent", consentSubForm);
+
+        var dnpmKlinikAnamneseForm = new Procedure(this.onkostarApi);
+        dnpmKlinikAnamneseForm.setId(2);
+        dnpmKlinikAnamneseForm.setPatientId(123);
+
+        when(onkostarApi.getProcedure(anyInt())).thenReturn(dnpmKlinikAnamneseForm);
+
+        this.service.applyConsent(excelForm);
+
+        verify(onkostarApi, times(0)).saveProcedure(any(Procedure.class), anyBoolean());
+    }
+
+    @Test
+    void testShouldUpdateRelatedDnpmKlinikAnamneseFormOnFormSave() throws Exception {
+
+        var consentSubForm = new Procedure(this.onkostarApi);
+        consentSubForm.setId(1);
+        consentSubForm.setPatientId(123);
+        consentSubForm.setStartDate(Date.from(Instant.parse("2023-04-03T12:00:00Z")));
+        consentSubForm.setValue("datum", new Item("datum", Date.from(Instant.parse("2023-04-03T12:00:00Z"))));
+        consentSubForm.setValue("status", new Item("status", "accepted"));
+
+        var excelForm = new Procedure(this.onkostarApi);
+        excelForm.setId(111);
+        excelForm.setPatientId(123);
+        excelForm.setValue("refdnpmklinikanamnese", new Item("refdnpmklinikanamnese", 2));
+        excelForm.addSubProcedure("ufdnpmconsent", consentSubForm);
+
+        var dnpmKlinikAnamneseForm = new Procedure(this.onkostarApi);
+        dnpmKlinikAnamneseForm.setId(2);
+        dnpmKlinikAnamneseForm.setPatientId(123);
+
+        when(onkostarApi.getProcedure(anyInt())).thenReturn(dnpmKlinikAnamneseForm);
+
+        this.service.applyConsent(excelForm);
+
+        var argumentCaptor = ArgumentCaptor.forClass(Procedure.class);
+        verify(onkostarApi, times(1)).saveProcedure(argumentCaptor.capture(), anyBoolean());
+
+        var savedForm = argumentCaptor.getValue();
+        assertThat(savedForm).isExactlyInstanceOf(Procedure.class);
+        assertThat(savedForm.getValue("ConsentStatusEinwilligungDNPM").getString()).isEqualTo("accepted");
+        assertThat(savedForm.getValue("ConsentDatumEinwilligungDNPM").getDate()).isEqualTo("2023-04-03T12:00:00Z");
+    }
+
+}