Forenkler sjekk for om IM skal oppdatere vilkårsgrunnlag

Co-authored-by: Erik Maximilian Forsman <[email protected]>
Co-authored-by: Hege Haavaldsen <[email protected]>

Author: 3 people

sykepenger-model/src/main/kotlin/no/nav/helse/hendelser/Inntektsmelding.kt

@@ -191,6 +191,14 @@ class Inntektsmelding(
     }
     private fun håndtertNå() = håndtertInntekt
     internal fun jurist(jurist: MaskinellJurist) = jurist.medInntektsmelding(this.meldingsreferanseId())
-    internal fun skalIkkeOppdatereVilkårsgrunnlag(sykdomstidslinjeperiode: Periode?) =
-        sykdomstidslinjeperiode != null && beregnetInntektsdato !in sykdomstidslinjeperiode && inntektsdato == null
+
+    internal fun skalOppdatereVilkårsgrunnlag(sykdomstidslinjeperiode: Periode?): Boolean {
+        if (forventerInntektsmelding()) return true
+        if (sykdomstidslinjeperiode == null) return false // har ikke noe sykdom for arbeidsgiveren
+        return beregnetInntektsdato in sykdomstidslinjeperiode
+    }
+
+    private fun forventerInntektsmelding(): Boolean {
+        return inntektsdato != null // inntektmelding fra portal, vi har bedt om IM og forventer IM
+    }
 }

sykepenger-model/src/main/kotlin/no/nav/helse/person/Arbeidsgiver.kt

