Merge pull request #42 from navikt/fjerne-link-paa-inaktive-oppgaver

Ikke sende med lenke på inaktive oppgaver
navikt · Nov 8, 2024 · 92fec65 · 92fec65
2 parents 3a4c135 + 9b982e9
commit 92fec65
Show file tree

Hide file tree

Showing 3 changed files with 30 additions and 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
@@ -18,14 +18,15 @@ 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,
                 forstBehandlet = varsel.opprettet,
                 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
@@ -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"
+                }
+            }
+
+        }
+    }
+
 }
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
@@ -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"
         ),