Fikse tester

navikt · Dec 4, 2024 · e8dc9bb · e8dc9bb
1 parent d22b515
commit e8dc9bb
Show file tree

Hide file tree

Showing 4 changed files with 41 additions and 53 deletions.
diff --git a/src/main/kotlin/no/nav/bidrag/behandling/dto/v2/behandling/BehandlingDtoV2.kt b/src/main/kotlin/no/nav/bidrag/behandling/dto/v2/behandling/BehandlingDtoV2.kt
@@ -280,10 +280,10 @@ data class IkkeAktiveInntekter(
     val ingenEndringer
         get() =
             barnetillegg.isEmpty() &&
-                utvidetBarnetrygd.isEmpty() &&
-                kontantstøtte.isEmpty() &&
-                småbarnstillegg.isEmpty() &&
-                årsinntekter.isEmpty()
+                    utvidetBarnetrygd.isEmpty() &&
+                    kontantstøtte.isEmpty() &&
+                    småbarnstillegg.isEmpty() &&
+                    årsinntekter.isEmpty()
 }
 
 data class Grunnlagsinnhentingsfeil(
@@ -384,7 +384,7 @@ data class PeriodeAndreVoksneIHusstanden(
     val totalAntallHusstandsmedlemmer: Int,
     @Schema(
         description =
-            "Detaljer om husstandsmedlemmer som bor hos BP for gjeldende periode. " +
+        "Detaljer om husstandsmedlemmer som bor hos BP for gjeldende periode. " +
                 "Antall hustandsmedlemmer er begrenset til maks 10 personer",
     )
     val husstandsmedlemmer: List<AndreVoksneIHusstandenDetaljerDto> = emptyList(),
@@ -544,26 +544,16 @@ fun Grunnlagsdatatype.tilInntektrapporteringYtelse() =
 
 fun Grunnlagsdatatype.innhentesForRolle(behandling: Behandling) =
     when (this) {
-        Grunnlagsdatatype.BARNETILSYN ->
-            when (this.behandlingstypeMotRolletyper[behandling.tilType()]!!.first()) {
-                Rolletype.BIDRAGSMOTTAKER -> behandling.bidragsmottaker
-                Rolletype.BIDRAGSPLIKTIG -> behandling.bidragspliktig
-                else -> null
-            }
-
-        Grunnlagsdatatype.BOFORHOLD ->
-            when (this.behandlingstypeMotRolletyper[behandling.tilType()]!!.first()) {
-                Rolletype.BIDRAGSMOTTAKER -> behandling.bidragsmottaker
-                Rolletype.BIDRAGSPLIKTIG -> behandling.bidragspliktig
-                else -> null
-            }
-
-        Grunnlagsdatatype.BOFORHOLD_ANDRE_VOKSNE_I_HUSSTANDEN ->
-            when (this.behandlingstypeMotRolletyper[behandling.tilType()]!!.first()) {
-                Rolletype.BIDRAGSMOTTAKER -> behandling.bidragsmottaker
-                Rolletype.BIDRAGSPLIKTIG -> behandling.bidragspliktig
-                else -> null
+        Grunnlagsdatatype.BARNETILSYN, Grunnlagsdatatype.BOFORHOLD, Grunnlagsdatatype.BOFORHOLD_ANDRE_VOKSNE_I_HUSSTANDEN -> {
+            val t = this.behandlingstypeMotRolletyper[behandling.tilType()]
+            t?.let {
+                when (it.first()) {
+                    Rolletype.BIDRAGSMOTTAKER -> behandling.bidragsmottaker
+                    Rolletype.BIDRAGSPLIKTIG -> behandling.bidragspliktig
+                    else -> null
+                }
             }
+        }
 
         else -> null
     }
diff --git a/src/main/kotlin/no/nav/bidrag/behandling/transformers/underhold/Utvidelser.kt b/src/main/kotlin/no/nav/bidrag/behandling/transformers/underhold/Utvidelser.kt
@@ -34,16 +34,16 @@ fun Barnetilsyn.tilStønadTilBarnetilsynDto(): StønadTilBarnetilsynDto =
         id = this.id,
         periode = DatoperiodeDto(this.fom, this.tom),
         skolealder =
-            when (this.under_skolealder) {
-                true -> Skolealder.UNDER
-                false -> Skolealder.OVER
-                else -> null
-            },
+        when (this.under_skolealder) {
+            true -> Skolealder.UNDER
+            false -> Skolealder.OVER
+            else -> null
+        },
         tilsynstype =
-            when (this.omfang) {
-                Tilsynstype.IKKE_ANGITT -> null
-                else -> this.omfang
-            },
+        when (this.omfang) {
+            Tilsynstype.IKKE_ANGITT -> null
+            else -> this.omfang
+        },
         kilde = this.kilde,
     )
 
@@ -56,7 +56,7 @@ fun Underholdskostnad.harIkkeBarnetilsynITabellFraFør(personident: String) =
         .first()
         .personident
         ?.verdi == personident &&
-        this.barnetilsyn.isEmpty()
+            this.barnetilsyn.isEmpty()
 
 fun BarnDto.annetBarnMedSammeNavnOgFødselsdatoEksistererFraFør(behandling: Behandling) =
     behandling.underholdskostnader
@@ -78,11 +78,11 @@ fun BarnetilsynGrunnlagDto.tilBarnetilsyn(u: Underholdskostnad) =
         kilde = Kilde.OFFENTLIG,
         omfang = this.tilsynstype ?: Tilsynstype.IKKE_ANGITT,
         under_skolealder =
-            when (this.skolealder) {
-                Skolealder.OVER -> false
-                Skolealder.UNDER -> true
-                else -> null
-            },
+        when (this.skolealder) {
+            Skolealder.OVER -> false
+            Skolealder.UNDER -> true
+            else -> null
+        },
     )
 
 fun Set<Underholdskostnad>.justerePerioderEtterVirkningsdato() = forEach { it.justerePerioder() }
@@ -166,7 +166,7 @@ fun Behandling.aktivereBarnetilsynHvisIngenEndringerMåAksepteres() {
     val aktiveGrunnlag = grunnlag.hentAlleAktiv().toSet()
     if (ikkeAktiveGrunnlag.isEmpty()) return
     val endringerSomMåBekreftes = ikkeAktiveGrunnlag.henteEndringerIBarnetilsyn(aktiveGrunnlag, this)
-    val rolleInnhentetFor = Grunnlagsdatatype.BARNETILSYN.innhentesForRolle(this)!!
+    val rolleInnhentetFor = Grunnlagsdatatype.BARNETILSYN.innhentesForRolle(this)
 
     underholdskostnader
         .filter { it.person.rolle.isNotEmpty() }
@@ -177,13 +177,13 @@ fun Behandling.aktivereBarnetilsynHvisIngenEndringerMåAksepteres() {
                 ikkeAktiveGrunnlag
                     .hentGrunnlagForType(
                         Grunnlagsdatatype.BARNETILSYN,
-                        rolleInnhentetFor.personident!!.verdi,
+                        rolleInnhentetFor?.personident!!.verdi,
                     ).find { it.gjelder != null && it.gjelder == u.person.personident!!.verdi } ?: return@forEach
 
             log.info {
                 "Ikke-aktive grunnlag type ${Grunnlagsdatatype.BOFORHOLD} med id ${ikkeaktivtGrunnlag.id} " +
-                    " for rolle ${rolleInnhentetFor.rolletype} i behandling ${this.id} har ingen " +
-                    "endringer som må aksepteres av saksbehandler. Aktiverer automatisk det nyinnhenta grunnlaget."
+                        " for rolle ${rolleInnhentetFor.rolletype} i behandling ${this.id} har ingen " +
+                        "endringer som må aksepteres av saksbehandler. Aktiverer automatisk det nyinnhenta grunnlaget."
             }
 
             ikkeaktivtGrunnlag.aktiv = LocalDateTime.now()
@@ -194,11 +194,9 @@ fun Behandling.aktivereBarnetilsynHvisIngenEndringerMåAksepteres() {
 
 private fun Behandling.aktivereOriginaltBarnetilsynsgrunnlagHvisAktivertForAlleBarn() {
     val grunnlagsdatatype = Grunnlagsdatatype.BARNETILSYN
-    val nyesteOriginaleBarnetilsynsgrunnlag =
-        this.henteNyesteGrunnlag(
-            Grunnlagstype(grunnlagsdatatype, false),
-            grunnlagsdatatype.innhentesForRolle(this)!!,
-        )
+    val nyesteOriginaleBarnetilsynsgrunnlag = grunnlagsdatatype.innhentesForRolle(this)?.let {
+        this.henteNyesteGrunnlag(Grunnlagstype(grunnlagsdatatype, false), it)
+    }
 
     nyesteOriginaleBarnetilsynsgrunnlag?.let {
         if (nyesteOriginaleBarnetilsynsgrunnlag.aktiv == null &&

diff --git a/.../kotlin/no/nav/bidrag/behandling/controller/behandling/OppretteBehandlingFraVedtakTest.kt b/.../kotlin/no/nav/bidrag/behandling/controller/behandling/OppretteBehandlingFraVedtakTest.kt
@@ -71,7 +71,7 @@ class OppretteBehandlingFraVedtakTest : BehandlingControllerTest() {
         assertSoftly(behandling) {
             roller shouldHaveSize 3
             inntekter shouldHaveSize 15
-            grunnlag shouldHaveSize 36
+            grunnlag shouldHaveSize 37
             innkrevingstype shouldBe Innkrevingstype.MED_INNKREVING
             opprinneligVedtakstidspunkt shouldHaveSize 1
             opprinneligVedtakstidspunkt shouldContain LocalDateTime.parse("2024-02-23T15:34:27.275019")

diff --git a/src/test/kotlin/no/nav/bidrag/behandling/service/VedtakTilBehandlingForskuddTest.kt b/src/test/kotlin/no/nav/bidrag/behandling/service/VedtakTilBehandlingForskuddTest.kt
@@ -325,9 +325,9 @@ class VedtakTilBehandlingForskuddTest : CommonVedtakTilBehandlingTest() {
         }
 
         assertSoftly(behandling.grunnlag) {
-            this shouldHaveSize 24
+            this shouldHaveSize 25
             filter { it.erBearbeidet && it.rolle.rolletype == Rolletype.BIDRAGSMOTTAKER }.shouldHaveSize(
-                13,
+                14,
             )
             filter { it.erBearbeidet && it.rolle.rolletype == Rolletype.BARN }.shouldHaveSize(1)
             filter { !it.erBearbeidet && it.rolle.rolletype == Rolletype.BIDRAGSMOTTAKER }.shouldHaveSize(
@@ -591,7 +591,7 @@ class VedtakTilBehandlingForskuddTest : CommonVedtakTilBehandlingTest() {
 
     private fun Behandling.validerGrunnlag() {
         assertSoftly(grunnlagListe) {
-            size shouldBe 24
+            size shouldBe 25
             filtrerEtterTypeOgIdent(
                 Grunnlagsdatatype.SKATTEPLIKTIGE_INNTEKTER,
                 testdataBarn2.ident,
@@ -653,7 +653,7 @@ class VedtakTilBehandlingForskuddTest : CommonVedtakTilBehandlingTest() {
                 Grunnlagsdatatype.BARNETILSYN,
                 testdataBM.ident,
                 true,
-            ) shouldHaveSize 1
+            ) shouldHaveSize 2
             filtrerEtterTypeOgIdent(
                 Grunnlagsdatatype.SMÅBARNSTILLEGG,
                 testdataBM.ident,