From 2b5842a4f9b9ee7b711bf5d9074657263f79016c Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Elias=20Andreassen=20Th=C3=B8gersen?= Date: Wed, 23 Oct 2024 11:07:52 +0200 Subject: [PATCH] =?UTF-8?q?=E2=9A=B0=EF=B8=8F=20Fjerner=20funksjoner=20som?= =?UTF-8?q?=20ikke=20lenger=20er=20i=20bruk?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Co-authored-by: Christian Bülow Skovborg --- .../helse/db/HistorikkinnslagRepository.kt | 9 ------- .../helse/db/PgHistorikkinnslagRepository.kt | 19 ------------- .../db/TransactionalPeriodehistorikkDao.kt | 27 ------------------- .../db/PgHistorikkinnslagRepositoryTest.kt | 20 -------------- .../modell/PeriodehistorikkApiDaoTest.kt | 26 ------------------ 5 files changed, 101 deletions(-) delete mode 100644 spesialist-selve/src/test/kotlin/no/nav/helse/modell/PeriodehistorikkApiDaoTest.kt diff --git a/spesialist-selve/src/main/kotlin/no/nav/helse/db/HistorikkinnslagRepository.kt b/spesialist-selve/src/main/kotlin/no/nav/helse/db/HistorikkinnslagRepository.kt index 85379da74..9ec48b4aa 100644 --- a/spesialist-selve/src/main/kotlin/no/nav/helse/db/HistorikkinnslagRepository.kt +++ b/spesialist-selve/src/main/kotlin/no/nav/helse/db/HistorikkinnslagRepository.kt @@ -1,7 +1,6 @@ package no.nav.helse.db import no.nav.helse.modell.periodehistorikk.HistorikkinnslagDto -import no.nav.helse.spesialist.api.periodehistorikk.PeriodehistorikkType import java.util.UUID interface HistorikkinnslagRepository { @@ -15,14 +14,6 @@ interface HistorikkinnslagRepository { generasjonId: UUID, ) - fun lagre( - historikkType: PeriodehistorikkType, - saksbehandlerOid: UUID? = null, - utbetalingId: UUID, - notatId: Int? = null, - json: String = "{}", - ) - fun migrer( tidligereUtbetalingId: UUID, utbetalingId: UUID, diff --git a/spesialist-selve/src/main/kotlin/no/nav/helse/db/PgHistorikkinnslagRepository.kt b/spesialist-selve/src/main/kotlin/no/nav/helse/db/PgHistorikkinnslagRepository.kt index f62e79e04..07fd7d6e7 100644 --- a/spesialist-selve/src/main/kotlin/no/nav/helse/db/PgHistorikkinnslagRepository.kt +++ b/spesialist-selve/src/main/kotlin/no/nav/helse/db/PgHistorikkinnslagRepository.kt @@ -3,7 +3,6 @@ package no.nav.helse.db import kotliquery.sessionOf import no.nav.helse.mediator.oppgave.OppgaveDao import no.nav.helse.modell.periodehistorikk.HistorikkinnslagDto -import no.nav.helse.spesialist.api.periodehistorikk.PeriodehistorikkType import java.util.UUID import javax.sql.DataSource @@ -37,22 +36,4 @@ class PgHistorikkinnslagRepository( TransactionalPeriodehistorikkDao(session).migrer(tidligereUtbetalingId, utbetalingId) } } - - override fun lagre( - historikkType: PeriodehistorikkType, - saksbehandlerOid: UUID?, - utbetalingId: UUID, - notatId: Int?, - json: String, - ) { - sessionOf(dataSource).use { session -> - TransactionalPeriodehistorikkDao(session).lagre( - historikkType, - saksbehandlerOid, - utbetalingId, - notatId, - json, - ) - } - } } diff --git a/spesialist-selve/src/main/kotlin/no/nav/helse/db/TransactionalPeriodehistorikkDao.kt b/spesialist-selve/src/main/kotlin/no/nav/helse/db/TransactionalPeriodehistorikkDao.kt index 1c6a3acd8..55b7c07a9 100644 --- a/spesialist-selve/src/main/kotlin/no/nav/helse/db/TransactionalPeriodehistorikkDao.kt +++ b/spesialist-selve/src/main/kotlin/no/nav/helse/db/TransactionalPeriodehistorikkDao.kt @@ -12,7 +12,6 @@ import no.nav.helse.modell.periodehistorikk.TotrinnsvurderingFerdigbehandlet import no.nav.helse.modell.periodehistorikk.TotrinnsvurderingRetur import no.nav.helse.modell.periodehistorikk.VedtaksperiodeReberegnet import no.nav.helse.spesialist.api.graphql.schema.NotatType -import no.nav.helse.spesialist.api.periodehistorikk.PeriodehistorikkType import org.intellij.lang.annotations.Language import java.util.UUID @@ -100,32 +99,6 @@ class TransactionalPeriodehistorikkDao( is VedtaksperiodeReberegnet -> "VEDTAKSPERIODE_REBEREGNET" // TODO: Mangler å migrere typen i databasen } - override fun lagre( - historikkType: PeriodehistorikkType, - saksbehandlerOid: UUID?, - utbetalingId: UUID, - notatId: Int?, - json: String, - ) { - @Language("PostgreSQL") - val statement = """ - INSERT INTO periodehistorikk (type, saksbehandler_oid, utbetaling_id, notat_id, json) - VALUES (:type, :saksbehandler_oid, :utbetaling_id, :notat_id, :json::json) - """ - session.run( - queryOf( - statement, - mapOf( - "type" to historikkType.name, - "saksbehandler_oid" to saksbehandlerOid, - "utbetaling_id" to utbetalingId, - "notat_id" to notatId, - "json" to json, - ), - ).asUpdate, - ) - } - override fun migrer( tidligereUtbetalingId: UUID, utbetalingId: UUID, diff --git a/spesialist-selve/src/test/kotlin/no/nav/helse/db/PgHistorikkinnslagRepositoryTest.kt b/spesialist-selve/src/test/kotlin/no/nav/helse/db/PgHistorikkinnslagRepositoryTest.kt index a92498ead..29a7d6c70 100644 --- a/spesialist-selve/src/test/kotlin/no/nav/helse/db/PgHistorikkinnslagRepositoryTest.kt +++ b/spesialist-selve/src/test/kotlin/no/nav/helse/db/PgHistorikkinnslagRepositoryTest.kt @@ -3,12 +3,10 @@ package no.nav.helse.db import DatabaseIntegrationTest import no.nav.helse.modell.periodehistorikk.HistorikkinnslagDto import no.nav.helse.modell.saksbehandler.SaksbehandlerDto -import no.nav.helse.spesialist.api.periodehistorikk.PeriodehistorikkType import org.junit.jupiter.api.Assertions.assertEquals import org.junit.jupiter.api.Test class PgHistorikkinnslagRepositoryTest : DatabaseIntegrationTest() { - @Test fun `lagre periodehistorikk ved hjelp av oppgaveId`() { opprettPerson() @@ -25,22 +23,4 @@ class PgHistorikkinnslagRepositoryTest : DatabaseIntegrationTest() { assertEquals(1, result.size) } - - @Test - fun `lagre periodehistorikk ved hjelp av utbetalingId`() { - opprettPerson() - opprettArbeidsgiver() - opprettSaksbehandler() - opprettVedtaksperiode() - opprettOppgave() - - pgHistorikkinnslagRepository.lagre( - PeriodehistorikkType.TOTRINNSVURDERING_TIL_GODKJENNING, - SAKSBEHANDLER_OID, - UTBETALING_ID - ) - val result = periodehistorikkApiDao.finn(UTBETALING_ID) - - assertEquals(1, result.size) - } } diff --git a/spesialist-selve/src/test/kotlin/no/nav/helse/modell/PeriodehistorikkApiDaoTest.kt b/spesialist-selve/src/test/kotlin/no/nav/helse/modell/PeriodehistorikkApiDaoTest.kt deleted file mode 100644 index 6a598de87..000000000 --- a/spesialist-selve/src/test/kotlin/no/nav/helse/modell/PeriodehistorikkApiDaoTest.kt +++ /dev/null @@ -1,26 +0,0 @@ -package no.nav.helse.modell - -import DatabaseIntegrationTest -import no.nav.helse.spesialist.api.periodehistorikk.PeriodehistorikkType -import org.junit.jupiter.api.Assertions.assertEquals -import org.junit.jupiter.api.Test -import java.util.UUID - -class PeriodehistorikkApiDaoTest : DatabaseIntegrationTest() { - - @Test - fun `lagre og finn periodehistorikk`() { - val periodeId = UUID.randomUUID() - opprettSaksbehandler() - - pgHistorikkinnslagRepository.lagre( - PeriodehistorikkType.TOTRINNSVURDERING_TIL_GODKJENNING, - SAKSBEHANDLER_OID, - periodeId - ) - val result = periodehistorikkApiDao.finn(periodeId) - - assertEquals(1, result.size) - } - -}