EY-4669 Fjern alle requires med internfeilexception check

navikt · Nov 13, 2024 · 389bae8 · 389bae8
1 parent 0c38459
commit 389bae8
Show file tree

Hide file tree

Showing 34 changed files with 228 additions and 97 deletions.
diff --git a/apps/etterlatte-api/src/main/kotlin/no/nav/etterlatte/samordning/vedtak/Tjenestepensjon.kt b/apps/etterlatte-api/src/main/kotlin/no/nav/etterlatte/samordning/vedtak/Tjenestepensjon.kt
@@ -4,12 +4,13 @@ import no.nav.etterlatte.libs.common.feilhaandtering.ForespoerselException
 import no.nav.etterlatte.libs.common.feilhaandtering.IkkeFunnetException
 import no.nav.etterlatte.libs.common.feilhaandtering.IkkeTillattException
 import no.nav.etterlatte.libs.common.feilhaandtering.UgyldigForespoerselException
+import no.nav.etterlatte.libs.common.feilhaandtering.checkInternFeil
 
 data class Tjenestepensjonnummer(
     val value: String,
 ) {
     init {
-        require(value == value.replace(Regex("[^0-9]"), ""))
+        checkInternFeil(value == value.replace(Regex("[^0-9]"), "")) { "Feil innhold i Tjenestepensjonnummer" }
     }
 }
 

diff --git a/apps/etterlatte-behandling/src/main/kotlin/behandling/BehandlingDao.kt b/apps/etterlatte-behandling/src/main/kotlin/behandling/BehandlingDao.kt
@@ -24,6 +24,7 @@ import no.nav.etterlatte.libs.common.behandling.Revurderingaarsak
 import no.nav.etterlatte.libs.common.behandling.TidligereFamiliepleier
 import no.nav.etterlatte.libs.common.behandling.Utlandstilknytning
 import no.nav.etterlatte.libs.common.behandling.Virkningstidspunkt
+import no.nav.etterlatte.libs.common.feilhaandtering.checkInternFeil
 import no.nav.etterlatte.libs.common.grunnlag.Grunnlagsopplysning
 import no.nav.etterlatte.libs.common.gyldigSoeknad.GyldighetsResultat
 import no.nav.etterlatte.libs.common.sak.BehandlingOgSak
@@ -261,7 +262,9 @@ class BehandlingDao(
                     stmt.setString(18, opphoerFraOgMed?.let { fom -> objectMapper.writeValueAsString(fom) })
                     stmt.setJsonb(19, tidligereFamiliepleier)
                 }
-                require(stmt.executeUpdate() == 1)
+                checkInternFeil(stmt.executeUpdate() == 1) {
+                    "Kunne ikke opprette behandling for ${behandling.id}"
+                }
             }
         }
 
@@ -281,7 +284,9 @@ class BehandlingDao(
             stmt.setObject(1, objectMapper.writeValueAsString(gyldighetsproeving))
             stmt.setTidspunkt(2, Tidspunkt.now().toLocalDatetimeUTC().toTidspunkt())
             stmt.setObject(3, behandlingId)
-            require(stmt.executeUpdate() == 1)
+            checkInternFeil(stmt.executeUpdate() == 1) {
+                "Kunne ikke lagreGyldighetsproeving behandling for $behandlingId"
+            }
         }
     }
 
@@ -300,7 +305,9 @@ class BehandlingDao(
             stmt.setString(1, status.name)
             stmt.setTidspunkt(2, sistEndret.toTidspunkt())
             stmt.setObject(3, behandlingId)
-            require(stmt.executeUpdate() == 1)
+            checkInternFeil(stmt.executeUpdate() == 1) {
+                "Kunne ikke lagreStatus behandling for $behandlingId"
+            }
         }
     }
 
@@ -313,7 +320,9 @@ class BehandlingDao(
 
             stmt.setString(1, objectMapper.writeValueAsString(boddEllerArbeidetUtlandet))
             stmt.setObject(2, behandlingId)
-            require(stmt.executeUpdate() == 1)
+            checkInternFeil(stmt.executeUpdate() == 1) {
+                "Kunne ikke lagreBoddEllerArbeidetUtlandet behandling for $behandlingId"
+            }
         }
     }
 
