1302: Bruk objekter for OppgaveId og JournalpostId istedet for strings

Author: Henreich

app/src/main/kotlin/no/nav/tiltakspenger/vedtak/repository/meldekort/BrukersMeldekortPostgresRepo.kt

@@ -2,6 +2,8 @@ package no.nav.tiltakspenger.vedtak.repository.meldekort
 
 import kotliquery.Row
 import kotliquery.Session
+import no.nav.tiltakspenger.felles.OppgaveId
+import no.nav.tiltakspenger.felles.journalføring.JournalpostId
 import no.nav.tiltakspenger.libs.common.MeldekortId
 import no.nav.tiltakspenger.libs.common.MeldeperiodeId
 import no.nav.tiltakspenger.libs.common.SakId
@@ -72,7 +74,10 @@ class BrukersMeldekortPostgresRepo(
         }
     }
 
-    override fun hentForMeldeperiodeId(meldeperiodeId: MeldeperiodeId, sessionContext: SessionContext?): BrukersMeldekort? {
+    override fun hentForMeldeperiodeId(
+        meldeperiodeId: MeldeperiodeId,
+        sessionContext: SessionContext?,
+    ): BrukersMeldekort? {
         return sessionFactory.withSession(sessionContext) { session ->
             hentForMeldeperiodeId(meldeperiodeId, session)
         }
@@ -155,8 +160,8 @@ class BrukersMeldekortPostgresRepo(
                 )!!,
                 sakId = SakId.fromString(row.string("sak_id")),
                 dager = row.string("dager").toMeldekortDager(),
-                journalpostId = row.string("journalpost_id"),
-                oppgaveId = row.stringOrNull("oppgave_id"),
+                journalpostId = JournalpostId(row.string("journalpost_id")),
+                oppgaveId = row.stringOrNull("oppgave_id")?.let { OppgaveId(it) },
             )
         }
     }

app/src/test/kotlin/no/nav/tiltakspenger/meldekort/service/OppgaveMeldekortServiceTest.kt

@@ -7,7 +7,6 @@ import io.mockk.every
 import io.mockk.mockk
 import kotlinx.coroutines.test.runTest
 import no.nav.tiltakspenger.common.JournalpostIdGenerator
-import no.nav.tiltakspenger.felles.journalføring.JournalpostId
 import no.nav.tiltakspenger.libs.common.SakId
 import no.nav.tiltakspenger.meldekort.domene.BrukersMeldekort
 import no.nav.tiltakspenger.meldekort.domene.BrukersMeldekortRepo
