Feilfiks opphørsdato periodisering boforhold (#475)

Author: ugur93

bidrag-beregn-core/src/main/kotlin/no/nav/bidrag/beregn/core/boforhold/BeregnBoforholdService.kt

@@ -6,6 +6,7 @@ import no.nav.bidrag.beregn.core.dto.PeriodeCore
 import no.nav.bidrag.beregn.core.dto.VoksneIHusstandenPeriodeCore
 import no.nav.bidrag.beregn.core.mapping.tilGrunnlag
 import no.nav.bidrag.beregn.core.service.mapper.CoreMapper
+import no.nav.bidrag.beregn.core.util.justerPeriodeTilOpphørsdato
 import no.nav.bidrag.domene.enums.grunnlag.Grunnlagstype
 import no.nav.bidrag.domene.enums.person.Bostatuskode
 import no.nav.bidrag.domene.tid.ÅrMånedsperiode
@@ -141,13 +142,12 @@ internal class BeregnBoforholdService : CoreMapper() {
     fun justerPerioderForOpphørsdato(periodeliste: List<PeriodeCore>, opphørsdato: LocalDate?): List<PeriodeCore> {
         if (opphørsdato == null) return periodeliste
         // Antar at opphørsdato er måneden perioden skal opphøre
-        val justerOpphørsdato = opphørsdato.withDayOfMonth(1).minusDays(1)
         return periodeliste.filter {
             it.datoFom.isBefore(opphørsdato)
         }
             .map { grunnlag ->
-                if (grunnlag.datoTil == null || grunnlag.datoTil.isAfter(justerOpphørsdato)) {
-                    grunnlag.copy(datoTil = justerOpphørsdato)
+                if (grunnlag.datoTil == null || grunnlag.datoTil.isAfter(opphørsdato)) {
+                    grunnlag.copy(datoTil = justerPeriodeTilOpphørsdato(opphørsdato))
                 } else {
                     grunnlag
                 }

bidrag-beregn-core/src/main/kotlin/no/nav/bidrag/beregn/core/inntekt/service/BeregnInntektService.kt

@@ -6,6 +6,7 @@ import com.fasterxml.jackson.module.kotlin.registerKotlinModule
 import no.nav.bidrag.beregn.core.bo.Periode
 import no.nav.bidrag.beregn.core.util.InntektUtil.erKapitalinntekt
 import no.nav.bidrag.beregn.core.util.InntektUtil.justerKapitalinntekt
+import no.nav.bidrag.beregn.core.util.justerPeriodeTilOpphørsdato
 import no.nav.bidrag.commons.service.sjablon.SjablonProvider
 import no.nav.bidrag.commons.util.secureLogger
 import no.nav.bidrag.domene.enums.inntekt.Inntektstype
@@ -159,7 +160,7 @@ internal class BeregnInntektService {
         }
             .map { grunnlag ->
                 if (grunnlag.periode.til == null || grunnlag.periode.til!!.isAfter(periode.til)) {
-                    grunnlag.copy(periode = grunnlag.periode.copy(til = periode.til))
+                    grunnlag.copy(periode = grunnlag.periode.copy(til = justerPeriodeTilOpphørsdato(periode.til)))
                 } else {
                     grunnlag
                 }

bidrag-beregn-core/src/main/kotlin/no/nav/bidrag/beregn/core/util/PeriodeUtil.kt

@@ -4,6 +4,7 @@ import no.nav.bidrag.beregn.core.bo.Avvik
 import no.nav.bidrag.beregn.core.bo.Periode
 import no.nav.bidrag.domene.enums.beregning.Avvikstype
 import java.time.LocalDate
+import java.time.YearMonth
 
 object PeriodeUtil {
     // Validerer at datoer er gyldige
@@ -122,3 +123,23 @@ object PeriodeUtil {
         return avvikListe
     }
 }
+
+val sluttenAvForrigeMåned get() = LocalDate.now().sluttenAvForrigeMåned
+
+val LocalDate?.sluttenAvForrigeMåned get() = this?.withDayOfMonth(1)?.minusDays(1)
+
+// Antar opphørsdato er til dato og ikke til-og.med (starten av måneden det opphøres)
+fun justerPeriodeTilOpphørsdato(oppphørsdato: LocalDate?) = if (oppphørsdato == null ||
+    oppphørsdato.withDayOfMonth(1).minusDays(1).isAfter(sluttenAvForrigeMåned)
+) {
+    null
+} else {
+    oppphørsdato.sluttenAvForrigeMåned
+}
+fun justerPeriodeTilOpphørsdato(oppphørsdato: YearMonth?) = if (oppphørsdato == null ||
+    oppphørsdato.isAfter(YearMonth.from(sluttenAvForrigeMåned))
+) {
+    null
+} else {
+    oppphørsdato
+}

bidrag-boforhold/src/main/kotlin/no/nav/bidrag/boforhold/utils/BoforholdPeriodeUtils.kt

@@ -1,19 +1,19 @@
 package no.nav.bidrag.boforhold.utils
 
+import no.nav.bidrag.beregn.core.util.justerPeriodeTilOpphørsdato
 import no.nav.bidrag.boforhold.dto.BoforholdResponseV2
 import no.nav.bidrag.boforhold.dto.Bostatus
 import java.time.LocalDate
 
 fun List<BoforholdResponseV2>.justerBoforholdPerioderForOpphørsdato(opphørsdato: LocalDate?): List<BoforholdResponseV2> {
     if (opphørsdato == null) return this
     // Antar at opphørsdato er måneden perioden skal opphøre
-    val justerOpphørsdato = opphørsdato.withDayOfMonth(1).minusDays(1)
     return filter {
         it.periodeFom.isBefore(opphørsdato)
     }
         .map { grunnlag ->
-            if (grunnlag.periodeTom == null || grunnlag.periodeTom.isAfter(justerOpphørsdato)) {
-                grunnlag.copy(periodeTom = justerOpphørsdato)
+            if (grunnlag.periodeTom == null || grunnlag.periodeTom.isAfter(opphørsdato)) {
+                grunnlag.copy(periodeTom = justerPeriodeTilOpphørsdato(opphørsdato))
             } else {
                 grunnlag
             }
@@ -22,13 +22,12 @@ fun List<BoforholdResponseV2>.justerBoforholdPerioderForOpphørsdato(opphørsdat
 fun List<Bostatus>.justerBostatusPerioderForOpphørsdato(opphørsdato: LocalDate?): List<Bostatus> {
     if (opphørsdato == null) return this
     // Antar at opphørsdato er måneden perioden skal opphøre
-    val justerOpphørsdato = opphørsdato.withDayOfMonth(1).minusDays(1)
     return filter {
-        it.periodeFom!!.isBefore(justerOpphørsdato)
+        it.periodeFom!!.isBefore(opphørsdato)
     }
         .map { grunnlag ->
-            if (grunnlag.periodeTom == null || grunnlag.periodeTom.isAfter(justerOpphørsdato)) {
-                grunnlag.copy(periodeTom = justerOpphørsdato)
+            if (grunnlag.periodeTom == null || grunnlag.periodeTom.isAfter(opphørsdato)) {
+                grunnlag.copy(periodeTom = justerPeriodeTilOpphørsdato(opphørsdato))
             } else {
                 grunnlag
             }