@@ -736,8 +736,8 @@ internal class Arbeidsgiver private constructor(
         inntektsmelding.leggTilRefusjon(refusjonshistorikk)
         val sykdomstidslinjeperiode = sykdomstidslinje().periode()
         val skjæringstidspunkt = person.skjæringstidspunkt(inntektsdato.somPeriode())
-        if (inntektsmelding.skalIkkeOppdatereVilkårsgrunnlag(sykdomstidslinjeperiode)) {
-            return inntektsmelding.info("Lagrer ikke inntekt på skjæringstidspunkt fordi inntektdato er oppgitt til å være utenfor den perioden arbeidsgiver har sykdom for")
+        if (!inntektsmelding.skalOppdatereVilkårsgrunnlag(sykdomstidslinjeperiode)) {
+            return inntektsmelding.info("Inntektsmelding oppdaterer ikke vilkårsgrunnlag")
         }
         finnAlternativInntektsdato(inntektsdato, skjæringstidspunkt)?.let {
             inntektsmelding.addInntekt(inntektshistorikk, it)

sykepenger-model/src/test/kotlin/no/nav/helse/spleis/e2e/flere_arbeidsgivere/FlereArbeidsgivereGhostTest.kt

@@ -83,6 +83,7 @@ import no.nav.helse.økonomi.Inntekt.Companion.månedlig
 import no.nav.helse.økonomi.Inntekt.Companion.årlig
 import no.nav.helse.økonomi.Prosentdel.Companion.prosent
 import org.junit.jupiter.api.Assertions.assertEquals
+import org.junit.jupiter.api.Assertions.assertFalse
 import org.junit.jupiter.api.Assertions.assertTrue
 import org.junit.jupiter.api.Test
 import org.junit.jupiter.api.assertThrows
@@ -143,7 +144,7 @@ internal class FlereArbeidsgivereGhostTest : AbstractEndToEndTest() {
         håndterUtbetalingsgodkjenning(2.vedtaksperiode, orgnummer = a1)
         håndterUtbetalt(orgnummer = a1)
 
-        assertInfoSomFølgeAv("Lagrer ikke inntekt på skjæringstidspunkt fordi inntektdato er oppgitt til å være utenfor den perioden arbeidsgiver har sykdom for") {
+        assertInfoSomFølgeAv("Inntektsmelding oppdaterer ikke vilkårsgrunnlag") {
             håndterInntektsmelding(listOf(mandag den 29.januar til 13.februar), orgnummer = a2)
         }
         // IM replayes, og ettersom 27. og 28 blir friskedager pga. IM beregnes skjæringstidspunktet til 29.januar. Når A1 sin søknad kommer dekker den "hullet" med sykdom slik at skjæringstidspunktet blir 1.januar
@@ -190,47 +191,38 @@ internal class FlereArbeidsgivereGhostTest : AbstractEndToEndTest() {
             orgnummer = ghost,
             begrunnelseForReduksjonEllerIkkeUtbetalt = "IkkeFravaer",
         )
-        håndterYtelser(2.vedtaksperiode, orgnummer = a1)
-        håndterSimulering(2.vedtaksperiode, orgnummer = a1)
         håndterUtbetalingsgodkjenning(2.vedtaksperiode, orgnummer = a1)
         håndterUtbetalt(orgnummer = a1)
 
         val ghostInntektEtterIm = inspektør.vilkårsgrunnlag(1.januar)!!.inspektør.sykepengegrunnlag.inspektør.arbeidsgiverInntektsopplysninger.single { it.gjelder(ghost) }.inspektør.inntektsopplysning
-        assertTrue(ghostInntektEtterIm is InntektsmeldingInntekt)
+        assertFalse(ghostInntektEtterIm is InntektsmeldingInntekt)
+        assertTrue(ghostInntektEtterIm is SkattSykepengegrunnlag)
 
         // Så kjem søknaden på ghosten læll
         val ghostSøknad = håndterSøknad(Sykdom(1.januar, 31.januar, 100.prosent), orgnummer = ghost)
         assertSisteTilstand(1.vedtaksperiode, AVSLUTTET_UTEN_UTBETALING, orgnummer = a1)
         assertSisteTilstand(2.vedtaksperiode, AVVENTER_REVURDERING, orgnummer = a1)
         assertSisteTilstand(1.vedtaksperiode, AVVENTER_HISTORIKK, orgnummer = ghost)
+        val ghostInntektEtterImNå = inspektør.vilkårsgrunnlag(1.januar)!!.inspektør.sykepengegrunnlag.inspektør.arbeidsgiverInntektsopplysninger.single { it.gjelder(ghost) }.inspektør.inntektsopplysning
+        assertTrue(ghostInntektEtterImNå is InntektsmeldingInntekt)
+        assertFalse(ghostInntektEtterImNå is SkattSykepengegrunnlag)
 
         håndterYtelser(1.vedtaksperiode, orgnummer = ghost)
         håndterSimulering(1.vedtaksperiode, orgnummer = ghost)
         håndterUtbetalingsgodkjenning(1.vedtaksperiode, orgnummer = ghost)
         håndterUtbetalt(orgnummer = ghost)
 
         håndterYtelser(2.vedtaksperiode, orgnummer = a1)
+        håndterSimulering(2.vedtaksperiode, orgnummer = a1)
         håndterUtbetalingsgodkjenning(2.vedtaksperiode, orgnummer = a1)
+        håndterUtbetalt(orgnummer = a1)
 
         assertSisteTilstand(1.vedtaksperiode, AVSLUTTET_UTEN_UTBETALING, a1)
         assertSisteTilstand(2.vedtaksperiode, AVSLUTTET, a1)
         assertSisteTilstand(1.vedtaksperiode, AVSLUTTET, ghost)
 
-
-        assertForventetFeil(
-            forklaring = """
-                Perioden på A1 som går i AvventerRevurdering kaller på gjennoppta behandling før periode på Ghost i AvventerIM får replayet
-                IM. Da ender vi opp med å gå videre uten å ha referanser til inntektsmelding som er brukt, og får ikke validert inneholdet i IM
-            """,
-            nå = {
-                assertIngenVarsler(1.vedtaksperiode.filter(ghost))
-                assertEquals(listOf(Dokumentsporing.søknad(ghostSøknad)), inspektør(ghost).hendelser(1.vedtaksperiode))
-            },
-            ønsket = {
-                assertVarsel(RV_IM_8, 1.vedtaksperiode.filter(ghost))
-                assertEquals(listOf(Dokumentsporing.søknad(ghostSøknad), Dokumentsporing.inntektsmeldingDager(ghostIM), Dokumentsporing.inntektsmeldingInntekt(ghostIM)), inspektør(ghost).hendelser(1.vedtaksperiode))
-            }
-        )
+        assertVarsel(RV_IM_8, 1.vedtaksperiode.filter(ghost))
+        assertEquals(listOf(Dokumentsporing.søknad(ghostSøknad), Dokumentsporing.inntektsmeldingDager(ghostIM), Dokumentsporing.inntektsmeldingInntekt(ghostIM)), inspektør(ghost).hendelser(1.vedtaksperiode))
     }
 
     @Test

sykepenger-model/src/test/kotlin/no/nav/helse/spleis/e2e/flere_arbeidsgivere/FlereUkjenteArbeidsgivereTest.kt

@@ -92,7 +92,7 @@ internal class FlereUkjenteArbeidsgivereTest : AbstractEndToEndTest() {
 
 
         assertFunksjonellFeil(RV_SV_2, 1.vedtaksperiode.filter(a2))
-        assertSisteTilstand(1.vedtaksperiode, AVVENTER_HISTORIKK_REVURDERING, orgnummer = a1)
+        assertSisteTilstand(1.vedtaksperiode, AVSLUTTET, orgnummer = a1)
         assertTrue(inspektør(a2).periodeErForkastet(1.vedtaksperiode))
     }
 
@@ -119,7 +119,7 @@ internal class FlereUkjenteArbeidsgivereTest : AbstractEndToEndTest() {
         }
 
         val overstyringerIgangsatt = observatør.overstyringIgangsatt
-        assertEquals(3, overstyringerIgangsatt.size)
+        assertEquals(2, overstyringerIgangsatt.size)
 
         overstyringerIgangsatt.first().also { event ->
             assertEquals(PersonObserver.OverstyringIgangsatt(
@@ -134,19 +134,6 @@ internal class FlereUkjenteArbeidsgivereTest : AbstractEndToEndTest() {
             ), event)
         }
 
-        overstyringerIgangsatt[1].also { event ->
-            assertEquals(PersonObserver.OverstyringIgangsatt(
-                årsak = "KORRIGERT_INNTEKTSMELDING_INNTEKTSOPPLYSNINGER",
-                skjæringstidspunkt = 1.januar,
-                periodeForEndring = 1.januar til 1.januar,
-                berørtePerioder = listOf(
-                    VedtaksperiodeData(a1, 1.vedtaksperiode.id(a1), 1.januar til 31.januar, 1.januar, "REVURDERING"),
-                    VedtaksperiodeData(a1, 2.vedtaksperiode.id(a1), 1.februar til 28.februar, 1.januar, "REVURDERING"),
-                    VedtaksperiodeData(a1, 3.vedtaksperiode.id(a1), 1.mars til 31.mars, 1.januar, "REVURDERING")
-                )
-            ), event)
-        }
-
         overstyringerIgangsatt.last().also { event ->
             assertEquals(PersonObserver.OverstyringIgangsatt(
                 årsak = "KORRIGERT_INNTEKTSMELDING_INNTEKTSOPPLYSNINGER",
@@ -160,7 +147,7 @@ internal class FlereUkjenteArbeidsgivereTest : AbstractEndToEndTest() {
             ), event)
         }
 
-        assertTilstander(1.vedtaksperiode, AVVENTER_HISTORIKK_REVURDERING, orgnummer = a1)
+        assertTilstander(1.vedtaksperiode, AVVENTER_GODKJENNING_REVURDERING, orgnummer = a1)
         assertTilstander(2.vedtaksperiode, AVVENTER_REVURDERING, orgnummer = a1)
         assertTilstander(3.vedtaksperiode, AVVENTER_REVURDERING, orgnummer = a1)
         assertForkastetPeriodeTilstander(1.vedtaksperiode, START, TIL_INFOTRYGD, orgnummer = a2)
@@ -310,8 +297,6 @@ internal class FlereUkjenteArbeidsgivereTest : AbstractEndToEndTest() {
             AVVENTER_GODKJENNING,
             AVVENTER_BLOKKERENDE_PERIODE,
             AVVENTER_HISTORIKK,
-            AVVENTER_BLOKKERENDE_PERIODE,
-            AVVENTER_HISTORIKK,
             AVVENTER_SIMULERING,
             AVVENTER_GODKJENNING, orgnummer = a1
         )
@@ -363,8 +348,6 @@ internal class FlereUkjenteArbeidsgivereTest : AbstractEndToEndTest() {
             AVSLUTTET,
             AVVENTER_REVURDERING,
             AVVENTER_HISTORIKK_REVURDERING,
-            AVVENTER_REVURDERING,
-            AVVENTER_HISTORIKK_REVURDERING,
             AVVENTER_SIMULERING_REVURDERING,
             AVVENTER_GODKJENNING_REVURDERING,
             TIL_UTBETALING,

sykepenger-model/src/test/kotlin/no/nav/helse/spleis/e2e/inntektsmelding/InntektsmeldingE2ETest.kt

@@ -336,6 +336,16 @@ internal class InntektsmeldingE2ETest : AbstractEndToEndTest() {
         assertSisteTilstand(1.vedtaksperiode, AVVENTER_VILKÅRSPRØVING)
     }
 
+    @Test
+    fun `inntektsmelding i det blå`() {
+        nyttVedtak(1.januar, 31.januar)
+        forlengVedtak(1.februar, 28.februar)
+
+        håndterInntektsmelding(listOf(1.oktober til 16.oktober), orgnummer = a2)
+        assertSisteTilstand(1.vedtaksperiode, AVSLUTTET)
+        assertSisteTilstand(2.vedtaksperiode, AVSLUTTET)
+    }
+
     @Test
     fun `Skal ikke bruke inntekt fra gammel inntektsmelding`() {
         håndterInntektsmelding(arbeidsgiverperioder = listOf(1.januar til 16.januar))