From 5b4ecb51e116e37d36b20f7694bf784690ee744e Mon Sep 17 00:00:00 2001
From: Hanna Prinz <hanna.prinz@digitalservice.bund.de>
Date: Fri, 24 Nov 2023 15:33:12 +0100
Subject: [PATCH] Fix failing tests

RISDEV-2741
---
 .../transformer/EnsuingDecisionTransformer.java |  4 ++--
 .../transformer/PendingDecisionTransformer.java | 11 +++++++++++
 frontend/src/components/EnsuingDecisions.vue    | 17 ++++++++++++++++-
 frontend/src/domain/ensuingDecision.ts          |  2 +-
 .../active-citations.spec.ts                    | 10 +++++-----
 .../categories/ensuing-decisions.spec.ts        |  8 ++++----
 .../categories/previous-decisions.spec.ts       |  8 ++++----
 7 files changed, 43 insertions(+), 17 deletions(-)

diff --git a/backend/src/main/java/de/bund/digitalservice/ris/caselaw/adapter/transformer/EnsuingDecisionTransformer.java b/backend/src/main/java/de/bund/digitalservice/ris/caselaw/adapter/transformer/EnsuingDecisionTransformer.java
index fd7e14c4c4..7173e960cd 100644
--- a/backend/src/main/java/de/bund/digitalservice/ris/caselaw/adapter/transformer/EnsuingDecisionTransformer.java
+++ b/backend/src/main/java/de/bund/digitalservice/ris/caselaw/adapter/transformer/EnsuingDecisionTransformer.java
@@ -10,9 +10,9 @@ public static EnsuingDecision transformToDomain(EnsuingDecisionDTO ensuingDecisi
         .uuid(ensuingDecisionDTO.getId())
         .documentNumber(ensuingDecisionDTO.getDocumentNumber())
         .referencedDocumentationUnitId(
-            ensuingDecisionDTO.getReferencedDocumentationUnitId() == null
+            ensuingDecisionDTO.getReferencedDocumentationUnit() == null
                 ? null
-                : ensuingDecisionDTO.getReferencedDocumentationUnitId())
+                : ensuingDecisionDTO.getReferencedDocumentationUnit().getId())
         .court(getCourtFromDTO(ensuingDecisionDTO.getCourt()))
         .fileNumber(getFileNumber(ensuingDecisionDTO.getFileNumber()))
         .documentType(getDocumentTypeFromDTO(ensuingDecisionDTO.getDocumentType()))
diff --git a/backend/src/main/java/de/bund/digitalservice/ris/caselaw/adapter/transformer/PendingDecisionTransformer.java b/backend/src/main/java/de/bund/digitalservice/ris/caselaw/adapter/transformer/PendingDecisionTransformer.java
index d68854d7a5..b75a939607 100644
--- a/backend/src/main/java/de/bund/digitalservice/ris/caselaw/adapter/transformer/PendingDecisionTransformer.java
+++ b/backend/src/main/java/de/bund/digitalservice/ris/caselaw/adapter/transformer/PendingDecisionTransformer.java
@@ -1,5 +1,6 @@
 package de.bund.digitalservice.ris.caselaw.adapter.transformer;
 
+import de.bund.digitalservice.ris.caselaw.adapter.database.jpa.DocumentationUnitDTO;
 import de.bund.digitalservice.ris.caselaw.adapter.database.jpa.PendingDecisionDTO;
 import de.bund.digitalservice.ris.caselaw.domain.EnsuingDecision;
 