@@ -32,7 +31,7 @@ class OppgaveMeldekortServiceTest {
     @Test
     fun `skal opprette oppgave for gyldig meldekort`() {
         runTest {
-            val journalpostId = JournalpostIdGenerator().neste().toString()
+            val journalpostId = JournalpostIdGenerator().neste()
             val meldekort = mockk<BrukersMeldekort>()
             every { meldekort.journalpostId } returns journalpostId
             every { meldekort.sakId } returns SakId.random()
@@ -45,7 +44,7 @@ class OppgaveMeldekortServiceTest {
             coVerify {
                 oppgaveGateway.opprettOppgave(
                     any(),
-                    JournalpostId(journalpostId),
+                    journalpostId,
                     Oppgavebehov.NYTT_MELDEKORT,
                 )
             }
@@ -69,7 +68,7 @@ class OppgaveMeldekortServiceTest {
     fun `ingen oppgave opprettes om det ikke finnes en tilknyttet sak`() {
         runTest {
             val meldekort = mockk<BrukersMeldekort>()
-            every { meldekort.journalpostId } returns JournalpostIdGenerator().neste().toString()
+            every { meldekort.journalpostId } returns JournalpostIdGenerator().neste()
             every { meldekort.sakId } returns SakId.random()
             coEvery { brukersMeldekortRepo.hentMeldekortSomIKkeSkalGodkjennesAutomatisk() } returns listOf(meldekort)
             coEvery { sakRepo.hentForSakId(any()) } returns null

app/src/test/kotlin/no/nav/tiltakspenger/vedtak/repository/meldekort/MeldekortBrukerPostgresRepoTest.kt

@@ -37,7 +37,7 @@ class MeldekortBrukerPostgresRepoTest {
                     meldeperiode = meldeperiode,
                     sakId = nyttBrukersMeldekort.sakId,
                     dager = nyttBrukersMeldekort.dager,
-                    journalpostId = nyttBrukersMeldekort.journalpostId.toString(),
+                    journalpostId = nyttBrukersMeldekort.journalpostId,
                     oppgaveId = nyttBrukersMeldekort.oppgaveId,
                 ),
             )

domene/src/main/kotlin/no/nav/tiltakspenger/meldekort/domene/BrukersMeldekort.kt

@@ -1,5 +1,7 @@
 package no.nav.tiltakspenger.meldekort.domene
 
+import no.nav.tiltakspenger.felles.OppgaveId
+import no.nav.tiltakspenger.felles.journalføring.JournalpostId
 import no.nav.tiltakspenger.libs.common.HendelseVersjon
 import no.nav.tiltakspenger.libs.common.MeldekortId
 import no.nav.tiltakspenger.libs.common.MeldeperiodeKjedeId
@@ -23,8 +25,8 @@ data class BrukersMeldekort(
     val meldeperiode: Meldeperiode,
     val sakId: SakId,
     val dager: List<BrukersMeldekortDag>,
-    val journalpostId: String?,
-    val oppgaveId: String?,
+    val journalpostId: JournalpostId?,
+    val oppgaveId: OppgaveId?,
 ) {
     val meldeperiodeKjedeId: MeldeperiodeKjedeId = meldeperiode.meldeperiodeKjedeId
     val meldeperiodeVersjon: HendelseVersjon = meldeperiode.versjon

domene/src/main/kotlin/no/nav/tiltakspenger/meldekort/domene/NyttBrukersMeldekort.kt

@@ -1,5 +1,6 @@
 package no.nav.tiltakspenger.meldekort.domene
 
+import no.nav.tiltakspenger.felles.OppgaveId
 import no.nav.tiltakspenger.felles.journalføring.JournalpostId
 import no.nav.tiltakspenger.libs.common.MeldekortId
 import no.nav.tiltakspenger.libs.common.MeldeperiodeId
@@ -23,5 +24,5 @@ data class NyttBrukersMeldekort(
     val sakId: SakId,
     val dager: List<BrukersMeldekortDag>,
     val journalpostId: JournalpostId,
-    val oppgaveId: String?,
+    val oppgaveId: OppgaveId?,
 )

domene/src/main/kotlin/no/nav/tiltakspenger/meldekort/service/OppgaveMeldekortService.kt

@@ -1,7 +1,6 @@
 package no.nav.tiltakspenger.meldekort.service
 
 import mu.KotlinLogging
-import no.nav.tiltakspenger.felles.journalføring.JournalpostId
 import no.nav.tiltakspenger.meldekort.domene.BrukersMeldekortRepo
 import no.nav.tiltakspenger.saksbehandling.ports.OppgaveGateway
 import no.nav.tiltakspenger.saksbehandling.ports.Oppgavebehov
@@ -21,7 +20,7 @@ class OppgaveMeldekortService(
                 ?: log.warn { "Fant ikke journalpostId for meldekortId ${meldekort.id}" }.let { return@forEach }
             val sak = sakRepo.hentForSakId(meldekort.sakId)
                 ?: log.warn { "Fant ikke sak for sakId ${meldekort.sakId}" }.let { return@forEach }
-            oppgaveGateway.opprettOppgave(sak.fnr, JournalpostId(journalpostId), Oppgavebehov.NYTT_MELDEKORT)
+            oppgaveGateway.opprettOppgave(sak.fnr, journalpostId, Oppgavebehov.NYTT_MELDEKORT)
         }
     }
 }

test-common/src/main/kotlin/no/nav/tiltakspenger/fakes/repos/BrukersMeldekortFakeRepo.kt

@@ -23,7 +23,7 @@ class BrukersMeldekortFakeRepo(val meldeperiodeFakeRepo: MeldeperiodeFakeRepo) :
             meldeperiode = meldeperiode,
             mottatt = brukersMeldekort.mottatt,
             dager = brukersMeldekort.dager,
-            journalpostId = brukersMeldekort.journalpostId.toString(),
+            journalpostId = brukersMeldekort.journalpostId,
             oppgaveId = brukersMeldekort.oppgaveId,
         )
     }

test-common/src/main/kotlin/no/nav/tiltakspenger/objectmothers/MeldekortMother.kt

@@ -423,7 +423,7 @@ interface MeldekortMother {
             meldeperiode = meldeperiode,
             sakId = sakId,
             dager = dager,
-            journalpostId = JournalpostIdGenerator().neste().toString(),
+            journalpostId = JournalpostIdGenerator().neste(),
             oppgaveId = null,
         )
     }