PEK-1081 Use brukFremtidigInntekt flag

src/main/kotlin/no/nav/pensjon/simulator/afp/folketrygdberegnet/api/direct/acl/v0/spec/TpoFolketrygdberegnetAfpSpecMapperV0.kt

@@ -37,7 +37,8 @@ object TpoFolketrygdberegnetAfpSpecMapperV0 {
             foedselAar = 0,
             utlandAntallAar = source.utenlandsopphold ?: 0,
             utlandPeriodeListe = mutableListOf(),
-            fremtidigInntektListe = null, // NB: not mutableListOf()
+            fremtidigInntektListe = mutableListOf(),
+            brukFremtidigInntekt = false,
             inntektOver1GAntallAar = 0, // used for anonym only
             flyktning = null,
             epsHarInntektOver2G = source.eps2G == true,

src/main/kotlin/no/nav/pensjon/simulator/afp/folketrygdberegnet/api/viapen/acl/v1/spec/FolketrygdberegnetAfpSpecMapperV1.kt

@@ -37,7 +37,8 @@ object FolketrygdberegnetAfpSpecMapperV1 {
             foedselAar = 0,
             utlandAntallAar = source.utenlandsopphold ?: 0,
             utlandPeriodeListe = mutableListOf(),
-            fremtidigInntektListe = null, // NB: not mutableListOf()
+            fremtidigInntektListe = mutableListOf(),
+            brukFremtidigInntekt = false,
             inntektOver1GAntallAar = 0, // used for anonym only
             flyktning = null,
             epsHarInntektOver2G = source.eps2G == true,

src/main/kotlin/no/nav/pensjon/simulator/afp/offentlig/livsvarig/LivsvarigOffentligAfpService.kt

@@ -24,7 +24,8 @@ class LivsvarigOffentligAfpService(
         pid: Pid,
         foedselsdato: LocalDate,
         forventetAarligInntektBeloep: Int,
-        fremtidigeInntekter: List<FremtidigInntekt>?,
+        fremtidigeInntekter: List<FremtidigInntekt>,
+        brukFremtidigInntekt: Boolean,
         virkningDato: LocalDate,
     ): LivsvarigOffentligAfpResult? {
         if (valid(foedselsdato.year, virkningDato.year).not()) return null
@@ -33,8 +34,8 @@ class LivsvarigOffentligAfpService(
         val til: LocalDate = sisteAarMedAfpOpptjeningInntekt(foedselsdato)
 
         val fremtidigInntektListe: List<Inntekt> =
-            fremtidigeInntekter?.map { Inntekt(it.aarligInntektBeloep, it.fom) }
-                ?: forventedeInntekter(fom, til, forventetAarligInntektBeloep)
+            if (brukFremtidigInntekt) fremtidigeInntekter.map { Inntekt(it.aarligInntektBeloep, it.fom) }
+            else forventedeInntekter(fom, til, forventetAarligInntektBeloep)
 
         return client.simuler(
             LivsvarigOffentligAfpSpec(

src/main/kotlin/no/nav/pensjon/simulator/alderspensjon/AlderspensjonSpecMapper.kt

@@ -41,6 +41,7 @@ object AlderspensjonSpecMapper {
             utlandAntallAar = source.antallAarUtenlandsEtter16,
             utlandPeriodeListe = mutableListOf(), // utenlandsopphold is in V4 specified by utlandAntallAar
             fremtidigInntektListe = source.fremtidigInntektListe.map(::fremtidigInntekt).toMutableList(),
+            brukFremtidigInntekt = true,
             inntektOver1GAntallAar = 0,
             flyktning = false,
             epsHarInntektOver2G = source.epsHarInntektOver2G,

src/main/kotlin/no/nav/pensjon/simulator/alderspensjon/api/nav/direct/acl/v3/spec/NavSimuleringSpecMapperV3.kt

@@ -55,7 +55,8 @@ object NavSimuleringSpecMapperV3 {
             flyktning = false,
             utlandPeriodeListe = utlandPeriodeListe.toMutableList(),
             // Inntekt angis før/etter gradert/helt uttak istedenfor via liste:
-            fremtidigInntektListe = null, // NB: Must be null (not empty list)
+            fremtidigInntektListe = mutableListOf(),
+            brukFremtidigInntekt = false,
             rettTilOffentligAfpFom = null,
             afpOrdning = null,
             afpInntektMaanedFoerUttak = null,

src/main/kotlin/no/nav/pensjon/simulator/alderspensjon/api/nav/direct/anonym/acl/v1/spec/AnonymSimuleringSpecMapperV1.kt

@@ -34,7 +34,8 @@ object AnonymSimuleringSpecMapperV1 {
             simulerForTp = false,
             flyktning = false,
             utlandPeriodeListe = mutableListOf(),
-            fremtidigInntektListe = null, // NB: not mutableListOf()
+            fremtidigInntektListe = mutableListOf(),
+            brukFremtidigInntekt = false,
             rettTilOffentligAfpFom = null,
             afpOrdning = null,
             afpInntektMaanedFoerUttak = null,

src/main/kotlin/no/nav/pensjon/simulator/alderspensjon/api/nav/viapen/acl/v2/spec/NavSimuleringSpecMapperV2.kt

@@ -37,7 +37,8 @@ object NavSimuleringSpecMapperV2 {
             foedselAar = source.fodselsar ?: 0,
             utlandAntallAar = source.utenlandsopphold ?: 0,
             utlandPeriodeListe = source.utenlandsperiodeForSimuleringList.map(::utlandPeriode).toMutableList(),
-            fremtidigInntektListe = null, // not used in PSELV; NB: not mutableListOf()
+            fremtidigInntektListe = mutableListOf(),
+            brukFremtidigInntekt = false,
             inntektOver1GAntallAar = 0, // used for anonym only
             flyktning = source.flyktning,
             epsHarInntektOver2G = source.eps2G == true,

src/main/kotlin/no/nav/pensjon/simulator/alderspensjon/api/tpo/viapen/acl/v1/TpoSimuleringSpecMapperV1.kt

@@ -28,7 +28,8 @@ object TpoSimuleringSpecMapperV1 {
             foedselAar = 0,
             utlandAntallAar = source.utenlandsopphold ?: 0,
             utlandPeriodeListe = mutableListOf(),
-            fremtidigInntektListe = null, // V2, V3 only; NB: not mutableListOf()
+            fremtidigInntektListe = mutableListOf(), // V2, V3 only
+            brukFremtidigInntekt = false,
             inntektOver1GAntallAar = 0,
             flyktning = null,
             epsHarInntektOver2G = source.eps2G == true,

src/main/kotlin/no/nav/pensjon/simulator/alderspensjon/api/tpo/viapen/acl/v2/TpoSimuleringSpecMapperV2.kt

@@ -31,6 +31,7 @@ object TpoSimuleringSpecMapperV2 {
             utlandPeriodeListe = mutableListOf(),
             fremtidigInntektListe = source.fremtidigInntektList.orEmpty()
                 .map(TpoSimuleringSpecMapperV2::inntekt).toMutableList(), // V2, V3 only
+            brukFremtidigInntekt = true,
             inntektOver1GAntallAar = 0,
             flyktning = null,
             epsHarInntektOver2G = source.eps2G == true,

src/main/kotlin/no/nav/pensjon/simulator/alderspensjon/api/tpo/viapen/acl/v3/TpoSimuleringSpecMapperV3.kt

@@ -30,6 +30,7 @@ object TpoSimuleringSpecMapperV3 {
             utlandAntallAar = source.utenlandsopphold ?: 0,
             utlandPeriodeListe = mutableListOf(),
             fremtidigInntektListe = source.fremtidigInntektList.orEmpty().map(::inntekt).toMutableList(),
+            brukFremtidigInntekt = true,
             inntektOver1GAntallAar = 0,
             flyktning = null,
             epsHarInntektOver2G = source.eps2G == true,

src/main/kotlin/no/nav/pensjon/simulator/alderspensjon/spec/SimuleringSpecValidator.kt

@@ -7,7 +7,7 @@ import no.nav.pensjon.simulator.tech.web.BadRequestException
 object SimuleringSpecValidator {
 
     fun validate(spec: SimuleringSpec){
-        spec.fremtidigInntektListe?.let(::validateInntekt)
+        validateInntekt(spec.fremtidigInntektListe)
         validateUttak(spec)
     }
 
@@ -34,9 +34,5 @@ object SimuleringSpecValidator {
             if (spec.foersteUttakDato.isAfter(spec.heltUttakDato))
             throw BadRequestException("Andre uttak (100 %) starter ikke etter første uttak (gradert)")
         }
-
-      //  if (spec.heltUttakDato?.let { spec.foersteUttakDato.isAfter(it).not() } == true) {
-        //    throw BadRequestException("Andre uttak (100 %) starter ikke etter første uttak (gradert)")
-       // }
     }
 }

src/main/kotlin/no/nav/pensjon/simulator/beholdning/FolketrygdBeholdningService.kt

@@ -108,6 +108,7 @@ class FolketrygdBeholdningService(
                 epsHarPensjon = beholdningSpec.epsHarPensjon,
                 epsHarInntektOver2G = beholdningSpec.epsHarInntektOver2G,
                 fremtidigInntektListe = beholdningSpec.fremtidigInntektListe.map(::fremtidigInntekt).toMutableList(),
+                brukFremtidigInntekt = true,
                 type = if (erFoerstegangsuttak) SimuleringType.ALDER else SimuleringType.ENDR_ALDER, // inkluderAfpPrivat = false
                 foedselAar = 0, // only for anonym
                 forventetInntektBeloep = 0, // inntekt instead given by fremtidigInntektListe

src/main/kotlin/no/nav/pensjon/simulator/core/SimulatorCore.kt

@@ -185,6 +185,7 @@ class SimulatorCore(
                             foedselsdato = it,
                             forventetAarligInntektBeloep = spec.forventetInntektBeloep,
                             fremtidigeInntekter = spec.fremtidigInntektListe,
+                            brukFremtidigInntekt = spec.brukFremtidigInntekt,
                             virkningDato = spec.rettTilOffentligAfpFom ?: spec.foersteUttakDato!!
                         )
                     }

src/main/kotlin/no/nav/pensjon/simulator/core/krav/KravhodeCreator.kt

@@ -310,10 +310,9 @@ class KravhodeCreator(
         }
 
         val persongrunnlag = kravhode.hentPersongrunnlagForSoker()
-        val brukFremtidigInntekt = spec.fremtidigInntektListe != null // NB: true if empty list
         val inntektListe: MutableList<Inntekt>
 
-        if (brukFremtidigInntekt) {
+        if (spec.brukFremtidigInntekt) {
             val gjeldendeAar = SISTE_GYLDIGE_OPPTJENING_AAR + 1
             val sisteOpptjeningAar = MAX_OPPTJENING_ALDER + foedselAar(person, spec)
             val fom: LocalDate = foersteDag(gjeldendeAar)
@@ -708,7 +707,7 @@ class KravhodeCreator(
             sisteOpptjeningAar: Int,
             fom: LocalDate
         ): List<Inntektsgrunnlag> {
-            val fremtidigInntektListe = spec.fremtidigInntektListe ?: mutableListOf()
+            val fremtidigInntektListe = spec.fremtidigInntektListe
 
             // NB: The original fremtidigInntektListe is here modified (as is done in PEN):
             if (fremtidigInntektListe.isEmpty() || doesNotHaveFremtidigInntektBeforeFom(spec, fom)) {
@@ -772,7 +771,7 @@ class KravhodeCreator(
             }
 
         private fun doesNotHaveFremtidigInntektBeforeFom(spec: SimuleringSpec, fom: LocalDate) =
-            spec.fremtidigInntektListe.orEmpty().none { isBeforeByDay(it.fom, fom, true) }
+            spec.fremtidigInntektListe.none { isBeforeByDay(it.fom, fom, true) }
 
         private fun foedselsdato(person: PenPerson?, spec: SimuleringSpec): LocalDate =
             person?.foedselsdato ?: spec.foedselDato ?: foersteDag(spec.foedselAar)

src/main/kotlin/no/nav/pensjon/simulator/core/spec/SimuleringSpec.kt

@@ -33,7 +33,8 @@ data class SimuleringSpec(
     val foedselAar: Int,
     val utlandAntallAar: Int, // PEN: SimuleringEtter2011.utenlandsopphold
     val utlandPeriodeListe: MutableList<UtlandPeriode>,
-    val fremtidigInntektListe: MutableList<FremtidigInntekt>?, // must be nullable
+    val fremtidigInntektListe: MutableList<FremtidigInntekt>,
+    val brukFremtidigInntekt: Boolean,
     val inntektOver1GAntallAar: Int,
     val flyktning: Boolean?,
     val epsHarInntektOver2G: Boolean,
@@ -130,6 +131,7 @@ data class SimuleringSpec(
             utlandAntallAar = utlandAntallAar,
             utlandPeriodeListe = utlandPeriodeListe,
             fremtidigInntektListe = fremtidigInntektListe,
+            brukFremtidigInntekt = brukFremtidigInntekt,
             inntektOver1GAntallAar = inntektOver1GAntallAar,
             flyktning = flyktning,
             epsHarInntektOver2G = epsHarInntektOver2G,

src/main/kotlin/no/nav/pensjon/simulator/core/spec/SimuleringSpecUtil.kt

@@ -92,6 +92,7 @@ object SimuleringSpecUtil {
             utlandAntallAar = source.utlandAntallAar,
             utlandPeriodeListe = source.utlandPeriodeListe,
             fremtidigInntektListe = source.fremtidigInntektListe,
+            brukFremtidigInntekt = source.brukFremtidigInntekt,
             inntektOver1GAntallAar = source.inntektOver1GAntallAar,
             flyktning = source.flyktning,
             epsHarInntektOver2G = source.epsHarInntektOver2G,
@@ -127,6 +128,7 @@ object SimuleringSpecUtil {
             utlandAntallAar = source.utlandAntallAar,
             utlandPeriodeListe = source.utlandPeriodeListe,
             fremtidigInntektListe = source.fremtidigInntektListe,
+            brukFremtidigInntekt = source.brukFremtidigInntekt,
             inntektOver1GAntallAar = source.inntektOver1GAntallAar,
             flyktning = source.flyktning,
             epsHarInntektOver2G = source.epsHarInntektOver2G,

src/main/kotlin/no/nav/pensjon/simulator/uttak/api/acl/UttakSpecMapperV1.kt

@@ -35,6 +35,7 @@ object UttakSpecMapperV1 {
             utlandAntallAar = 0, // not taken into account
             utlandPeriodeListe = mutableListOf(), // not taken into account
             fremtidigInntektListe = source.fremtidigInntektListe.orEmpty().map(::inntekt).toMutableList(),
+            brukFremtidigInntekt = true,
             inntektOver1GAntallAar = 0,
             flyktning = false,
             epsHarInntektOver2G = false,

src/test/kotlin/no/nav/pensjon/simulator/afp/offentlig/livsvarig/LivsvarigOffentligAfpServiceTest.kt

@@ -25,7 +25,8 @@ class LivsvarigOffentligAfpServiceTest : FunSpec({
             pid,
             foedselsdato = LocalDate.of(1962, 12, 31),
             forventetAarligInntektBeloep = 0,
-            fremtidigeInntekter = null,
+            fremtidigeInntekter = emptyList(),
+            brukFremtidigInntekt = false,
             virkningDato
         ) shouldBe null
 
@@ -37,7 +38,8 @@ class LivsvarigOffentligAfpServiceTest : FunSpec({
             pid,
             foedselsdato = LocalDate.of(1970, 1, 1),
             forventetAarligInntektBeloep = 0,
-            fremtidigeInntekter = null,
+            fremtidigeInntekter = emptyList(),
+            brukFremtidigInntekt = false,
             virkningDato = LocalDate.of(2031, 12, 31)
         ) shouldBe null
 
@@ -49,7 +51,8 @@ class LivsvarigOffentligAfpServiceTest : FunSpec({
             pid,
             foedselsdato,
             forventetAarligInntektBeloep = 123000,
-            fremtidigeInntekter = null,
+            fremtidigeInntekter = emptyList(),
+            brukFremtidigInntekt = false,
             virkningDato
         )
 
@@ -72,6 +75,7 @@ class LivsvarigOffentligAfpServiceTest : FunSpec({
             foedselsdato,
             forventetAarligInntektBeloep = 123000,
             fremtidigeInntekter = emptyList(),
+            brukFremtidigInntekt = true,
             virkningDato
         )
 
@@ -94,6 +98,7 @@ class LivsvarigOffentligAfpServiceTest : FunSpec({
                 FremtidigInntekt(aarligInntektBeloep = 234000, fom = LocalDate.of(2024, 2, 1)),
                 FremtidigInntekt(aarligInntektBeloep = 123000, fom = LocalDate.of(2025, 3, 1))
             ),
+            brukFremtidigInntekt = true,
             virkningDato
         )
 

src/test/kotlin/no/nav/pensjon/simulator/alderspensjon/AlderspensjonSpecMapperTest.kt

@@ -79,6 +79,7 @@ private fun simuleringSpec(type: SimuleringType, livsvarigOffentligAfpRettFom: L
                 fom = LocalDate.of(2029, 5, 6)
             )
         ),
+        brukFremtidigInntekt = true,
         inntektOver1GAntallAar = 0,
         flyktning = false,
         epsHarInntektOver2G = false,

src/test/kotlin/no/nav/pensjon/simulator/alderspensjon/api/nav/direct/acl/v3/spec/NavSimuleringSpecMapperV3Test.kt

@@ -78,7 +78,8 @@ class NavSimuleringSpecMapperV3Test : FunSpec({
                     arbeidet = true
                 )
             ),
-            fremtidigInntektListe = null, // null in this context
+            fremtidigInntektListe = mutableListOf(), // empty in this context
+            brukFremtidigInntekt = false,
             inntektOver1GAntallAar = 0,
             flyktning = false,
             epsHarInntektOver2G = false,

src/test/kotlin/no/nav/pensjon/simulator/alderspensjon/api/nav/viapen/acl/v2/spec/NavSimuleringSpecMapperV2Test.kt

@@ -104,7 +104,8 @@ class NavSimuleringSpecMapperV2Test : FunSpec({
                             arbeidet = true
                         )
                     ),
-                    fremtidigInntektListe = null, // ikke brukt i PSELV
+                    fremtidigInntektListe = mutableListOf(), // ikke brukt i PSELV
+                    brukFremtidigInntekt = false,
                     inntektOver1GAntallAar = 0, // kun for anonym simulering
                     flyktning = false,
                     epsHarInntektOver2G = true,

src/test/kotlin/no/nav/pensjon/simulator/alderspensjon/spec/SimuleringSpecValidatorTest.kt

@@ -0,0 +1,65 @@
+package no.nav.pensjon.simulator.alderspensjon.spec
+
+import io.kotest.assertions.throwables.shouldThrow
+import io.kotest.core.spec.style.FunSpec
+import io.kotest.matchers.shouldBe
+import no.nav.pensjon.simulator.core.krav.FremtidigInntekt
+import no.nav.pensjon.simulator.tech.web.BadRequestException
+import no.nav.pensjon.simulator.testutil.TestObjects.simuleringSpec
+import java.time.LocalDate
+
+class SimuleringSpecValidatorTest : FunSpec({
+
+    test("validate should throw exception if: En fremtidig inntekt har f.o.m.-dato som ikke er den 1. i måneden") {
+        shouldThrow<BadRequestException> {
+            SimuleringSpecValidator.validate(
+                simuleringSpec(
+                    inntektSpecListe = listOf(
+                        FremtidigInntekt(
+                            aarligInntektBeloep = 1,
+                            fom = LocalDate.of(2025, 1, 2) // ikke den 1. i måneden
+                        )
+                    )
+                )
+            )
+        }.message shouldBe "En fremtidig inntekt har f.o.m.-dato som ikke er den 1. i måneden"
+    }
+
+    test("validate should throw exception if: To fremtidige inntekter har samme f.o.m.-dato") {
+        shouldThrow<BadRequestException> {
+            SimuleringSpecValidator.validate(
+                simuleringSpec(
+                    inntektSpecListe = listOf(
+                        FremtidigInntekt(
+                            aarligInntektBeloep = 1,
+                            fom = LocalDate.of(2025, 1, 1)
+                        ),
+                        FremtidigInntekt(
+                            aarligInntektBeloep = 2,
+                            fom = LocalDate.of(2025, 1, 1) // samme f.o.m.-dato
+                        )
+                    )
+                )
+            )
+        }.message shouldBe "To fremtidige inntekter har samme f.o.m.-dato"
+    }
+
+    test("validate should throw exception if: Dato for første uttak mangler") {
+        shouldThrow<BadRequestException> {
+            SimuleringSpecValidator.validate(
+                simuleringSpec(foersteUttakDato = null)
+            )
+        }.message shouldBe "Dato for første uttak mangler"
+    }
+
+    test("validate should throw exception if: Andre uttak (100 %) starter ikke etter første uttak (gradert)") {
+        shouldThrow<BadRequestException> {
+            SimuleringSpecValidator.validate(
+                simuleringSpec(
+                    foersteUttakDato = LocalDate.of(2032, 6, 2)
+                    // heltUttakDato = LocalDate.of(2032, 6, 1) // starter ikke etter første uttak
+                )
+            )
+        }.message shouldBe "Andre uttak (100 %) starter ikke etter første uttak (gradert)"
+    }
+})

src/test/kotlin/no/nav/pensjon/simulator/beholdning/FolketrygdBeholdningServiceTest.kt

@@ -132,6 +132,7 @@ private fun simuleringSpec() =
                 fom = LocalDate.of(2027, 8, 1)
             )
         ),
+        brukFremtidigInntekt = true,
         type = SimuleringType.ALDER,
         foedselAar = 0, // only for anonym
         forventetInntektBeloep = 0, // inntekt instead given by fremtidigInntektListe

src/test/kotlin/no/nav/pensjon/simulator/core/krav/KravUtilTest.kt

@@ -90,7 +90,8 @@ private fun simuleringSpec(utlandPeriodeListe: MutableList<UtlandPeriode>) =
         foedselAar = 0,
         utlandAntallAar = 0,
         utlandPeriodeListe = utlandPeriodeListe,
-        fremtidigInntektListe = null,
+        fremtidigInntektListe = mutableListOf(),
+        brukFremtidigInntekt = false,
         inntektOver1GAntallAar = 0,
         flyktning = null,
         epsHarInntektOver2G = false,

src/test/kotlin/no/nav/pensjon/simulator/testutil/TestObjects.kt

@@ -26,12 +26,13 @@ object TestObjects {
         type: SimuleringType = SimuleringType.ALDER_M_AFP_PRIVAT,
         sivilstatus: SivilstatusType = SivilstatusType.UGIF,
         epsHarPensjon: Boolean = false,
+        foersteUttakDato: LocalDate? = LocalDate.of(2029, 1, 1),
         inntektSpecListe: List<FremtidigInntekt> = emptyList()
     ) = SimuleringSpec(
         type,
         sivilstatus,
         epsHarPensjon,
-        foersteUttakDato = LocalDate.of(2029, 1, 1),
+        foersteUttakDato,
         heltUttakDato = LocalDate.of(2032, 6, 1),
         pid = pid,
         foedselDato = null,
@@ -67,6 +68,7 @@ object TestObjects {
             )
         ),
         fremtidigInntektListe = inntektSpecListe.toMutableList(),
+        brukFremtidigInntekt = true,
         inntektOver1GAntallAar = 0,
         flyktning = false,
         epsHarInntektOver2G = true,