From 9b982e9cab2bf84633eaac9869f9f7c49a082630 Mon Sep 17 00:00:00 2001
From: Amirhossien Lotfi <Amirhossien.lotfi@nav.no>
Date: Fri, 8 Nov 2024 20:05:47 +0100
Subject: [PATCH] =?UTF-8?q?Ikke=20sende=20med=20lenke=20p=C3=A5=20inaktive?=
 =?UTF-8?q?=20oppgaver?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 .../nav/tms/varsel/api/varsel/alleVarsler.kt  |  3 ++-
 .../tms/varsel/api/varsel/AlleVarslerTest.kt  | 27 +++++++++++++++++++
 .../varsel/api/varsel/alleVarslerTestData.kt  |  2 +-
 3 files changed, 30 insertions(+), 2 deletions(-)

diff --git a/src/main/kotlin/no/nav/tms/varsel/api/varsel/alleVarsler.kt b/src/main/kotlin/no/nav/tms/varsel/api/varsel/alleVarsler.kt
index facdb88..73b5a50 100644
--- a/src/main/kotlin/no/nav/tms/varsel/api/varsel/alleVarsler.kt
+++ b/src/main/kotlin/no/nav/tms/varsel/api/varsel/alleVarsler.kt
@@ -18,6 +18,7 @@ data class Varsel(
     companion object {
         fun fromVarsel(varsel: VarselAuthority.Varsel): Varsel {
             val isMasked = varsel.innhold == null
+            val isInaktivertOppgave = varsel.type == VarselType.oppgave && !varsel.aktiv
 
             return Varsel(
                 eventId = varsel.varselId,
@@ -25,7 +26,7 @@ data class Varsel(
                 isMasked = isMasked,
                 spraakkode = varsel.innhold?.spraakkode,
                 tekst = varsel.innhold?.tekst,
-                link = varsel.innhold?.link,
+                link = if(isInaktivertOppgave) null else varsel.innhold?.link,
                 eksternVarslingSendt = varsel.eksternVarslingSendt,
                 eksternVarslingKanaler = varsel.eksternVarslingKanaler,
                 type = if (varsel.type == VarselType.oppgave) VarselType.oppgave else VarselType.beskjed,
diff --git a/src/test/kotlin/no/nav/tms/varsel/api/varsel/AlleVarslerTest.kt b/src/test/kotlin/no/nav/tms/varsel/api/varsel/AlleVarslerTest.kt
index 6048cd6..bf2635f 100644
--- a/src/test/kotlin/no/nav/tms/varsel/api/varsel/AlleVarslerTest.kt
+++ b/src/test/kotlin/no/nav/tms/varsel/api/varsel/AlleVarslerTest.kt
@@ -70,4 +70,31 @@ class AlleVarslerTest {
         }
     }
 
+    @Test
+    fun `Ikke sende med lenke på inaktive oppgaver`() {
+        val incomingVarselList = listOf(
+            AlleVarslerTestData.incomingVarsel(type = VarselType.beskjed),
+            AlleVarslerTestData.incomingVarsel(type = VarselType.innboks),
+            AlleVarslerTestData.incomingVarsel(type = VarselType.oppgave),
+            AlleVarslerTestData.incomingVarsel(type = VarselType.oppgave, aktiv = false),
+            AlleVarslerTestData.incomingVarsel(type = VarselType.beskjed, aktiv = false),
+            AlleVarslerTestData.incomingVarsel(type = VarselType.innboks, aktiv = false)
+        )
+
+        AlleVarsler.fromVarsler(incomingVarselList).apply {
+            aktive.oppgaver[0].link shouldBe "www.nav.no/test"
+            aktive.beskjeder.forEach() {
+                it.link shouldBe "www.nav.no/test"
+            }
+            inaktive.forEach() {
+                if(it.type == VarselType.oppgave) {
+                    it.link shouldBe null
+                } else {
+                    it.link shouldBe "www.nav.no/test"
+                }
+            }
+
+        }
+    }
+
 }
\ No newline at end of file
diff --git a/src/test/kotlin/no/nav/tms/varsel/api/varsel/alleVarslerTestData.kt b/src/test/kotlin/no/nav/tms/varsel/api/varsel/alleVarslerTestData.kt
index b9a9be5..da9e0df 100644
--- a/src/test/kotlin/no/nav/tms/varsel/api/varsel/alleVarslerTestData.kt
+++ b/src/test/kotlin/no/nav/tms/varsel/api/varsel/alleVarslerTestData.kt
@@ -8,7 +8,7 @@ object AlleVarslerTestData {
         varselId: String = "123143",
         aktiv: Boolean = true,
         innhold: VarselAuthority.Innhold? = VarselAuthority.Innhold(
-            link = "test . no",
+            link = "www.nav.no/test",
             spraakkode = "nb",
             tekst = "Varsel test tekst"
         ),