Skip to content

Commit 9b982e9

Browse files
committed
Ikke sende med lenke på inaktive oppgaver
1 parent 3a4c135 commit 9b982e9

File tree

3 files changed

+30
-2
lines changed

3 files changed

+30
-2
lines changed

src/main/kotlin/no/nav/tms/varsel/api/varsel/alleVarsler.kt

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -18,14 +18,15 @@ data class Varsel(
1818
companion object {
1919
fun fromVarsel(varsel: VarselAuthority.Varsel): Varsel {
2020
val isMasked = varsel.innhold == null
21+
val isInaktivertOppgave = varsel.type == VarselType.oppgave && !varsel.aktiv
2122

2223
return Varsel(
2324
eventId = varsel.varselId,
2425
forstBehandlet = varsel.opprettet,
2526
isMasked = isMasked,
2627
spraakkode = varsel.innhold?.spraakkode,
2728
tekst = varsel.innhold?.tekst,
28-
link = varsel.innhold?.link,
29+
link = if(isInaktivertOppgave) null else varsel.innhold?.link,
2930
eksternVarslingSendt = varsel.eksternVarslingSendt,
3031
eksternVarslingKanaler = varsel.eksternVarslingKanaler,
3132
type = if (varsel.type == VarselType.oppgave) VarselType.oppgave else VarselType.beskjed,

src/test/kotlin/no/nav/tms/varsel/api/varsel/AlleVarslerTest.kt

Lines changed: 27 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -70,4 +70,31 @@ class AlleVarslerTest {
7070
}
7171
}
7272

73+
@Test
74+
fun `Ikke sende med lenke på inaktive oppgaver`() {
75+
val incomingVarselList = listOf(
76+
AlleVarslerTestData.incomingVarsel(type = VarselType.beskjed),
77+
AlleVarslerTestData.incomingVarsel(type = VarselType.innboks),
78+
AlleVarslerTestData.incomingVarsel(type = VarselType.oppgave),
79+
AlleVarslerTestData.incomingVarsel(type = VarselType.oppgave, aktiv = false),
80+
AlleVarslerTestData.incomingVarsel(type = VarselType.beskjed, aktiv = false),
81+
AlleVarslerTestData.incomingVarsel(type = VarselType.innboks, aktiv = false)
82+
)
83+
84+
AlleVarsler.fromVarsler(incomingVarselList).apply {
85+
aktive.oppgaver[0].link shouldBe "www.nav.no/test"
86+
aktive.beskjeder.forEach() {
87+
it.link shouldBe "www.nav.no/test"
88+
}
89+
inaktive.forEach() {
90+
if(it.type == VarselType.oppgave) {
91+
it.link shouldBe null
92+
} else {
93+
it.link shouldBe "www.nav.no/test"
94+
}
95+
}
96+
97+
}
98+
}
99+
73100
}

src/test/kotlin/no/nav/tms/varsel/api/varsel/alleVarslerTestData.kt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@ object AlleVarslerTestData {
88
varselId: String = "123143",
99
aktiv: Boolean = true,
1010
innhold: VarselAuthority.Innhold? = VarselAuthority.Innhold(
11-
link = "test . no",
11+
link = "www.nav.no/test",
1212
spraakkode = "nb",
1313
tekst = "Varsel test tekst"
1414
),

0 commit comments

Comments
 (0)