@@ -8,6 +9,10 @@ public static EnsuingDecision transformToDomain(PendingDecisionDTO pendingDecisi
     return EnsuingDecision.builder()
         .uuid(pendingDecisionDTO.getId())
         .documentNumber(pendingDecisionDTO.getDocumentNumber())
+        .referencedDocumentationUnitId(
+            pendingDecisionDTO.getReferencedDocumentationUnit() == null
+                ? null
+                : pendingDecisionDTO.getReferencedDocumentationUnit().getId())
         .court(getCourtFromDTO(pendingDecisionDTO.getCourt()))
         .fileNumber(getFileNumber(pendingDecisionDTO.getFileNumber()))
         .documentType(getDocumentTypeFromDTO(pendingDecisionDTO.getDocumentType()))
@@ -27,6 +32,12 @@ public static PendingDecisionDTO transformToDTO(EnsuingDecision ensuingDecision)
         .court(getCourtFromDomain(ensuingDecision.getCourt()))
         .date(ensuingDecision.getDecisionDate())
         .documentNumber(ensuingDecision.getDocumentNumber())
+        .referencedDocumentationUnit(
+            ensuingDecision.getReferencedDocumentationUnitId() == null
+                ? null
+                : DocumentationUnitDTO.builder()
+                    .id(ensuingDecision.getReferencedDocumentationUnitId())
+                    .build())
         .documentType(getDocumentTypeFromDomain(ensuingDecision.getDocumentType()))
         .fileNumber(getFileNumber(ensuingDecision.getFileNumber()))
         .note(ensuingDecision.getNote())
diff --git a/frontend/src/components/EnsuingDecisions.vue b/frontend/src/components/EnsuingDecisions.vue
index 97aa7f1373..605ca65d1f 100644
--- a/frontend/src/components/EnsuingDecisions.vue
+++ b/frontend/src/components/EnsuingDecisions.vue
@@ -1,5 +1,6 @@
 <script lang="ts" setup>
 import { h, computed } from "vue"
+import { RouterLink } from "vue-router"
 import EnsuingDecisionInputGroup from "./EnsuingDecisionInputGroup.vue"
 import EditableList from "@/components/EditableListCaselaw.vue"
 import EnsuingDecision from "@/domain/ensuingDecision"
@@ -25,7 +26,21 @@ const ensuingDecisions = computed({
 const defaultValue = new EnsuingDecision()
 
 function decisionSummarizer(dataEntry: EnsuingDecision) {
-  if (
+  if (dataEntry.isReadOnly) {
+    return h(
+      RouterLink,
+      {
+        class: ["ds-link-01-bold", "underline"],
+        target: "_blank",
+        tabindex: -1,
+        to: {
+          name: "caselaw-documentUnit-documentNumber-categories",
+          params: { documentNumber: dataEntry.documentNumber },
+        },
+      },
+      () => dataEntry.renderDecision,
+    )
+  } else if (
     !dataEntry.hasMissingRequiredFields ||
     (dataEntry.missingRequiredFields.length === 1 &&
       dataEntry.missingRequiredFields[0] === "decisionDate" &&
diff --git a/frontend/src/domain/ensuingDecision.ts b/frontend/src/domain/ensuingDecision.ts
index 14ce5f2a35..ecfc2d10b5 100644
--- a/frontend/src/domain/ensuingDecision.ts
+++ b/frontend/src/domain/ensuingDecision.ts
@@ -52,7 +52,7 @@ export default class EnsuingDecision
   }
 
   get isReadOnly(): boolean {
-    return false
+    return this.hasForeignSource
   }
 
   get isEmpty(): boolean {
diff --git a/frontend/test/e2e/caselaw/categories/contentRelatedIndexing/active-citations.spec.ts b/frontend/test/e2e/caselaw/categories/contentRelatedIndexing/active-citations.spec.ts
index 27fad2a099..b8a4568572 100644
--- a/frontend/test/e2e/caselaw/categories/contentRelatedIndexing/active-citations.spec.ts
+++ b/frontend/test/e2e/caselaw/categories/contentRelatedIndexing/active-citations.spec.ts
@@ -234,7 +234,7 @@ test.describe("active citations", () => {
       court: prefilledDocumentUnit.coreData.court?.label,
       fileNumber: prefilledDocumentUnit.coreData.fileNumbers?.[0],
       documentType: prefilledDocumentUnit.coreData.documentType?.jurisShortcut,
-      decisionDate: "01.01.2020",
+      decisionDate: "31.12.2019",
     })
 
     const activeCitationContainer = page.getByLabel("Aktivzitierung")
@@ -245,7 +245,7 @@ test.describe("active citations", () => {
 
     //citation style ignored in search results
     const result = page.getByText(
-      `AG Aachen, 01.01.2020, ${prefilledDocumentUnit.coreData.fileNumbers?.[0]}, Anerkenntnisurteil, ${prefilledDocumentUnit.documentNumber}`,
+      `AG Aachen, 31.12.2019, ${prefilledDocumentUnit.coreData.fileNumbers?.[0]}, Anerkenntnisurteil, ${prefilledDocumentUnit.documentNumber}`,
     )
 
     await expect(result).toBeVisible()
@@ -253,7 +253,7 @@ test.describe("active citations", () => {
 
     //make sure to have citation style in list
     const listItem = page.getByText(
-      `Änderung, AG Aachen, 01.01.2020, ${prefilledDocumentUnit.coreData.fileNumbers?.[0]}, Anerkenntnisurteil, ${prefilledDocumentUnit.documentNumber}`,
+      `Änderung, AG Aachen, 31.12.2019, ${prefilledDocumentUnit.coreData.fileNumbers?.[0]}, Anerkenntnisurteil, ${prefilledDocumentUnit.documentNumber}`,
     )
     await expect(listItem).toBeVisible()
     await expect(page.getByLabel("Eintrag löschen")).toBeVisible()
@@ -268,7 +268,7 @@ test.describe("active citations", () => {
       court: prefilledDocumentUnit.coreData.court?.label,
       fileNumber: prefilledDocumentUnit.coreData.fileNumbers?.[0],
       documentType: prefilledDocumentUnit.coreData.documentType?.jurisShortcut,
-      decisionDate: "01.01.2020",
+      decisionDate: "31.12.2019",
     })
 
     await activeCitationContainer.getByLabel("Nach Entscheidung suchen").click()
@@ -312,7 +312,7 @@ test.describe("active citations", () => {
       court: prefilledDocumentUnit.coreData.court?.label,
       fileNumber: prefilledDocumentUnit.coreData.fileNumbers?.[0],
       documentType: prefilledDocumentUnit.coreData.documentType?.jurisShortcut,
-      decisionDate: "01.01.2020",
+      decisionDate: "31.12.2019",
     })
     await page.getByLabel("Aktivzitierung speichern").click()
 
diff --git a/frontend/test/e2e/caselaw/categories/ensuing-decisions.spec.ts b/frontend/test/e2e/caselaw/categories/ensuing-decisions.spec.ts
index d0387e400f..db8f715a60 100644
--- a/frontend/test/e2e/caselaw/categories/ensuing-decisions.spec.ts
+++ b/frontend/test/e2e/caselaw/categories/ensuing-decisions.spec.ts
@@ -318,7 +318,7 @@ test.describe("ensuing decisions", () => {
       court: prefilledDocumentUnit.coreData.court?.label,
       fileNumber: prefilledDocumentUnit.coreData.fileNumbers?.[0],
       documentType: prefilledDocumentUnit.coreData.documentType?.jurisShortcut,
-      decisionDate: "01.01.2020",
+      decisionDate: "31.12.2019",
     })
     const ensuingDecisionContainer = page.getByLabel("Nachgehende Entscheidung")
     await ensuingDecisionContainer
@@ -328,7 +328,7 @@ test.describe("ensuing decisions", () => {
     await expect(page.getByText("1 Ergebnis gefunden.")).toBeVisible()
 
     const result = page.getByText(
-      `AG Aachen, 01.01.2020, ${prefilledDocumentUnit.coreData.fileNumbers?.[0]}, AnU, ${prefilledDocumentUnit.documentNumber}`,
+      `nachgehend, AG Aachen, 31.12.2019, ${prefilledDocumentUnit.coreData.fileNumbers?.[0]}, AnU, ${prefilledDocumentUnit.documentNumber}`,
     )
 
     await expect(result).toBeVisible()
@@ -336,7 +336,7 @@ test.describe("ensuing decisions", () => {
 
     //make sure to have citation style in list
     const listItem = page.getByText(
-      `AG Aachen, 01.01.2020, ${prefilledDocumentUnit.coreData.fileNumbers?.[0]}, AnU, ${prefilledDocumentUnit.documentNumber}`,
+      `nachgehend, AG Aachen, 31.12.2019, ${prefilledDocumentUnit.coreData.fileNumbers?.[0]}, AnU, ${prefilledDocumentUnit.documentNumber}`,
     )
     await expect(listItem).toBeVisible()
     await expect(page.getByLabel("Eintrag löschen")).toBeVisible()
@@ -350,7 +350,7 @@ test.describe("ensuing decisions", () => {
       court: prefilledDocumentUnit.coreData.court?.label,
       fileNumber: prefilledDocumentUnit.coreData.fileNumbers?.[0],
       documentType: prefilledDocumentUnit.coreData.documentType?.jurisShortcut,
-      decisionDate: "01.01.2020",
+      decisionDate: "31.12.2019",
     })
 
     await ensuingDecisionContainer
diff --git a/frontend/test/e2e/caselaw/categories/previous-decisions.spec.ts b/frontend/test/e2e/caselaw/categories/previous-decisions.spec.ts
index 465083e801..87aa185b7c 100644
--- a/frontend/test/e2e/caselaw/categories/previous-decisions.spec.ts
+++ b/frontend/test/e2e/caselaw/categories/previous-decisions.spec.ts
@@ -253,7 +253,7 @@ test.describe("previous decisions", () => {
       court: prefilledDocumentUnit.coreData.court?.label,
       fileNumber: prefilledDocumentUnit.coreData.fileNumbers?.[0],
       documentType: prefilledDocumentUnit.coreData.documentType?.jurisShortcut,
-      decisionDate: "01.01.2020",
+      decisionDate: "31.12.2019",
     })
     const previousDecisionContainer = page.getByLabel("Vorgehende Entscheidung")
     await previousDecisionContainer
@@ -263,7 +263,7 @@ test.describe("previous decisions", () => {
     await expect(page.getByText("1 Ergebnis gefunden.")).toBeVisible()
 
     const result = page.getByText(
-      `AG Aachen, 01.01.2020, ${prefilledDocumentUnit.coreData.fileNumbers?.[0]}, AnU, ${prefilledDocumentUnit.documentNumber}`,
+      `AG Aachen, 31.12.2019, ${prefilledDocumentUnit.coreData.fileNumbers?.[0]}, AnU, ${prefilledDocumentUnit.documentNumber}`,
     )
 
     await expect(result).toBeVisible()
@@ -271,7 +271,7 @@ test.describe("previous decisions", () => {
 
     //make sure to have citation style in list
     const listItem = page.getByText(
-      `AG Aachen, 01.01.2020, ${prefilledDocumentUnit.coreData.fileNumbers?.[0]}, AnU, ${prefilledDocumentUnit.documentNumber}`,
+      `AG Aachen, 31.12.2019, ${prefilledDocumentUnit.coreData.fileNumbers?.[0]}, AnU, ${prefilledDocumentUnit.documentNumber}`,
     )
     await expect(listItem).toBeVisible()
     await expect(page.getByLabel("Eintrag löschen")).toBeVisible()
@@ -285,7 +285,7 @@ test.describe("previous decisions", () => {
       court: prefilledDocumentUnit.coreData.court?.label,
       fileNumber: prefilledDocumentUnit.coreData.fileNumbers?.[0],
       documentType: prefilledDocumentUnit.coreData.documentType?.jurisShortcut,
-      decisionDate: "01.01.2020",
+      decisionDate: "31.12.2019",
     })
 
     await previousDecisionContainer