små fikses fra feedback av PR

Author: ramnav990

app/src/main/kotlin/no/nav/tiltakspenger/saksbehandling/meldekort/domene/MeldekortBehandling.kt

@@ -98,7 +98,6 @@ sealed interface MeldekortBehandling {
                     meldeperiode = meldeperiode,
                     meldekortId = this.id,
                     sakId = this.sakId,
-                    maksDagerMedTiltakspengerForPeriode = meldeperiode.antallDagerForPeriode,
                     tiltakstypePerioder = tiltakstypePerioder,
                 ),
             )
@@ -201,7 +200,6 @@ sealed interface MeldekortBehandling {
                     tiltakstypePerioder = tiltakstypePerioder,
                     meldekortId = this.id,
                     sakId = this.sakId,
-                    maksDagerMedTiltakspengerForPeriode = meldeperiode.antallDagerForPeriode,
                 ),
                 saksbehandler = saksbehandler,
                 navkontor = this.navkontor,
@@ -335,21 +333,15 @@ fun Sak.opprettMeldekortBehandling(
             "Forrige meldekortbehandling i kjeden må være godkjent for å opprette en ny behandling/korrigering (kjede $kjedeId på sak ${this.id})"
         }
     }
-
     val meldekortId = MeldekortId.random()
     val meldeperiode = hentSisteMeldeperiodeForKjede(kjedeId)
-    require(this.vedtaksliste.innvilgelsesperioder.isNotEmpty()) { "Må ha minst én periode som gir rett til tiltakspegner for å opprette meldekortbehandling" }
 
     val overlappendePeriode = this.vedtaksliste.innvilgelsesperioder.overlappendePerioder(
         listOf(meldeperiode.periode),
     ).singleOrNullOrThrow()
 
     requireNotNull(overlappendePeriode) { "Meldeperioden må overlappe med innvilgelsesperioden(e)" }
 
-    // TODO jah: Behandlingen må ta inn periodisert antall dager og ikke bruke tidligere vedtak her. Tror ikke maksDagerMedTiltakspengerForPeriode brukes til noe; kanskje den bør bort fra beregningen?
-    val vedtak = this.vedtaksliste.tidslinjeForPeriode(overlappendePeriode).single().verdi
-    val maksDagerMedTiltakspengerForPeriode = vedtak?.behandling?.maksDagerMedTiltakspengerForPeriode
-
     // TODO abn: må støtte flere brukers meldekort på samme kjede før vi åpner for korrigering fra bruker
     val brukersMeldekort = this.brukersMeldekort.find { it.kjedeId == kjedeId }
 
@@ -372,8 +364,6 @@ fun Sak.opprettMeldekortBehandling(
             meldeperiode = meldeperiode,
             meldekortId = meldekortId,
             sakId = this.id,
-            // TODO jah: Behandlingen må ta inn periodisert antall dager og ikke bruke tidligere vedtak her. Tror ikke maksDagerMedTiltakspengerForPeriode brukes til noe; kanskje den bør bort fra beregningen?
-            maksDagerMedTiltakspengerForPeriode = maksDagerMedTiltakspengerForPeriode!!,
             tiltakstypePerioder = this.vedtaksliste.tiltakstypeperioder,
         ),
     )

app/src/main/kotlin/no/nav/tiltakspenger/saksbehandling/meldekort/domene/Meldeperiode.kt

