Begynn på test av post-endepunkt

Co-authored-by: Arselan Sultani <[email protected]>
Co-authored-by: Vegard Skjefstad <[email protected]>

Author: 3 people

dp-inntekt-api/src/main/kotlin/no/nav/dagpenger/inntekt/db/InntektStore.kt

@@ -56,7 +56,7 @@ data class StoreInntektCommand(
 
 data class InntektPersonMapping(
     val inntektId: InntektId,
-    val aktoerId: String,
+    val aktørId: String,
     val fnr: String? = null,
     val kontekstId: String,
     val beregningsdato: LocalDate,

dp-inntekt-api/src/main/kotlin/no/nav/dagpenger/inntekt/db/PostgresInntektStore.kt

@@ -105,7 +105,7 @@ internal class PostgresInntektStore(
                 ).map { row ->
                     InntektPersonMapping(
                         inntektId = InntektId(row.string("inntektid")),
-                        aktoerId = row.string("aktorid"),
+                        aktørId = row.string("aktorid"),
                         fnr = row.string("fnr"),
                         kontekstId = row.string("kontekstid"),
                         beregningsdato = row.zonedDateTime("beregningsdato").toLocalDate(),

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

@@ -21,7 +21,7 @@ data class GUIInntekt(
 )
 
 data class Inntektsmottaker(
-    val fnr: String?,
+    val pnr: String?,
     val navn: String?,
 )
 

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

@@ -70,11 +70,11 @@ private fun mapToInntektkomponentResponse(inntekterDto: InntekterDto): Inntektko
                                 inntektMaaned.skattemessigBosattLand,
                                 inntektMaaned.utbetaltIMaaned,
                                 // TODO: Er det mottaker som skal brukes her?
-                                mapToAktoerNaturligIdent(inntekterDto.mottaker.fnr),
+                                mapToAktoerNaturligIdent(inntekterDto.mottaker.pnr),
                                 // TODO: Er det virksomhet som skal brukes her?
                                 mapToAktoerOrganisasjon(virksomhet),
                                 mapToAktoerOrganisasjon(virksomhet),
-                                mapToAktoerNaturligIdent(inntekterDto.mottaker.fnr),
+                                mapToAktoerNaturligIdent(inntekterDto.mottaker.pnr),
                                 inntektMaaned.inngaarIGrunnlagForTrekk,
                                 inntektMaaned.utloeserArbeidsgiveravgift,
                                 inntektMaaned.informasjonsstatus,
@@ -90,7 +90,7 @@ private fun mapToInntektkomponentResponse(inntekterDto: InntekterDto): Inntektko
 
     return InntektkomponentResponse(
         arbeidsInntektMaaneder,
-        mapToAktoerNaturligIdent(inntekterDto.mottaker.fnr),
+        mapToAktoerNaturligIdent(inntekterDto.mottaker.pnr),
     )
 }
 

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

@@ -175,14 +175,13 @@ fun Route.uklassifisertInntekt(
                         inntekterDto = call.receive<InntekterDto>(),
                         inntektId = inntektId,
                     ).let {
-                        val person = personOppslag.hentPerson(it.inntekt.ident.identifikator)
                         val inntektPersonMapping = inntektStore.getInntektPersonMapping(inntektId)
                         inntektStore.storeInntekt(
                             StoreInntektCommand(
                                 inntektparametre =
                                     Inntektparametre(
-                                        aktørId = person.aktørId,
-                                        fødselsnummer = person.fødselsnummer,
+                                        aktørId = inntektPersonMapping.aktørId,
+                                        fødselsnummer = it.inntekt.ident.identifikator,
                                         regelkontekst = RegelKontekst(inntektPersonMapping.kontekstId, inntektPersonMapping.kontekstType),
                                         beregningsdato = inntektPersonMapping.beregningsdato,
                                     ),
@@ -195,7 +194,7 @@ fun Route.uklassifisertInntekt(
                             ),
                         )
                     }.let {
-                        call.respond(HttpStatusCode.OK, it.inntektId)
+                        call.respond(HttpStatusCode.OK, it.inntektId.id)
                     }.also {
                         inntektKorrigeringCounter.inc()
                     }

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

@@ -166,7 +166,7 @@ val inntektkomponentResponse =
 
 val mottaker =
     Inntektsmottaker(
-        fnr = "2044350291600",
+        pnr = "2044350291600",
         navn = "Ola Nordmann",
     )
 

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

@@ -29,6 +29,8 @@ import no.nav.security.mock.oauth2.MockOAuth2Server
 @Suppress("ktlint:standard:function-naming")
 internal object TestApplication {
     private const val ISSUER_ID = "default"
+    const val TEST_OAUTH_USER = "user"
+
     val mockOAuth2Server: MockOAuth2Server by lazy {
         MockOAuth2Server().also {
             it.start()
@@ -37,7 +39,7 @@ internal object TestApplication {
     val testOAuthToken: String by lazy {
         mockOAuth2Server.issueToken(
             issuerId = ISSUER_ID,
-            subject = "user",
+            subject = TEST_OAUTH_USER,
         ).serialize()
     }
 

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

@@ -2,6 +2,7 @@ package no.nav.dagpenger.inntekt.v1
 
 import com.fasterxml.jackson.module.kotlin.readValue
 import de.huxhorn.sulky.ulid.ULID
+import io.kotest.matchers.nulls.shouldNotBeNull
 import io.kotest.matchers.shouldBe
 import io.ktor.client.request.get
 import io.ktor.client.statement.bodyAsText
@@ -10,10 +11,13 @@ import io.ktor.http.HttpStatusCode
 import io.mockk.coEvery
 import io.mockk.every
 import io.mockk.mockk
+import io.mockk.slot
+import io.mockk.verify
 import kotlinx.coroutines.runBlocking
 import no.nav.dagpenger.inntekt.Problem
 import no.nav.dagpenger.inntekt.db.DetachedInntekt
 import no.nav.dagpenger.inntekt.db.InntektId
+import no.nav.dagpenger.inntekt.db.InntektPersonMapping
 import no.nav.dagpenger.inntekt.db.InntektStore
 import no.nav.dagpenger.inntekt.db.Inntektparametre
 import no.nav.dagpenger.inntekt.db.ManueltRedigert
@@ -38,6 +42,7 @@ import no.nav.dagpenger.inntekt.oppslag.Person
 import no.nav.dagpenger.inntekt.oppslag.PersonOppslag
 import no.nav.dagpenger.inntekt.oppslag.enhetsregister.EnhetsregisterClient
 import no.nav.dagpenger.inntekt.serder.jacksonObjectMapper
+import no.nav.dagpenger.inntekt.v1.TestApplication.TEST_OAUTH_USER
 import no.nav.dagpenger.inntekt.v1.TestApplication.autentisert
 import no.nav.dagpenger.inntekt.v1.TestApplication.mockInntektApi
 import no.nav.dagpenger.inntekt.v1.TestApplication.withMockAuthServerAndTestApplication
@@ -452,7 +457,7 @@ internal class UklassifisertInntektRouteTest {
         }
 
     @Test
-    fun `Get request for uklassifisert inntekt with inntektID should return 200 ok`() {
+    fun `Get request for uklassifisert inntekt med inntektID returnerer 200 ok`() {
         val enhetsregisterClientMock = mockk<EnhetsregisterClient>(relaxed = true)
         return withMockAuthServerAndTestApplication(
             mockInntektApi(
@@ -496,4 +501,53 @@ internal class UklassifisertInntektRouteTest {
             assertEquals("", storedInntekt.virksomheter[1].virksomhetsnavn)
         }
     }
+
+    @Test
+    fun `Post request for uklassifisert inntekt med inntektId lagrer og returnerer ny ID`() {
+        return withMockAuthServerAndTestApplication(
+            moduleFunction =
+                mockInntektApi(
+                    inntektskomponentClient = inntektskomponentClientMock,
+                    inntektStore = inntektStoreMock,
+                ),
+        ) {
+            val body =
+                UklassifisertInntektRouteTest::class.java
+                    .getResource("/test-data/expected-uklassifisert-post-body.json")
+                    ?.readText()
+            val inntekterDto = jacksonObjectMapper.readValue<InntekterDto>(body!!)
+
+            val inntektPersonMapping =
+                InntektPersonMapping(
+                    inntektId = inntektId,
+                    aktørId = "123456789",
+                    fnr = null,
+                    kontekstId = "kontekstId",
+                    beregningsdato = LocalDate.now(),
+                    timestamp = LocalDateTime.now(),
+                    kontekstType = "kontekstType",
+                )
+            every { inntektStoreMock.getInntektPersonMapping(any()) } returns inntektPersonMapping
+
+            val storeInntektCommandSlot = slot<StoreInntektCommand>()
+            every { inntektStoreMock.storeInntekt(capture(storeInntektCommandSlot), any()) } returns storedInntekt
+
+            val response =
+                autentisert(
+                    httpMethod = HttpMethod.Post,
+                    endepunkt = "$uklassifisertInntekt/${inntektId.id}",
+                    body = body,
+                )
+
+            response.bodyAsText() shouldBe storedInntekt.inntektId.id
+            verify(exactly = 1) { inntektStoreMock.storeInntekt(any(), any()) }
+            storeInntektCommandSlot.captured.inntektparametre.aktørId shouldBe inntektPersonMapping.aktørId
+            storeInntektCommandSlot.captured.inntektparametre.fødselsnummer shouldBe inntekterDto.mottaker.pnr
+            storeInntektCommandSlot.captured.inntektparametre.regelkontekst.id shouldBe inntektPersonMapping.kontekstId
+            storeInntektCommandSlot.captured.inntektparametre.regelkontekst.type shouldBe inntektPersonMapping.kontekstType
+            storeInntektCommandSlot.captured.inntektparametre.beregningsdato shouldBe inntektPersonMapping.beregningsdato
+            storeInntektCommandSlot.captured.manueltRedigert.shouldNotBeNull()
+            storeInntektCommandSlot.captured.manueltRedigert!!.redigertAv shouldBe TEST_OAUTH_USER
+        }
+    }
 }

dp-inntekt-api/src/test/resources/test-data/expected-uklassifisert-post-body.json

@@ -0,0 +1,44 @@
+{
+  "virksomheter": [
+    {
+      "virksomhetsnummer": "123456789",
+      "virksomhetsnavn": "Test Virksomhet",
+      "periode": {
+        "fra": "2023-01",
+        "til": "2023-12"
+      },
+      "inntekter": [
+        {
+          "belop": 50000.00,
+          "fordel": "kontantytelse",
+          "beskrivelse": "bil",
+          "inntektskilde": "A-ordningen",
+          "inntektsstatus": "LoependeInnrapportert",
+          "inntektsperiodetype": "Maaned",
+          "leveringstidspunkt": "2023-01",
+          "opptjeningsland": "NO",
+          "opptjeningsperiode": null,
+          "skattemessigBosattLand": "NO",
+          "utbetaltIMaaned": "2023-01",
+          "inntektsinnsender": null,
+          "virksomhet": null,
+          "inntektsmottaker": null,
+          "inngaarIGrunnlagForTrekk": true,
+          "utloeserArbeidsgiveravgift": true,
+          "informasjonsstatus": "InngaarAlltid",
+          "inntektType": "LOENNSINNTEKT",
+          "tilleggsinformasjon": null,
+          "redigert": true,
+          "begrunnelse": "Standard",
+          "aarMaaned": "2023-01"
+        }
+      ],
+      "totalBeløp": 50000.00,
+      "avvikListe": []
+    }
+  ],
+  "mottaker": {
+    "pnr": "19876543210",
+    "navn": "Navn Navnesen"
+  }
+}