Skip to content

Commit 092a8f9

Browse files
geiralundgeiralund
committed
Utvidet kontekstid rad for å få plass til uuid
navikt/dagpenger#1017
1 parent e6a5ec6 commit 092a8f9

File tree

2 files changed

+85
-42
lines changed

2 files changed

+85
-42
lines changed

‎dp-inntekt-api/src/main/resources/db/migration/V10__Kontekstid_utvidelse.sql‎

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1 @@
1+
ALTER TABLE inntekt_v1_person_mapping ALTER COLUMN kontekstid TYPE VARCHAR(40);

‎dp-inntekt-api/src/test/kotlin/no/nav/dagpenger/inntekt/db/PostgresTest.kt‎

Lines changed: 84 additions & 42 deletions
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@ import io.kotest.core.spec.style.StringSpec
55
import io.kotest.matchers.shouldBe
66
import io.kotest.matchers.shouldNotBe
77
import io.kotest.property.Arb
8-
import io.kotest.property.arbitrary.arb
8+
import io.kotest.property.arbitrary.arbitrary
99
import io.kotest.property.arbitrary.localDateTime
1010
import io.kotest.property.arbitrary.next
1111
import io.kotest.property.arbitrary.string
@@ -34,7 +34,7 @@ internal class PostgresTest {
3434
fun `Migration scripts are applied successfully`() {
3535
withCleanDb {
3636
val migrations = migrate(DataSource.instance)
37-
assertEquals(14, migrations, "Wrong number of migrations")
37+
assertEquals(15, migrations, "Wrong number of migrations")
3838
}
3939
}
4040

@@ -52,7 +52,7 @@ internal class PostgresTest {
5252
fun `Migration of testdata `() {
5353
withCleanDb {
5454
val migrations = migrate(DataSource.instance, locations = listOf("db/migration", "db/testdata"))
55-
assertEquals(19, migrations, "Wrong number of migrations")
55+
assertEquals(20, migrations, "Wrong number of migrations")
5656
}
5757
}
5858

@@ -105,9 +105,17 @@ internal class PostgresInntektStoreTest {
105105
with(PostgresInntektStore(DataSource.instance)) {
106106

107107
val aktør1 =
108-
Inntektparametre(aktørId = aktørId1, beregningsdato = LocalDate.now(), regelkontekst = RegelKontekst("1234", "vedtak"))
108+
Inntektparametre(
109+
aktørId = aktørId1,
110+
beregningsdato = LocalDate.now(),
111+
regelkontekst = RegelKontekst("1234", "vedtak")
112+
)
109113
val aktør2 =
110-
Inntektparametre(aktørId = aktørId2, beregningsdato = LocalDate.now(), regelkontekst = RegelKontekst("1234", "vedtak"))
114+
Inntektparametre(
115+
aktørId = aktørId2,
116+
beregningsdato = LocalDate.now(),
117+
regelkontekst = RegelKontekst("1234", "vedtak")
118+
)
111119
storeInntekt(
112120
StoreInntektCommand(
113121
inntektparametre = aktør1,
@@ -132,8 +140,20 @@ internal class PostgresInntektStoreTest {
132140
getInntektId(aktør1) shouldNotBe null
133141
getInntektId(aktør2) shouldNotBe null
134142
assertNotEquals(getInntektId(aktør2), getInntektId(aktør1))
135-
getInntektId(Inntektparametre(aktørId = aktørId2, beregningsdato = LocalDate.now(), regelkontekst = RegelKontekst("464664", "vedtak"))) shouldBe null
136-
getInntektId(Inntektparametre(aktørId = "3535535335", beregningsdato = LocalDate.now(), regelkontekst = RegelKontekst("1234", "vedtak"))) shouldBe null
143+
getInntektId(
144+
Inntektparametre(
145+
aktørId = aktørId2,
146+
beregningsdato = LocalDate.now(),
147+
regelkontekst = RegelKontekst("464664", "vedtak")
148+
)
149+
) shouldBe null
150+
getInntektId(
151+
Inntektparametre(
152+
aktørId = "3535535335",
153+
beregningsdato = LocalDate.now(),
154+
regelkontekst = RegelKontekst("1234", "vedtak")
155+
)
156+
) shouldBe null
137157
}
138158
}
139159
}
@@ -149,9 +169,17 @@ internal class PostgresInntektStoreTest {
149169
with(PostgresInntektStore(DataSource.instance)) {
150170

151171
val aktør1 =
152-
Inntektparametre(aktørId = aktørId1, beregningsdato = LocalDate.now(), regelkontekst = RegelKontekst("1234", "veiledning"))
172+
Inntektparametre(
173+
aktørId = aktørId1,
174+
beregningsdato = LocalDate.now(),
175+
regelkontekst = RegelKontekst("1234", "veiledning")
176+
)
153177
val aktør2 =
154-
Inntektparametre(aktørId = aktørId1, beregningsdato = LocalDate.now(), regelkontekst = RegelKontekst("1234", "saksbehandling"))
178+
Inntektparametre(
179+
aktørId = aktørId1,
180+
beregningsdato = LocalDate.now(),
181+
regelkontekst = RegelKontekst("1234", "saksbehandling")
182+
)
155183
storeInntekt(
156184
StoreInntektCommand(
157185
inntektparametre = aktør1,
@@ -223,12 +251,17 @@ internal class PostgresInntektStoreTest {
223251

224252
storeInntekt(
225253
StoreInntektCommand(
226-
inntektparametre = Inntektparametre(aktørId = "1234", beregningsdato = LocalDate.now(), regelkontekst = RegelKontekst("12345", "vedtak")),
254+
inntektparametre = Inntektparametre(
255+
aktørId = "1234",
256+
beregningsdato = LocalDate.now(),
257+
regelkontekst = RegelKontekst("12345", "vedtak")
258+
),
227259
inntekt = hentInntektListeResponse
228260
)
229261
)
230262

231-
val inntektId = getInntektId(Inntektparametre("1234", LocalDate.now(), RegelKontekst("12345", "vedtak")))
263+
val inntektId =
264+
getInntektId(Inntektparametre("1234", LocalDate.now(), RegelKontekst("12345", "vedtak")))
232265
val storedInntekt = inntektId?.let { getInntekt(it) }!!
233266
assertNotNull(storedInntekt.inntektId)
234267
assertTrue("Inntekstliste should be in the same state") { hentInntektListeResponse == storedInntekt.inntekt }
@@ -310,7 +343,11 @@ internal class PostgresInntektStoreTest {
310343

311344
val inntekt = storeInntekt(
312345
StoreInntektCommand(
313-
inntektparametre = Inntektparametre("1234", LocalDate.of(2019, 4, 14), RegelKontekst("12345", "vedtak")),
346+
inntektparametre = Inntektparametre(
347+
"1234",
348+
LocalDate.of(2019, 4, 14),
349+
RegelKontekst("12345", "vedtak")
350+
),
314351
inntekt = hentInntektListeResponse
315352
)
316353
)
@@ -346,7 +383,13 @@ internal class PostgresInntektStoreTest {
346383
migrate(DataSource.instance, locations = listOf("db/migration", "unit-testdata"))
347384

348385
with(PostgresInntektStore(DataSource.instance)) {
349-
val inntektsid = getInntektId(Inntektparametre("AKTØR_ID", LocalDate.of(2019, 1, 1), RegelKontekst("-1337", "forskudd")))
386+
val inntektsid = getInntektId(
387+
Inntektparametre(
388+
"AKTØR_ID",
389+
LocalDate.of(2019, 1, 1),
390+
RegelKontekst("-1337", "forskudd")
391+
)
392+
)
350393
assertNotNull(inntektsid)
351394
}
352395
}
@@ -409,39 +452,38 @@ internal class InntektsStorePropertyTest : StringSpec() {
409452
}
410453
}
411454

412-
private val storeInntekCommandGenerator = arb {
413-
val stringArb = Arb.string(10, 11)
414-
generateSequence {
415-
StoreInntektCommand(
416-
inntektparametre = Inntektparametre(
417-
aktørId = stringArb.next(it),
418-
regelkontekst = RegelKontekst(stringArb.next(it), "vedtak"),
419-
fødselnummer = stringArb.next(it),
420-
beregningsdato = Arb.localDateTime(minYear = 2010, maxYear = LocalDate.now().year).next(it).toLocalDate()
421-
),
422-
inntekt = InntektkomponentResponse(
423-
arbeidsInntektMaaned = emptyList(),
424-
ident = Aktoer(AktoerType.AKTOER_ID, "1234")
425-
)
455+
private val storeInntekCommandGenerator = arbitrary {
456+
val aktørId = Arb.string(10, 11)
457+
val kontekstId = Arb.string(5, 40)
458+
StoreInntektCommand(
459+
inntektparametre = Inntektparametre(
460+
aktørId = aktørId.next(it),
461+
regelkontekst = RegelKontekst(kontekstId.next(it), "vedtak"),
462+
fødselnummer = aktørId.next(it),
463+
beregningsdato = Arb.localDateTime(minYear = 2010, maxYear = LocalDate.now().year).next(it)
464+
.toLocalDate()
465+
),
466+
inntekt = InntektkomponentResponse(
467+
arbeidsInntektMaaned = emptyList(),
468+
ident = Aktoer(AktoerType.AKTOER_ID, "1234")
426469
)
427-
}
470+
)
428471
}
429472

430-
private val storeInntektCommandGeneratorWithSameVedtakidAndBeregningsDato = arb {
473+
private val storeInntektCommandGeneratorWithSameVedtakidAndBeregningsDato = arbitrary {
431474
val stringArb = Arb.string(10, 11)
432-
generateSequence {
433-
StoreInntektCommand(
434-
inntektparametre = Inntektparametre(
435-
aktørId = stringArb.next(it),
436-
regelkontekst = RegelKontekst("12345", "vedtak"),
437-
fødselnummer = null,
438-
beregningsdato = LocalDate.now()
439-
),
440-
inntekt = InntektkomponentResponse(
441-
arbeidsInntektMaaned = emptyList(),
442-
ident = Aktoer(AktoerType.AKTOER_ID, "1234")
443-
)
475+
val kontekstId = Arb.string(5, 40)
476+
StoreInntektCommand(
477+
inntektparametre = Inntektparametre(
478+
aktørId = stringArb.next(it),
479+
regelkontekst = RegelKontekst(kontekstId.next(it), "vedtak"),
480+
fødselnummer = null,
481+
beregningsdato = LocalDate.now()
482+
),
483+
inntekt = InntektkomponentResponse(
484+
arbeidsInntektMaaned = emptyList(),
485+
ident = Aktoer(AktoerType.AKTOER_ID, "1234")
444486
)
445-
}
487+
)
446488
}
447489
}

0 commit comments

Comments
 (0)