1302: Småfix og testfix

Henreich · Henreich · commit 433d9ef7ddbf · 2025-02-24T08:06:24.000+01:00
diff --git a/app/src/main/kotlin/no/nav/tiltakspenger/vedtak/App.kt b/app/src/main/kotlin/no/nav/tiltakspenger/vedtak/App.kt
@@ -79,7 +79,7 @@ internal fun start(
             applicationContext.behandlingContext.journalførVedtaksbrevService.journalfør()
             applicationContext.behandlingContext.distribuerVedtaksbrevService.distribuer()
             applicationContext.sendTilDatadelingService.send(Configuration.isNais())
-            applicationContext.meldekortContext.oppgaveMeldekortService.opprettOppgaveForMeldekortSomIkkeGodkjennesMaskinelt()
+            applicationContext.meldekortContext.oppgaveMeldekortService.opprettOppgaveForMeldekortSomIkkeGodkjennesAutomatisk()
             if (Configuration.isNais()) {
                 applicationContext.endretTiltaksdeltakerJobb.opprettOppgaveForEndredeDeltakere()
                 applicationContext.endretTiltaksdeltakerJobb.opprydning()
diff --git a/app/src/main/kotlin/no/nav/tiltakspenger/vedtak/repository/meldekort/BrukersMeldekortPostgresRepo.kt b/app/src/main/kotlin/no/nav/tiltakspenger/vedtak/repository/meldekort/BrukersMeldekortPostgresRepo.kt
@@ -30,22 +30,28 @@ class BrukersMeldekortPostgresRepo(
                         meldeperiode_versjon,
                         sak_id,
                         mottatt,
-                        dager
+                        dager,
+                        journalpost_id,
+                        oppgave_id
                     ) values (
                         :id,
                         :meldeperiode_hendelse_id,
                         (SELECT id FROM meldeperiode WHERE hendelse_id = :meldeperiode_hendelse_id),
                         (SELECT versjon FROM meldeperiode WHERE hendelse_id = :meldeperiode_hendelse_id),
                         :sak_id,
                         :mottatt,
-                        to_jsonb(:dager::jsonb)
+                        to_jsonb(:dager::jsonb),
+                        :journalpost_id,
+                        :oppgave_id
                     )
                     """,
                     "id" to brukersMeldekort.id.toString(),
                     "meldeperiode_hendelse_id" to brukersMeldekort.meldeperiodeHendelseId.toString(),
                     "sak_id" to brukersMeldekort.sakId.toString(),
                     "mottatt" to brukersMeldekort.mottatt,
                     "dager" to brukersMeldekort.toDagerJson(),
+                    "journalpost_id" to brukersMeldekort.journalpostId.toString(),
+                    "oppgave_id" to brukersMeldekort.oppgaveId,
                 ).asUpdate,
             )
         }
@@ -72,7 +78,7 @@ class BrukersMeldekortPostgresRepo(
         }
     }
 
-    override fun hentDeUtenTilknyttetOppgave(sessionContext: SessionContext?): List<BrukersMeldekort> {
+    override fun hentMeldekortSomIKkeSkalGodkjennesAutomatisk(sessionContext: SessionContext?): List<BrukersMeldekort> {
         return sessionFactory.withSession(sessionContext) { session ->
             session.run(
                 sqlQuery(
diff --git a/app/src/main/kotlin/no/nav/tiltakspenger/vedtak/routes/meldekort/frameldekortapi/MottaMeldekortRoutes.kt b/app/src/main/kotlin/no/nav/tiltakspenger/vedtak/routes/meldekort/frameldekortapi/MottaMeldekortRoutes.kt
@@ -69,5 +69,6 @@ private fun BrukerutfyltMeldekortDTO.toDomain(): NyttBrukersMeldekort {
             )
         },
         journalpostId = JournalpostId(this.journalpostId),
+        oppgaveId = null,
     )
 }
diff --git a/app/src/test/kotlin/no/nav/tiltakspenger/LocalApplicationContext.kt b/app/src/test/kotlin/no/nav/tiltakspenger/LocalApplicationContext.kt
@@ -186,6 +186,8 @@ class LocalApplicationContext : ApplicationContext(gitHash = "fake-git-hash") {
             personService = personContext.personService,
             entraIdSystemtokenClient = entraIdSystemtokenClient,
             navkontorService = navkontorService,
+            oppgaveGateway = oppgaveGateway,
+            sakRepo = sakContext.sakRepo,
         ) {}
     }
     override val behandlingContext by lazy {
diff --git a/app/src/test/kotlin/no/nav/tiltakspenger/meldekort/service/OppgaveMeldekortServiceTest.kt b/app/src/test/kotlin/no/nav/tiltakspenger/meldekort/service/OppgaveMeldekortServiceTest.kt
@@ -0,0 +1,97 @@
+package no.nav.tiltakspenger.meldekort.service
+
+import io.mockk.clearAllMocks
+import io.mockk.coEvery
+import io.mockk.coVerify
+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
+import no.nav.tiltakspenger.saksbehandling.ports.OppgaveGateway
+import no.nav.tiltakspenger.saksbehandling.ports.Oppgavebehov
+import no.nav.tiltakspenger.saksbehandling.ports.SakRepo
+import org.junit.jupiter.api.BeforeEach
+import org.junit.jupiter.api.Test
+
+class OppgaveMeldekortServiceTest {
+
+    private val oppgaveGateway = mockk<OppgaveGateway>(relaxed = true)
+    private val sakRepo = mockk<SakRepo>()
+    private val brukersMeldekortRepo = mockk<BrukersMeldekortRepo>()
+    private val service = OppgaveMeldekortService(oppgaveGateway, sakRepo, brukersMeldekortRepo)
+
+    @BeforeEach
+    fun setup() {
+        clearAllMocks()
+    }
+
+    @Test
+    fun `skal opprette oppgave for gyldig meldekort`() {
+        runTest {
+            val journalpostId = JournalpostIdGenerator().neste().toString()
+            val meldekort = mockk<BrukersMeldekort>()
+            every { meldekort.journalpostId } returns journalpostId
+            every { meldekort.sakId } returns SakId.random()
+
+            coEvery { brukersMeldekortRepo.hentMeldekortSomIKkeSkalGodkjennesAutomatisk() } returns listOf(meldekort)
+            coEvery { sakRepo.hentForSakId(any()) } returns mockk(relaxed = true)
+
+            service.opprettOppgaveForMeldekortSomIkkeGodkjennesAutomatisk()
+
+            coVerify {
+                oppgaveGateway.opprettOppgave(
+                    any(),
+                    JournalpostId(journalpostId),
+                    Oppgavebehov.NYTT_MELDEKORT,
+                )
+            }
+        }
+    }
+
+    @Test
+    fun `ingen oppgave blir opprettet om journalpostId ikke finnes`() {
+        runTest {
+            val meldekort = mockk<BrukersMeldekort>()
+            every { meldekort.journalpostId } returns null
+            coEvery { brukersMeldekortRepo.hentMeldekortSomIKkeSkalGodkjennesAutomatisk() } returns listOf(meldekort)
+
+            service.opprettOppgaveForMeldekortSomIkkeGodkjennesAutomatisk()
+
+            coVerify(exactly = 0) { oppgaveGateway.opprettOppgave(any(), any(), any()) }
+        }
+    }
+
+    @Test
+    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.sakId } returns SakId.random()
+            coEvery { brukersMeldekortRepo.hentMeldekortSomIKkeSkalGodkjennesAutomatisk() } returns listOf(meldekort)
+            coEvery { sakRepo.hentForSakId(any()) } returns null
+
+            service.opprettOppgaveForMeldekortSomIkkeGodkjennesAutomatisk()
+
+            coVerify(exactly = 0) { oppgaveGateway.opprettOppgave(any(), any(), any()) }
+        }
+    }
+
+    @Test
+    fun `skal opprette oppgaver for gyldige meldekort`() {
+        runTest {
+            val meldekort = (1..20).map { mockk<BrukersMeldekort>(relaxed = true) }
+            coEvery { brukersMeldekortRepo.hentMeldekortSomIKkeSkalGodkjennesAutomatisk() } returns meldekort
+            coEvery { sakRepo.hentForSakId(any()) } returns mockk(relaxed = true)
+
+            service.opprettOppgaveForMeldekortSomIkkeGodkjennesAutomatisk()
+
+            coVerify(exactly = meldekort.size) {
+                oppgaveGateway.opprettOppgave(any(), any(), Oppgavebehov.NYTT_MELDEKORT)
+            }
+        }
+    }
+}
diff --git a/app/src/test/kotlin/no/nav/tiltakspenger/vedtak/repository/meldekort/MeldekortBrukerPostgresRepoTest.kt b/app/src/test/kotlin/no/nav/tiltakspenger/vedtak/repository/meldekort/MeldekortBrukerPostgresRepoTest.kt
@@ -37,6 +37,8 @@ class MeldekortBrukerPostgresRepoTest {
                     meldeperiode = meldeperiode,
                     sakId = nyttBrukersMeldekort.sakId,
                     dager = nyttBrukersMeldekort.dager,
+                    journalpostId = nyttBrukersMeldekort.journalpostId.toString(),
+                    oppgaveId = nyttBrukersMeldekort.oppgaveId,
                 ),
             )
         }
diff --git a/domene/src/main/kotlin/no/nav/tiltakspenger/meldekort/domene/BrukersMeldekortRepo.kt b/domene/src/main/kotlin/no/nav/tiltakspenger/meldekort/domene/BrukersMeldekortRepo.kt
@@ -26,7 +26,7 @@ interface BrukersMeldekortRepo {
         sessionContext: SessionContext? = null,
     ): BrukersMeldekort?
 
-    fun hentDeUtenTilknyttetOppgave(
+    fun hentMeldekortSomIKkeSkalGodkjennesAutomatisk(
         sessionContext: SessionContext? = null,
     ): List<BrukersMeldekort>
 }
diff --git a/domene/src/main/kotlin/no/nav/tiltakspenger/meldekort/domene/NyttBrukersMeldekort.kt b/domene/src/main/kotlin/no/nav/tiltakspenger/meldekort/domene/NyttBrukersMeldekort.kt
@@ -24,4 +24,5 @@ data class NyttBrukersMeldekort(
     val sakId: SakId,
     val dager: List<BrukersMeldekortDag>,
     val journalpostId: JournalpostId,
+    val oppgaveId: String?,
 )
diff --git a/domene/src/main/kotlin/no/nav/tiltakspenger/meldekort/service/OppgaveMeldekortService.kt b/domene/src/main/kotlin/no/nav/tiltakspenger/meldekort/service/OppgaveMeldekortService.kt
@@ -15,8 +15,8 @@ class OppgaveMeldekortService(
     private val log = KotlinLogging.logger {}
 
     // På sikt vil man bare opprette oppgave for meldekort som trenger en manuell gjennomgang, for meldekort skal generelt godkjennes maskinelt.
-    suspend fun opprettOppgaveForMeldekortSomIkkeGodkjennesMaskinelt() {
-        brukersMeldekortRepo.hentDeUtenTilknyttetOppgave().forEach { meldekort ->
+    suspend fun opprettOppgaveForMeldekortSomIkkeGodkjennesAutomatisk() {
+        brukersMeldekortRepo.hentMeldekortSomIKkeSkalGodkjennesAutomatisk().forEach { meldekort ->
             val journalpostId = meldekort.journalpostId
                 ?: log.warn { "Fant ikke journalpostId for meldekortId ${meldekort.id}" }.let { return@forEach }
             val sak = sakRepo.hentForSakId(meldekort.sakId)
diff --git a/test-common/src/main/kotlin/no/nav/tiltakspenger/common/TestApplicationContext.kt b/test-common/src/main/kotlin/no/nav/tiltakspenger/common/TestApplicationContext.kt
@@ -195,6 +195,8 @@ class TestApplicationContext(
                 personService = personContext.personService,
                 entraIdSystemtokenClient = entraIdSystemtokenClient,
                 navkontorService = navkontorService,
+                oppgaveGateway = oppgaveGateway,
+                sakRepo = sakContext.sakRepo,
             ) {
             override val meldekortBehandlingRepo = meldekortBehandlingFakeRepo
             override val meldeperiodeRepo = meldeperiodeFakeRepo
diff --git a/test-common/src/main/kotlin/no/nav/tiltakspenger/fakes/repos/BrukersMeldekortFakeRepo.kt b/test-common/src/main/kotlin/no/nav/tiltakspenger/fakes/repos/BrukersMeldekortFakeRepo.kt
@@ -23,6 +23,8 @@ class BrukersMeldekortFakeRepo(val meldeperiodeFakeRepo: MeldeperiodeFakeRepo) :
             meldeperiode = meldeperiode,
             mottatt = brukersMeldekort.mottatt,
             dager = brukersMeldekort.dager,
+            journalpostId = brukersMeldekort.journalpostId.toString(),
+            oppgaveId = brukersMeldekort.oppgaveId,
         )
     }
 
@@ -43,7 +45,7 @@ class BrukersMeldekortFakeRepo(val meldeperiodeFakeRepo: MeldeperiodeFakeRepo) :
         return data.get().values.find { it.meldeperiode.id == hendelseId }
     }
 
-    override fun hentDeUtenTilknyttetOppgave(sessionContext: SessionContext?): List<BrukersMeldekort> {
-        TODO("Not yet implemented")
+    override fun hentMeldekortSomIKkeSkalGodkjennesAutomatisk(sessionContext: SessionContext?): List<BrukersMeldekort> {
+        return data.get().values.filter { it.oppgaveId == null }
     }
 }
diff --git a/test-common/src/main/kotlin/no/nav/tiltakspenger/objectmothers/MeldekortMother.kt b/test-common/src/main/kotlin/no/nav/tiltakspenger/objectmothers/MeldekortMother.kt
@@ -3,6 +3,7 @@ package no.nav.tiltakspenger.objectmothers
 import arrow.core.NonEmptyList
 import arrow.core.nonEmptyListOf
 import arrow.core.toNonEmptyListOrNull
+import no.nav.tiltakspenger.common.JournalpostIdGenerator
 import no.nav.tiltakspenger.felles.Navkontor
 import no.nav.tiltakspenger.felles.erHelg
 import no.nav.tiltakspenger.felles.nå
@@ -434,6 +435,8 @@ interface MeldekortMother {
             meldeperiode = meldeperiode,
             sakId = sakId,
             dager = dager,
+            journalpostId = JournalpostIdGenerator().neste().toString(),
+            oppgaveId = null,
         )
     }
 
@@ -458,6 +461,8 @@ interface MeldekortMother {
             meldeperiodeHendelseId = meldeperiodeId,
             sakId = sakId,
             dager = dager,
+            journalpostId = JournalpostIdGenerator().neste(),
+            oppgaveId = null,
         )
     }
 }

Original file line number	Diff line number	Diff line change
`@@ -69,5 +69,6 @@ private fun BrukerutfyltMeldekortDTO.toDomain(): NyttBrukersMeldekort {`
`69`	`69`	`)`
`70`	`70`	`},`
`71`	`71`	`journalpostId = JournalpostId(this.journalpostId),`
	`72`	`+ oppgaveId = null,`
`72`	`73`	`)`
`73`	`74`	`}`
Original file line number	Diff line number	Diff line change
`@@ -186,6 +186,8 @@ class LocalApplicationContext : ApplicationContext(gitHash = "fake-git-hash") {`
`186`	`186`	`personService = personContext.personService,`
`187`	`187`	`entraIdSystemtokenClient = entraIdSystemtokenClient,`
`188`	`188`	`navkontorService = navkontorService,`
	`189`	`+ oppgaveGateway = oppgaveGateway,`
	`190`	`+ sakRepo = sakContext.sakRepo,`
`189`	`191`	`) {}`
`190`	`192`	`}`
`191`	`193`	`override val behandlingContext by lazy {`
Original file line number	Diff line number	Diff line change
`@@ -37,6 +37,8 @@ class MeldekortBrukerPostgresRepoTest {`
`37`	`37`	`meldeperiode = meldeperiode,`
`38`	`38`	`sakId = nyttBrukersMeldekort.sakId,`
`39`	`39`	`dager = nyttBrukersMeldekort.dager,`
	`40`	`+ journalpostId = nyttBrukersMeldekort.journalpostId.toString(),`
	`41`	`+ oppgaveId = nyttBrukersMeldekort.oppgaveId,`
`40`	`42`	`),`
`41`	`43`	`)`
`42`	`44`	`}`
Original file line number	Diff line number	Diff line change
`@@ -26,7 +26,7 @@ interface BrukersMeldekortRepo {`
`26`	`26`	`sessionContext: SessionContext? = null,`
`27`	`27`	`): BrukersMeldekort?`
`28`	`28`
`29`		`- fun hentDeUtenTilknyttetOppgave(`
	`29`	`+ fun hentMeldekortSomIKkeSkalGodkjennesAutomatisk(`
`30`	`30`	`sessionContext: SessionContext? = null,`
`31`	`31`	`): List<BrukersMeldekort>`
`32`	`32`	`}`
Original file line number	Diff line number	Diff line change
`@@ -24,4 +24,5 @@ data class NyttBrukersMeldekort(`
`24`	`24`	`val sakId: SakId,`
`25`	`25`	`val dager: List<BrukersMeldekortDag>,`
`26`	`26`	`val journalpostId: JournalpostId,`
	`27`	`+ val oppgaveId: String?,`
`27`	`28`	`)`
Original file line number	Diff line number	Diff line change
`@@ -23,6 +23,8 @@ class BrukersMeldekortFakeRepo(val meldeperiodeFakeRepo: MeldeperiodeFakeRepo) :`
`23`	`23`	`meldeperiode = meldeperiode,`
`24`	`24`	`mottatt = brukersMeldekort.mottatt,`
`25`	`25`	`dager = brukersMeldekort.dager,`
	`26`	`+ journalpostId = brukersMeldekort.journalpostId.toString(),`
	`27`	`+ oppgaveId = brukersMeldekort.oppgaveId,`
`26`	`28`	`)`
`27`	`29`	`}`
`28`	`30`
`@@ -43,7 +45,7 @@ class BrukersMeldekortFakeRepo(val meldeperiodeFakeRepo: MeldeperiodeFakeRepo) :`
`43`	`45`	`return data.get().values.find { it.meldeperiode.id == hendelseId }`
`44`	`46`	`}`
`45`	`47`
`46`		`- override fun hentDeUtenTilknyttetOppgave(sessionContext: SessionContext?): List<BrukersMeldekort> {`
`47`		`- TODO("Not yet implemented")`
	`48`	`+ override fun hentMeldekortSomIKkeSkalGodkjennesAutomatisk(sessionContext: SessionContext?): List<BrukersMeldekort> {`
	`49`	`+ return data.get().values.filter { it.oppgaveId == null }`
`48`	`50`	`}`
`49`	`51`	`}`
Original file line number	Diff line number	Diff line change
`@@ -3,6 +3,7 @@ package no.nav.tiltakspenger.objectmothers`
`3`	`3`	`import arrow.core.NonEmptyList`
`4`	`4`	`import arrow.core.nonEmptyListOf`
`5`	`5`	`import arrow.core.toNonEmptyListOrNull`
	`6`	`+import no.nav.tiltakspenger.common.JournalpostIdGenerator`
`6`	`7`	`import no.nav.tiltakspenger.felles.Navkontor`
`7`	`8`	`import no.nav.tiltakspenger.felles.erHelg`
`8`	`9`	`import no.nav.tiltakspenger.felles.nå`
`@@ -434,6 +435,8 @@ interface MeldekortMother {`
`434`	`435`	`meldeperiode = meldeperiode,`
`435`	`436`	`sakId = sakId,`
`436`	`437`	`dager = dager,`
	`438`	`+ journalpostId = JournalpostIdGenerator().neste().toString(),`
	`439`	`+ oppgaveId = null,`
`437`	`440`	`)`
`438`	`441`	`}`
`439`	`442`
`@@ -458,6 +461,8 @@ interface MeldekortMother {`
`458`	`461`	`meldeperiodeHendelseId = meldeperiodeId,`
`459`	`462`	`sakId = sakId,`
`460`	`463`	`dager = dager,`
	`464`	`+ journalpostId = JournalpostIdGenerator().neste(),`
	`465`	`+ oppgaveId = null,`
`461`	`466`	`)`
`462`	`467`	`}`
`463`	`468`	`}`