Eksplisitt assert på tester som får flere skjæringstidspunkt

Author: fraadsbrandth

sykepenger-model/src/test/kotlin/no/nav/helse/dsl/UgyldigeSituasjonerObservatør.kt

@@ -15,7 +15,6 @@ import no.nav.helse.person.TilstandType
 import no.nav.helse.person.TilstandType.AVVENTER_BLOKKERENDE_PERIODE
 import no.nav.helse.person.TilstandType.AVVENTER_INFOTRYGDHISTORIKK
 import no.nav.helse.person.TilstandType.AVVENTER_INNTEKTSMELDING
-import no.nav.helse.person.TilstandType.REVURDERING_FEILET
 import no.nav.helse.person.VedtaksperiodeView
 import no.nav.helse.person.aktivitetslogg.Aktivitet
 import no.nav.helse.person.aktivitetslogg.Aktivitetslogg
@@ -24,7 +23,6 @@ import no.nav.helse.person.aktivitetslogg.Varselkode.RV_SV_1
 import no.nav.helse.person.arbeidsgiver
 import no.nav.helse.sykdomstidslinje.Dag
 import no.nav.helse.sykdomstidslinje.Dag.UkjentDag
-import org.junit.jupiter.api.Assertions.assertEquals
 import org.junit.jupiter.api.Assertions.assertTrue
 import org.junit.jupiter.api.assertThrows
 import kotlin.check
