Forslag til standard på require og check (#6643)

* Endre navngivning + enkel javadoc * bruke nye funksjoner i koden
navikt · Dec 18, 2024 · 58fe4db · 58fe4db
1 parent 8b4b55b
commit 58fe4db
Show file tree

Hide file tree

Showing 43 changed files with 204 additions and 185 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,13 +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
+import no.nav.etterlatte.libs.common.feilhaandtering.krev
 
 data class Tjenestepensjonnummer(
     val value: String,
 ) {
     init {
-        checkInternFeil(value == value.replace(Regex("[^0-9]"), "")) { "Feil innhold i Tjenestepensjonnummer" }
+        krev(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
@@ -26,7 +26,7 @@ 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.InternfeilException
-import no.nav.etterlatte.libs.common.feilhaandtering.checkInternFeil
+import no.nav.etterlatte.libs.common.feilhaandtering.krev
 import no.nav.etterlatte.libs.common.grunnlag.Grunnlagsopplysning
 import no.nav.etterlatte.libs.common.gyldigSoeknad.GyldighetsResultat
 import no.nav.etterlatte.libs.common.sak.BehandlingOgSak
@@ -262,7 +262,7 @@ class BehandlingDao(
                     stmt.setString(17, opphoerFraOgMed?.let { fom -> objectMapper.writeValueAsString(fom) })
                     stmt.setJsonb(18, tidligereFamiliepleier)
                 }
-                checkInternFeil(stmt.executeUpdate() == 1) {
+                krev(stmt.executeUpdate() == 1) {
                     "Kunne ikke opprette behandling for ${behandling.id}"
                 }
             }
@@ -284,7 +284,7 @@ class BehandlingDao(
             stmt.setObject(1, objectMapper.writeValueAsString(gyldighetsproeving))
             stmt.setTidspunkt(2, Tidspunkt.now().toLocalDatetimeUTC().toTidspunkt())
             stmt.setObject(3, behandlingId)
-            checkInternFeil(stmt.executeUpdate() == 1) {
+            krev(stmt.executeUpdate() == 1) {
                 "Kunne ikke lagreGyldighetsproeving behandling for $behandlingId"
             }
         }
@@ -311,7 +311,7 @@ class BehandlingDao(
             stmt.setString(1, status.name)
             stmt.setTidspunkt(2, sistEndret.toTidspunkt())
             stmt.setObject(3, behandlingId)
-            checkInternFeil(stmt.executeUpdate() == 1) {
+            krev(stmt.executeUpdate() == 1) {
                 "Kunne ikke lagreStatus behandling for $behandlingId"
             }
         }
@@ -329,7 +329,7 @@ class BehandlingDao(
             stmt.setString(1, aarsakTilAvbrytelse.name)
             stmt.setString(2, kommentar)
             stmt.setObject(3, behandlingId)
-            checkInternFeil(stmt.executeUpdate() == 1) {
+            krev(stmt.executeUpdate() == 1) {
                 "Kunne ikke lagreStatus behandling for $behandlingId"
             }
         }
@@ -344,7 +344,7 @@ class BehandlingDao(
 
             stmt.setString(1, objectMapper.writeValueAsString(boddEllerArbeidetUtlandet))
             stmt.setObject(2, behandlingId)
-            checkInternFeil(stmt.executeUpdate() == 1) {
+            krev(stmt.executeUpdate() == 1) {
                 "Kunne ikke lagreBoddEllerArbeidetUtlandet behandling for $behandlingId"
             }
         }
@@ -358,7 +358,7 @@ class BehandlingDao(
             val statement = prepareStatement("UPDATE behandling set utlandstilknytning = ? where id = ?")
             statement.setJsonb(1, utlandstilknytning)
             statement.setObject(2, behandlingId)
-            checkInternFeil(statement.executeUpdate() == 1) {
+            krev(statement.executeUpdate() == 1) {
                 "Kunne ikke lagreUtlandstilknytning behandling for $behandlingId"
             }
         }
@@ -372,7 +372,7 @@ class BehandlingDao(
             val statement = prepareStatement("UPDATE behandling set tidligere_familiepleier = ? where id = ?")
             statement.setJsonb(1, tidligereFamiliepleier)
             statement.setObject(2, behandlingId)
-            checkInternFeil(statement.executeUpdate() == 1) {
+            krev(statement.executeUpdate() == 1) {
                 "Kunne ikke lagreTidligereFamiliepleier behandling for $behandlingId"
             }
         }
@@ -483,7 +483,7 @@ class BehandlingDao(
             val statement = prepareStatement("UPDATE behandling set sende_brev = ? where id = ?")
             statement.setBoolean(1, skalSendeBrev)
             statement.setObject(2, behandlingId)
-            checkInternFeil(statement.executeUpdate() == 1) {
+            krev(statement.executeUpdate() == 1) {
                 "Kunne ikke send brev behandling for $behandlingId"
             }
         }

diff --git a/apps/etterlatte-behandling/src/main/kotlin/behandling/BehandlingService.kt b/apps/etterlatte-behandling/src/main/kotlin/behandling/BehandlingService.kt
@@ -46,7 +46,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.feilhaandtering.krev
 import no.nav.etterlatte.libs.common.grunnlag.Grunnlagsopplysning
 import no.nav.etterlatte.libs.common.grunnlag.NyeSaksopplysninger
 import no.nav.etterlatte.libs.common.grunnlag.lagOpplysning
@@ -639,7 +639,7 @@ internal class BehandlingServiceImpl(
     ) {
         inTransaction {
             val behandling = behandlingDao.hentBehandling(behandlingId)
-            checkInternFeil(behandling != null) {
+            krev(behandling != null) {
                 "Behandling finnes ikke $behandlingId"
             }
             when (behandling!!.type) {

diff --git a/...te-behandling/src/main/kotlin/behandling/aktivitetsplikt/AktivitetspliktOppgaveService.kt b/...te-behandling/src/main/kotlin/behandling/aktivitetsplikt/AktivitetspliktOppgaveService.kt
@@ -19,7 +19,7 @@ import no.nav.etterlatte.libs.common.behandling.UtlandstilknytningType
 import no.nav.etterlatte.libs.common.feilhaandtering.GenerellIkkeFunnetException
 import no.nav.etterlatte.libs.common.feilhaandtering.InternfeilException
 import no.nav.etterlatte.libs.common.feilhaandtering.UgyldigForespoerselException
-import no.nav.etterlatte.libs.common.feilhaandtering.checkNotNullorThrowInternFeil
+import no.nav.etterlatte.libs.common.feilhaandtering.krevIkkeNull
 import no.nav.etterlatte.libs.common.grunnlag.Grunnlagsopplysning
 import no.nav.etterlatte.libs.common.oppgave.OppgaveIntern
 import no.nav.etterlatte.libs.common.oppgave.OppgaveType
@@ -172,10 +172,10 @@ class AktivitetspliktOppgaveService(
 
         val aktivitetsgrad = mapAktivitetsgradstypeTilAktivtetsgrad(sisteAktivtetsgrad.aktivitetsgrad)
         val utbetaling =
-            checkNotNullorThrowInternFeil(brevdata.utbetaling) { "Mangler utbetaling for utbetaling for oppgave ${oppgave.id}" }
+            krevIkkeNull(brevdata.utbetaling) { "Mangler utbetaling for utbetaling for oppgave ${oppgave.id}" }
         val redusertEtterInntekt =
-            checkNotNullorThrowInternFeil(brevdata.redusertEtterInntekt) { "Mangler redusert-inntekt for oppgave ${oppgave.id}" }
-        val spraak = checkNotNullorThrowInternFeil(brevdata.spraak) { "Mangler spraak for oppgave ${oppgave.id}" }
+            krevIkkeNull(brevdata.redusertEtterInntekt) { "Mangler redusert-inntekt for oppgave ${oppgave.id}" }
+        val spraak = krevIkkeNull(brevdata.spraak) { "Mangler spraak for oppgave ${oppgave.id}" }
         val nasjonalEllerUtlandMapped = mapNasjonalEllerUtland(nasjonalEllerUtland.type)
 
         val brevparametere =

diff --git a/...tterlatte-behandling/src/main/kotlin/behandling/aktivitetsplikt/AktivitetspliktService.kt b/...tterlatte-behandling/src/main/kotlin/behandling/aktivitetsplikt/AktivitetspliktService.kt
@@ -35,7 +35,7 @@ import no.nav.etterlatte.libs.common.behandling.tilVirkningstidspunkt
 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.feilhaandtering.krev
 import no.nav.etterlatte.libs.common.grunnlag.Grunnlagsopplysning
 import no.nav.etterlatte.libs.common.grunnlag.hentDoedsdato
 import no.nav.etterlatte.libs.common.oppgave.OppgaveIntern
@@ -328,7 +328,7 @@ class AktivitetspliktService(
         if (aktivitetsgrad.id != null) {
             aktivitetspliktAktivitetsgradDao.oppdaterAktivitetsgrad(aktivitetsgrad, kilde, behandlingId)
         } else {
-            checkInternFeil(
+            krev(
                 aktivitetspliktAktivitetsgradDao.hentAktivitetsgradForBehandling(behandlingId).isEmpty(),
             ) { "Aktivitetsgrad finnes allerede for behandling $behandlingId" }
             val unntak = aktivitetspliktUnntakDao.hentUnntakForBehandling(behandlingId)
@@ -425,7 +425,7 @@ class AktivitetspliktService(
         if (unntak.id != null) {
             aktivitetspliktUnntakDao.oppdaterUnntak(unntak, kilde, behandlingId)
         } else {
-            checkInternFeil(
+            krev(
                 aktivitetspliktUnntakDao.hentUnntakForBehandling(behandlingId).isEmpty(),
             ) { "Unntak finnes allerede for behandling $behandlingId" }
 

diff --git a/.../src/main/kotlin/behandling/aktivitetsplikt/vurdering/AktivitetspliktAktivitetsgradDao.kt b/.../src/main/kotlin/behandling/aktivitetsplikt/vurdering/AktivitetspliktAktivitetsgradDao.kt
@@ -8,7 +8,7 @@ import no.nav.etterlatte.common.ConnectionAutoclosing
 import no.nav.etterlatte.libs.common.aktivitetsplikt.AktivitetspliktAktivitetsgradDto
 import no.nav.etterlatte.libs.common.aktivitetsplikt.VurdertAktivitetsgrad
 import no.nav.etterlatte.libs.common.feilhaandtering.InternfeilException
-import no.nav.etterlatte.libs.common.feilhaandtering.checkInternFeil
+import no.nav.etterlatte.libs.common.feilhaandtering.krev
 import no.nav.etterlatte.libs.common.grunnlag.Grunnlagsopplysning
 import no.nav.etterlatte.libs.common.sak.SakId
 import no.nav.etterlatte.libs.database.setSakId
@@ -33,7 +33,7 @@ class AktivitetspliktAktivitetsgradDao(
         oppgaveId: UUID? = null,
         behandlingId: UUID? = null,
     ) = connectionAutoclosing.hentConnection {
-        checkInternFeil(oppgaveId != null || behandlingId != null) {
+        krev(oppgaveId != null || behandlingId != null) {
             "Kan ikke opprette aktivitetsgrad som ikke er koblet på en behandling eller oppgave. $sakId"
         }
 

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,7 +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.feilhaandtering.krev
 import no.nav.etterlatte.libs.common.objectMapper
 import no.nav.etterlatte.libs.database.setJsonb
 import no.nav.etterlatte.libs.database.singleOrNull
@@ -87,7 +87,7 @@ class BehandlingInfoDao(
                     setJsonb(2, brevutfall)
                 }.run { executeUpdate() }
                     .also {
-                        checkInternFeil(it == 1) {
+                        krev(it == 1) {
                             "Kunne ikke lagreBrevutfall behandling for ${brevutfall.behandlingId}"
                         }
                     }.let {
@@ -130,7 +130,7 @@ class BehandlingInfoDao(
                     setJsonb(2, etterbetaling)
                 }.run { executeUpdate() }
                     .also {
-                        checkInternFeil(it == 1) {
+                        krev(it == 1) {
                             "Kunne ikke lagreBrevutfall behandling for ${etterbetaling.behandlingId}"
                         }
                     }.let {
@@ -153,7 +153,7 @@ class BehandlingInfoDao(
                     setObject(2, behandlingId)
                 }.run { executeUpdate() }
                     .also {
-                        checkInternFeil(it == 1) {
+                        krev(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,7 +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.common.feilhaandtering.krev
 import no.nav.etterlatte.libs.database.setJsonb
 import no.nav.etterlatte.libs.database.singleOrNull
 import java.sql.ResultSet
@@ -30,7 +30,7 @@ class BosattUtlandDao(
                 statement.setObject(2, bosattUtland.rinanummer)
                 statement.setJsonb(3, bosattUtland.mottatteSeder)
                 statement.setJsonb(4, bosattUtland.sendteSeder)
-                checkInternFeil(statement.executeUpdate() == 1) { "Kunne ikke lagre utland for behandlingId: ${bosattUtland.behandlingId}" }
+                krev(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,7 +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.feilhaandtering.krev
 import no.nav.etterlatte.libs.common.sak.DisttribuertEllerIverksatt
 import no.nav.etterlatte.libs.common.sak.KjoeringDistEllerIverksattRequest
 import no.nav.etterlatte.libs.common.sak.KjoeringRequest
@@ -74,7 +74,7 @@ class OmregningDao(
                 statement.setString(5, request.corrId)
                 statement.setString(6, request.feilendeSteg)
                 statement.executeUpdate().also {
-                    checkInternFeil(it > 0) {
+                    krev(it > 0) {
                         "Kunne ikke oppdaterKjoering for id sakid ${request.sakId}"
                     }
                 }
@@ -106,7 +106,7 @@ class OmregningDao(
                 statement.setBigDecimal(10, request.avkortingEtter)
                 statement.setBigDecimal(11, request.vedtakBeloep)
                 statement.executeUpdate().also {
-                    checkInternFeil(it > 0) {
+                    krev(it > 0) {
                         "Kunne ikke lagreKjoering for id sakid ${request.sakId}"
                     }
                 }
@@ -140,7 +140,7 @@ class OmregningDao(
                 statement.setString(3, status.name)
                 statement.setBoolean(4, true)
                 statement.executeUpdate().also {
-                    checkInternFeil(it > 0) {
+                    krev(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,7 +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.feilhaandtering.krev
 import no.nav.etterlatte.libs.common.objectMapper
 import no.nav.etterlatte.libs.common.sak.Sak
 import no.nav.etterlatte.libs.common.sak.SakId
@@ -196,7 +196,7 @@ class TilbakekrevingDao(
         }
         statement.setBoolean(7, tilbakekrevingBehandling.sendeBrev)
         statement.executeUpdate().also {
-            checkInternFeil(it == 1) {
+            krev(it == 1) {
                 "Kunne ikke lagre tilbaekreving behandling for sakid ${tilbakekrevingBehandling.sak.id}"
             }
         }
@@ -215,7 +215,7 @@ class TilbakekrevingDao(
             )
         statement.setObject(1, tilbakekrevingId)
         statement.executeUpdate().also {
-            checkInternFeil(it > 0) {
+            krev(it > 0) {
                 "Kunne ikke deleteTilbakekrevingsperioder behandling for id $tilbakekrevingId"
             }
         }

diff --git a/apps/etterlatte-behandling/src/main/kotlin/oppgaveGosys/GosysOppgaveService.kt b/apps/etterlatte-behandling/src/main/kotlin/oppgaveGosys/GosysOppgaveService.kt
@@ -12,7 +12,6 @@ import no.nav.etterlatte.inTransaction
 import no.nav.etterlatte.libs.common.Enhetsnummer
 import no.nav.etterlatte.libs.common.feilhaandtering.IkkeFunnetException
 import no.nav.etterlatte.libs.common.feilhaandtering.UgyldigForespoerselException
-import no.nav.etterlatte.libs.common.feilhaandtering.checkUgyldigForespoerselException
 import no.nav.etterlatte.libs.common.oppgave.OppgaveIntern
 import no.nav.etterlatte.libs.common.oppgave.OppgaveKilde
 import no.nav.etterlatte.libs.common.oppgave.OppgaveSaksbehandler
@@ -247,8 +246,11 @@ class GosysOppgaveServiceImpl(
             throw StoetterKunFlyttingAvJournalfoeringsoppgave()
         }
 
-        checkUgyldigForespoerselException(value = !gosysOppgave.journalpostId.isNullOrBlank(), code = "KAN_IKKE_FLYTTE_GOSYS_OPPGAVE") {
-            "Kan ikke flytte oppgave når journalpostId mangler (oppgaveId=${gosysOppgave.id})"
+        if (gosysOppgave.journalpostId.isNullOrBlank()) {
+            throw UgyldigForespoerselException(
+                code = "KAN_IKKE_FLYTTE_GOSYS_OPPGAVE",
+                detail = "Kan ikke flytte oppgave når journalpostId mangler (oppgaveId=${gosysOppgave.id})",
+            )
         }
 
         val nyOppgave =

diff --git a/apps/etterlatte-behandling/src/main/kotlin/sak/SakSkrivDao.kt b/apps/etterlatte-behandling/src/main/kotlin/sak/SakSkrivDao.kt
@@ -3,7 +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.feilhaandtering.krev
 import no.nav.etterlatte.libs.common.person.AdressebeskyttelseGradering
 import no.nav.etterlatte.libs.common.person.Folkeregisteridentifikator
 import no.nav.etterlatte.libs.common.sak.Sak
@@ -40,7 +40,7 @@ class SakSkrivDao(
                     logger.info(
                         "Oppdaterer adressebeskyttelse med: $adressebeskyttelseGradering for sak med id: $sakId, antall oppdatert er $it",
                     )
-                    checkInternFeil(it > 0) {
+                    krev(it > 0) {
                         "Kunne ikke oppdaterAdresseBeskyttelse for id sakid $sakId"
                     }
                 }

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
@@ -14,7 +14,7 @@ import kotlinx.coroutines.runBlocking
 import no.nav.etterlatte.grunnbeloep.Grunnbeloep
 import no.nav.etterlatte.libs.common.beregning.AarligInntektsjusteringAvkortingRequest
 import no.nav.etterlatte.libs.common.beregning.MottattInntektsjusteringAvkortigRequest
-import no.nav.etterlatte.libs.common.feilhaandtering.checkInternFeil
+import no.nav.etterlatte.libs.common.feilhaandtering.krev
 import java.util.UUID
 
 class BeregningService(
@@ -96,6 +96,6 @@ class BeregningService(
     suspend fun hentGrunnbeloep(): Grunnbeloep =
         beregningApp
             .get("$url/api/beregning/grunnbeloep")
-            .also { checkInternFeil(it.status.isSuccess()) { "Kunne ikke hente grunnbeloep" } }
+            .also { krev(it.status.isSuccess()) { "Kunne ikke hente grunnbeloep" } }
             .body<Grunnbeloep>()
 }