Skip to content

Commit 8522758

Browse files
JMLindsethArselanSultanivskjefst
committed
Begynn på test av post-endepunkt
Co-authored-by: Arselan Sultani <[email protected]> Co-authored-by: Vegard Skjefstad <[email protected]>
1 parent 19553b3 commit 8522758

File tree

8 files changed

+92
-12
lines changed

8 files changed

+92
-12
lines changed

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

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -56,7 +56,7 @@ data class StoreInntektCommand(
5656

5757
data class InntektPersonMapping(
5858
val inntektId: InntektId,
59-
val aktoerId: String,
59+
val aktørId: String,
6060
val fnr: String? = null,
6161
val kontekstId: String,
6262
val beregningsdato: LocalDate,

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

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -105,7 +105,7 @@ internal class PostgresInntektStore(
105105
).map { row ->
106106
InntektPersonMapping(
107107
inntektId = InntektId(row.string("inntektid")),
108-
aktoerId = row.string("aktorid"),
108+
aktørId = row.string("aktorid"),
109109
fnr = row.string("fnr"),
110110
kontekstId = row.string("kontekstid"),
111111
beregningsdato = row.zonedDateTime("beregningsdato").toLocalDate(),

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

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,7 @@ data class GUIInntekt(
2121
)
2222

2323
data class Inntektsmottaker(
24-
val fnr: String?,
24+
val pnr: String?,
2525
val navn: String?,
2626
)
2727

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

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -70,11 +70,11 @@ private fun mapToInntektkomponentResponse(inntekterDto: InntekterDto): Inntektko
7070
inntektMaaned.skattemessigBosattLand,
7171
inntektMaaned.utbetaltIMaaned,
7272
// TODO: Er det mottaker som skal brukes her?
73-
mapToAktoerNaturligIdent(inntekterDto.mottaker.fnr),
73+
mapToAktoerNaturligIdent(inntekterDto.mottaker.pnr),
7474
// TODO: Er det virksomhet som skal brukes her?
7575
mapToAktoerOrganisasjon(virksomhet),
7676
mapToAktoerOrganisasjon(virksomhet),
77-
mapToAktoerNaturligIdent(inntekterDto.mottaker.fnr),
77+
mapToAktoerNaturligIdent(inntekterDto.mottaker.pnr),
7878
inntektMaaned.inngaarIGrunnlagForTrekk,
7979
inntektMaaned.utloeserArbeidsgiveravgift,
8080
inntektMaaned.informasjonsstatus,
@@ -90,7 +90,7 @@ private fun mapToInntektkomponentResponse(inntekterDto: InntekterDto): Inntektko
9090

9191
return InntektkomponentResponse(
9292
arbeidsInntektMaaneder,
93-
mapToAktoerNaturligIdent(inntekterDto.mottaker.fnr),
93+
mapToAktoerNaturligIdent(inntekterDto.mottaker.pnr),
9494
)
9595
}
9696

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

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -175,14 +175,13 @@ fun Route.uklassifisertInntekt(
175175
inntekterDto = call.receive<InntekterDto>(),
176176
inntektId = inntektId,
177177
).let {
178-
val person = personOppslag.hentPerson(it.inntekt.ident.identifikator)
179178
val inntektPersonMapping = inntektStore.getInntektPersonMapping(inntektId)
180179
inntektStore.storeInntekt(
181180
StoreInntektCommand(
182181
inntektparametre =
183182
Inntektparametre(
184-
aktørId = person.aktørId,
185-
fødselsnummer = person.fødselsnummer,
183+
aktørId = inntektPersonMapping.aktørId,
184+
fødselsnummer = it.inntekt.ident.identifikator,
186185
regelkontekst = RegelKontekst(inntektPersonMapping.kontekstId, inntektPersonMapping.kontekstType),
187186
beregningsdato = inntektPersonMapping.beregningsdato,
188187
),
@@ -195,7 +194,7 @@ fun Route.uklassifisertInntekt(
195194
),
196195
)
197196
}.let {
198-
call.respond(HttpStatusCode.OK, it.inntektId)
197+
call.respond(HttpStatusCode.OK, it.inntektId.id)
199198
}.also {
200199
inntektKorrigeringCounter.inc()
201200
}

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

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -166,7 +166,7 @@ val inntektkomponentResponse =
166166

167167
val mottaker =
168168
Inntektsmottaker(
169-
fnr = "2044350291600",
169+
pnr = "2044350291600",
170170
navn = "Ola Nordmann",
171171
)
172172

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

Lines changed: 38 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -14,6 +14,7 @@ import kotlinx.coroutines.runBlocking
1414
import no.nav.dagpenger.inntekt.Problem
1515
import no.nav.dagpenger.inntekt.db.DetachedInntekt
1616
import no.nav.dagpenger.inntekt.db.InntektId
17+
import no.nav.dagpenger.inntekt.db.InntektPersonMapping
1718
import no.nav.dagpenger.inntekt.db.InntektStore
1819
import no.nav.dagpenger.inntekt.db.Inntektparametre
1920
import no.nav.dagpenger.inntekt.db.ManueltRedigert
@@ -452,7 +453,7 @@ internal class UklassifisertInntektRouteTest {
452453
}
453454

454455
@Test
455-
fun `Get request for uklassifisert inntekt with inntektID should return 200 ok`() {
456+
fun `Get request for uklassifisert inntekt med inntektID returnerer 200 ok`() {
456457
val enhetsregisterClientMock = mockk<EnhetsregisterClient>(relaxed = true)
457458
return withMockAuthServerAndTestApplication(
458459
mockInntektApi(
@@ -496,4 +497,40 @@ internal class UklassifisertInntektRouteTest {
496497
assertEquals("", storedInntekt.virksomheter[1].virksomhetsnavn)
497498
}
498499
}
500+
501+
@Test
502+
fun `Post request for uklassifisert inntekt med inntektId lagrer og returnerer ny ID`() {
503+
return withMockAuthServerAndTestApplication(
504+
moduleFunction =
505+
mockInntektApi(
506+
inntektskomponentClient = inntektskomponentClientMock,
507+
inntektStore = inntektStoreMock,
508+
),
509+
) {
510+
val body =
511+
UklassifisertInntektRouteTest::class.java
512+
.getResource("/test-data/expected-uklassifisert-post-body.json")
513+
?.readText()
514+
515+
every { inntektStoreMock.getInntektPersonMapping(any()) } returns
516+
InntektPersonMapping(
517+
inntektId = inntektId,
518+
aktørId = "1234567890",
519+
fnr = "12345678910",
520+
kontekstId = "kontekstId",
521+
beregningsdato = LocalDate.now(),
522+
timestamp = LocalDateTime.now(),
523+
kontekstType = "kontekstType",
524+
)
525+
526+
every { inntektStoreMock.storeInntekt(any(), any()) } returns storedInntekt
527+
528+
val response =
529+
autentisert(
530+
httpMethod = HttpMethod.Post,
531+
endepunkt = "$uklassifisertInntekt/${inntektId.id}",
532+
body = body,
533+
)
534+
}
535+
}
499536
}
Lines changed: 44 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,44 @@
1+
{
2+
"virksomheter": [
3+
{
4+
"virksomhetsnummer": "123456789",
5+
"virksomhetsnavn": "Test Virksomhet",
6+
"periode": {
7+
"fra": "2023-01",
8+
"til": "2023-12"
9+
},
10+
"inntekter": [
11+
{
12+
"belop": 50000.00,
13+
"fordel": "kontantytelse",
14+
"beskrivelse": "bil",
15+
"inntektskilde": "A-ordningen",
16+
"inntektsstatus": "LoependeInnrapportert",
17+
"inntektsperiodetype": "Maaned",
18+
"leveringstidspunkt": "2023-01",
19+
"opptjeningsland": "NO",
20+
"opptjeningsperiode": null,
21+
"skattemessigBosattLand": "NO",
22+
"utbetaltIMaaned": "2023-01",
23+
"inntektsinnsender": null,
24+
"virksomhet": null,
25+
"inntektsmottaker": null,
26+
"inngaarIGrunnlagForTrekk": true,
27+
"utloeserArbeidsgiveravgift": true,
28+
"informasjonsstatus": "InngaarAlltid",
29+
"inntektType": "LOENNSINNTEKT",
30+
"tilleggsinformasjon": null,
31+
"redigert": true,
32+
"begrunnelse": "Standard",
33+
"aarMaaned": "2023-01"
34+
}
35+
],
36+
"totalBeløp": 50000.00,
37+
"avvikListe": []
38+
}
39+
],
40+
"mottaker": {
41+
"pnr": "19876543210",
42+
"navn": "Navn Navnesen"
43+
}
44+
}

0 commit comments

Comments
 (0)