From b9159c274364b9b4cf557e60fa03285e8a04f40a Mon Sep 17 00:00:00 2001
From: Kuettner Tobias <tobias.kuettner@scandio.de>
Date: Wed, 25 Mar 2020 08:02:54 +0100
Subject: [PATCH 1/3] Make link between test and patient working

---
 client/src/api/Api.js                         |  30 +++--
 client/src/components/Navigation.vue          |   6 +-
 .../laboratory/LinkSampleAndPatient.vue       |  91 -------------
 .../laboratory/LinkTestAndPatient.vue         | 121 ++++++++++++++++++
 .../laboratory/LinkTestResultAndPatient.vue   |   2 +-
 client/src/components/laboratory/Login.vue    |   7 +-
 ...entPage.vue => LinkTestAndPatientPage.vue} |  18 +--
 client/src/routes/routes.js                   |   6 +-
 .../de/coronavirus/imis/TestDataLoader.java   |   7 +-
 .../imis/api/LabTestController.java           |  13 +-
 .../imis/api/dto/CreateLabTestDTO.java        |   3 +-
 .../imis/api/dto/UpdateTestStatusDTO.java     |   2 +-
 .../de/coronavirus/imis/domain/LabTest.java   |  13 +-
 .../imis/services/LabTestService.java         |  43 ++++---
 .../resources/sample_data/createLabTest.json  |   5 +-
 15 files changed, 204 insertions(+), 163 deletions(-)
 delete mode 100644 client/src/components/laboratory/LinkSampleAndPatient.vue
 create mode 100644 client/src/components/laboratory/LinkTestAndPatient.vue
 rename client/src/components/page/{LinkSampleAndPatientPage.vue => LinkTestAndPatientPage.vue} (60%)

diff --git a/client/src/api/Api.js b/client/src/api/Api.js
index 72ba6525..deb26f58 100644
--- a/client/src/api/Api.js
+++ b/client/src/api/Api.js
@@ -42,7 +42,23 @@ class Api {
   }
 
   /*
-   * ----------------------------------
+   * -----------IMPLEMENTED IN UI-----------
+   */
+
+  postInstitution(institution) {
+    return this.executeRequest("/institutions", METHOD.POST, institution);
+  }
+
+  postPatient(patient) {
+    return this.executeRequest("/patients", METHOD.POST, patient);
+  }
+
+  postLabTest(labtest) {
+    return this.executeRequest("/labtests", METHOD.POST, labtest);
+  }
+
+  /*
+   * ---------NOT IMPLEMENTED IN UI---------
    */
 
   postDoctorCreateAppointment({ doctorId, laboratoryId, patientId }) {
@@ -77,14 +93,6 @@ class Api {
     return this.executeRequest(`/test_reports/${testId}`, METHOD.DELETE);
   }
 
-  postInstitution(institution) {
-    return this.executeRequest("/institutions", METHOD.POST, institution);
-  }
-
-  postLabtest(labtest) {
-    return this.executeRequest("/labtest", METHOD.POST, labtest);
-  }
-
   putLabtest({ updatedTestStatus }) {
     return this.executeRequest("/labtest", METHOD.PUT, updatedTestStatus);
   }
@@ -97,10 +105,6 @@ class Api {
     return this.executeRequest("/patients", METHOD.GET);
   }
 
-  postPatient(patient) {
-    return this.executeRequest("/patients", METHOD.POST, patient);
-  }
-
   getPatient(id) {
     return this.executeRequest(`/patients/${id}`, METHOD.GET);
   }
diff --git a/client/src/components/Navigation.vue b/client/src/components/Navigation.vue
index c73827a2..1baf1759 100644
--- a/client/src/components/Navigation.vue
+++ b/client/src/components/Navigation.vue
@@ -19,15 +19,15 @@ const items = [
   },
   {
     key: "2",
-    url: "/link-sample-and-patient",
+    url: "/link-test-and-patient",
     type: "deployment-unit",
-    text: "Probe Zuordnen"
+    text: "Test Zuordnen"
   },
   {
     key: "3",
     url: "/link-test-result",
     type: "experiment",
-    text: "Laborresultate"
+    text: "Testresultat Zuordnen"
   },
   { key: "4", url: "/patient-overview", type: "user", text: "Patienten-Daten" },
   { key: "5", url: "/all-data", type: "team", text: "Alle Patienten" },
diff --git a/client/src/components/laboratory/LinkSampleAndPatient.vue b/client/src/components/laboratory/LinkSampleAndPatient.vue
deleted file mode 100644
index 2e7e5373..00000000
--- a/client/src/components/laboratory/LinkSampleAndPatient.vue
+++ /dev/null
@@ -1,91 +0,0 @@
-<template>
-  <a-card style="width: 500px; margin: 2rem auto; min-height: 300px">
-    <a-form
-      :label-col="{ span: 6 }"
-      :wrapper-col="{ span: 18 }"
-      :colon="false"
-      @submit="handleSubmit"
-    >
-      <a-form-item label="Patienten-ID">
-        <a-input
-          v-decorator="[
-            'note',
-            {
-              rules: [
-                {
-                  required: true,
-                  message: 'Bitte geben Sie die Patienten-ID ein.'
-                }
-              ]
-            }
-          ]"
-          placeholder="z.B 1337-4237-9438"
-        />
-      </a-form-item>
-      <a-form-item label="Proben-ID">
-        <a-input
-          v-decorator="[
-            'note',
-            {
-              rules: [
-                {
-                  required: true,
-                  message: 'Bitte geben Sie Ihre Proben-ID ein.'
-                }
-              ]
-            }
-          ]"
-          placeholder="z.B ae48-hr43-sk97"
-        />
-      </a-form-item>
-      <a-form-item label="Kommentar">
-        <a-textarea
-          placeholder="Kommentar hinzufügen"
-          :autoSize="{ minRows: 3, maxRows: 5 }"
-        />
-      </a-form-item>
-      <a-divider />
-      <a-form-item :wrapper-col="{ span: 24, offset: 0 }">
-        <a-button type="primary" html-type="submit">
-          Speichern
-        </a-button>
-      </a-form-item>
-    </a-form>
-  </a-card>
-</template>
-
-<script>
-// TODO: Use form from ant and to get the values from the <form> (see PatientDataCompontent)
-
-export default {
-  name: "LinkSampleAndPatient",
-  methods: {
-    handleSubmit(e) {
-      e.preventDefault();
-
-      // TODO: Send request to connect patient with sample
-      console.log("Handle connect patient with sample");
-
-      const patientId = "198424989813";
-      const sampleId = "91029310923";
-
-      // TODO: Handle response from server
-      const response = { type: "success" };
-      const { type } = response;
-
-      // Check notification type (success, info, warning, error)
-      const notification = {};
-      if (type === "success") {
-        notification.message = "Verknüpfung erfolgreich.";
-        notification.description = `Patienten ID: ${patientId} & Proben ID: ${sampleId}.`;
-      } else if (type === "error") {
-        notification.message = "Es is ein Fehler aufgetreten.";
-        notification.description = `Patienten ID: ${patientId} & Proben ID: ${sampleId}.`;
-      }
-      this.$notification[type](notification);
-    }
-  }
-};
-</script>
-
-<style></style>
diff --git a/client/src/components/laboratory/LinkTestAndPatient.vue b/client/src/components/laboratory/LinkTestAndPatient.vue
new file mode 100644
index 00000000..7cd4f878
--- /dev/null
+++ b/client/src/components/laboratory/LinkTestAndPatient.vue
@@ -0,0 +1,121 @@
+<template>
+  <a-card style="width: 500px; margin: 2rem auto; min-height: 300px">
+    <div v-if="!createdLabTest">
+      <a-form
+        :label-col="{ span: 6 }"
+        :wrapper-col="{ span: 18 }"
+        :colon="false"
+        :form="form"
+        @submit="handleSubmit"
+      >
+        <a-form-item label="Patienten-ID">
+          <a-input
+            v-decorator="[
+              'patientId',
+              {
+                rules: [
+                  {
+                    required: true,
+                    message: 'Bitte geben Sie die Patienten-ID ein.'
+                  }
+                ]
+              }
+            ]"
+            placeholder="z.B 1337-4237-9438"
+          />
+        </a-form-item>
+        <a-form-item label="Test-ID">
+          <a-input
+            v-decorator="[
+              'testId',
+              {
+                rules: [
+                  {
+                    required: true,
+                    message: 'Bitte geben Sie die Test-ID ein.'
+                  }
+                ]
+              }
+            ]"
+            placeholder="z.B ae48-hr43-sk97"
+          />
+        </a-form-item>
+        <a-form-item label="Kommentar">
+          <a-textarea
+            v-decorator="['comment']"
+            placeholder="Kommentar hinzufügen"
+            :autoSize="{ minRows: 3, maxRows: 5 }"
+          />
+        </a-form-item>
+        <a-divider />
+        <a-form-item :wrapper-col="{ span: 24, offset: 0 }">
+          <a-button type="primary" html-type="submit">
+            Speichern
+          </a-button>
+        </a-form-item>
+      </a-form>
+    </div>
+    <div v-else>
+      <div>Der Test wurde erfolgreich angelegt.</div>
+      <br />
+      <div>Die Test ID lautet: {{ createdLabTest.id }}</div>
+    </div>
+  </a-card>
+</template>
+
+<script>
+import Api from "../../api/Api";
+
+export default {
+  name: "LinkTestAndPatient",
+  props: {
+    laboratoryId: {
+      type: String,
+      required: true
+    }
+  },
+  data() {
+    return {
+      form: this.$form.createForm(this),
+      createdLabTest: null
+    };
+  },
+  methods: {
+    handleSubmit(e) {
+      e.preventDefault();
+
+      this.form.validateFields((err, values) => {
+        if (err) {
+          return;
+        }
+
+        const request = {
+          ...values,
+          laboratoryId: this.laboratoryId
+        };
+
+        Api.postLabTest(request)
+          .then(labTest => {
+            this.createdLabTest = labTest;
+
+            const notification = {
+              message: "Test angelegt und verknüpft.",
+              description:
+                "Der Test wurde erfolgreich angelegt und mit dem Patienten verknüpft."
+            };
+            this.$notification["success"](notification);
+          })
+          .catch(err => {
+            const notification = {
+              message: "Fehler beim Anlegen des Tests.",
+              description: err.message
+            };
+            this.$notification["error"](notification);
+          });
+      });
+    }
+  }
+};
+</script>
+
+<style></style>
diff --git a/client/src/components/laboratory/LinkTestResultAndPatient.vue b/client/src/components/laboratory/LinkTestResultAndPatient.vue
index fe2b9abd..4a9eb0d8 100644
--- a/client/src/components/laboratory/LinkTestResultAndPatient.vue
+++ b/client/src/components/laboratory/LinkTestResultAndPatient.vue
@@ -5,7 +5,7 @@
       :wrapper-col="{ span: 18 }"
       @submit="handleSubmit"
     >
