From 9809fd550f5ddd394081529318db6a5001b54bac Mon Sep 17 00:00:00 2001
From: Paul-Christian Volkmer <volkmer_p@ukw.de>
Date: Mon, 20 Mar 2023 22:18:37 +0100
Subject: [PATCH] =?UTF-8?q?Issue=20#11:=20Verwende=20=C3=9Cberschrift=20f?=
 =?UTF-8?q?=C3=BCr=20Fragestellung=20und=20Empfehlung?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 .../mtb/OsTumorkonferenzToProtocolMapper.java    |  2 +-
 ...umorkonferenzVarianteUkwToProtocolMapper.java |  8 ++++----
 .../DNPM/services/mtb/DefaultMtbServiceTest.java | 16 +++++++++-------
 3 files changed, 14 insertions(+), 12 deletions(-)

diff --git a/src/main/java/DNPM/services/mtb/OsTumorkonferenzToProtocolMapper.java b/src/main/java/DNPM/services/mtb/OsTumorkonferenzToProtocolMapper.java
index f6684a2..6379ecf 100644
--- a/src/main/java/DNPM/services/mtb/OsTumorkonferenzToProtocolMapper.java
+++ b/src/main/java/DNPM/services/mtb/OsTumorkonferenzToProtocolMapper.java
@@ -21,7 +21,7 @@ public class OsTumorkonferenzToProtocolMapper implements ProcedureToProtocolMapp
                 null != fragestellung && !fragestellung.getString().isBlank()
                 && null != empfehlung && !empfehlung.getString().isBlank()
         ) {
-            return Optional.of(String.format("%s\n%s", fragestellung.getString(), empfehlung.getString()));
+            return Optional.of(String.format("Fragestellung:\n%s\n\nEmpfehlung:\n%s", fragestellung.getString(), empfehlung.getString()));
         } else if (null != fragestellung && !fragestellung.getString().isBlank()) {
             return Optional.of(fragestellung.getString());
         } else if (null != empfehlung && !empfehlung.getString().isBlank()) {
diff --git a/src/main/java/DNPM/services/mtb/OsTumorkonferenzVarianteUkwToProtocolMapper.java b/src/main/java/DNPM/services/mtb/OsTumorkonferenzVarianteUkwToProtocolMapper.java
index de36415..78a94eb 100644
--- a/src/main/java/DNPM/services/mtb/OsTumorkonferenzVarianteUkwToProtocolMapper.java
+++ b/src/main/java/DNPM/services/mtb/OsTumorkonferenzVarianteUkwToProtocolMapper.java
@@ -24,14 +24,14 @@ public class OsTumorkonferenzVarianteUkwToProtocolMapper implements ProcedureToP
                     null != fragestellung && !fragestellung.getString().isBlank()
                             && null != empfehlung && !empfehlung.getString().isBlank()
             ) {
-                return String.format("%s\n%s", fragestellung.getString(), empfehlung.getString());
+                return String.format("Fragestellung:\n%s\n\nEmpfehlung:\n%s", fragestellung.getString().trim(), empfehlung.getString().trim());
             } else if (null != fragestellung && !fragestellung.getString().isBlank()) {
-                return fragestellung.getString();
+                return fragestellung.getString().trim();
             } else if (null != empfehlung && !empfehlung.getString().isBlank()) {
-                return empfehlung.getString();
+                return empfehlung.getString().trim();
             }
             return "";
-        }).collect(Collectors.joining("\n"));
+        }).collect(Collectors.joining("\n\n"));
 
         if (!result.isBlank()) {
             return Optional.of(result);
diff --git a/src/test/java/DNPM/services/mtb/DefaultMtbServiceTest.java b/src/test/java/DNPM/services/mtb/DefaultMtbServiceTest.java
index 6230edd..073f25c 100644
--- a/src/test/java/DNPM/services/mtb/DefaultMtbServiceTest.java
+++ b/src/test/java/DNPM/services/mtb/DefaultMtbServiceTest.java
@@ -22,22 +22,24 @@ import static org.mockito.Mockito.when;
 @ExtendWith(MockitoExtension.class)
 public class DefaultMtbServiceTest {
 
-    @Mock
     private IOnkostarApi onkostarApi;
 
-    @Mock
     private SettingsService settingsService;
 
     private DefaultMtbService service;
 
     @BeforeEach
-    void setup() {
+    void setup(
+            @Mock IOnkostarApi onkostarApi,
+            @Mock SettingsService settingsService
+    ) {
+        this.onkostarApi = onkostarApi;
+        this.settingsService = settingsService;
         this.service = new DefaultMtbService(settingsService);
     }
 
     @Test
     void testShouldReturnMtbProtocolForDefaultImplementation() {
-
         when(settingsService.getSID()).thenReturn(Optional.of("12345"));
         when(settingsService.multipleMtbsInMtbEpisode()).thenReturn(false);
 
@@ -53,12 +55,11 @@ public class DefaultMtbServiceTest {
 
         var actual = service.getProtocol(procedures);
 
-        assertThat(actual).isEqualTo("Test ok?\nRerun Test if not ok!");
+        assertThat(actual).isEqualTo("Fragestellung:\nTest ok?\n\nEmpfehlung:\nRerun Test if not ok!");
     }
 
     @Test
     void testShouldReturnEmptyMtbProtocolForMultipleMtb() {
-
         when(settingsService.getSID()).thenReturn(Optional.of("12345"));
         when(settingsService.multipleMtbsInMtbEpisode()).thenReturn(true);
 
@@ -103,7 +104,8 @@ public class DefaultMtbServiceTest {
         var actual = service.getProtocol(procedures);
 
         assertThat(actual).isEqualTo(
-                "Test ok?\nRerun Test if not ok!\nTest immer noch ok?\nDo not rerun Test if ok!"
+                "Fragestellung:\nTest ok?\n\nEmpfehlung:\nRerun Test if not ok!\n\n" +
+                        "Fragestellung:\nTest immer noch ok?\n\nEmpfehlung:\nDo not rerun Test if ok!"
         );
     }