enklere forkasting

Author: davidsteinsland

sykepenger-api/src/test/kotlin/no/nav/helse/spleis/graphql/SpeilBehandlingerBuilderTest.kt

@@ -169,16 +169,12 @@ internal class SpeilBehandlingerBuilderTest : AbstractE2ETest() {
         håndterVilkårsgrunnlagTilGodkjenning()
         håndterUtbetalingsgodkjenning(utbetalingGodkjent = false)
         generasjoner {
-            assertEquals(3, size)
+            assertEquals(2, size)
             0.generasjon {
-                assertEquals(1, size)
-                uberegnetPeriode(0) fra 1.januar til 10.januar medTilstand Annullert
-            }
-            1.generasjon {
                 assertEquals(1, size)
                 uberegnetPeriode(0) fra 1.januar til 10.januar medTilstand IngenUtbetaling medHendelser setOf(søknad, im)
             }
-            2.generasjon {
+            1.generasjon {
                 assertEquals(1, size)
                 uberegnetPeriode(0) fra 1.januar til 10.januar medTilstand IngenUtbetaling medHendelser setOf(søknad)
             }
@@ -1733,18 +1729,14 @@ internal class SpeilBehandlingerBuilderTest : AbstractE2ETest() {
         håndterUtbetalingsgodkjenning(utbetalingGodkjent = false)
 
         generasjoner {
-            assertEquals(3, size)
+            assertEquals(2, size)
 
             0.generasjon {
                 assertEquals(2, size)
                 uberegnetPeriode(0) fra 21.januar til 27.januar medTilstand Annullert
-                uberegnetPeriode(1) fra 10.januar til 20.januar medTilstand Annullert
+                uberegnetPeriode(1) fra 10.januar til 20.januar medTilstand IngenUtbetaling
             }
             1.generasjon {
-                assertEquals(1, size)
-                uberegnetPeriode(0) fra 10.januar til 20.januar medTilstand IngenUtbetaling
-            }
-            2.generasjon {
                 assertEquals(2, size)
                 uberegnetPeriode(0) fra 21.januar til 27.januar medTilstand IngenUtbetaling
                 uberegnetPeriode(1) fra 12.januar til 20.januar medTilstand IngenUtbetaling

sykepenger-mediators/src/test/kotlin/no/nav/helse/spleis/mediator/e2e/AktørEndringE2ETest.kt

@@ -53,12 +53,11 @@ internal class AktørEndringE2ETest : AbstractEndToEndMediatorTest() {
         assertEquals(1, antallPersonalias(FNR1))
         assertEquals(1, antallPersonalias(FNR2))
         val meldinger = testRapid.inspektør.meldinger("vedtaksperiode_endret")
-        assertEquals(5, meldinger.size)
+        assertEquals(4, meldinger.size)
         assertEquals(FNR1, meldinger[0].path("fødselsnummer").asText())
         assertEquals(FNR1, meldinger[1].path("fødselsnummer").asText())
         assertEquals(FNR2, meldinger[2].path("fødselsnummer").asText())
         assertEquals(FNR2, meldinger[3].path("fødselsnummer").asText())
-        assertEquals(FNR2, meldinger[4].path("fødselsnummer").asText())
 
         testRapid.inspektør.siste("vedtaksperiode_forkastet").also { melding ->
             assertEquals(FNR2, melding.path("fødselsnummer").asText())

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

@@ -1281,7 +1281,7 @@ internal class Vedtaksperiode private constructor(
     }
 
     private fun forkast(hendelse: Hendelse, aktivitetslogg: IAktivitetslogg) {
-        person.søppelbøtte(hendelse, aktivitetslogg, TIDLIGERE_OG_ETTERGØLGENDE(this))
+        person.søppelbøtte(hendelse, aktivitetslogg, OVERLAPPENDE_OG_ETTERGØLGENDE(this))
     }
 
     private fun registrerKontekst(aktivitetslogg: IAktivitetslogg): IAktivitetslogg {
@@ -1601,7 +1601,7 @@ internal class Vedtaksperiode private constructor(
     }
 
     private fun førsteFraværsdagerForForespørsel(): List<PersonObserver.FørsteFraværsdag> {
-        val deAndre = person.vedtaksperioder(MED_SAMME_AGP_OG_SKJÆRINGSTIDSPUNKT(this))
+        val deAndre = person.vedtaksperioder(MED_SKJÆRINGSTIDSPUNKT(this.skjæringstidspunkt))
             .filterNot { it.arbeidsgiver === this.arbeidsgiver }
             .groupBy { it.arbeidsgiver }
             .mapNotNull { (arbeidsgiver, perioder) ->
@@ -3309,27 +3309,9 @@ internal class Vedtaksperiode private constructor(
         }
 
         // Fredet funksjonsnavn
-        internal val TIDLIGERE_OG_ETTERGØLGENDE = fun(segSelv: Vedtaksperiode): VedtaksperiodeFilter {
-            val medSammeAGP = MED_SAMME_AGP_OG_SKJÆRINGSTIDSPUNKT(segSelv)
+        internal val OVERLAPPENDE_OG_ETTERGØLGENDE = fun(segSelv: Vedtaksperiode): VedtaksperiodeFilter {
             return fun(other: Vedtaksperiode): Boolean {
-                if (other.periode.start >= segSelv.periode.start) return true // Forkaster nyere perioder på tvers av arbeidsgivere
-                return medSammeAGP(other)
-            }
-        }
-        internal val MED_SAMME_AGP_OG_SKJÆRINGSTIDSPUNKT = fun(segSelv: Vedtaksperiode): VedtaksperiodeFilter {
-            val skjæringstidspunkt = segSelv.skjæringstidspunkt
-            val arbeidsgiverperiode = segSelv.behandlinger
-                .arbeidsgiverperiode()
-                .arbeidsgiverperioder
-                .periode()
-            return fun(other: Vedtaksperiode): Boolean {
-                val arbeidsgiverperiodeOther = other
-                    .behandlinger
-                    .arbeidsgiverperiode()
-                    .arbeidsgiverperioder
-                    .periode()
-                if (arbeidsgiverperiode != null && other.arbeidsgiver === segSelv.arbeidsgiver && arbeidsgiverperiodeOther?.overlapperMed(arbeidsgiverperiode) == true) return true // Forkaster samme arbeidsgiverperiode (kun for samme arbeidsgiver)
-                return other.skjæringstidspunkt == skjæringstidspunkt // Forkaster alt med samme skjæringstidspunkt på tvers av arbeidsgivere
+                return segSelv.periode.overlapperEllerStarterFør(other.periode)
             }
         }
 

sykepenger-model/src/test/kotlin/no/nav/helse/hendelser/PeriodeTest.kt

@@ -34,6 +34,31 @@ import org.junit.jupiter.api.assertThrows
 internal class PeriodeTest {
     private val periode = Periode(1.juli, 10.juli)
 
+    @Test
+    fun overlapperEllerStarterFør() {
+        val p1 = 3.januar til 5.januar
+
+        (1.januar til 2.januar).also { p2 ->
+            assertFalse(p1.overlapperEllerStarterFør(p2))
+            assertTrue(p2.overlapperEllerStarterFør(p1))
+        }
+
+        (3.januar til 4.januar).also { p2 ->
+            assertTrue(p1.overlapperEllerStarterFør(p2))
+            assertTrue(p2.overlapperEllerStarterFør(p1))
+        }
+
+        (5.januar til 6.januar).also { p2 ->
+            assertTrue(p1.overlapperEllerStarterFør(p2))
+            assertTrue(p2.overlapperEllerStarterFør(p1))
+        }
+
+        (6.januar til 7.januar).also { p2 ->
+            assertTrue(p1.overlapperEllerStarterFør(p2))
+            assertFalse(p2.overlapperEllerStarterFør(p1))
+        }
+    }
+
     @Test
     fun mursteinsperioder() {
         val perioder = listOf(

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

@@ -79,7 +79,7 @@ internal class ForkastForlengelseAvForkastetPeriodeTest : AbstractEndToEndTest()
         assertTrue(observatør.hendelseider(3.vedtaksperiode.id(a1)).isNotEmpty())
         assertTrue(observatør.hendelseider(4.vedtaksperiode.id(a1)).isNotEmpty())
         assertTrue(observatør.hendelseider(5.vedtaksperiode.id(a1)).isNotEmpty())
-        assertForkastetPeriodeTilstander(1.vedtaksperiode, START, AVVENTER_INFOTRYGDHISTORIKK, AVVENTER_INNTEKTSMELDING, AVVENTER_BLOKKERENDE_PERIODE, AVSLUTTET_UTEN_UTBETALING, AVVENTER_BLOKKERENDE_PERIODE, AVSLUTTET_UTEN_UTBETALING, TIL_INFOTRYGD)
+        assertTilstander(1.vedtaksperiode, START, AVVENTER_INFOTRYGDHISTORIKK, AVVENTER_INNTEKTSMELDING, AVVENTER_BLOKKERENDE_PERIODE, AVSLUTTET_UTEN_UTBETALING, AVVENTER_BLOKKERENDE_PERIODE, AVSLUTTET_UTEN_UTBETALING)
         assertForkastetPeriodeTilstander(2.vedtaksperiode, START, AVVENTER_INNTEKTSMELDING, TIL_INFOTRYGD)
         assertForkastetPeriodeTilstander(3.vedtaksperiode, START, TIL_INFOTRYGD)
         assertForkastetPeriodeTilstander(4.vedtaksperiode, START, TIL_INFOTRYGD)

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

@@ -30,7 +30,7 @@ internal class ForkasteAuuTest : AbstractDslTest() {
             nyPeriode(17.januar til 31.januar)
             nullstillTilstandsendringer()
             håndterAnmodningOmForkasting(2.vedtaksperiode)
-            assertForkastetPeriodeTilstander(1.vedtaksperiode, AVSLUTTET_UTEN_UTBETALING, TIL_INFOTRYGD)
+            assertTilstander(1.vedtaksperiode, AVSLUTTET_UTEN_UTBETALING)
             assertForkastetPeriodeTilstander(2.vedtaksperiode, AVVENTER_INNTEKTSMELDING, TIL_INFOTRYGD)
         }
     }
@@ -42,7 +42,7 @@ internal class ForkasteAuuTest : AbstractDslTest() {
             nyPeriode(18.januar til 31.januar)
             nullstillTilstandsendringer()
             håndterAnmodningOmForkasting(2.vedtaksperiode)
-            assertForkastetPeriodeTilstander(1.vedtaksperiode, AVSLUTTET_UTEN_UTBETALING, TIL_INFOTRYGD)
+            assertTilstander(1.vedtaksperiode, AVSLUTTET_UTEN_UTBETALING)
             assertForkastetPeriodeTilstander(2.vedtaksperiode, AVVENTER_INNTEKTSMELDING, TIL_INFOTRYGD)
         }
     }

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

@@ -10,6 +10,7 @@ import no.nav.helse.hendelser.til
 import no.nav.helse.januar
 import no.nav.helse.mars
 import no.nav.helse.person.TilstandType.AVSLUTTET
+import no.nav.helse.person.TilstandType.AVSLUTTET_UTEN_UTBETALING
 import no.nav.helse.person.TilstandType.AVVENTER_GODKJENNING
 import no.nav.helse.person.TilstandType.TIL_INFOTRYGD
 import no.nav.helse.person.aktivitetslogg.Varselkode
@@ -39,11 +40,8 @@ internal class VedtaksperiodeForkastetE2ETest : AbstractEndToEndTest() {
 
     @Test
     fun `forkaster kort periode`() {
-        håndterSykmelding(Sykmeldingsperiode(1.januar, 5.januar))
         håndterSøknad(1.januar til 5.januar)
-        håndterSykmelding(Sykmeldingsperiode(6.januar, 15.januar))
         håndterSøknad(6.januar til 15.januar)
-        håndterSykmelding(Sykmeldingsperiode(16.januar, 31.januar))
         håndterSøknad(16.januar til 31.januar)
         håndterInntektsmelding(listOf(1.januar til 16.januar))
         håndterVilkårsgrunnlag(3.vedtaksperiode)
@@ -52,10 +50,11 @@ internal class VedtaksperiodeForkastetE2ETest : AbstractEndToEndTest() {
         håndterUtbetalingsgodkjenning(3.vedtaksperiode)
         håndterUtbetalt()
         håndterAnnullerUtbetaling(a1)
-        assertEquals(3, observatør.forkastedePerioder())
+        håndterUtbetalt()
+        assertEquals(1, observatør.forkastedePerioder())
         assertEquals(AVSLUTTET, observatør.forkastet(3.vedtaksperiode.id(a1)).gjeldendeTilstand)
-        assertSisteTilstand(1.vedtaksperiode, TIL_INFOTRYGD)
-        assertSisteTilstand(2.vedtaksperiode, TIL_INFOTRYGD)
+        assertSisteTilstand(1.vedtaksperiode, AVSLUTTET_UTEN_UTBETALING)
+        assertSisteTilstand(2.vedtaksperiode, AVSLUTTET_UTEN_UTBETALING)
         assertSisteTilstand(3.vedtaksperiode, TIL_INFOTRYGD)
     }
 

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

@@ -115,7 +115,8 @@ internal class VilkårsgrunnlagE2ETest : AbstractDslTest() {
             assertVilkårsgrunnlagFraSpleisFor(1.januar)
 
             håndterAnnullering(inspektør.sisteUtbetaling().utbetalingId)
-            assertIngenVilkårsgrunnlagFraSpleis()
+            håndterUtbetalt()
+            assertVilkårsgrunnlagFraSpleisFor(1.januar)
         }
     }
 

sykepenger-model/src/test/kotlin/no/nav/helse/spleis/e2e/infotrygd/ForlengelseFraInfotrygdTest.kt

@@ -23,6 +23,7 @@ import no.nav.helse.spleis.e2e.AbstractEndToEndTest
 import no.nav.helse.spleis.e2e.assertForkastetPeriodeTilstander
 import no.nav.helse.spleis.e2e.assertFunksjonellFeil
 import no.nav.helse.spleis.e2e.assertSisteTilstand
+import no.nav.helse.spleis.e2e.assertTilstander
 import no.nav.helse.spleis.e2e.assertVarsel
 import no.nav.helse.spleis.e2e.håndterSykmelding
 import no.nav.helse.spleis.e2e.håndterSøknad
@@ -57,7 +58,7 @@ internal class ForlengelseFraInfotrygdTest : AbstractEndToEndTest() {
         håndterUtbetalingshistorikkEtterInfotrygdendring(ArbeidsgiverUtbetalingsperiode(a1, 1.januar, 31.januar, 100.prosent, 1000.daglig))
 
         assertVarsel(Varselkode.RV_IT_3, 1.vedtaksperiode.filter())
-        assertForkastetPeriodeTilstander(1.vedtaksperiode, START, AVVENTER_INFOTRYGDHISTORIKK, AVVENTER_INNTEKTSMELDING, TIL_INFOTRYGD)
+        assertTilstander(1.vedtaksperiode, START, AVVENTER_INFOTRYGDHISTORIKK, AVVENTER_INNTEKTSMELDING)
         assertForkastetPeriodeTilstander(2.vedtaksperiode, START, AVVENTER_INNTEKTSMELDING, TIL_INFOTRYGD)
         assertForkastetPeriodeTilstander(3.vedtaksperiode, START, AVVENTER_INNTEKTSMELDING, AVVENTER_BLOKKERENDE_PERIODE, TIL_INFOTRYGD)
     }

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

@@ -114,9 +114,10 @@ internal class InntektsmeldingOgFerieE2ETest : AbstractEndToEndTest() {
         nyttVedtak(21.januar til 31.januar, arbeidsgiverperiode = listOf(1.januar til 5.januar, 10.januar til 20.januar), orgnummer = a1, vedtaksperiodeIdInnhenter = 4.vedtaksperiode)
         nullstillTilstandsendringer()
         håndterAnnullerUtbetaling(a1)
-        assertForkastetPeriodeTilstander(1.vedtaksperiode, AVSLUTTET_UTEN_UTBETALING, TIL_INFOTRYGD, orgnummer = a1)
-        assertForkastetPeriodeTilstander(2.vedtaksperiode, AVSLUTTET_UTEN_UTBETALING, TIL_INFOTRYGD, orgnummer = a1)
-        assertForkastetPeriodeTilstander(3.vedtaksperiode, AVSLUTTET_UTEN_UTBETALING, TIL_INFOTRYGD, orgnummer = a1)
+        håndterUtbetalt(orgnummer = a1)
+        assertTilstander(1.vedtaksperiode, AVSLUTTET_UTEN_UTBETALING, AVVENTER_BLOKKERENDE_PERIODE, AVSLUTTET_UTEN_UTBETALING, orgnummer = a1)
+        assertTilstander(2.vedtaksperiode, AVSLUTTET_UTEN_UTBETALING, AVVENTER_BLOKKERENDE_PERIODE, AVSLUTTET_UTEN_UTBETALING, orgnummer = a1)
+        assertTilstander(3.vedtaksperiode, AVSLUTTET_UTEN_UTBETALING, AVVENTER_BLOKKERENDE_PERIODE, AVSLUTTET_UTEN_UTBETALING, orgnummer = a1)
         assertForkastetPeriodeTilstander(4.vedtaksperiode, AVSLUTTET, TIL_INFOTRYGD, orgnummer = a1)
     }
 

sykepenger-model/src/test/kotlin/no/nav/helse/spleis/e2e/revurdering/ReberegningAvAvsluttetUtenUtbetalingNyE2ETest.kt

@@ -211,7 +211,7 @@ internal class ReberegningAvAvsluttetUtenUtbetalingNyE2ETest : AbstractEndToEndT
 
         nullstillTilstandsendringer()
         håndterUtbetalingsgodkjenning(2.vedtaksperiode, utbetalingGodkjent = false)
-        assertForkastetPeriodeTilstander(1.vedtaksperiode, AVSLUTTET_UTEN_UTBETALING, TIL_INFOTRYGD)
+        assertTilstander(1.vedtaksperiode, AVSLUTTET_UTEN_UTBETALING)
         assertForkastetPeriodeTilstander(2.vedtaksperiode, AVVENTER_GODKJENNING, TIL_INFOTRYGD)
     }
 
@@ -918,8 +918,8 @@ internal class ReberegningAvAvsluttetUtenUtbetalingNyE2ETest : AbstractEndToEndT
         nullstillTilstandsendringer()
         håndterUtbetalingsgodkjenning(2.vedtaksperiode, utbetalingGodkjent = false, orgnummer = a1)
 
-        assertForkastetPeriodeTilstander(1.vedtaksperiode, AVSLUTTET_UTEN_UTBETALING, TIL_INFOTRYGD, orgnummer = a1)
-        assertForkastetPeriodeTilstander(1.vedtaksperiode, AVSLUTTET_UTEN_UTBETALING, TIL_INFOTRYGD, orgnummer = a2)
+        assertTilstander(1.vedtaksperiode, AVSLUTTET_UTEN_UTBETALING, orgnummer = a1)
+        assertTilstander(1.vedtaksperiode, AVSLUTTET_UTEN_UTBETALING, orgnummer = a2)
         assertForkastetPeriodeTilstander(2.vedtaksperiode, AVVENTER_GODKJENNING, TIL_INFOTRYGD, orgnummer = a1)
         assertForkastetPeriodeTilstander(2.vedtaksperiode, AVVENTER_BLOKKERENDE_PERIODE, TIL_INFOTRYGD, orgnummer = a2)
         assertForkastetPeriodeTilstander(3.vedtaksperiode, AVVENTER_BLOKKERENDE_PERIODE, TIL_INFOTRYGD, orgnummer = a1)

sykepenger-primitiver/src/main/kotlin/no/nav/helse/hendelser/Periode.kt

@@ -132,6 +132,9 @@ class Periode(fom: LocalDate, tom: LocalDate) : ClosedRange<LocalDate>, Iterable
     fun slutterEtter(other: LocalDate) =
         other <= this.endInclusive
 
+    fun overlapperEllerStarterFør(other: Periode) =
+        this.start <= other.endInclusive
+
     fun utenfor(other: Periode) =
         this.start < other.start || this.endInclusive > other.endInclusive