-      <a-form-item label="Proben-ID">
+      <a-form-item label="Test-ID">
         <a-input
           v-decorator="[
             'note',
diff --git a/client/src/components/laboratory/Login.vue b/client/src/components/laboratory/Login.vue
index fb0953a2..7734b9b0 100644
--- a/client/src/components/laboratory/Login.vue
+++ b/client/src/components/laboratory/Login.vue
@@ -43,8 +43,7 @@
       </a-form-item>
     </a-form>
     <p class="test-acess">
-      Testzugang: Kennung <i><b>1234</b></i
-      >, Password <i><b>asdf</b></i>
+      Testzugang mit Password <i><b>asdf</b></i>
     </p>
   </a-card>
 </template>
@@ -71,8 +70,8 @@ export default {
           return;
         }
 
-        if (values.id === "1234" && values.password === "asdf") {
-          this.$emit("on-login-success");
+        if (values.id && values.password === "asdf") {
+          this.$emit("on-login-success", values.id);
         } else {
           this.$notification["error"]({
             message: "Login Fehler",
diff --git a/client/src/components/page/LinkSampleAndPatientPage.vue b/client/src/components/page/LinkTestAndPatientPage.vue
similarity index 60%
rename from client/src/components/page/LinkSampleAndPatientPage.vue
rename to client/src/components/page/LinkTestAndPatientPage.vue
index b955e8a4..034cb65e 100644
--- a/client/src/components/page/LinkSampleAndPatientPage.vue
+++ b/client/src/components/page/LinkTestAndPatientPage.vue
@@ -5,17 +5,17 @@
         <Login
           @on-login-success="onLoginSuccess"
           :title="
-            'Bitte melden Sie sich hier mit Kennung und Passwort an um eine Probe einem Patienten zuzuordnen.'
+            'Bitte melden Sie sich hier mit Laborkennung und Passwort an um einen Test einem Patienten zuzuordnen.'
           "
         />
       </a-tab-pane>
       <a-tab-pane
-        tab="2. Probe zuordnen"
+        tab="2. Test zuordnen"
         key="2"
         forceRender
         :disabled="!isLoggedIn"
       >
-        <LinkSampleAndPatient />
+        <LinkTestAndPatient :laboratoryId="laboratoryId" />
       </a-tab-pane>
     </a-tabs>
   </div>
@@ -23,22 +23,24 @@
 
 <script>
 import Login from "../laboratory/Login";
-import LinkSampleAndPatient from "../laboratory/LinkSampleAndPatient";
+import LinkTestAndPatient from "../laboratory/LinkTestAndPatient";
 
 export default {
-  name: "LinkSampleAndPatientPage",
+  name: "LinkTestAndPatientPage",
   components: {
     Login,
-    LinkSampleAndPatient
+    LinkTestAndPatient
   },
   data() {
     return {
       activeKey: "1",
-      isLoggedIn: false
+      isLoggedIn: false,
+      laboratoryId: ""
     };
   },
   methods: {
-    onLoginSuccess() {
+    onLoginSuccess(laboratoryId) {
+      this.laboratoryId = laboratoryId;
       this.isLoggedIn = true;
       this.activeKey = "2";
     }
diff --git a/client/src/routes/routes.js b/client/src/routes/routes.js
index a3165056..d146c8b1 100644
--- a/client/src/routes/routes.js
+++ b/client/src/routes/routes.js
@@ -1,5 +1,5 @@
 import LinkTestResultAndPatientPage from "../components/page/LinkTestResultAndPatientPage";
-import LinkSampleAndPatientPage from "../components/page/LinkSampleAndPatientPage";
+import LinkTestAndPatientPage from "../components/page/LinkTestAndPatientPage";
 import RegisterInstitutionPage from "../components/page/RegisterInstitutionPage";
 import RegisterPatientPage from "../components/page/RegisterPatientPage";
 
@@ -34,7 +34,7 @@ export const routes = [
     component: ViewAllDataComponent
   },
   {
-    path: "/link-sample-and-patient",
-    component: LinkSampleAndPatientPage
+    path: "/link-test-and-patient",
+    component: LinkTestAndPatientPage
   }
 ];
diff --git a/server/src/main/java/de/coronavirus/imis/TestDataLoader.java b/server/src/main/java/de/coronavirus/imis/TestDataLoader.java
index 64197ab0..c6ccd5d3 100644
--- a/server/src/main/java/de/coronavirus/imis/TestDataLoader.java
+++ b/server/src/main/java/de/coronavirus/imis/TestDataLoader.java
@@ -47,7 +47,6 @@ static Object makeDTO(String testFileName, Class clazz)
         Resource resource = new ClassPathResource("sample_data" + File.separator + testFileName);
         File file = resource.getFile();
 
-
         byte[] encoded = Files.readAllBytes(file.toPath());
 
         String str = new String(encoded, Charset.defaultCharset());
@@ -62,7 +61,6 @@ public void run(ApplicationArguments args) {
                 patientService.addPatient(createPersonDTO);
             }
 
-
             // SETUP OUR WORLD
             var createLaboratoryDTO = (CreateInstitutionDTO) makeDTO("createLaboratory.json", CreateInstitutionDTO.class);
             var laboratory = institutionService.createLaboratoryInstitution(createLaboratoryDTO);
@@ -87,8 +85,9 @@ public void run(ApplicationArguments args) {
             eventService.createScheduledEvent(person, laboratory.getId(), doctorsOffice.getId());
 
             // LAB RECEIVES SAMPLE AND PROCESSES IT
-            final String labInternalTestId = "42";
-            var labTest = labTestService.createLabTest(person.getId(), laboratory.getId(), labInternalTestId);
+            final String testId = "42";
+            final String comment = "comment";
+            var labTest = labTestService.createLabTest(person.getId(), laboratory.getId(), testId, comment);
 
 
             // LAB HAS RESULT AND SOTRES IT
diff --git a/server/src/main/java/de/coronavirus/imis/api/LabTestController.java b/server/src/main/java/de/coronavirus/imis/api/LabTestController.java
index 8c956ca1..a94617cd 100644
--- a/server/src/main/java/de/coronavirus/imis/api/LabTestController.java
+++ b/server/src/main/java/de/coronavirus/imis/api/LabTestController.java
@@ -17,7 +17,7 @@
 import org.springframework.web.bind.annotation.RestController;
 
 @RestController
-@RequestMapping("/labtest")
+@RequestMapping("/labtests")
 @AllArgsConstructor
 public class LabTestController {
 
@@ -25,19 +25,22 @@ public class LabTestController {
 
     @PostMapping
     public ResponseEntity<LabTest> createTestForPatient(@RequestBody CreateLabTestDTO createLabTestRequest) {
-        return ResponseEntity.ok(service.createLabTest(createLabTestRequest.getPatientId(),
+        return ResponseEntity.ok(service.createLabTest(
+                createLabTestRequest.getPatientId(),
                 createLabTestRequest.getLaboratoryId(),
-                createLabTestRequest.getLabInternalId()));
+                createLabTestRequest.getTestId(),
+                createLabTestRequest.getComment())
+        );
     }
 
     @GetMapping("/patient/{id}")
     public Set<LabTest> getLabTestForPatient(@PathVariable("id") String patientId) {
-        return service.getAllLabTestForPatient(patientId);
+        return service.getAllLabsTestForPatient(patientId);
     }
 
     @PutMapping("/{id}")
     public PatientEvent updateTestStatus(@PathVariable("id") String id, @RequestBody UpdateTestStatusDTO statusDTO) {
-        return service.updateTestStatus(id, statusDTO.getUpdatedTestStatus());
+        return service.updateTestStatus(id, statusDTO.getStatus());
     }
 
 
diff --git a/server/src/main/java/de/coronavirus/imis/api/dto/CreateLabTestDTO.java b/server/src/main/java/de/coronavirus/imis/api/dto/CreateLabTestDTO.java
index f9a80099..4c267611 100644
--- a/server/src/main/java/de/coronavirus/imis/api/dto/CreateLabTestDTO.java
+++ b/server/src/main/java/de/coronavirus/imis/api/dto/CreateLabTestDTO.java
@@ -6,5 +6,6 @@
 public class CreateLabTestDTO {
     private String patientId;
     private String laboratoryId;
-    private String labInternalId;
+    private String testId;
+    private String comment;
 }
diff --git a/server/src/main/java/de/coronavirus/imis/api/dto/UpdateTestStatusDTO.java b/server/src/main/java/de/coronavirus/imis/api/dto/UpdateTestStatusDTO.java
index 5cf1c2e3..1b97f483 100644
--- a/server/src/main/java/de/coronavirus/imis/api/dto/UpdateTestStatusDTO.java
+++ b/server/src/main/java/de/coronavirus/imis/api/dto/UpdateTestStatusDTO.java
@@ -4,5 +4,5 @@
 
 @Data
 public class UpdateTestStatusDTO {
-    private String updatedTestStatus;
+    private String status;
 }
diff --git a/server/src/main/java/de/coronavirus/imis/domain/LabTest.java b/server/src/main/java/de/coronavirus/imis/domain/LabTest.java
index 8a6be1fe..45e71073 100644
--- a/server/src/main/java/de/coronavirus/imis/domain/LabTest.java
+++ b/server/src/main/java/de/coronavirus/imis/domain/LabTest.java
@@ -1,19 +1,17 @@
 package de.coronavirus.imis.domain;
 
+import com.fasterxml.jackson.annotation.JsonIgnore;
+import com.fasterxml.jackson.databind.annotation.JsonPOJOBuilder;
 import javax.persistence.Entity;
 import javax.persistence.GeneratedValue;
 import javax.persistence.Id;
 import javax.persistence.ManyToOne;
 import javax.persistence.OneToOne;
-
-import org.hibernate.annotations.GenericGenerator;
-
-import com.fasterxml.jackson.annotation.JsonIgnore;
-import com.fasterxml.jackson.databind.annotation.JsonPOJOBuilder;
 import lombok.AllArgsConstructor;
 import lombok.Builder;
 import lombok.Data;
 import lombok.NoArgsConstructor;
+import org.hibernate.annotations.GenericGenerator;
 
 @Entity
 @Data
@@ -27,8 +25,7 @@ public class LabTest {
     @GenericGenerator(name = "uuid2", strategy = "uuid2")
     private String id;
 
-
-    private String laborTestID;
+    private String testId;
 
     @ManyToOne
     @JsonIgnore
@@ -36,6 +33,8 @@ public class LabTest {
 
     private TestStatus testStatus;
 
+    private String comment;
+
     @OneToOne
     private TestReport report;
 
diff --git a/server/src/main/java/de/coronavirus/imis/services/LabTestService.java b/server/src/main/java/de/coronavirus/imis/services/LabTestService.java
index 28375449..8d11c737 100644
--- a/server/src/main/java/de/coronavirus/imis/services/LabTestService.java
+++ b/server/src/main/java/de/coronavirus/imis/services/LabTestService.java
@@ -1,17 +1,5 @@
 package de.coronavirus.imis.services;
 
-import java.util.List;
-import java.util.Objects;
-import java.util.Optional;
-import java.util.Set;
-import java.util.stream.Collectors;
-
-import javax.transaction.Transactional;
-
-import org.springframework.stereotype.Component;
-
-import lombok.RequiredArgsConstructor;
-
 import de.coronavirus.imis.domain.EventType;
 import de.coronavirus.imis.domain.LabTest;
 import de.coronavirus.imis.domain.Laboratory;
@@ -23,32 +11,47 @@
 import de.coronavirus.imis.repositories.LabTestRepository;
 import de.coronavirus.imis.repositories.LaboratoryRepository;
 import de.coronavirus.imis.repositories.PatientEventRepository;
+import java.util.List;
+import java.util.Objects;
+import java.util.Optional;
+import java.util.Set;
+import java.util.stream.Collectors;
+import javax.transaction.Transactional;
+import lombok.RequiredArgsConstructor;
+import org.springframework.stereotype.Component;
 
 @Component
 @RequiredArgsConstructor
 public class LabTestService {
+
     private final PatientService patientService;
     private final PatientEventService eventService;
     private final LaboratoryRepository laboratoryRepository;
     private final LabTestRepository labTestRepository;
     private final PatientEventRepository eventRepository;
 
-
     @Transactional
-    public LabTest createLabTest(String patientId, String labId, String labInternalId) {
-        final Patient patient = patientService.findPatientById(patientId).orElseThrow(PatientNotFoundException::new);
-        final Laboratory laboratory = laboratoryRepository.findById(labId).orElseThrow(LaboratoryNotFoundException::new);
-        final LabTest labTest = LabTest.builder().
-                laboratory(laboratory).testStatus(TestStatus.TEST_SUBMITTED)
-                .laborTestID(labInternalId)
+    public LabTest createLabTest(String patientId, String laboratoryId, String testId, String comment) {
+        final Patient patient = patientService
+                .findPatientById(patientId)
+                .orElseThrow(PatientNotFoundException::new);
+        final Laboratory laboratory = laboratoryRepository
+                .findById(laboratoryId)
+                .orElseThrow(LaboratoryNotFoundException::new);
+        final LabTest labTest = LabTest.builder()
+                .laboratory(laboratory)
+                .testStatus(TestStatus.TEST_SUBMITTED)
+                .testId(testId)
+                .comment(comment)
                 .build();
+
         labTestRepository.save(labTest);
         eventService.createLabTestEvent(patient, labTest, Optional.empty());
         return labTest;
     }
 
     @Transactional
-    public Set<LabTest> getAllLabTestForPatient(String patiendId) {
+    public Set<LabTest> getAllLabsTestForPatient(String patiendId) {
         final Patient patient = patientService.findPatientById(patiendId).orElseThrow(PatientNotFoundException::new);
         final var events = eventService.getAllForPatient(patient);
         return events.stream().map(PatientEvent::getLabTest).collect(Collectors.toSet());
diff --git a/server/src/main/resources/sample_data/createLabTest.json b/server/src/main/resources/sample_data/createLabTest.json
index fedc2ac8..a6c0d89e 100644
--- a/server/src/main/resources/sample_data/createLabTest.json
+++ b/server/src/main/resources/sample_data/createLabTest.json
@@ -1,5 +1,6 @@
 {
-  "labInternalId": "string",
+  "testId": "string",
   "laboratoryId": "string",
-  "patientId": "string"
+  "patientId": "string",
+  "comment": "string"
 }
\ No newline at end of file

From 162daa76ef286c28828ba00e53f1872a288491a2 Mon Sep 17 00:00:00 2001
From: DavidB <klausbart@gmx.net>
Date: Wed, 25 Mar 2020 19:55:53 +0100
Subject: [PATCH 2/3] Website should now always be 100% viewport height

---
 client/src/App.vue | 14 ++++++++------
 1 file changed, 8 insertions(+), 6 deletions(-)

diff --git a/client/src/App.vue b/client/src/App.vue
index 4e4f6093..b5032a4a 100644
--- a/client/src/App.vue
+++ b/client/src/App.vue
@@ -6,15 +6,12 @@
     </div>
 
     <a-layout id="components-layout-demo-responsive">
-      <div class="logo" />
-      <Navigation />
       <a-layout>
-        <a-layout-header
-          :style="{ background: '#fff', padding: 0, height: 'auto' }"
-        >
+        <a-layout-header style="height: auto">
+          <Navigation />
         </a-layout-header>
         <a-layout-content
-          style="margin: '0px auto'; box-sizing: border-box; padding: 2rem 4rem; width: 100%; min-height: calc(100vh - 46px - 69px);"
+          style="margin: auto; box-sizing: border-box; padding: 0rem 4rem; width: 100%;"
         >
           <div :style="{ padding: '0px', minHeight: '360px', width: '100%' }">
             <router-view></router-view>
@@ -51,6 +48,11 @@ export default {
 .prototype-warning {
   border: 2px solid red;
   font-size: 20px;
+  height: 35px;
   background-color: #ffdddd;
 }
+
+  #components-layout-demo-responsive {
+    height: calc(100vh - 35px);
+  }
 </style>

From 950aeaeb4dc5d0241a7ea3ea430c37da7b1ef2e2 Mon Sep 17 00:00:00 2001
From: Kuettner Tobias <tobias.kuettner@scandio.de>
Date: Wed, 25 Mar 2020 20:39:12 +0100
Subject: [PATCH 3/3] Implement Linking Test Result an Test / Patient Implement
 File Upload

---
 client/src/api/Api.js                         |  20 +-
 .../laboratory/LinkTestAndPatient.vue         |   3 +-
 .../laboratory/LinkTestResultAndPatient.vue   | 196 ++++++++++++------
 client/src/components/laboratory/Login.vue    |   1 -
 .../de/coronavirus/imis/TestDataLoader.java   |   2 +-
 .../imis/api/LabTestController.java           |  11 +-
 .../imis/api/TestReportController.java        |  41 +---
 .../imis/api/dto/UpdateTestStatusDTO.java     |   2 +
 .../coronavirus/imis/domain/Laboratory.java   |  10 +-
 .../coronavirus/imis/domain/TestReport.java   |   4 +-
 .../repositories/TestReportRepository.java    |   6 +-
 .../imis/services/LabTestService.java         |  38 +++-
 .../imis/services/TestReportService.java      |  17 +-
 13 files changed, 209 insertions(+), 142 deletions(-)

diff --git a/client/src/api/Api.js b/client/src/api/Api.js
index deb26f58..475cd8e4 100644
--- a/client/src/api/Api.js
+++ b/client/src/api/Api.js
@@ -45,16 +45,20 @@ class Api {
    * -----------IMPLEMENTED IN UI-----------
    */
 
-  postInstitution(institution) {
-    return this.executeRequest("/institutions", METHOD.POST, institution);
+  postInstitution(request) {
+    return this.executeRequest("/institutions", METHOD.POST, request);
   }
 
-  postPatient(patient) {
-    return this.executeRequest("/patients", METHOD.POST, patient);
+  postPatient(request) {
+    return this.executeRequest("/patients", METHOD.POST, request);
   }
 
-  postLabTest(labtest) {
-    return this.executeRequest("/labtests", METHOD.POST, labtest);
+  postLabTest(request) {
+    return this.executeRequest("/labtests", METHOD.POST, request);
+  }
+
+  putLabTest(testId, request) {
+    return this.executeRequest(`/labtests/${testId}`, METHOD.PUT, request);
   }
 
   /*
@@ -93,10 +97,6 @@ class Api {
     return this.executeRequest(`/test_reports/${testId}`, METHOD.DELETE);
   }
 
-  putLabtest({ updatedTestStatus }) {
-    return this.executeRequest("/labtest", METHOD.PUT, updatedTestStatus);
-  }
-
   getLabtestByPatient(patientId) {
     return this.executeRequest(`/labtest/patient/${patientId}`, METHOD.GET);
   }
diff --git a/client/src/components/laboratory/LinkTestAndPatient.vue b/client/src/components/laboratory/LinkTestAndPatient.vue
index 7cd4f878..0d56584a 100644
--- a/client/src/components/laboratory/LinkTestAndPatient.vue
+++ b/client/src/components/laboratory/LinkTestAndPatient.vue
@@ -58,7 +58,8 @@
     <div v-else>
       <div>Der Test wurde erfolgreich angelegt.</div>
       <br />
-      <div>Die Test ID lautet: {{ createdLabTest.id }}</div>
+      <div>Test ID: {{ createdLabTest.id }}</div>
+      <div>Test Status: {{ createdLabTest.testStatus }}</div>
     </div>
   </a-card>
 </template>
diff --git a/client/src/components/laboratory/LinkTestResultAndPatient.vue b/client/src/components/laboratory/LinkTestResultAndPatient.vue
index 4a9eb0d8..5db3a0a3 100644
--- a/client/src/components/laboratory/LinkTestResultAndPatient.vue
+++ b/client/src/components/laboratory/LinkTestResultAndPatient.vue
@@ -1,84 +1,150 @@
 <template>
   <a-card style="width: 500px; margin: 2rem auto; min-height: 300px">
-    <a-form
-      :label-col="{ span: 6 }"
-      :wrapper-col="{ span: 18 }"
-      @submit="handleSubmit"
-    >
-      <a-form-item label="Test-ID">
-        <a-input
-          v-decorator="[
-            'note',
-            {
-              rules: [
-                { required: true, message: 'Bitte geben Sie Ihre Test-ID ein.' }
-              ]
-            }
-          ]"
-          placeholder="z.B 1337-4237-9438"
-        />
-      </a-form-item>
-      <a-form-item label="Testresultat">
-        <a-radio-group v-decorator="['radio-group']">
-          <a-radio value="true">
-            Positiv
-          </a-radio>
-          <a-radio value="false">
-            Negativ
-          </a-radio>
-        </a-radio-group>
-      </a-form-item>
-      <a-form-item label="Kommentar">
-        <a-textarea
-          placeholder="Kommentar hinzufügen"
-          :autoSize="{ minRows: 3, maxRows: 5 }"
-        />
-      </a-form-item>
-      <a-form-item :wrapper-col="{ span: 24 }">
-        <!-- TODO: FILE UPLOAD! -->
-        <a-upload action="https://www.mocky.io/v2/5cc8019d300000980a055e76">
-          <a-button> <a-icon type="upload" />Test Report hochladen</a-button>
-        </a-upload>
-      </a-form-item>
-      <a-divider />
-      <a-form-item :wrapper-col="{ span: 24, offset: 0 }">
-        <a-button type="primary" html-type="submit">
-          Speichern
-        </a-button>
-      </a-form-item>
-    </a-form>
+    <div v-if="!updatedLabTest">
+      <a-form
+        :form="form"
+        :label-col="{ span: 6 }"
+        :wrapper-col="{ span: 18 }"
+        @submit="handleSubmit"
+      >
+        <a-form-item label="Test-ID">
+          <a-input
+            v-decorator="[
+              'testId',
+              {
+                rules: [
+                  {
+                    required: true,
+                    message: 'Bitte geben Sie Ihre Test-ID ein.'
+                  }
+                ]
+              }
+            ]"
+            placeholder="z.B 1337-4237-9438"
+          />
+        </a-form-item>
+        <a-form-item label="Testresultat">
+          <a-radio-group
+            v-decorator="[
+              'testResult',
+              {
+                rules: [
+                  {
+                    required: true,
+                    message: 'Bitte geben Sie das Testresultat an.'
+                  }
+                ]
+              }
+            ]"
+          >
+            <a-radio value="positive">
+              Positiv
+            </a-radio>
+            <a-radio value="negative">
+              Negativ
+            </a-radio>
+          </a-radio-group>
+        </a-form-item>
+        <a-form-item label="Kommentar">
+          <a-textarea
+            placeholder="Kommentar hinzufügen"
+            :autoSize="{ minRows: 3, maxRows: 5 }"
+            v-decorator="['comment']"
+          />
+        </a-form-item>
+        <a-form-item :wrapper-col="{ span: 24 }">
+          <a-upload
+            :accept="'pdf'"
+            :beforeUpload="beforeUpload"
+            :multiple="false"
+          >
+            <a-button> <a-icon type="upload" />Test Report hochladen</a-button>
+          </a-upload>
+        </a-form-item>
+        <a-divider />
+        <a-form-item :wrapper-col="{ span: 24, offset: 0 }">
+          <a-button type="primary" html-type="submit">
+            Speichern
+          </a-button>
+        </a-form-item>
+      </a-form>
+    </div>
+    <div v-else>
+      <div>Der Test wurde erfolgreich aktualisiert.</div>
+      <br />
+      <div>Test ID: {{ updatedLabTest.id }}</div>
+      <div>Test Status: {{ updatedLabTest.testStatus }}</div>
+    </div>
   </a-card>
 </template>
 
 <script>
-// TODO: Use form from ant and to get the values from the <form> (see PatientDataCompontent)
+import Api from "../../api/Api";
 
 export default {
   name: "LinkTestResultAndPatient",
+  data() {
+    return {
+      form: this.$form.createForm(this),
+      updatedLabTest: null,
+      fileBytes: null
+    };
+  },
   methods: {
+    beforeUpload(file) {
+      const setFileBytes = fileBytes => {
+        this.fileBytes = fileBytes;
+      };
+
+      const reader = new FileReader();
+      reader.onload = function(e) {
+        const utf8 = unescape(encodeURIComponent(e.target.result));
+        const array = [];
+        for (let i = 0; i < utf8.length; i++) {
+          array.push(utf8.charCodeAt(i));
+        }
+        setFileBytes(array);
+      };
+      reader.readAsDataURL(file);
+
+      return false;
+    },
     handleSubmit(e) {
       e.preventDefault();
 
-      // TODO: Send request to connect patient with test result
-      console.log("Handle connect patient with test result");
+      this.form.validateFields((err, values) => {
+        if (err) {
+          return;
+        }
 
-      const patientId = "198424989813";
-      const testResult = "negative";
+        const { testId } = values;
+        const request = {
+          status:
+            values.testResult === "positive"
+              ? "TEST_POSITIVE"
+              : "TEST_NEGATIVE",
+          comment: values.comment,
+          file: this.fileBytes
+        };
 
-      // TODO: Handle response from server
-      const response = { type: "success" };
-      const { type } = response;
+        Api.putLabTest(testId, request)
+          .then(labTest => {
+            this.updatedLabTest = labTest;
 
-      // Check notification type (success, info, warning, error)
-      const notification = {};
-      if (type === "success") {
-        notification.message = "Verknüpfung erfolgreich.";
-        notification.description = `Patienten ID: ${patientId} & Test Resultat: ${testResult}.`;
-      } else if (type === "error") {
-        notification.message = "Es is ein Fehler aufgetreten.";
-        notification.description = `Patienten ID: ${patientId} & Test Resultat: ${testResult}.`;
-      }
-      this.$notification[type](notification);
+            const notification = {
+              message: "Testergebnis hinzugefügt.",
+              description: "Das Testergebnis wurde erfolgreich hinzugefügt."
+            };
+            this.$notification["success"](notification);
+          })
+          .catch(err => {
+            const notification = {
+              message: "Fehler beim Hinzufügen des Testergebnisses.",
+              description: err.message
+            };
+            this.$notification["error"](notification);
+          });
+      });
     }
   }
 };
diff --git a/client/src/components/laboratory/Login.vue b/client/src/components/laboratory/Login.vue
index 7734b9b0..4d439091 100644
--- a/client/src/components/laboratory/Login.vue
+++ b/client/src/components/laboratory/Login.vue
@@ -49,7 +49,6 @@
 </template>
 
 <script>
-// TODO: Use form from ant and to get the values from the <form> (see PatientDataCompontent)
 
 export default {
   name: "Login",
diff --git a/server/src/main/java/de/coronavirus/imis/TestDataLoader.java b/server/src/main/java/de/coronavirus/imis/TestDataLoader.java
index c6ccd5d3..568b7e90 100644
--- a/server/src/main/java/de/coronavirus/imis/TestDataLoader.java
+++ b/server/src/main/java/de/coronavirus/imis/TestDataLoader.java
@@ -92,7 +92,7 @@ public void run(ApplicationArguments args) {
 
             // LAB HAS RESULT AND SOTRES IT
             // FIXME: 22.03.20 report cannot be attached
-            labTestService.updateTestStatus(labTest.getId(), "TEST_POSITIVE");
+            labTestService.updateTestStatus(labTest.getId(), "TEST_POSITIVE", comment, null);
 
             // HEALTH OFFICE WANTS TO SEE ALL DATA
             var allPatients = patientService.getAllPatients();
diff --git a/server/src/main/java/de/coronavirus/imis/api/LabTestController.java b/server/src/main/java/de/coronavirus/imis/api/LabTestController.java
index a94617cd..0d232951 100644
--- a/server/src/main/java/de/coronavirus/imis/api/LabTestController.java
+++ b/server/src/main/java/de/coronavirus/imis/api/LabTestController.java
@@ -39,9 +39,12 @@ public Set<LabTest> getLabTestForPatient(@PathVariable("id") String patientId) {
     }
 
     @PutMapping("/{id}")
-    public PatientEvent updateTestStatus(@PathVariable("id") String id, @RequestBody UpdateTestStatusDTO statusDTO) {
-        return service.updateTestStatus(id, statusDTO.getStatus());
+    public ResponseEntity<LabTest> updateTestStatus(@PathVariable("id") String id, @RequestBody UpdateTestStatusDTO statusDTO) {
+        return ResponseEntity.ok(service.updateTestStatus(
+                id,
+                statusDTO.getStatus(),
+                statusDTO.getComment(),
+                statusDTO.getFile())
+        );
     }
-
-
 }
diff --git a/server/src/main/java/de/coronavirus/imis/api/TestReportController.java b/server/src/main/java/de/coronavirus/imis/api/TestReportController.java
index cc8090c4..a84b132d 100644
--- a/server/src/main/java/de/coronavirus/imis/api/TestReportController.java
+++ b/server/src/main/java/de/coronavirus/imis/api/TestReportController.java
@@ -2,12 +2,13 @@
 
 import de.coronavirus.imis.domain.TestReport;
 import de.coronavirus.imis.services.TestReportService;
+import java.util.List;
 import lombok.RequiredArgsConstructor;
 import org.springframework.http.ResponseEntity;
-import org.springframework.web.bind.annotation.*;
-import org.springframework.web.multipart.MultipartFile;
-
-import java.util.List;
+import org.springframework.web.bind.annotation.GetMapping;
+import org.springframework.web.bind.annotation.PathVariable;
+import org.springframework.web.bind.annotation.RequestMapping;
+import org.springframework.web.bind.annotation.RestController;
 
 @RestController
 @RequestMapping("/test_reports")
@@ -17,39 +18,15 @@ public class TestReportController {
     // Instance of the storage service.
     private final TestReportService testReportService;
 
-    /***
-     * Endpoint to upload a file.
-     * @param file: The file to upload.
-     * @param id: The id of the test report the file belongs to.
-     * @return A redirection.
-     */
-    @PostMapping("/{id}")
-    public ResponseEntity<TestReport> uploadTestReport(
-            @RequestParam("file") MultipartFile file,
-            @PathVariable("id") String id
-    ) {
-        TestReport testReport;
-        try {
-            testReport = testReportService
-                    .addTestReport(id, file.getBytes());
-        } catch (Exception e) {
-            return ResponseEntity.status(500).build();
-        }
-
-        return ResponseEntity.ok().body(testReport);
-    }
-
     /***
      * Endpoint to receive a test report.
-     * @param id: The id of the test report to retrieve.
+     * @param testId: The testId of the test report to retrieve.
      * @return The test report.
      */
-    @GetMapping("/{id}")
-    public ResponseEntity<TestReport> getTestReport(
-            @PathVariable String id
-    ) {
+    @GetMapping("/{testId}")
+    public ResponseEntity<TestReport> getTestReport(@PathVariable String testId) {
         return testReportService
-                .findTestReportById(id)
+                .findTestReportByTestId(testId)
                 .map(ResponseEntity::ok)
                 .orElse(ResponseEntity.notFound().build());
     }
diff --git a/server/src/main/java/de/coronavirus/imis/api/dto/UpdateTestStatusDTO.java b/server/src/main/java/de/coronavirus/imis/api/dto/UpdateTestStatusDTO.java
index 1b97f483..47efd3ff 100644
--- a/server/src/main/java/de/coronavirus/imis/api/dto/UpdateTestStatusDTO.java
+++ b/server/src/main/java/de/coronavirus/imis/api/dto/UpdateTestStatusDTO.java
@@ -5,4 +5,6 @@
 @Data
 public class UpdateTestStatusDTO {
     private String status;
+    private String comment;
+    private byte[] file;
 }
diff --git a/server/src/main/java/de/coronavirus/imis/domain/Laboratory.java b/server/src/main/java/de/coronavirus/imis/domain/Laboratory.java
index fdc9238b..6f8e80b2 100644
--- a/server/src/main/java/de/coronavirus/imis/domain/Laboratory.java
+++ b/server/src/main/java/de/coronavirus/imis/domain/Laboratory.java
@@ -1,21 +1,19 @@
 package de.coronavirus.imis.domain;
 
 import java.util.List;
-
 import javax.persistence.Entity;
 import javax.persistence.OneToMany;
-
-import lombok.Data;
 import lombok.EqualsAndHashCode;
+import lombok.Getter;
+import lombok.Setter;
 
 @EqualsAndHashCode(callSuper = true)
 @Entity
-@Data
+@Getter
+@Setter
 public class Laboratory extends Institution {
 
 
     @OneToMany(mappedBy = "laboratory")
     private List<LabTest> assignedLabTest;
-
-
 }
diff --git a/server/src/main/java/de/coronavirus/imis/domain/TestReport.java b/server/src/main/java/de/coronavirus/imis/domain/TestReport.java
index 591588bc..630780a0 100644
--- a/server/src/main/java/de/coronavirus/imis/domain/TestReport.java
+++ b/server/src/main/java/de/coronavirus/imis/domain/TestReport.java
@@ -2,6 +2,7 @@
 
 
 import javax.persistence.Entity;
+import javax.persistence.GeneratedValue;
 import javax.persistence.Id;
 import javax.persistence.Lob;
 
@@ -18,9 +19,8 @@
 @AllArgsConstructor
 public class TestReport {
 
-    // ID of the test.
     @Id
-    private String id;
+    private String testId;
 
     // Binary of the test files.
     @Lob
diff --git a/server/src/main/java/de/coronavirus/imis/repositories/TestReportRepository.java b/server/src/main/java/de/coronavirus/imis/repositories/TestReportRepository.java
index a52ce8eb..2b012f7c 100644
--- a/server/src/main/java/de/coronavirus/imis/repositories/TestReportRepository.java
+++ b/server/src/main/java/de/coronavirus/imis/repositories/TestReportRepository.java
@@ -1,6 +1,7 @@
 package de.coronavirus.imis.repositories;
 
 import de.coronavirus.imis.domain.TestReport;
+import java.util.Optional;
 import org.springframework.data.jpa.repository.JpaRepository;
 import org.springframework.stereotype.Repository;
 
@@ -8,5 +9,6 @@
  * Repository for file storage.
  */
 @Repository
-public interface TestReportRepository extends JpaRepository<TestReport, String>
-{}
+public interface TestReportRepository extends JpaRepository<TestReport, String> {
+    Optional<TestReport> findByTestId(String testId);
+}
diff --git a/server/src/main/java/de/coronavirus/imis/services/LabTestService.java b/server/src/main/java/de/coronavirus/imis/services/LabTestService.java
index 8d11c737..30fbc27e 100644
--- a/server/src/main/java/de/coronavirus/imis/services/LabTestService.java
+++ b/server/src/main/java/de/coronavirus/imis/services/LabTestService.java
@@ -2,6 +2,7 @@
 
 import de.coronavirus.imis.domain.EventType;
 import de.coronavirus.imis.domain.LabTest;
+import de.coronavirus.imis.domain.LabTestNotFoundException;
 import de.coronavirus.imis.domain.Laboratory;
 import de.coronavirus.imis.domain.LaboratoryNotFoundException;
 import de.coronavirus.imis.domain.Patient;
@@ -18,14 +19,17 @@
 import java.util.stream.Collectors;
 import javax.transaction.Transactional;
 import lombok.RequiredArgsConstructor;
+import lombok.extern.slf4j.Slf4j;
 import org.springframework.stereotype.Component;
 
 @Component
 @RequiredArgsConstructor
+@Slf4j
 public class LabTestService {
 
     private final PatientService patientService;
     private final PatientEventService eventService;
+    private final TestReportService testReportService;
     private final LaboratoryRepository laboratoryRepository;
     private final LabTestRepository labTestRepository;
     private final PatientEventRepository eventRepository;
@@ -58,22 +62,38 @@ public Set<LabTest> getAllLabsTestForPatient(String patiendId) {
     }
 
     @Transactional
-    public PatientEvent updateTestStatus(String testId, final String statusString) {
+    public LabTest updateTestStatus(final String testId, final String statusString, final String comment, final byte[] file) {
         TestStatus statusToSet = TestStatus.valueOf(statusString.toUpperCase());
-        var labTest = labTestRepository.findById(testId).orElseThrow();
-        final List<PatientEvent> event = eventService.getForLabTest(labTest);
+
+        var labTest = labTestRepository.findById(testId)
+                .orElseThrow(LabTestNotFoundException::new);
         labTest.setTestStatus(statusToSet);
+
+        final List<PatientEvent> event = eventService.getForLabTest(labTest);
         var eventType = testStatusToEvent(statusToSet);
-        var patient = event.stream().map(PatientEvent::getPatient).findFirst().orElseThrow();
-        var doctor = event.stream().map(PatientEvent::getResponsibleDoctor)
-                .filter(Objects::nonNull).findFirst();
+
+        var patient = event.stream()
+                .map(PatientEvent::getPatient)
+                .findFirst()
+                .orElseThrow();
 
         var changeEvent = new PatientEvent()
                 .setEventType(eventType)
                 .setLabTest(labTest)
-                .setPatient(patient);
-        doctor.ifPresent(changeEvent::setResponsibleDoctor);
-        return eventRepository.save(changeEvent);
+                .setPatient(patient)
+                .setComment(comment);
+
+        event.stream()
+                .map(PatientEvent::getResponsibleDoctor)
+                .filter(Objects::nonNull)
+                .findFirst()
+                .ifPresent(changeEvent::setResponsibleDoctor);
+
+        eventRepository.save(changeEvent);
+
+        Optional.ofNullable(file).ifPresent((bytes) -> testReportService.addTestReport(labTest.getId(), bytes));
+
+        return labTest;
     }
 
     private EventType testStatusToEvent(TestStatus input) {
diff --git a/server/src/main/java/de/coronavirus/imis/services/TestReportService.java b/server/src/main/java/de/coronavirus/imis/services/TestReportService.java
index 248cf5a2..f5372f3d 100644
--- a/server/src/main/java/de/coronavirus/imis/services/TestReportService.java
+++ b/server/src/main/java/de/coronavirus/imis/services/TestReportService.java
@@ -2,10 +2,9 @@
 
 import de.coronavirus.imis.domain.TestReport;
 import de.coronavirus.imis.repositories.TestReportRepository;
-import org.springframework.stereotype.Service;
-
 import java.util.List;
 import java.util.Optional;
+import org.springframework.stereotype.Service;
 
 /***
  * Implements functionality to save files related to a test to the database.
@@ -33,21 +32,21 @@ public List<TestReport> getAllTestReports() {
     }
 
     /***
-     * Returns a test report by its id from the database.
-     * @param id: The id of the desired test report.
+     * Returns a test report by its testId from the database.
+     * @param testId: The testId of the desired test report.
      * @return the desired test report.
      */
-    public Optional<TestReport> findTestReportById(String id) {
-        return testReportRepository.findById(id);
+    public Optional<TestReport> findTestReportByTestId(String testId) {
+        return testReportRepository.findByTestId(testId);
     }
 
     /***
      * Creates a test report and saves it in the database.
-     * @param id: Id of the test report.
+     * @param testId: Id of the test report.
      * @param file: Binary data of the test report.
      */
-    public TestReport addTestReport(String id, byte[] file) {
-        TestReport testReport = new TestReport(id, file);
+    public TestReport addTestReport(String testId, byte[] file) {
+        TestReport testReport = new TestReport(testId, file);
         testReportRepository.save(testReport);
         return testReport;
     }