Skip to content

Commit 8b92c8e

Browse files
authored
Fiks av spørring der aktørid ble ignorert fordi fødselsnummer er null (#90)
resolves navikt/dagpenger#401
1 parent 00acd62 commit 8b92c8e

File tree

2 files changed

+43
-1
lines changed

2 files changed

+43
-1
lines changed

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

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -95,7 +95,7 @@ internal class PostgresInntektStore(private val dataSource: DataSource) : Inntek
9595
SELECT inntektId
9696
FROM inntekt_V1_person_mapping
9797
WHERE aktørId = ?
98-
AND fnr = ? OR fnr IS NULL
98+
AND (fnr = ? OR fnr IS NULL)
9999
AND vedtakId = ?
100100
AND beregningsdato = ?
101101
ORDER BY timestamp DESC LIMIT 1

src/test/kotlin/no/nav/dagpenger/inntekt/db/PostgresTest.kt

Lines changed: 42 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,8 @@
11
package no.nav.dagpenger.inntekt.db
22

3+
import io.kotest.assertions.assertSoftly
34
import io.kotest.matchers.shouldBe
5+
import io.kotest.matchers.shouldNotBe
46
import java.time.LocalDate
57
import kotlin.test.assertEquals
68
import kotlin.test.assertFalse
@@ -84,6 +86,46 @@ internal class PostgresInntektStoreTest {
8486
}
8587
}
8688

89+
@Test
90+
fun ` Should insert different inntekt based on different aktørid and same vedtak id `() {
91+
92+
val aktørId1 = "1234"
93+
val aktørId2 = "5678"
94+
95+
withMigratedDb {
96+
with(PostgresInntektStore(DataSource.instance)) {
97+
98+
storeInntekt(
99+
StoreInntektCommand(
100+
inntektparametre = Inntektparametre(aktørId = aktørId1, vedtakId = "1234", beregningsdato = LocalDate.now()),
101+
inntekt = InntektkomponentResponse(
102+
emptyList(),
103+
Aktoer(AktoerType.AKTOER_ID, aktørId1)
104+
)
105+
)
106+
)
107+
108+
storeInntekt(
109+
StoreInntektCommand(
110+
inntektparametre = Inntektparametre(aktørId = aktørId2, vedtakId = "1234", beregningsdato = LocalDate.now()),
111+
inntekt = InntektkomponentResponse(
112+
emptyList(),
113+
Aktoer(AktoerType.AKTOER_ID, aktørId2)
114+
)
115+
)
116+
)
117+
118+
assertSoftly {
119+
getInntektId(Inntektparametre(aktørId = aktørId1, vedtakId = "1234", beregningsdato = LocalDate.now())) shouldNotBe null
120+
getInntektId(Inntektparametre(aktørId = aktørId2, vedtakId = "1234", beregningsdato = LocalDate.now())) shouldNotBe null
121+
getInntektId(Inntektparametre(aktørId = aktørId2, vedtakId = "1234", beregningsdato = LocalDate.now())) shouldNotBe getInntektId(Inntektparametre(aktørId = aktørId1, vedtakId = "1234", beregningsdato = LocalDate.now()))
122+
getInntektId(Inntektparametre(aktørId = aktørId2, vedtakId = "464664", beregningsdato = LocalDate.now())) shouldBe null
123+
getInntektId(Inntektparametre(aktørId = "3535535335", vedtakId = "1234", beregningsdato = LocalDate.now())) shouldBe null
124+
}
125+
}
126+
}
127+
}
128+
87129
@Test
88130
fun `Successful insert of inntekter which is manuelt redigert`() {
89131
withMigratedDb {

0 commit comments

Comments
 (0)