@@ -325,7 +334,9 @@ class BehandlingDao(
             val statement = prepareStatement("UPDATE behandling set utlandstilknytning = ? where id = ?")
             statement.setJsonb(1, utlandstilknytning)
             statement.setObject(2, behandlingId)
-            require(statement.executeUpdate() == 1)
+            checkInternFeil(statement.executeUpdate() == 1) {
+                "Kunne ikke lagreUtlandstilknytning behandling for $behandlingId"
+            }
         }
     }
 
@@ -337,7 +348,9 @@ class BehandlingDao(
             val statement = prepareStatement("UPDATE behandling set tidligere_familiepleier = ? where id = ?")
             statement.setJsonb(1, tidligereFamiliepleier)
             statement.setObject(2, behandlingId)
-            require(statement.executeUpdate() == 1)
+            checkInternFeil(statement.executeUpdate() == 1) {
+                "Kunne ikke lagreTidligereFamiliepleier behandling for $behandlingId"
+            }
         }
     }
 
@@ -446,7 +459,9 @@ class BehandlingDao(
             val statement = prepareStatement("UPDATE behandling set sende_brev = ? where id = ?")
             statement.setBoolean(1, skalSendeBrev)
             statement.setObject(2, behandlingId)
-            require(statement.executeUpdate() == 1)
+            checkInternFeil(statement.executeUpdate() == 1) {
+                "Kunne ikke send brev behandling for $behandlingId"
+            }
         }
     }
 
@@ -503,4 +518,5 @@ private fun ResultSet.toTidligereFamiliepleier(): TidligereFamiliepleier? =
 val objectMapper: ObjectMapper =
     jacksonObjectMapper().registerModule(JavaTimeModule()).disable(SerializationFeature.WRITE_DATES_AS_TIMESTAMPS)
 
+// TODO: fjerne denne bruke, blir veldig dårlig feilmelding
 fun PreparedStatement.updateSuccessful() = require(this.executeUpdate() == 1)
diff --git a/apps/etterlatte-behandling/src/main/kotlin/behandling/BehandlingService.kt b/apps/etterlatte-behandling/src/main/kotlin/behandling/BehandlingService.kt
@@ -44,6 +44,7 @@ import no.nav.etterlatte.libs.common.behandling.Virkningstidspunkt
 import no.nav.etterlatte.libs.common.feilhaandtering.IkkeFunnetException
 import no.nav.etterlatte.libs.common.feilhaandtering.InternfeilException
 import no.nav.etterlatte.libs.common.feilhaandtering.UgyldigForespoerselException
+import no.nav.etterlatte.libs.common.feilhaandtering.checkInternFeil
 import no.nav.etterlatte.libs.common.grunnlag.Grunnlagsopplysning
 import no.nav.etterlatte.libs.common.grunnlag.NyeSaksopplysninger
 import no.nav.etterlatte.libs.common.grunnlag.lagOpplysning