@@ -145,10 +143,7 @@ internal class UgyldigeSituasjonerObservatør(private val person: Person): Perso
     }
 
     private fun sjekkUgyldigeVentesituasjoner(event: PersonObserver.VedtaksperiodeVenterEvent) {
-        // En linje å kommentere inn om man kjeder seg 🫠
-        //if (event.trengerNyInntektsmeldingEtterFlyttetSkjæringstidspunkt()) error("vedtaksperiode på ${event.organisasjonsnummer} venter på ${event.venterPå}")
         if (event.venterPå.venteårsak.hva != "HJELP") return // Om vi venter på noe annet enn hjelp er det OK 👍
-        if (event.venterPå.venteårsak.hvorfor == "FLERE_SKJÆRINGSTIDSPUNKT") return // Dette kan skje :(
         """
         Har du endret/opprettet en vedtaksperiodetilstand uten å vurdere konsekvensene av 'venteårsak'? 
         Eller har du klart å skriv en test vi ikke støtter? 

sykepenger-model/src/test/kotlin/no/nav/helse/spleis/e2e/ManglerVilkårsgrunnlagE2ETest.kt

@@ -1,6 +1,7 @@
 package no.nav.helse.spleis.e2e
 
 import no.nav.helse.desember
+import no.nav.helse.dsl.UgyldigeSituasjonerObservatør.Companion.assertUgyldigSituasjon
 import no.nav.helse.februar
 import no.nav.helse.hendelser.inntektsmelding.ALTINN
 import no.nav.helse.hendelser.Sykmeldingsperiode
@@ -112,7 +113,9 @@ internal class ManglerVilkårsgrunnlagE2ETest : AbstractEndToEndTest() {
         }
 
         observatør.vedtaksperiodeVenter.clear()
-        håndterSøknad(10.januar til 26.januar)
+        assertUgyldigSituasjon("En vedtaksperiode i AVVENTER_REVURDERING trenger hjelp fordi FLERE_SKJÆRINGSTIDSPUNKT!"){
+            håndterSøknad(10.januar til 26.januar)
+        }
         observatør.assertVenter(2.vedtaksperiode.id(a1), venterPåHva = HJELP, fordi = FLERE_SKJÆRINGSTIDSPUNKT)
 
         inspektør.sykdomstidslinje.inspektør.also { sykdomstidslinjeInspektør ->

sykepenger-model/src/test/kotlin/no/nav/helse/spleis/e2e/overstyring/OverstyrArbeidsgiverperiodeTest.kt

@@ -2,6 +2,7 @@ package no.nav.helse.spleis.e2e.overstyring
 
 import no.nav.helse.april
 import no.nav.helse.dsl.AbstractDslTest
+import no.nav.helse.dsl.UgyldigeSituasjonerObservatør.Companion.assertUgyldigSituasjon
 import no.nav.helse.februar
 import no.nav.helse.hendelser.Dagtype
 import no.nav.helse.hendelser.ManuellOverskrivingDag
@@ -20,6 +21,7 @@ import no.nav.helse.person.TilstandType.AVVENTER_VILKÅRSPRØVING
 import no.nav.helse.person.Venteårsak.Hva.HJELP
 import no.nav.helse.person.Venteårsak.Hvorfor.FLERE_SKJÆRINGSTIDSPUNKT
 import no.nav.helse.spleis.e2e.VedtaksperiodeVenterTest.Companion.assertVenter
+import no.nav.helse.spleis.e2e.håndterSøknad
 import no.nav.helse.økonomi.Prosentdel.Companion.prosent
 import org.junit.jupiter.api.Assertions.assertEquals
 import org.junit.jupiter.api.Test
@@ -44,10 +46,12 @@ internal class OverstyrArbeidsgiverperiodeTest : AbstractDslTest() {
             assertEquals(17.januar til 15.februar, inspektør.periode(1.vedtaksperiode))
             // drar agp tilbake to dager, men glemmer å omgjøre 1. februar til sykdom
             observatør.vedtaksperiodeVenter.clear()
-            håndterOverstyrTidslinje(listOf(
-                ManuellOverskrivingDag(15.januar, Dagtype.Sykedag, 100),
-                ManuellOverskrivingDag(16.januar, Dagtype.Sykedag, 100)
-            ))
+            assertUgyldigSituasjon("En vedtaksperiode i AVVENTER_REVURDERING trenger hjelp fordi FLERE_SKJÆRINGSTIDSPUNKT!"){
+                håndterOverstyrTidslinje(listOf(
+                    ManuellOverskrivingDag(15.januar, Dagtype.Sykedag, 100),
+                    ManuellOverskrivingDag(16.januar, Dagtype.Sykedag, 100)
+                ))
+            }
             observatør.assertVenter(1.vedtaksperiode, venterPåHva = HJELP, fordi = FLERE_SKJÆRINGSTIDSPUNKT)
             assertTilstander(1.vedtaksperiode, AVSLUTTET, AVVENTER_REVURDERING)
 

sykepenger-model/src/test/kotlin/no/nav/helse/spleis/e2e/overstyring/OverstyrTidslinjeTest.kt

@@ -2,6 +2,7 @@ package no.nav.helse.spleis.e2e.overstyring
 
 import java.time.LocalDate
 import no.nav.helse.august
+import no.nav.helse.dsl.UgyldigeSituasjonerObservatør.Companion.assertUgyldigSituasjon
 import no.nav.helse.erHelg
 import no.nav.helse.februar
 import no.nav.helse.hendelser.Dagtype
@@ -239,7 +240,9 @@ internal class OverstyrTidslinjeTest : AbstractEndToEndTest() {
         assertEquals(Dag.Arbeidsdag::class, inspektør.sykdomstidslinje[6.februar]::class)
         nullstillTilstandsendringer()
         observatør.vedtaksperiodeVenter.clear()
-        håndterInntektsmelding(listOf(16.januar til 31.januar), avsendersystem = ALTINN)
+        assertUgyldigSituasjon("En vedtaksperiode i AVVENTER_INNTEKTSMELDING trenger hjelp fordi FLERE_SKJÆRINGSTIDSPUNKT!"){
+            håndterInntektsmelding(listOf(16.januar til 31.januar), avsendersystem = ALTINN)
+        }
         observatør.assertVenter(1.vedtaksperiode.id(ORGNUMMER), venterPåHva = HJELP, fordi = FLERE_SKJÆRINGSTIDSPUNKT)
 
         assertTilstander(1.vedtaksperiode, AVSLUTTET_UTEN_UTBETALING, AVVENTER_INNTEKTSMELDING)