Lagre inntekt (WiP, squash før merge)

Co-authored-by: John Martin Lindseth <[email protected]>

Author: vskjefst

dp-inntekt-api/src/main/kotlin/no/nav/dagpenger/inntekt/mapping/MapToInntektFrontend.kt

@@ -115,6 +115,7 @@ data class InntektMaaned(
     val opptjeningsperiode: Periode? = null,
     val skattemessigBosattLand: String? = null,
     val utbetaltIMaaned: YearMonth,
+    val opplysningspliktig: Aktoer? = null,
     val inntektsinnsender: Aktoer? = null,
     val virksomhet: Aktoer? = null,
     val inntektsmottaker: Aktoer? = null,

dp-inntekt-api/src/main/kotlin/no/nav/dagpenger/inntekt/v1/models/InntekterDto.kt

@@ -28,6 +28,25 @@ fun InntekterDto.mapToStoredInntekt(inntektId: String): StoredInntekt =
 private fun mapToInntektkomponentResponse(inntekterDto: InntekterDto): InntektkomponentResponse {
     val arbeidsInntektMaaneder: MutableList<ArbeidsInntektMaaned> = mutableListOf()
 
+    /*
+    Dette må mappes om fra gruppering på virksomhet, til gruppering på aarMaaned (dvs hvert aarMaaned inneholder
+    inntekt for alle virksomhetene som har utbetalt inntekt for gitt aarMaaned). Strukturen som kommer fra frontend er
+    gruppert på virksomhet, og alle utbetalte inntekter for alle aarMaaned ligger inne i hver virksomhet.
+
+    Vi mistenker også at Avvik i datastrukturen som sendes til frontend (og som frontend da sender tilbake til backend),
+    ikke inneholder all informasjon og/eller ligger på feil sted i strukturen.
+
+    Løsningsforslag:
+
+    Lag en Map<YearMonth, List<Inntekt>>(), og map InntektMaaned-objekter til Inntekt-objekter, og legg i riktig
+    "bucket" i mappen -> opprett et ArbeidsInntektMaaned-objekt per key i map (key == maanedAar,
+    liste i value == arbeidsInntektInformasjon.
+
+    Hvis vi blir skikkelig tøffe: https://kotlinlang.org/docs/collection-grouping.html
+
+    I tillegg må vi se på mapping av avvik når vi har fått det på riktig plass i InntektDto.
+     */
+
     inntekterDto.virksomheter.forEach { virksomhet ->
         arbeidsInntektMaaneder.addAll(
             virksomhet.inntekter?.map {
@@ -48,12 +67,10 @@ private fun mapToInntektkomponentResponse(inntekterDto: InntekterDto): Inntektko
                                 inntektMaaned.opptjeningsperiode,
                                 inntektMaaned.skattemessigBosattLand,
                                 inntektMaaned.utbetaltIMaaned,
-                                // TODO: Er det mottaker som skal brukes her?
-                                mapToAktoerNaturligIdent(inntekterDto.mottaker.pnr),
-                                // TODO: Er det virksomhet som skal brukes her?
-                                mapToAktoerOrganisasjon(virksomhet),
-                                mapToAktoerOrganisasjon(virksomhet),
-                                mapToAktoerNaturligIdent(inntekterDto.mottaker.pnr),
+                                inntektMaaned.opplysningspliktig,
+                                inntektMaaned.inntektsinnsender,
+                                inntektMaaned.virksomhet,
+                                inntektMaaned.inntektsmottaker,
                                 inntektMaaned.inngaarIGrunnlagForTrekk,
                                 inntektMaaned.utloeserArbeidsgiveravgift,
                                 inntektMaaned.informasjonsstatus,
@@ -73,7 +90,5 @@ private fun mapToInntektkomponentResponse(inntekterDto: InntekterDto): Inntektko
     )
 }
 
-private fun mapToAktoerOrganisasjon(virksomhet: Virksomhet): Aktoer = Aktoer(AktoerType.ORGANISASJON, virksomhet.virksomhetsnummer)
-
 private fun mapToAktoerNaturligIdent(fnr: String?): Aktoer =
     Aktoer(AktoerType.NATURLIG_IDENT, fnr ?: throw IllegalArgumentException("Fødselsenummer mangler"))

dp-inntekt-api/src/test/kotlin/no/nav/dagpenger/inntekt/mapping/MapFromGUIInntektTest.kt

@@ -0,0 +1,44 @@
+package no.nav.dagpenger.inntekt.v1.models
+
+import com.fasterxml.jackson.module.kotlin.readValue
+import de.huxhorn.sulky.ulid.ULID
+import io.kotest.matchers.collections.shouldBeEmpty
+import io.kotest.matchers.collections.shouldHaveSize
+import io.kotest.matchers.nulls.shouldNotBeNull
+import io.kotest.matchers.shouldBe
+import no.nav.dagpenger.inntekt.inntektskomponenten.v1.AktoerType.NATURLIG_IDENT
+import no.nav.dagpenger.inntekt.serder.jacksonObjectMapper
+import org.junit.jupiter.api.Test
+import java.time.YearMonth
+
+class InntekterDtoTest {
+    @Test
+    fun `mapToStoredInntekt mapper til forventet resultat`() {
+        val inntektId = ULID().nextULID()
+        val inntekterDto =
+            jacksonObjectMapper.readValue<InntekterDto>(
+                this::class.java.getResource("/test-data/expected-uklassifisert-post-body.json")?.readText()!!,
+            )
+
+        val storedInntekt =
+            inntekterDto.mapToStoredInntekt(
+                inntektId = inntektId,
+            )
+
+        storedInntekt.manueltRedigert shouldBe true
+        storedInntekt.timestamp.shouldNotBeNull()
+        storedInntekt.inntektId.id shouldBe inntektId
+        storedInntekt.inntekt.ident.aktoerType shouldBe NATURLIG_IDENT
+        storedInntekt.inntekt.ident.identifikator shouldBe inntekterDto.mottaker.pnr
+        storedInntekt.inntekt.arbeidsInntektMaaned?.shouldHaveSize(2)
+        storedInntekt.inntekt.arbeidsInntektMaaned?.first().let {
+            it?.aarMaaned shouldBe YearMonth.of(2023, 1)
+            it?.avvikListe.shouldBeEmpty()
+            it?.arbeidsInntektInformasjon?.inntektListe?.shouldHaveSize(2)
+        }
+        storedInntekt.inntekt.arbeidsInntektMaaned?.get(1).let {
+            it?.aarMaaned shouldBe YearMonth.of(2024, 1)
+            it?.avvikListe.shouldBeEmpty()
+        }
+    }
+}

dp-inntekt-api/src/test/kotlin/no/nav/dagpenger/inntekt/v1/models/InntekterDtoTest.kt

@@ -5,7 +5,7 @@
       "virksomhetsnavn": "Test Virksomhet",
       "periode": {
         "fra": "2023-01",
-        "til": "2023-12"
+        "til": "2024-12"
       },
       "inntekter": [
         {
@@ -20,6 +20,7 @@
           "opptjeningsperiode": null,
           "skattemessigBosattLand": "NO",
           "utbetaltIMaaned": "2023-01",
+          "opplysningspliktig": null,
           "inntektsinnsender": null,
           "virksomhet": null,
           "inntektsmottaker": null,
@@ -31,9 +32,34 @@
           "redigert": true,
           "begrunnelse": "Standard",
           "aarMaaned": "2023-01"
+        },
+        {
+          "belop": 10000.00,
+          "fordel": "kontantytelse",
+          "beskrivelse": "bil",
+          "inntektskilde": "A-ordningen",
+          "inntektsstatus": "LoependeInnrapportert",
+          "inntektsperiodetype": "Maaned",
+          "leveringstidspunkt": "2024-01",
+          "opptjeningsland": "SE",
+          "opptjeningsperiode": null,
+          "skattemessigBosattLand": "NO",
+          "utbetaltIMaaned": "2024-01",
+          "opplysningspliktig": null,
+          "inntektsinnsender": null,
+          "virksomhet": null,
+          "inntektsmottaker": null,
+          "inngaarIGrunnlagForTrekk": true,
+          "utloeserArbeidsgiveravgift": true,
+          "informasjonsstatus": "InngaarAlltid",
+          "inntektType": "NAERINGSINNTEKT",
+          "tilleggsinformasjon": null,
+          "redigert": true,
+          "begrunnelse": "Ikke Standard",
+          "aarMaaned": "2024-01"
         }
       ],
-      "totalBeløp": 50000.00,
+      "totalBeløp": 60000.00,
       "avvikListe": []
     }
   ],