@@ -27,8 +27,8 @@ data class Meldeperiode(
     val girRett: Map<LocalDate, Boolean>,
     val sendtTilMeldekortApi: LocalDateTime?,
 ) : Comparable<Meldeperiode> {
-    val ingenDagerGirRett = girRett.values.none { it }
     val antallDagerSomGirRett = girRett.values.count { it }
+    val ingenDagerGirRett = antallDagerSomGirRett == 0
 
     fun helePeriodenErSperret(): Boolean {
         return girRett.values.toList().all { !it }
@@ -40,7 +40,12 @@ data class Meldeperiode(
     }
 
     fun erLik(meldeperiode: Meldeperiode): Boolean {
-        return this.kjedeId == meldeperiode.kjedeId && this.sakId == meldeperiode.sakId && this.saksnummer == meldeperiode.saksnummer && this.fnr == meldeperiode.fnr && this.periode == meldeperiode.periode && this.antallDagerForPeriode == meldeperiode.antallDagerForPeriode && this.girRett == meldeperiode.girRett
+        // feltene vi har lyst til å ignorere
+        return this == meldeperiode.copy(
+            id = this.id,
+            opprettet = this.opprettet,
+            versjon = this.versjon,
+        )
     }
 
     override fun compareTo(other: Meldeperiode): Int {

app/src/main/kotlin/no/nav/tiltakspenger/saksbehandling/meldekort/domene/MeldeperiodeBeregning.kt

@@ -127,7 +127,6 @@ sealed interface MeldeperiodeBeregning : List<MeldeperiodeBeregningDag> {
                 meldeperiode: Meldeperiode,
                 meldekortId: MeldekortId,
                 sakId: SakId,
-                maksDagerMedTiltakspengerForPeriode: Int,
                 tiltakstypePerioder: Periodisering<TiltakstypeSomGirRett?>,
             ): IkkeUtfyltMeldeperiode {
                 val dager =
@@ -146,7 +145,7 @@ sealed interface MeldeperiodeBeregning : List<MeldeperiodeBeregningDag> {
                         }
                     }
                 return if (dager.any { it is MeldeperiodeBeregningDag.IkkeUtfylt }) {
-                    IkkeUtfyltMeldeperiode(sakId, maksDagerMedTiltakspengerForPeriode, dager.toNonEmptyListOrNull()!!)
+                    IkkeUtfyltMeldeperiode(sakId, meldeperiode.antallDagerForPeriode, dager.toNonEmptyListOrNull()!!)
                 } else {
                     throw IllegalStateException("Alle dagene i en meldekortperiode er SPERRET. Dette har vi ikke støtte for i MVP.")
                 }

app/src/main/kotlin/no/nav/tiltakspenger/saksbehandling/repository/meldekort/MeldekortBehandlingPostgresRepo.kt

@@ -178,19 +178,19 @@ class MeldekortBehandlingPostgresRepo(
             row: Row,
             session: Session,
         ): MeldekortBehandling {
+            val meldeperiodeId = MeldeperiodeId.fromString(row.string("meldeperiode_id"))
+            val meldeperiode = MeldeperiodePostgresRepo.hentForMeldeperiodeId(meldeperiodeId, session)
+
             val id = MeldekortId.fromString(row.string("id"))
             val sakId = SakId.fromString(row.string("sak_id"))
             val saksnummer = Saksnummer(row.string("saksnummer"))
             val navkontorEnhetsnummer = row.string("navkontor")
             val navkontorNavn = row.stringOrNull("navkontor_navn")
             val fnr = Fnr.fromString(row.string("fnr"))
-            val maksDagerMedTiltakspengerForPeriode = Behandling.MAKS_DAGER_MED_TILTAKSPENGER_FOR_PERIODE
+            val maksDagerMedTiltakspengerForPeriode = meldeperiode?.antallDagerForPeriode ?: Behandling.MAKS_DAGER_MED_TILTAKSPENGER_FOR_PERIODE
             val opprettet = row.localDateTime("opprettet")
             val type = row.string("type").tilMeldekortBehandlingType()
 
-            val meldeperiodeId = MeldeperiodeId.fromString(row.string("meldeperiode_id"))
-            val meldeperiode = MeldeperiodePostgresRepo.hentForMeldeperiodeId(meldeperiodeId, session)
-
             val navkontor = Navkontor(kontornummer = navkontorEnhetsnummer, kontornavn = navkontorNavn)
 
             val saksbehandler = row.string("saksbehandler")

app/src/main/kotlin/no/nav/tiltakspenger/saksbehandling/repository/meldekort/MeldeperiodePostgresRepo.kt

@@ -73,9 +73,7 @@ internal class MeldeperiodePostgresRepo(
 
     override fun lagre(meldeperioder: List<Meldeperiode>, sessionContext: SessionContext?) {
         sessionFactory.withSessionContext(sessionContext) { sc ->
-            meldeperioder.forEach {
-                lagre(it, sc)
-            }
+            meldeperioder.forEach { lagre(it, sc) }
         }
     }
 

app/src/main/kotlin/no/nav/tiltakspenger/saksbehandling/saksbehandling/domene/behandling/SakRevurderingEx.kt

@@ -33,7 +33,7 @@ suspend fun Sak.startRevurdering(
     }
 
     // TODO - dette gjelder bare så lenge dette er en stans revurdering
-    // Her har vi ikke valgt revurderingsperioden, men revurderingsperioden kan aldri være større enn innvilgelseprioden
+    // Her har vi ikke valgt revurderingsperioden, men ved forlengelse vil den kunne være større.
     val saksopplysningsperiode = this.vedtaksliste.innvilgelsesperiode!!
 
     val revurdering = Behandling.opprettRevurdering(

app/src/main/kotlin/no/nav/tiltakspenger/saksbehandling/saksbehandling/service/behandling/IverksettBehandlingService.kt

@@ -141,9 +141,7 @@ class IverksettBehandlingService(
             statistikkSakRepo.lagre(sakStatistikk, tx)
             statistikkStønadRepo.lagre(stønadStatistikk, tx)
             oppdaterteMeldekort.forEach { meldekortBehandlingRepo.oppdater(it, tx) }
-            meldeperioder.forEach {
-                meldeperiodeRepo.lagre(it, tx)
-            }
+            meldeperiodeRepo.lagre(meldeperioder, tx)
         }
         return oppdatertSak.copy(meldekortBehandlinger = oppdaterteMeldekortbehandlinger)
     }
@@ -164,7 +162,7 @@ class IverksettBehandlingService(
             statistikkSakRepo.lagre(sakStatistikk, tx)
             statistikkStønadRepo.lagre(stønadStatistikk, tx)
             oppdaterteMeldekort.forEach { meldekortBehandlingRepo.oppdater(it, tx) }
-            oppdaterteMeldeperioder.forEach { meldeperiodeRepo.lagre(it, tx) }
+            meldeperiodeRepo.lagre(oppdaterteMeldeperioder, tx)
         }
         return oppdatertSak.copy(meldekortBehandlinger = oppdaterteMeldekortbehandlinger)
     }

app/src/test/kotlin/no/nav/tiltakspenger/saksbehandling/meldekort/domene/MeldeperiodeKjederTest.kt

@@ -1,7 +1,9 @@
 package no.nav.tiltakspenger.saksbehandling.meldekort.domene
 
 import io.kotest.matchers.shouldBe
+import no.nav.tiltakspenger.libs.common.Fnr
 import no.nav.tiltakspenger.libs.common.SakId
+import no.nav.tiltakspenger.libs.common.random
 import no.nav.tiltakspenger.libs.periodisering.Periode
 import no.nav.tiltakspenger.saksbehandling.felles.april
 import no.nav.tiltakspenger.saksbehandling.felles.desember
@@ -123,9 +125,11 @@ class MeldeperiodeKjederTest {
 
     @Test
     fun `genererer meldeperioder for et stansvedtak`() {
+        val fnr = Fnr.random()
+        val sakId = SakId.random()
         val periode = Periode(2.januar(2023), 17.januar(2023))
-        val innvilgelseVedtak = ObjectMother.nyRammevedtakInnvilgelse(periode = periode)
-        val stansVedtak = ObjectMother.nyRammevedtakStans(periode = periode)
+        val innvilgelseVedtak = ObjectMother.nyRammevedtakInnvilgelse(fnr = fnr, sakId = sakId, periode = periode)
+        val stansVedtak = ObjectMother.nyRammevedtakStans(fnr = fnr, sakId = sakId, periode = periode)
         val vedtaksliste = Vedtaksliste(listOf(innvilgelseVedtak, stansVedtak))
 
         val kjeder = MeldeperiodeKjeder(emptyList())
@@ -140,8 +144,10 @@ class MeldeperiodeKjederTest {
 
     @Test
     fun `genererer meldeperioder for innvilgelse, og deretter annulerer ved stans`() {
+        val fnr = Fnr.random()
+        val sakId = SakId.random()
         val periode = Periode(2.januar(2023), 17.januar(2023))
-        val innvilgelseVedtak = ObjectMother.nyRammevedtakInnvilgelse(periode = periode)
+        val innvilgelseVedtak = ObjectMother.nyRammevedtakInnvilgelse(fnr = fnr, sakId = sakId, periode = periode)
         val v1 = Vedtaksliste(listOf(innvilgelseVedtak))
         val kjederV1 = MeldeperiodeKjeder(emptyList())
 
@@ -162,7 +168,7 @@ class MeldeperiodeKjederTest {
             it.first.meldeperioder.last().antallDagerForPeriode shouldBe 2
         }
 
-        val stansVedtak = ObjectMother.nyRammevedtakStans(periode = periode)
+        val stansVedtak = ObjectMother.nyRammevedtakStans(fnr = fnr, sakId = sakId, periode = periode)
         val v2 = Vedtaksliste(listOf(innvilgelseVedtak, stansVedtak))
 
         val actual = nyeKjederV1.genererMeldeperioder(v2, LocalDate.MAX)

app/src/test/kotlin/no/nav/tiltakspenger/saksbehandling/objectmothers/MeldekortMother.kt

@@ -195,7 +195,6 @@ interface MeldekortMother {
             meldeperiode = meldeperiode,
             meldekortId = meldekortId,
             sakId = sakId,
-            maksDagerMedTiltakspengerForPeriode = maksDagerMedTiltakspengerForPeriode,
             tiltakstypePerioder = tiltakstypePerioder,
         )
     }
@@ -343,7 +342,6 @@ interface MeldekortMother {
                         meldeperiode = meldeperiode,
                         tiltakstypePerioder = tiltakstypePerioder,
                         meldekortId = meldekortId,
-                        maksDagerMedTiltakspengerForPeriode = kommando.dager.size,
                     ),
                     ikkeRettTilTiltakspengerTidspunkt = null,
                     meldeperiode = meldeperiode,
@@ -397,7 +395,6 @@ interface MeldekortMother {
                     meldeperiode = meldeperiode,
                     tiltakstypePerioder = tiltakstypePerioder,
                     meldekortId = meldekortId,
-                    maksDagerMedTiltakspengerForPeriode = kommando.dager.size,
                 ),
                 ikkeRettTilTiltakspengerTidspunkt = null,
                 meldeperiode = meldeperiode,

app/src/test/kotlin/no/nav/tiltakspenger/saksbehandling/objectmothers/ObjectMother.kt

@@ -1,14 +1,9 @@
 package no.nav.tiltakspenger.saksbehandling.objectmothers
 
 import no.nav.tiltakspenger.libs.common.AccessToken
-import no.nav.tiltakspenger.libs.common.Fnr
-import no.nav.tiltakspenger.libs.common.SakId
-import no.nav.tiltakspenger.libs.common.random
 import no.nav.tiltakspenger.saksbehandling.felles.Navkontor
 import no.nav.tiltakspenger.saksbehandling.felles.OppgaveId
-import no.nav.tiltakspenger.saksbehandling.felles.januar
 import no.nav.tiltakspenger.saksbehandling.saksbehandling.domene.personopplysninger.Navn
-import no.nav.tiltakspenger.saksbehandling.saksbehandling.domene.sak.Saksnummer
 import java.time.Instant
 
 object ObjectMother :
@@ -24,12 +19,6 @@ object ObjectMother :
     TiltakMother,
     SaksopplysningerMother,
     BarnetilleggMother {
-    val saksId = SakId.random()
-    val saksnummer = Saksnummer.genererSaknummer(1.januar(2024), "1111")
-    val fnr = Fnr.random()
-    val saksbehandler = this.saksbehandler()
-    val beslutter = this.beslutter()
-
     fun navn() = Navn("Fornavn", "Mellomnavn", "Etternavn")
     fun navkontor() = Navkontor(kontornummer = "0220", kontornavn = "Nav Asker")
     fun oppgaveId(oppgaveId: OppgaveId = OppgaveId("100")) = oppgaveId

app/src/test/kotlin/no/nav/tiltakspenger/saksbehandling/objectmothers/RammevedtakMother.kt

@@ -1,14 +1,17 @@
 package no.nav.tiltakspenger.saksbehandling.objectmothers
 
+import no.nav.tiltakspenger.libs.common.Fnr
 import no.nav.tiltakspenger.libs.common.SakId
 import no.nav.tiltakspenger.libs.common.VedtakId
 import no.nav.tiltakspenger.libs.common.nå
+import no.nav.tiltakspenger.libs.common.random
 import no.nav.tiltakspenger.libs.periodisering.Periode
 import no.nav.tiltakspenger.saksbehandling.distribusjon.DistribusjonId
 import no.nav.tiltakspenger.saksbehandling.felles.januar
 import no.nav.tiltakspenger.saksbehandling.felles.journalføring.JournalpostId
 import no.nav.tiltakspenger.saksbehandling.saksbehandling.domene.behandling.Behandling
 import no.nav.tiltakspenger.saksbehandling.saksbehandling.domene.behandling.Behandlingstype
+import no.nav.tiltakspenger.saksbehandling.saksbehandling.domene.sak.Saksnummer
 import no.nav.tiltakspenger.saksbehandling.saksbehandling.domene.vedtak.Rammevedtak
 import no.nav.tiltakspenger.saksbehandling.saksbehandling.domene.vedtak.Vedtakstype
 import java.time.LocalDate
@@ -20,9 +23,11 @@ interface RammevedtakMother {
         opprettet: LocalDateTime = nå(),
         sakId: SakId = SakId.random(),
         periode: Periode = ObjectMother.virkningsperiode(),
+        fnr: Fnr = Fnr.random(),
         behandling: Behandling = ObjectMother.nyVedtattBehandling(
             sakId = sakId,
             virkningsperiode = periode,
+            fnr = fnr,
         ),
         vedtaksdato: LocalDate = 2.januar(2023),
         vedtaksType: Vedtakstype = Vedtakstype.INNVILGELSE,
@@ -51,13 +56,14 @@ interface RammevedtakMother {
     fun nyRammevedtakInnvilgelse(
         id: VedtakId = VedtakId.random(),
         opprettet: LocalDateTime = nå(),
-        sakId: SakId = ObjectMother.saksId,
+        sakId: SakId = SakId.random(),
+        fnr: Fnr = Fnr.random(),
         periode: Periode = ObjectMother.virkningsperiode(),
         behandling: Behandling = ObjectMother.nyVedtattBehandling(
             sakId = sakId,
             virkningsperiode = periode,
-            saksnummer = ObjectMother.saksnummer,
-            fnr = ObjectMother.fnr,
+            saksnummer = Saksnummer.genererSaknummer(løpenr = "1001"),
+            fnr = fnr,
         ),
         vedtaksdato: LocalDate = 2.januar(2023),
         journalpostId: JournalpostId? = null,
@@ -85,13 +91,14 @@ interface RammevedtakMother {
     fun nyRammevedtakStans(
         id: VedtakId = VedtakId.random(),
         opprettet: LocalDateTime = nå(),
-        sakId: SakId = ObjectMother.saksId,
+        sakId: SakId = SakId.random(),
+        fnr: Fnr = Fnr.random(),
         periode: Periode = ObjectMother.virkningsperiode(),
         behandling: Behandling = ObjectMother.nyVedtattBehandling(
             sakId = sakId,
             virkningsperiode = periode,
-            saksnummer = ObjectMother.saksnummer,
-            fnr = ObjectMother.fnr,
+            saksnummer = Saksnummer.genererSaknummer(løpenr = "1001"),
+            fnr = fnr,
             behandlingstype = Behandlingstype.REVURDERING,
             søknad = null,
         ),

app/src/test/kotlin/no/nav/tiltakspenger/saksbehandling/objectmothers/SakMother.kt

@@ -126,11 +126,12 @@ interface SakMother {
     }
 
     fun nySakMedVedtak(
-        sakId: SakId = ObjectMother.saksId,
-        fnr: Fnr = ObjectMother.fnr,
-        saksnummer: Saksnummer = ObjectMother.saksnummer,
+        sakId: SakId = SakId.random(),
+        fnr: Fnr = Fnr.random(),
+        saksnummer: Saksnummer = Saksnummer.genererSaknummer(løpenr = "1001"),
         saksbehandler: Saksbehandler = saksbehandler(),
         virkningsperiode: Periode = virkningsperiode(),
+        beslutter: Saksbehandler = ObjectMother.beslutter(),
     ): Triple<Sak, Vedtak, Behandling> {
         val (sak, førstegangsbehandling) = this.sakMedOpprettetBehandling(
             sakId = sakId,
@@ -154,10 +155,10 @@ interface SakMother {
                     Pair(virkningsperiode, it.eksternDeltagelseId)
                 }.toList(),
             ),
-        ).taBehandling(ObjectMother.beslutter)
+        ).taBehandling(beslutter)
             .iverksett(
-                utøvendeBeslutter = ObjectMother.beslutter,
-                attestering = ObjectMother.godkjentAttestering(ObjectMother.beslutter),
+                utøvendeBeslutter = beslutter,
+                attestering = ObjectMother.godkjentAttestering(beslutter),
             )
 
         val sakMedIverksattBehandling = sak.copy(behandlinger = Behandlinger(iverksattBehandling))

app/src/test/kotlin/no/nav/tiltakspenger/saksbehandling/repository/meldekort/MeldekortBehandlingRepoImplTest.kt

@@ -17,6 +17,9 @@ class MeldekortBehandlingRepoImplTest {
     @Test
     fun `kan lagre og hente`() {
         withMigratedDb { testDataHelper ->
+            val meldekortRepo = testDataHelper.meldekortRepo
+            val sakRepo = testDataHelper.sakRepo
+
             val (sak) = testDataHelper.persisterIverksattFørstegangsbehandling(
                 deltakelseFom = 2.januar(2023),
                 deltakelseTom = 2.april(2023),
@@ -27,12 +30,7 @@ class MeldekortBehandlingRepoImplTest {
                 saksnummer = sak.saksnummer,
                 meldeperiode = sak.meldeperiodeKjeder.first().first(),
                 periode = sak.meldeperiodeKjeder.first().first().periode,
-            )
-
-            val meldekortRepo = testDataHelper.meldekortRepo
-            val sakRepo = testDataHelper.sakRepo
-
-            meldekortRepo.lagre(meldekort)
+            ).also { meldekortRepo.lagre(it) }
 
             testDataHelper.sessionFactory.withSession {
                 MeldekortBehandlingPostgresRepo.hentForMeldekortId(meldekort.id, it)!! shouldBe meldekort
@@ -47,16 +45,11 @@ class MeldekortBehandlingRepoImplTest {
                 oppdatertSak.meldeperiodeKjeder.last().kjedeId,
                 ObjectMother.navkontor(),
                 ObjectMother.saksbehandler(),
-            )
-
-            meldekortRepo.lagre(nesteMeldekort)
+            ).also { meldekortRepo.lagre(it) }
 
             val hentForMeldekortId2 =
                 testDataHelper.sessionFactory.withSession {
-                    MeldekortBehandlingPostgresRepo.hentForMeldekortId(
-                        nesteMeldekort.id,
-                        it,
-                    )
+                    MeldekortBehandlingPostgresRepo.hentForMeldekortId(nesteMeldekort.id, it)
                 }
             hentForMeldekortId2 shouldBe nesteMeldekort
         }
@@ -92,7 +85,10 @@ class MeldekortBehandlingRepoImplTest {
             meldekortRepo.oppdater(oppdatertMeldekortBehandling)
 
             testDataHelper.sessionFactory.withSession {
-                MeldekortBehandlingPostgresRepo.hentForMeldekortId(meldekortBehandling.id, it)!! shouldBe oppdatertMeldekortBehandling
+                MeldekortBehandlingPostgresRepo.hentForMeldekortId(
+                    meldekortBehandling.id,
+                    it,
+                )!! shouldBe oppdatertMeldekortBehandling
             }
         }
     }