@@ -617,10 +618,10 @@ internal class BehandlingServiceImpl(
     ) {
         inTransaction {
             val behandling = behandlingDao.hentBehandling(behandlingId)
-            require(behandling != null) {
+            checkInternFeil(behandling != null) {
                 "Behandling finnes ikke $behandlingId"
             }
-            when (behandling.type) {
+            when (behandling!!.type) {
                 BehandlingType.FØRSTEGANGSBEHANDLING -> throw KanIkkeEndreSendeBrevForFoerstegangsbehandling()
                 BehandlingType.REVURDERING -> {
                     behandlingDao.lagreSendeBrev(behandlingId, skalSendeBrev)

diff --git a/...tterlatte-behandling/src/main/kotlin/behandling/aktivitetsplikt/AktivitetspliktService.kt b/...tterlatte-behandling/src/main/kotlin/behandling/aktivitetsplikt/AktivitetspliktService.kt
@@ -31,6 +31,7 @@ import no.nav.etterlatte.libs.common.behandling.Revurderingaarsak
 import no.nav.etterlatte.libs.common.behandling.tilVirkningstidspunkt
 import no.nav.etterlatte.libs.common.feilhaandtering.InternfeilException
 import no.nav.etterlatte.libs.common.feilhaandtering.UgyldigForespoerselException
+import no.nav.etterlatte.libs.common.feilhaandtering.checkInternFeil
 import no.nav.etterlatte.libs.common.grunnlag.Grunnlagsopplysning
 import no.nav.etterlatte.libs.common.grunnlag.hentDoedsdato
 import no.nav.etterlatte.libs.common.oppgave.OppgaveIntern
@@ -290,7 +291,7 @@ class AktivitetspliktService(
         if (aktivitetsgrad.id != null) {
             aktivitetspliktAktivitetsgradDao.oppdaterAktivitetsgrad(aktivitetsgrad, kilde, behandlingId)
         } else {
-            require(
+            checkInternFeil(
                 aktivitetspliktAktivitetsgradDao.hentAktivitetsgradForBehandling(behandlingId).isEmpty(),
             ) { "Aktivitetsgrad finnes allerede for behandling $behandlingId" }
             val unntak = aktivitetspliktUnntakDao.hentUnntakForBehandling(behandlingId)
@@ -385,7 +386,7 @@ class AktivitetspliktService(
         if (unntak.id != null) {
             aktivitetspliktUnntakDao.oppdaterUnntak(unntak, kilde, behandlingId)
         } else {
-            require(
+            checkInternFeil(
                 aktivitetspliktUnntakDao.hentUnntakForBehandling(behandlingId).isEmpty(),
             ) { "Unntak finnes allerede for behandling $behandlingId" }
 

diff --git a/apps/etterlatte-behandling/src/main/kotlin/behandling/behandlinginfo/BehandlingInfoDao.kt b/apps/etterlatte-behandling/src/main/kotlin/behandling/behandlinginfo/BehandlingInfoDao.kt
@@ -5,6 +5,7 @@ import no.nav.etterlatte.behandling.utland.SluttbehandlingUtlandBehandlinginfo
 import no.nav.etterlatte.common.ConnectionAutoclosing
 import no.nav.etterlatte.libs.common.behandling.Brevutfall
 import no.nav.etterlatte.libs.common.feilhaandtering.InternfeilException
+import no.nav.etterlatte.libs.common.feilhaandtering.checkInternFeil
 import no.nav.etterlatte.libs.common.objectMapper
 import no.nav.etterlatte.libs.database.setJsonb
 import no.nav.etterlatte.libs.database.singleOrNull
@@ -85,8 +86,11 @@ class BehandlingInfoDao(
                     setObject(1, brevutfall.behandlingId)
                     setJsonb(2, brevutfall)
                 }.run { executeUpdate() }
-                    .also { require(it == 1) }
-                    .let {
+                    .also {
+                        checkInternFeil(it == 1) {
+                            "Kunne ikke lagreBrevutfall behandling for ${brevutfall.behandlingId}"
+                        }
+                    }.let {
                         hentBrevutfall(brevutfall.behandlingId)
                             ?: throw InternfeilException("Feilet under lagring av brevutfall")
                     }
@@ -125,8 +129,11 @@ class BehandlingInfoDao(
                     setObject(1, etterbetaling.behandlingId)
                     setJsonb(2, etterbetaling)
                 }.run { executeUpdate() }
-                    .also { require(it == 1) }
-                    .let {
+                    .also {
+                        checkInternFeil(it == 1) {
+                            "Kunne ikke lagreBrevutfall behandling for ${etterbetaling.behandlingId}"
+                        }
+                    }.let {
                         hentEtterbetaling(etterbetaling.behandlingId)
                             ?: throw InternfeilException("Feilet under lagring av etterbetaling")
                     }
@@ -145,7 +152,11 @@ class BehandlingInfoDao(
                     setJsonb(1, null)
                     setObject(2, behandlingId)
                 }.run { executeUpdate() }
-                    .also { require(it == 1) }
+                    .also {
+                        checkInternFeil(it == 1) {
+                            "Kunne ikke slettEtterbetaling behandling for $behandlingId"
+                        }
+                    }
             }
         }
 

diff --git a/apps/etterlatte-behandling/src/main/kotlin/behandling/bosattutland/BosattUtlandDao.kt b/apps/etterlatte-behandling/src/main/kotlin/behandling/bosattutland/BosattUtlandDao.kt
@@ -4,6 +4,7 @@ import com.fasterxml.jackson.module.kotlin.readValue
 import no.nav.etterlatte.behandling.hendelse.getUUID
 import no.nav.etterlatte.behandling.objectMapper
 import no.nav.etterlatte.common.ConnectionAutoclosing
+import no.nav.etterlatte.libs.common.feilhaandtering.checkInternFeil
 import no.nav.etterlatte.libs.database.setJsonb
 import no.nav.etterlatte.libs.database.singleOrNull
 import java.sql.ResultSet
@@ -29,7 +30,7 @@ class BosattUtlandDao(
                 statement.setObject(2, bosattUtland.rinanummer)
                 statement.setJsonb(3, bosattUtland.mottatteSeder)
                 statement.setJsonb(4, bosattUtland.sendteSeder)
-                require(statement.executeUpdate() == 1)
+                checkInternFeil(statement.executeUpdate() == 1) { "Kunne ikke lagre utland for behandlingId: ${bosattUtland.behandlingId}" }
             }
         }
 

diff --git a/apps/etterlatte-behandling/src/main/kotlin/behandling/omregning/OmregningDao.kt b/apps/etterlatte-behandling/src/main/kotlin/behandling/omregning/OmregningDao.kt
@@ -1,6 +1,7 @@
 package no.nav.etterlatte.behandling.omregning
 
 import no.nav.etterlatte.common.ConnectionAutoclosing
+import no.nav.etterlatte.libs.common.feilhaandtering.checkInternFeil
 import no.nav.etterlatte.libs.common.sak.KjoeringRequest
 import no.nav.etterlatte.libs.common.sak.KjoeringStatus
 import no.nav.etterlatte.libs.common.sak.LagreKjoeringRequest
@@ -69,7 +70,11 @@ class OmregningDao(
                 statement.setString(4, request.begrunnelse)
                 statement.setString(5, request.corrId)
                 statement.setString(6, request.feilendeSteg)
-                statement.executeUpdate().also { require(it == 1) }
+                statement.executeUpdate().also {
+                    checkInternFeil(it > 0) {
+                        "Kunne ikke oppdaterKjoering for id sakid ${request.sakId}"
+                    }
+                }
             }
         }
     }
@@ -97,7 +102,11 @@ class OmregningDao(
                 statement.setBigDecimal(9, request.avkortingFoer)
                 statement.setBigDecimal(10, request.avkortingEtter)
                 statement.setBigDecimal(11, request.vedtakBeloep)
-                statement.executeUpdate().also { require(it == 1) }
+                statement.executeUpdate().also {
+                    checkInternFeil(it > 0) {
+                        "Kunne ikke lagreKjoering for id sakid ${request.sakId}"
+                    }
+                }
             }
         }
     }

diff --git a/apps/etterlatte-behandling/src/main/kotlin/behandling/tilbakekreving/TilbakekrevingDao.kt b/apps/etterlatte-behandling/src/main/kotlin/behandling/tilbakekreving/TilbakekrevingDao.kt
@@ -4,6 +4,7 @@ import com.fasterxml.jackson.module.kotlin.readValue
 import no.nav.etterlatte.behandling.hendelse.getUUID
 import no.nav.etterlatte.common.ConnectionAutoclosing
 import no.nav.etterlatte.libs.common.Enhetsnummer
+import no.nav.etterlatte.libs.common.feilhaandtering.checkInternFeil
 import no.nav.etterlatte.libs.common.objectMapper
 import no.nav.etterlatte.libs.common.sak.Sak
 import no.nav.etterlatte.libs.common.sak.SakId
@@ -197,7 +198,11 @@ class TilbakekrevingDao(
             statement.setJsonb(6, vurdering)
         }
         statement.setBoolean(7, tilbakekrevingBehandling.sendeBrev)
-        statement.executeUpdate().also { require(it == 1) }
+        statement.executeUpdate().also {
+            checkInternFeil(it == 1) {
+                "Kunne ikke lagre tilbaekreving behandling for sakid ${tilbakekrevingBehandling.sak.id}"
+            }
+        }
     }
 
     private fun deleteTilbakekrevingsperioder(
@@ -212,7 +217,11 @@ class TilbakekrevingDao(
                 """.trimIndent(),
             )
         statement.setObject(1, tilbakekrevingId)
-        statement.executeUpdate().also { require(it > 0) }
+        statement.executeUpdate().also {
+            checkInternFeil(it > 0) {
+                "Kunne ikke deleteTilbakekrevingsperioder behandling for id $tilbakekrevingId"
+            }
+        }
     }
 
     private fun insertTilbakekrevingsperioder(

diff --git a/apps/etterlatte-behandling/src/main/kotlin/sak/SakSkrivDao.kt b/apps/etterlatte-behandling/src/main/kotlin/sak/SakSkrivDao.kt
@@ -3,6 +3,7 @@ package no.nav.etterlatte.sak
 import no.nav.etterlatte.grunnlagsendring.SakMedEnhet
 import no.nav.etterlatte.libs.common.Enhetsnummer
 import no.nav.etterlatte.libs.common.behandling.SakType
+import no.nav.etterlatte.libs.common.feilhaandtering.checkInternFeil
 import no.nav.etterlatte.libs.common.person.AdressebeskyttelseGradering
 import no.nav.etterlatte.libs.common.sak.Sak
 import no.nav.etterlatte.libs.common.sak.SakId
@@ -38,7 +39,9 @@ class SakSkrivDao(
                     logger.info(
                         "Oppdaterer adressebeskyttelse med: $adressebeskyttelseGradering for sak med id: $sakId, antall oppdatert er $it",
                     )
-                    require(it == 1)
+                    checkInternFeil(it > 0) {
+                        "Kunne ikke oppdaterAdresseBeskyttelse for id sakid $sakId"
+                    }
                 }
             }
         }

diff --git a/...latte-behandling/src/test/kotlin/behandling/aktivitetsplikt/AktivitetspliktServiceTest.kt b/...latte-behandling/src/test/kotlin/behandling/aktivitetsplikt/AktivitetspliktServiceTest.kt
@@ -40,6 +40,7 @@ import no.nav.etterlatte.libs.common.behandling.Prosesstype
 import no.nav.etterlatte.libs.common.behandling.Revurderingaarsak
 import no.nav.etterlatte.libs.common.behandling.SakType
 import no.nav.etterlatte.libs.common.behandling.tilVirkningstidspunkt
+import no.nav.etterlatte.libs.common.feilhaandtering.InternfeilException
 import no.nav.etterlatte.libs.common.grunnlag.Grunnlagsopplysning
 import no.nav.etterlatte.libs.common.oppgave.OppgaveIntern
 import no.nav.etterlatte.libs.common.oppgave.OppgaveKilde
@@ -457,7 +458,7 @@ class AktivitetspliktServiceTest {
                     every { status } returns BehandlingStatus.VILKAARSVURDERT
                 }
 
-            assertThrows<IllegalArgumentException> {
+            assertThrows<InternfeilException> {
                 service.upsertUnntakForBehandling(unntak, behandlingId, sakId, brukerTokenInfo)
             }
         }

diff --git a/...tte-behandling/src/test/kotlin/behandling/generellbehandling/GenerellBehandlingDaoTest.kt b/...tte-behandling/src/test/kotlin/behandling/generellbehandling/GenerellBehandlingDaoTest.kt
@@ -3,6 +3,7 @@ package no.nav.etterlatte.behandling.generellbehandling
 import no.nav.etterlatte.ConnectionAutoclosingTest
 import no.nav.etterlatte.DatabaseExtension
 import no.nav.etterlatte.behandling.sakId1
+import no.nav.etterlatte.libs.common.feilhaandtering.InternfeilException
 import no.nav.etterlatte.libs.common.generellbehandling.DokumentMedSendtDato
 import no.nav.etterlatte.libs.common.generellbehandling.GenerellBehandling
 import no.nav.etterlatte.libs.common.generellbehandling.Innhold
@@ -48,7 +49,7 @@ internal class GenerellBehandlingDaoTest(
 
     @Test
     fun `Assert skal catche at man oppretter med feil type`() {
-        assertThrows<IllegalArgumentException> {
+        assertThrows<InternfeilException> {
             GenerellBehandling(
                 UUID.randomUUID(),
                 sakId1,

diff --git a/...atte-beregning-kafka/src/main/kotlin/no/nav/etterlatte/beregningkafka/BeregningService.kt b/...atte-beregning-kafka/src/main/kotlin/no/nav/etterlatte/beregningkafka/BeregningService.kt
@@ -13,6 +13,7 @@ import io.ktor.http.isSuccess
 import kotlinx.coroutines.runBlocking
 import no.nav.etterlatte.grunnbeloep.Grunnbeloep
 import no.nav.etterlatte.libs.common.beregning.AarligInntektsjusteringAvkortingRequest
+import no.nav.etterlatte.libs.common.feilhaandtering.checkInternFeil
 import java.util.UUID
 
 class BeregningService(
@@ -86,6 +87,6 @@ class BeregningService(
     suspend fun hentGrunnbeloep(): Grunnbeloep =
         beregningApp
             .get("$url/api/beregning/grunnbeloep")
-            .also { require(it.status.isSuccess()) }
+            .also { checkInternFeil(it.status.isSuccess()) { "Kunne ikke hente grunnbeloep" } }
             .body<Grunnbeloep>()
 }