Alt med hvorfor != 'OVERSTYRING_IGANGSATT' inngår i alarm 🚨

navikt · Aug 5, 2024 · 0581233 · 0581233
1 parent 85ca132
commit 0581233
Show file tree

Hide file tree

Showing 3 changed files with 32 additions and 5 deletions.
diff --git a/src/main/kotlin/no/nav/helse/ventetilstand/VedtaksperiodeVentetilstandDao.kt b/src/main/kotlin/no/nav/helse/ventetilstand/VedtaksperiodeVentetilstandDao.kt
@@ -70,7 +70,7 @@ internal class VedtaksperiodeVentetilstandDao(private val dataSource: DataSource
         @Language("PostgreSQL")
         private val STUCK = """
             SELECT * FROM vedtaksperiode_venter
-            -- Må ha ventet minst 5 minutter før vi anser det som stuck. Disse er mest sannsynlig i transit.
+                -- Må ha ventet minst 5 minutter før vi anser det som stuck. Disse er mest sannsynlig i transit.
                 -- 'ventetSiden' er sist gang vedtaksperioden endret tilstand, så i tilstandene AVVENTER_BLOKKERENDE & AVENTER_REVURDERING blir det ofte feil å ta utgangspunkt i det tidspunktet.
                 -- F.eks. kan en periode vært i AVVENTER_BLOKKERENDE i 2 måneder og ventet på Godkjenning. Om perioden foran overstyres, og denne spørringen
                 -- kjøres mens vi venter på Utbetaling så vil perioden melde om at den har ventet 2 måneder på på utbetaling.
@@ -80,6 +80,9 @@ internal class VedtaksperiodeVentetilstandDao(private val dataSource: DataSource
             AND (
                 -- Om vi venter på en av disse tingene skal det alltid være alarm
                 (venterPaHva in ('BEREGNING', 'UTBETALING', 'HJELP'))
+                -- Om hvorfor er satt til noe skal det være alarm så fremt det ikke er på grunn av overstyring igangsatt
+                    OR
+                (venterPaHvorfor IS NOT NULL AND venterPaHvorfor != 'OVERSTYRING_IGANGSATT')
                     OR
                 -- Om vi venter for alltid skal alarmen gå når vi har ventet 3 måneder, så fremt det ikke venter på godkjenning fra saksbehandler eller inntektsmelding
                 (venterForAlltid = true AND ventetSiden < (now() AT TIME ZONE 'Europe/Oslo') - INTERVAL '3 MONTHS' AND venterPaHva not in ('GODKJENNING', 'INNTEKTSMELDING'))

diff --git a/src/test/kotlin/no/nav/helse/AbstractVedtaksperiodeVentetilstandTest.kt b/src/test/kotlin/no/nav/helse/AbstractVedtaksperiodeVentetilstandTest.kt
@@ -50,7 +50,7 @@ internal abstract class AbstractVedtaksperiodeVentetilstandTest(
         venterPå: String = "GODKJENNING",
         hendelseId: UUID = UUID.randomUUID(),
         fødselsnummer: String = "11111111111",
-        venterPåHvorfor: String = "TESTOLINI",
+        venterPåHvorfor: String? = "TESTOLINI",
         ventetSiden: LocalDateTime = LocalDateTime.parse("2023-03-04T21:34:17.96322")
     ) = """
         {
@@ -66,7 +66,7 @@ internal abstract class AbstractVedtaksperiodeVentetilstandTest(
             "organisasjonsnummer": "987654321",
             "venteårsak": {
               "hva": "$venterPå",
-              "hvorfor": "$venterPåHvorfor"
+              "hvorfor": ${venterPåHvorfor?.let { "\"$it\"" }}
             }
           },
           "@id": "$hendelseId",

diff --git a/src/test/kotlin/no/nav/helse/VedtaksperiodeVentetilstandTest.kt b/src/test/kotlin/no/nav/helse/VedtaksperiodeVentetilstandTest.kt
@@ -144,14 +144,38 @@ internal class VedtaksperiodeVentetilstandTest : AbstractVedtaksperiodeVentetils
     }
 
     @Test
-    fun `ingen alarm når vi er stuck pga inntektsmelding - vi får ikke gjort noe med dem uansett`() {
+    fun `ingen alarm når vi er stuck pga inntektsmelding UTEN hvorfor satt - vi får ikke gjort noe med dem uansett`() {
         assertEquals(emptyList<VedtaksperiodeVenter>(), stuck())
         val vedtaksperiodeId = UUID.randomUUID()
-        val vedtaksperiodeVenter = vedtaksperiodeVenter(vedtaksperiodeId, UUID.randomUUID(), "INNTEKTSMELDING", UUID.randomUUID())
+        val vedtaksperiodeVenter = vedtaksperiodeVenter(vedtaksperiodeId, UUID.randomUUID(), "INNTEKTSMELDING", UUID.randomUUID(), venterPåHvorfor = null)
         river.sendTestMessage(vedtaksperiodeVenter)
         assertEquals(emptyList<VedtaksperiodeVenter>(), stuck())
     }
 
+    @Test
+    fun `alarm når vi er stuck pga inntektsmelding MED hvorfor satt`() {
+        assertEquals(emptyList<VedtaksperiodeVenter>(), stuck())
+        val vedtaksperiodeId = UUID.randomUUID()
+        val vedtaksperiodeVenter = vedtaksperiodeVenter(vedtaksperiodeId, UUID.randomUUID(), "INNTEKTSMELDING", UUID.randomUUID(), venterPåHvorfor = "noe")
+        river.sendTestMessage(vedtaksperiodeVenter)
+        val stuck = stuck().single()
+        assertEquals(vedtaksperiodeId, stuck.vedtaksperiodeId)
+        assertEquals("INNTEKTSMELDING", stuck.venterPå.hva)
+        assertEquals("noe", stuck.venterPå.hvorfor)
+    }
+
+    @Test
+    fun `alarm når hvorfor er satt til noe som helst utenom overstyring igangsatt`() {
+        assertEquals(emptyList<VedtaksperiodeVenter>(), stuck())
+        val vedtaksperiodeId = UUID.randomUUID()
+        val vedtaksperiodeVenter = vedtaksperiodeVenter(vedtaksperiodeId, UUID.randomUUID(), "tja", UUID.randomUUID(), venterPåHvorfor = "tjo")
+        river.sendTestMessage(vedtaksperiodeVenter)
+        val stuck = stuck().single()
+        assertEquals(vedtaksperiodeId, stuck.vedtaksperiodeId)
+        assertEquals("tja", stuck.venterPå.hva)
+        assertEquals("tjo", stuck.venterPå.hvorfor)
+    }
+
     @Test
     fun `Ignorer meldinger med lik informasjon`() {
         val vedtaksperiodeId = UUID.randomUUID()