♻️ Rydder i periodehistorikkDao

navikt · Nov 12, 2024 · 5c915e2 · 5c915e2
1 parent b2a1684
commit 5c915e2
Show file tree

Hide file tree

Showing 11 changed files with 25 additions and 25 deletions.
diff --git a/spesialist-selve/src/main/kotlin/no/nav/helse/db/PeriodehistorikkDao.kt b/spesialist-selve/src/main/kotlin/no/nav/helse/db/PeriodehistorikkDao.kt
@@ -4,12 +4,12 @@ import no.nav.helse.modell.periodehistorikk.HistorikkinnslagDto
 import java.util.UUID
 
 interface PeriodehistorikkDao {
-    fun lagre(
+    fun lagreMedOppgaveId(
         historikkinnslag: HistorikkinnslagDto,
         oppgaveId: Long,
     )
 
-    fun lagre(
+    fun lagreMedGenerasjonId(
         historikkinnslag: HistorikkinnslagDto,
         generasjonId: UUID,
     )

diff --git a/spesialist-selve/src/main/kotlin/no/nav/helse/db/PgPeriodehistorikkDao.kt b/spesialist-selve/src/main/kotlin/no/nav/helse/db/PgPeriodehistorikkDao.kt
@@ -22,24 +22,24 @@ class PgPeriodehistorikkDao(
     constructor(session: Session) : this(MedSession(session))
     constructor(dataSource: DataSource) : this(MedDataSource(dataSource))
 
-    override fun lagre(
+    override fun lagreMedOppgaveId(
         historikkinnslag: HistorikkinnslagDto,
         oppgaveId: Long,
     ) {
         val generasjonId = PgOppgaveDao(queryRunner).finnGenerasjonId(oppgaveId)
-        lagre(historikkinnslag, generasjonId)
+        lagreMedGenerasjonId(historikkinnslag, generasjonId)
     }
 
-    override fun lagre(
+    override fun lagreMedGenerasjonId(
         historikkinnslag: HistorikkinnslagDto,
         generasjonId: UUID,
     ) {
         when (historikkinnslag) {
-            is FjernetFraPåVent -> lagre(historikkinnslag, generasjonId, null)
-            is LagtPåVent -> lagre(historikkinnslag, generasjonId, null)
-            is TotrinnsvurderingFerdigbehandlet -> lagre(historikkinnslag, generasjonId, null)
-            is AvventerTotrinnsvurdering -> lagre(historikkinnslag, generasjonId, null)
-            is TotrinnsvurderingAutomatiskRetur -> lagre(historikkinnslag, generasjonId, null)
+            is FjernetFraPåVent -> lagre(historikkinnslag, generasjonId)
+            is LagtPåVent -> lagre(historikkinnslag, generasjonId)
+            is TotrinnsvurderingFerdigbehandlet -> lagre(historikkinnslag, generasjonId)
+            is AvventerTotrinnsvurdering -> lagre(historikkinnslag, generasjonId)
+            is TotrinnsvurderingAutomatiskRetur -> lagre(historikkinnslag, generasjonId)
             is TotrinnsvurderingRetur -> {
                 val notatId =
                     PgNotatDao(queryRunner)
@@ -53,15 +53,15 @@ class PgPeriodehistorikkDao(
                 lagre(historikkinnslag, generasjonId, notatId)
             }
 
-            is AutomatiskBehandlingStanset -> lagre(historikkinnslag, generasjonId, null)
-            is VedtaksperiodeReberegnet -> lagre(historikkinnslag, generasjonId, null)
+            is AutomatiskBehandlingStanset -> lagre(historikkinnslag, generasjonId)
+            is VedtaksperiodeReberegnet -> lagre(historikkinnslag, generasjonId)
         }
     }
 
     private fun lagre(
         historikkinnslag: HistorikkinnslagDto,
         generasjonId: UUID,
-        notatId: Int?,
+        notatId: Int? = null,
     ) {
         asSQL(
             """

diff --git a/spesialist-selve/src/main/kotlin/no/nav/helse/mediator/GodkjenningService.kt b/spesialist-selve/src/main/kotlin/no/nav/helse/mediator/GodkjenningService.kt
@@ -106,7 +106,7 @@ internal class GodkjenningService(
                 val beslutter = totrinnsvurdering.beslutter?.let { saksbehandlerRepository.finnSaksbehandler(it)?.toDto() }
                 checkNotNull(beslutter) { "Forventer at beslutter er satt" }
                 val innslag = HistorikkinnslagDto.totrinnsvurderingFerdigbehandletInnslag(beslutter)
-                periodehistorikkDao.lagre(innslag, godkjenningDTO.oppgavereferanse)
+                periodehistorikkDao.lagreMedOppgaveId(innslag, godkjenningDTO.oppgavereferanse)
             }
         }
     }

diff --git a/spesialist-selve/src/main/kotlin/no/nav/helse/mediator/SaksbehandlerMediator.kt b/spesialist-selve/src/main/kotlin/no/nav/helse/mediator/SaksbehandlerMediator.kt
@@ -270,7 +270,7 @@ internal class SaksbehandlerMediator(
                     frist = handling.frist,
                     dialogRef = dialogRef,
                 )
-            periodehistorikkDao.lagre(innslag, handling.oppgaveId)
+            periodehistorikkDao.lagreMedOppgaveId(innslag, handling.oppgaveId)
             oppgaveService.leggPåVent(handling, saksbehandler)
             PåVentRepository(påVentDao).leggPåVent(saksbehandler.oid(), handling, dialogRef)
         } catch (e: Modellfeil) {
@@ -288,7 +288,7 @@ internal class SaksbehandlerMediator(
         }
         try {
             val innslag = HistorikkinnslagDto.fjernetFraPåVentInnslag(saksbehandler.toDto())
-            periodehistorikkDao.lagre(innslag, handling.oppgaveId)
+            periodehistorikkDao.lagreMedOppgaveId(innslag, handling.oppgaveId)
             oppgaveService.fjernFraPåVent(handling.oppgaveId)
             PåVentRepository(påVentDao).fjernFraPåVent(handling.oppgaveId)
         } catch (e: Modellfeil) {

diff --git a/.../src/main/kotlin/no/nav/helse/modell/kommando/VedtaksperiodeReberegnetPeriodehistorikk.kt b/.../src/main/kotlin/no/nav/helse/modell/kommando/VedtaksperiodeReberegnetPeriodehistorikk.kt
@@ -10,7 +10,7 @@ internal class VedtaksperiodeReberegnetPeriodehistorikk(
 ) : Command {
     override fun execute(context: CommandContext): Boolean {
         val innslag = HistorikkinnslagDto.vedtaksperiodeReberegnet()
-        periodehistorikkDao.lagre(historikkinnslag = innslag, generasjonId = vedtaksperiode.gjeldendeGenerasjonId)
+        periodehistorikkDao.lagreMedGenerasjonId(historikkinnslag = innslag, generasjonId = vedtaksperiode.gjeldendeGenerasjonId)
         return true
     }
 }
diff --git a/...kotlin/no/nav/helse/modell/stoppautomatiskbehandling/StansAutomatiskBehandlingMediator.kt b/...kotlin/no/nav/helse/modell/stoppautomatiskbehandling/StansAutomatiskBehandlingMediator.kt
@@ -96,7 +96,7 @@ internal class StansAutomatiskBehandlingMediator(
         val oppgaveId = oppgaveDao.finnOppgaveId(fødselsnummer)
         if (oppgaveId != null) {
             val innslag = HistorikkinnslagDto.automatiskBehandlingStanset()
-            periodehistorikkDao.lagre(innslag, oppgaveId)
+            periodehistorikkDao.lagreMedOppgaveId(innslag, oppgaveId)
         } else {
             sikkerlogg.info("Fant ikke oppgave for $fødselsnummer. Fikk ikke lagret historikkinnslag om stans av automatisk behandling")
         }

diff --git a/...t-selve/src/main/kotlin/no/nav/helse/modell/totrinnsvurdering/TotrinnsvurderingService.kt b/...t-selve/src/main/kotlin/no/nav/helse/modell/totrinnsvurdering/TotrinnsvurderingService.kt
@@ -28,7 +28,7 @@ class TotrinnsvurderingService(
         oppgaveDao.finnIdForAktivOppgave(vedtaksperiodeId)?.let {
             totrinnsvurderingDao.settErRetur(vedtaksperiodeId)
             val innslag = HistorikkinnslagDto.totrinnsvurderingAutomatiskRetur()
-            periodehistorikkDao.lagre(innslag, it)
+            periodehistorikkDao.lagreMedOppgaveId(innslag, it)
         }
     }
 
@@ -44,15 +44,15 @@ class TotrinnsvurderingService(
                 saksbehandler = saksbehandlerFraApi.toDto(),
                 dialogRef = dialogRef,
             )
-        periodehistorikkDao.lagre(innslag, oppgaveId)
+        periodehistorikkDao.lagreMedOppgaveId(innslag, oppgaveId)
     }
 
     override fun avventerTotrinnsvurdering(
         oppgaveId: Long,
         saksbehandlerFraApi: SaksbehandlerFraApi,
     ) {
         val innslag = HistorikkinnslagDto.avventerTotrinnsvurdering(saksbehandlerFraApi.toDto())
-        periodehistorikkDao.lagre(innslag, oppgaveId)
+        periodehistorikkDao.lagreMedOppgaveId(innslag, oppgaveId)
     }
 
     override fun erBeslutterOppgave(oppgaveId: Long): Boolean = hentAktiv(oppgaveId)?.erBeslutteroppgave() ?: false

diff --git a/spesialist-selve/src/test/kotlin/no/nav/helse/db/PgPeriodehistorikkDaoTest.kt b/spesialist-selve/src/test/kotlin/no/nav/helse/db/PgPeriodehistorikkDaoTest.kt
@@ -18,7 +18,7 @@ class PgPeriodehistorikkDaoTest : DatabaseIntegrationTest() {
         val saksbehandler = SaksbehandlerDto(SAKSBEHANDLER_EPOST, SAKSBEHANDLER_OID, SAKSBEHANDLER_NAVN, SAKSBEHANDLER_IDENT)
         val historikkinnslag = HistorikkinnslagDto.fjernetFraPåVentInnslag(saksbehandler)
 
-        historikkinnslagRepository.lagre(historikkinnslag, oppgaveId)
+        historikkinnslagRepository.lagreMedOppgaveId(historikkinnslag, oppgaveId)
         val result = periodehistorikkApiDao.finn(UTBETALING_ID)
 
         assertEquals(1, result.size)

diff --git a/.../test/kotlin/no/nav/helse/modell/kommando/VedtaksperiodeReberegnetPeriodehistorikkTest.kt b/.../test/kotlin/no/nav/helse/modell/kommando/VedtaksperiodeReberegnetPeriodehistorikkTest.kt
@@ -15,6 +15,6 @@ class VedtaksperiodeReberegnetPeriodehistorikkTest {
         val repository = mockk<PeriodehistorikkDao>(relaxed = true)
         val command = VedtaksperiodeReberegnetPeriodehistorikk(mockk(relaxed = true), repository)
         assertTrue(command.execute(context))
-        verify(exactly = 1) { repository.lagre(any<VedtaksperiodeReberegnet>(), any<UUID>()) }
+        verify(exactly = 1) { repository.lagreMedGenerasjonId(any<VedtaksperiodeReberegnet>(), any<UUID>()) }
     }
 }
diff --git a/...in/no/nav/helse/modell/stoppautomatiskbehandling/StansAutomatiskBehandlingMediatorTest.kt b/...in/no/nav/helse/modell/stoppautomatiskbehandling/StansAutomatiskBehandlingMediatorTest.kt
@@ -81,7 +81,7 @@ class StansAutomatiskBehandlingMediatorTest {
 
         verify(exactly = 1) { stansAutomatiskBehandlingDao.lagreFraISyfo(melding) }
         verify(exactly = 1) {
-            periodehistorikkDao.lagre(
+            periodehistorikkDao.lagreMedOppgaveId(
                 historikkinnslag = any<AutomatiskBehandlingStanset>(),
                 oppgaveId = any(),
             )

diff --git a/...lve/src/test/kotlin/no/nav/helse/modell/totrinnsvurdering/TotrinnsvurderingServiceTest.kt b/...lve/src/test/kotlin/no/nav/helse/modell/totrinnsvurdering/TotrinnsvurderingServiceTest.kt
@@ -38,7 +38,7 @@ class TotrinnsvurderingServiceTest {
         verify(exactly = 1) { totrinnsvurderingDao.settErRetur(vedtaksperiodeId) }
         verify(exactly = 1) { oppgaveDao.finnIdForAktivOppgave(vedtaksperiodeId) }
         verify(exactly = 1) {
-            periodehistorikkDao.lagre(
+            periodehistorikkDao.lagreMedOppgaveId(
                 historikkinnslag = any<TotrinnsvurderingAutomatiskRetur>(),
                 oppgaveId = oppgaveId,
             )