From 58fe4dbd3ec8330ad334e16366aba3ec00d7d4e4 Mon Sep 17 00:00:00 2001 From: "Nicolas N." Date: Wed, 18 Dec 2024 14:54:01 +0100 Subject: [PATCH] =?UTF-8?q?Forslag=20til=20standard=20p=C3=A5=20require=20?= =?UTF-8?q?og=20check=20(#6643)?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit * Endre navngivning + enkel javadoc * bruke nye funksjoner i koden --- .../samordning/vedtak/Tjenestepensjon.kt | 4 +- .../main/kotlin/behandling/BehandlingDao.kt | 18 +++--- .../kotlin/behandling/BehandlingService.kt | 4 +- .../AktivitetspliktOppgaveService.kt | 8 +-- .../aktivitetsplikt/AktivitetspliktService.kt | 6 +- .../AktivitetspliktAktivitetsgradDao.kt | 4 +- .../behandlinginfo/BehandlingInfoDao.kt | 8 +-- .../bosattutland/BosattUtlandDao.kt | 4 +- .../behandling/omregning/OmregningDao.kt | 8 +-- .../tilbakekreving/TilbakekrevingDao.kt | 6 +- .../oppgaveGosys/GosysOppgaveService.kt | 8 ++- .../src/main/kotlin/sak/SakSkrivDao.kt | 4 +- .../beregningkafka/BeregningService.kt | 4 +- .../kotlin/beregning/BeregningRepository.kt | 12 ++-- .../beregning/grunnlag/RegulerOverstyrt.kt | 8 +-- .../no/nav/etterlatte/brev/BrevService.kt | 5 +- .../nav/etterlatte/brev/db/BrevRepository.kt | 46 +++++++------- .../etterlatte/brev/dokarkiv/Journalpost.kt | 4 +- .../brev/vedtaksbrev/VedtaksbrevService.kt | 4 +- .../main/kotlin/grunnlag/GrunnlagService.kt | 7 +-- .../src/main/kotlin/grunnlag/OpplysningDao.kt | 4 +- .../statistikk/domain/SoeknadStatistikk.kt | 4 +- .../features/soeknad/OpprettSoeknad.kt | 4 +- .../TilbakekrevingHendelseRepository.kt | 6 +- .../main/kotlin/trygdetid/TrygdetidService.kt | 4 +- .../utbetaling/avstemming/AvstemmingDao.kt | 6 +- .../avstemming/Grensesnittavstemming.kt | 4 +- .../iverksetting/utbetaling/UtbetalingDao.kt | 4 +- .../VedtakOgBeregningSammenligner.kt | 8 +-- .../VedtaksvurderingRepository.kt | 16 ++--- .../outbox/OutboxRepository.kt | 4 +- .../libs/common/behandling/Klage.kt | 4 +- .../generellbehandling/GenerellBehandling.kt | 6 +- .../src/main/kotlin/Prosent.kt | 4 +- .../no/nav/etterlatte/brev/Brevkoder.kt | 4 +- .../src/main/kotlin/JdbcUtils.kt | 8 +-- .../src/main/kotlin/kafka/KafkaProdusent.kt | 4 +- .../src/main/kotlin/VedtakDto.kt | 4 +- .../ListenerMedLoggingOgFeilhaandtering.kt | 4 +- .../src/main/kotlin/Enhetsnummer.kt | 4 +- .../kotlin/feilhaandtering/Preconditions.kt | 63 +++++++++++++++++++ .../src/main/kotlin/feilhaandtering/Utils.kt | 46 -------------- .../soeknad/InnsendtSoeknadTestData.kt | 2 +- 43 files changed, 204 insertions(+), 185 deletions(-) create mode 100644 libs/saksbehandling-common/src/main/kotlin/feilhaandtering/Preconditions.kt delete mode 100644 libs/saksbehandling-common/src/main/kotlin/feilhaandtering/Utils.kt 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 index 2455c9e201d..5ee6e42039a 100644 --- 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 index 96cfd57d4d4..59fa5091891 100644 --- 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 index 582515bedc0..a709cde96ab 100644 --- 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/apps/etterlatte-behandling/src/main/kotlin/behandling/aktivitetsplikt/AktivitetspliktOppgaveService.kt b/apps/etterlatte-behandling/src/main/kotlin/behandling/aktivitetsplikt/AktivitetspliktOppgaveService.kt index 169a135edbd..8650f8d90bd 100644 --- a/apps/etterlatte-behandling/src/main/kotlin/behandling/aktivitetsplikt/AktivitetspliktOppgaveService.kt +++ b/apps/etterlatte-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/apps/etterlatte-behandling/src/main/kotlin/behandling/aktivitetsplikt/AktivitetspliktService.kt b/apps/etterlatte-behandling/src/main/kotlin/behandling/aktivitetsplikt/AktivitetspliktService.kt index 8db7a09d5cf..26bb2456869 100644 --- a/apps/etterlatte-behandling/src/main/kotlin/behandling/aktivitetsplikt/AktivitetspliktService.kt +++ b/apps/etterlatte-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/apps/etterlatte-behandling/src/main/kotlin/behandling/aktivitetsplikt/vurdering/AktivitetspliktAktivitetsgradDao.kt b/apps/etterlatte-behandling/src/main/kotlin/behandling/aktivitetsplikt/vurdering/AktivitetspliktAktivitetsgradDao.kt index 6a6fbedcc0e..af2a5f5ba8d 100644 --- a/apps/etterlatte-behandling/src/main/kotlin/behandling/aktivitetsplikt/vurdering/AktivitetspliktAktivitetsgradDao.kt +++ b/apps/etterlatte-behandling/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 index a16943b5821..59a1eb87d14 100644 --- 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 index 530372e14cb..303c0b74d39 100644 --- 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 index 93a4cdea68a..c05b8fbd99d 100644 --- 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 index e2d1326b1df..3a936a5aeaa 100644 --- 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 index 09f74f7e4c7..9d491130b2d 100644 --- 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 index 4c19dbed9fb..8567d2a4c1f 100644 --- 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/apps/etterlatte-beregning-kafka/src/main/kotlin/no/nav/etterlatte/beregningkafka/BeregningService.kt b/apps/etterlatte-beregning-kafka/src/main/kotlin/no/nav/etterlatte/beregningkafka/BeregningService.kt index a4239b664ed..bc8a8c627ee 100644 --- a/apps/etterlatte-beregning-kafka/src/main/kotlin/no/nav/etterlatte/beregningkafka/BeregningService.kt +++ b/apps/etterlatte-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() } diff --git a/apps/etterlatte-beregning/src/main/kotlin/beregning/BeregningRepository.kt b/apps/etterlatte-beregning/src/main/kotlin/beregning/BeregningRepository.kt index f76ce4b2bde..527e32d695f 100644 --- a/apps/etterlatte-beregning/src/main/kotlin/beregning/BeregningRepository.kt +++ b/apps/etterlatte-beregning/src/main/kotlin/beregning/BeregningRepository.kt @@ -11,7 +11,7 @@ import no.nav.etterlatte.libs.common.beregning.BeregningsMetode import no.nav.etterlatte.libs.common.beregning.Beregningsperiode import no.nav.etterlatte.libs.common.beregning.Beregningstype import no.nav.etterlatte.libs.common.beregning.OverstyrtBeregningKategori -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.Metadata import no.nav.etterlatte.libs.common.objectMapper @@ -220,19 +220,19 @@ private fun toBeregningsperiode(row: Row): BeregningsperiodeDAO = private fun toBeregning(beregningsperioder: List): Beregning { val base = beregningsperioder.first().apply { - checkInternFeil(beregningsperioder.all { it.beregningId == beregningId }) { + krev(beregningsperioder.all { it.beregningId == beregningId }) { "Beregningen inneholder forskjellige beregningsIder $beregningId for beregning $beregningId" } - checkInternFeil(beregningsperioder.all { it.behandlingId == behandlingId }) { + krev(beregningsperioder.all { it.behandlingId == behandlingId }) { "Beregningen inneholder forskjellige behandlingIder $behandlingId for beregning $beregningId" } - checkInternFeil(beregningsperioder.all { it.type == type }) { + krev(beregningsperioder.all { it.type == type }) { "Beregningen inneholder forskjellige typer $type for beregning $beregningId" } - checkInternFeil(beregningsperioder.all { it.beregnetDato == beregnetDato }) { + krev(beregningsperioder.all { it.beregnetDato == beregnetDato }) { "Beregningen inneholder forskjellige beregnetDatoer $beregnetDato for beregning $beregningId" } - checkInternFeil(beregningsperioder.all { it.grunnlagMetadata == grunnlagMetadata }) { + krev(beregningsperioder.all { it.grunnlagMetadata == grunnlagMetadata }) { "Beregningen inneholder forskjellige grunnlagMetadata $grunnlagMetadata for beregning $beregningId" } } diff --git a/apps/etterlatte-beregning/src/main/kotlin/beregning/grunnlag/RegulerOverstyrt.kt b/apps/etterlatte-beregning/src/main/kotlin/beregning/grunnlag/RegulerOverstyrt.kt index 536eb32b122..ffef7696ff4 100644 --- a/apps/etterlatte-beregning/src/main/kotlin/beregning/grunnlag/RegulerOverstyrt.kt +++ b/apps/etterlatte-beregning/src/main/kotlin/beregning/grunnlag/RegulerOverstyrt.kt @@ -4,7 +4,7 @@ import no.nav.etterlatte.beregning.regler.overstyr.RegulerManuellBeregningGrunnl import no.nav.etterlatte.beregning.regler.overstyr.grunnbeloepUtenGrunnlag import no.nav.etterlatte.beregning.regler.overstyr.regulerOverstyrtKroneavrundet import no.nav.etterlatte.grunnbeloep.Grunnbeloep -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.objectMapper import no.nav.etterlatte.libs.common.tidspunkt.Tidspunkt @@ -89,17 +89,17 @@ private fun utledGrunnbeloep(reguleringsmaaned: YearMonth) = ).let { resultat -> when (resultat) { is RegelkjoeringResultat.Suksess -> { - checkInternFeil(resultat.periodiserteResultater.size == 2) { + krev(resultat.periodiserteResultater.size == 2) { "Fikk uventet antall perioder for utleding av grunnlag: ${resultat.periodiserteResultater.size}" } resultat.periodiserteResultater.let { val gammelG: Grunnbeloep = it[0].resultat.verdi val forrigeGrunnbeloepDato = reguleringsmaaned.minusYears(1) - checkInternFeil(gammelG.dato == forrigeGrunnbeloepDato) { + krev(gammelG.dato == forrigeGrunnbeloepDato) { "Dato til utledet forrige grunnbeløp er ikke forventet dato $forrigeGrunnbeloepDato" } val nyG: Grunnbeloep = it[1].resultat.verdi - checkInternFeil(nyG.dato == reguleringsmaaned) { + krev(nyG.dato == reguleringsmaaned) { "Dato til utledet nytt grunnbeløp er ikke forventet dato $reguleringsmaaned" } Pair(gammelG, nyG) diff --git a/apps/etterlatte-brev-api/src/main/kotlin/no/nav/etterlatte/brev/BrevService.kt b/apps/etterlatte-brev-api/src/main/kotlin/no/nav/etterlatte/brev/BrevService.kt index cdd8720ca78..5e29f41a32f 100644 --- a/apps/etterlatte-brev-api/src/main/kotlin/no/nav/etterlatte/brev/BrevService.kt +++ b/apps/etterlatte-brev-api/src/main/kotlin/no/nav/etterlatte/brev/BrevService.kt @@ -26,7 +26,7 @@ import no.nav.etterlatte.brev.vedtaksbrev.UgyldigAntallMottakere import no.nav.etterlatte.brev.vedtaksbrev.UgyldigMottakerKanIkkeFerdigstilles import no.nav.etterlatte.libs.common.feilhaandtering.InternfeilException import no.nav.etterlatte.libs.common.feilhaandtering.UgyldigForespoerselException -import no.nav.etterlatte.libs.common.feilhaandtering.checkUgyldigForespoerselException +import no.nav.etterlatte.libs.common.feilhaandtering.sjekk import no.nav.etterlatte.libs.common.logging.sikkerlogger import no.nav.etterlatte.libs.common.sak.Sak import no.nav.etterlatte.libs.common.sak.SakId @@ -404,7 +404,8 @@ class BrevService( logger.info("Sjekker om brev med id=$id kan slettes") val brev = sjekkOmBrevKanEndres(id) - checkUgyldigForespoerselException(value = brev.behandlingId == null, code = "BREV_KAN_IKKE_SLETTES") { + + sjekk(brev.behandlingId == null) { "Brev med id=$id er et vedtaksbrev og kan ikke slettes" } diff --git a/apps/etterlatte-brev-api/src/main/kotlin/no/nav/etterlatte/brev/db/BrevRepository.kt b/apps/etterlatte-brev-api/src/main/kotlin/no/nav/etterlatte/brev/db/BrevRepository.kt index 330ec9f5ccf..4b5526a1c5f 100644 --- a/apps/etterlatte-brev-api/src/main/kotlin/no/nav/etterlatte/brev/db/BrevRepository.kt +++ b/apps/etterlatte-brev-api/src/main/kotlin/no/nav/etterlatte/brev/db/BrevRepository.kt @@ -40,7 +40,7 @@ import no.nav.etterlatte.brev.model.Spraak import no.nav.etterlatte.brev.model.Status import no.nav.etterlatte.brev.model.opprettBrevFra import no.nav.etterlatte.libs.common.deserialize -import no.nav.etterlatte.libs.common.feilhaandtering.checkInternFeil +import no.nav.etterlatte.libs.common.feilhaandtering.krev import no.nav.etterlatte.libs.common.person.MottakerFoedselsnummer import no.nav.etterlatte.libs.common.sak.SakId import no.nav.etterlatte.libs.common.tidspunkt.Tidspunkt @@ -119,13 +119,13 @@ class BrevRepository( ), ).asUpdate, ).also { - checkInternFeil(it == 1) { "Brev fikk ikke oppdatert payload brevid: $id" } + krev(it == 1) { "Brev fikk ikke oppdatert payload brevid: $id" } } tx .lagreHendelse(id, Status.OPPDATERT, payload.toJson(), bruker) .also { - checkInternFeil(it == 1) { "Brev fikk ikke oppdatert hendelse brevid: $id" } + krev(it == 1) { "Brev fikk ikke oppdatert hendelse brevid: $id" } } } @@ -145,13 +145,13 @@ class BrevRepository( ), ).asUpdate, ).also { - checkInternFeil(it == 1) { "Brev fikk ikke oppdatert vedlegg id: $id" } + krev(it == 1) { "Brev fikk ikke oppdatert vedlegg id: $id" } } tx .lagreHendelse(id, Status.OPPDATERT, payload.toJson(), bruker) .also { - checkInternFeil(it == 1) { "Brev fikk ikke oppdatert hendelse for vedlegg id: $id" } + krev(it == 1) { "Brev fikk ikke oppdatert hendelse for vedlegg id: $id" } } } @@ -201,13 +201,13 @@ class BrevRepository( ), ).asUpdate, ).also { - checkInternFeil(it == 1) { "Brev fikk ikke oppdatert mottaker id: $id" } + krev(it == 1) { "Brev fikk ikke oppdatert mottaker id: $id" } } tx .lagreHendelse(id, Status.OPPDATERT, mottaker.toJson(), bruker) .also { - checkInternFeil(it == 1) { "Brev fikk ikke oppdater hendelse for oppdatert mottaker id: $id" } + krev(it == 1) { "Brev fikk ikke oppdater hendelse for oppdatert mottaker id: $id" } } } @@ -219,13 +219,13 @@ class BrevRepository( tx .run(queryOf("DELETE FROM mottaker WHERE id = ? AND brev_id = ?", mottakerId, brevId).asUpdate) .also { - checkInternFeil(it == 1) { "Brev fikk ikke slettet mottaker id: $brevId" } + krev(it == 1) { "Brev fikk ikke slettet mottaker id: $brevId" } } tx .lagreHendelse(brevId, Status.OPPDATERT, "mottaker med id=$mottakerId fjernet fra brevet ", bruker) .also { - checkInternFeil(it == 1) { "Brev fikk ikke oppdater hendelse for slettet mottaker id: $brevId" } + krev(it == 1) { "Brev fikk ikke oppdater hendelse for slettet mottaker id: $brevId" } } } @@ -244,13 +244,13 @@ class BrevRepository( ), ).asUpdate, ).also { - checkInternFeil(it == 1) { "Brev fikk ikke oppdatert tittel id: $brevId" } + krev(it == 1) { "Brev fikk ikke oppdatert tittel id: $brevId" } } tx .lagreHendelse(brevId, Status.OPPDATERT, tittel.toJson(), bruker) .also { - checkInternFeil(it == 1) { "Brev fikk ikke oppdater hendelse for oppdatert tittel id: $brevId" } + krev(it == 1) { "Brev fikk ikke oppdater hendelse for oppdatert tittel id: $brevId" } } } @@ -269,13 +269,13 @@ class BrevRepository( ), ).asUpdate, ).also { - checkInternFeil(it == 1) { "Brev fikk ikke oppdatert språk id: $brevId" } + krev(it == 1) { "Brev fikk ikke oppdatert språk id: $brevId" } } tx .lagreHendelse(brevId, Status.OPPDATERT, spraak.toJson(), bruker) .also { - checkInternFeil(it == 1) { "Brev fikk ikke oppdater hendelse for oppdatert språk id: $brevId" } + krev(it == 1) { "Brev fikk ikke oppdater hendelse for oppdatert språk id: $brevId" } } } @@ -295,11 +295,11 @@ class BrevRepository( ), ).asUpdate, ).also { oppdatert -> - checkInternFeil(oppdatert == 1) { "Pdf ble ikke opprettet id: $brevId" } + krev(oppdatert == 1) { "Pdf ble ikke opprettet id: $brevId" } } tx.lagreHendelse(brevId, Status.FERDIGSTILT, bruker = bruker).also { - checkInternFeil(it == 1) { "Brev fikk ikke oppdater hendelse ferdigstilt for lagre pdf id: $brevId" } + krev(it == 1) { "Brev fikk ikke oppdater hendelse ferdigstilt for lagre pdf id: $brevId" } } } } @@ -318,7 +318,7 @@ class BrevRepository( "bytes" to pdf.bytes, ), ).asUpdate, - ).also { oppdatert -> checkInternFeil(oppdatert == 1) { "Fikk ikke lagret pdf id: $brevId" } } + ).also { oppdatert -> krev(oppdatert == 1) { "Fikk ikke lagret pdf id: $brevId" } } } } @@ -328,7 +328,7 @@ class BrevRepository( ) { using(sessionOf(ds)) { it.lagreHendelse(brevId, Status.FERDIGSTILT, bruker = bruker).also { - checkInternFeil(it == 1) { "Hendelse ferdigstilt ble ikke gjort for id $brevId" } + krev(it == 1) { "Hendelse ferdigstilt ble ikke gjort for id $brevId" } } } } @@ -340,7 +340,7 @@ class BrevRepository( ) { using(sessionOf(ds)) { it.lagreHendelse(brevId, Status.UTGAATT, "${bruker.ident()}: $kommentar".toJson(), bruker).also { - checkInternFeil(it == 1) { "Hendelse utgått ble ikke gjort for id $brevId" } + krev(it == 1) { "Hendelse utgått ble ikke gjort for id $brevId" } } } } @@ -371,7 +371,7 @@ class BrevRepository( ulagretBrev.mottakere .sumOf { tx.opprettMottaker(brevId, it) } .also { opprettet -> - checkInternFeil(opprettet == ulagretBrev.mottakere.size) { + krev(opprettet == ulagretBrev.mottakere.size) { "Mottaker ble ikke opprettet for id $brevId" } } @@ -388,11 +388,11 @@ class BrevRepository( "payload_vedlegg" to ulagretBrev.innholdVedlegg?.toJson(), ), ).asUpdate, - ).also { opprettet -> checkInternFeil(opprettet == 1) { "Innhold ble ikke opprettet for id $brevId" } } + ).also { opprettet -> krev(opprettet == 1) { "Innhold ble ikke opprettet for id $brevId" } } tx .lagreHendelse(brevId, Status.OPPRETTET, ulagretBrev.opprettet, bruker = bruker) - .also { oppdatert -> checkInternFeil(oppdatert == 1) { "Hendelse ble ikke satt til opprettet for id $brevId" } } + .also { oppdatert -> krev(oppdatert == 1) { "Hendelse ble ikke satt til opprettet for id $brevId" } } opprettBrevFra(brevId, ulagretBrev) } @@ -408,7 +408,7 @@ class BrevRepository( journalpostResponse.journalpostId, mottakerId, ).asUpdate, - ).also { oppdatert -> checkInternFeil(oppdatert == 1) { "Journalpost ble ikke lagre med ny id for: $mottakerId" } } + ).also { oppdatert -> krev(oppdatert == 1) { "Journalpost ble ikke lagre med ny id for: $mottakerId" } } } fun settBrevJournalfoert( @@ -431,7 +431,7 @@ class BrevRepository( distResponse.bestillingsId, mottakerId, ).asUpdate, - ).also { oppdatert -> checkInternFeil(oppdatert == 1) { "feilet på oppdatering av mottaker for id $mottakerId" } } + ).also { oppdatert -> krev(oppdatert == 1) { "feilet på oppdatering av mottaker for id $mottakerId" } } } fun settBrevDistribuert( diff --git a/apps/etterlatte-brev-api/src/main/kotlin/no/nav/etterlatte/brev/dokarkiv/Journalpost.kt b/apps/etterlatte-brev-api/src/main/kotlin/no/nav/etterlatte/brev/dokarkiv/Journalpost.kt index 45fe5ed0ea0..f7f8b37925a 100644 --- a/apps/etterlatte-brev-api/src/main/kotlin/no/nav/etterlatte/brev/dokarkiv/Journalpost.kt +++ b/apps/etterlatte-brev-api/src/main/kotlin/no/nav/etterlatte/brev/dokarkiv/Journalpost.kt @@ -2,7 +2,7 @@ package no.nav.etterlatte.brev.dokarkiv import com.fasterxml.jackson.annotation.JsonInclude import no.nav.etterlatte.libs.common.Enhetsnummer -import no.nav.etterlatte.libs.common.feilhaandtering.checkInternFeil +import no.nav.etterlatte.libs.common.feilhaandtering.krev import org.slf4j.LoggerFactory interface OpprettJournalpost { @@ -42,7 +42,7 @@ data class JournalpostRequest( override val tittel: String, ) : OpprettJournalpost { init { - checkInternFeil(journalposttype != JournalPostType.NOTAT) { + krev(journalposttype != JournalPostType.NOTAT) { "${this::class.simpleName} skal ikke brukes til opprettelse av Notat. " + "Bruk ${OpprettNotatJournalpostRequest::class.simpleName} i stedet." } diff --git a/apps/etterlatte-brev-api/src/main/kotlin/no/nav/etterlatte/brev/vedtaksbrev/VedtaksbrevService.kt b/apps/etterlatte-brev-api/src/main/kotlin/no/nav/etterlatte/brev/vedtaksbrev/VedtaksbrevService.kt index 603220d2b7c..b5a2aebe16f 100644 --- a/apps/etterlatte-brev-api/src/main/kotlin/no/nav/etterlatte/brev/vedtaksbrev/VedtaksbrevService.kt +++ b/apps/etterlatte-brev-api/src/main/kotlin/no/nav/etterlatte/brev/vedtaksbrev/VedtaksbrevService.kt @@ -21,7 +21,7 @@ import no.nav.etterlatte.brev.varselbrev.BrevDataMapperRedigerbartUtfallVarsel import no.nav.etterlatte.libs.common.behandling.SakType 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 import no.nav.etterlatte.libs.common.logging.sikkerlogger import no.nav.etterlatte.libs.common.sak.SakId import no.nav.etterlatte.libs.common.toJson @@ -61,7 +61,7 @@ class VedtaksbrevService( behandlingId: UUID, brukerTokenInfo: BrukerTokenInfo, ): Brev { - checkInternFeil(db.hentBrevForBehandling(behandlingId, Brevtype.VEDTAK).firstOrNull() == null) { + krev(db.hentBrevForBehandling(behandlingId, Brevtype.VEDTAK).firstOrNull() == null) { "Vedtaksbrev finnes allerede på behandling (id=$behandlingId) og kan ikke opprettes på nytt" } diff --git a/apps/etterlatte-grunnlag/src/main/kotlin/grunnlag/GrunnlagService.kt b/apps/etterlatte-grunnlag/src/main/kotlin/grunnlag/GrunnlagService.kt index 00cbb25546d..3ab28056ff6 100644 --- a/apps/etterlatte-grunnlag/src/main/kotlin/grunnlag/GrunnlagService.kt +++ b/apps/etterlatte-grunnlag/src/main/kotlin/grunnlag/GrunnlagService.kt @@ -12,8 +12,7 @@ import no.nav.etterlatte.libs.common.deserialize import no.nav.etterlatte.libs.common.feilhaandtering.GenerellIkkeFunnetException import no.nav.etterlatte.libs.common.feilhaandtering.IkkeFunnetException import no.nav.etterlatte.libs.common.feilhaandtering.IkkeTillattException -import no.nav.etterlatte.libs.common.feilhaandtering.InternfeilException -import no.nav.etterlatte.libs.common.feilhaandtering.checkNotNullOrThrowException +import no.nav.etterlatte.libs.common.feilhaandtering.sjekkIkkeNull import no.nav.etterlatte.libs.common.grunnlag.Grunnlag import no.nav.etterlatte.libs.common.grunnlag.Grunnlagsopplysning import no.nav.etterlatte.libs.common.grunnlag.OppdaterGrunnlagRequest @@ -390,8 +389,8 @@ class RealGrunnlagService( ) opplysningDao.laasGrunnlagVersjonForBehandling(skalLaasesId) - return checkNotNullOrThrowException(opplysningDao.hentBehandlingVersjon(skalLaasesId)) { - InternfeilException("Fant ikke låst grunnlagsversjon vi akkurat la inn :(") + return sjekkIkkeNull(opplysningDao.hentBehandlingVersjon(skalLaasesId)) { + "Fant ikke låst grunnlagsversjon vi akkurat la inn :(" } } diff --git a/apps/etterlatte-grunnlag/src/main/kotlin/grunnlag/OpplysningDao.kt b/apps/etterlatte-grunnlag/src/main/kotlin/grunnlag/OpplysningDao.kt index ce060b887cf..3071f4317e1 100644 --- a/apps/etterlatte-grunnlag/src/main/kotlin/grunnlag/OpplysningDao.kt +++ b/apps/etterlatte-grunnlag/src/main/kotlin/grunnlag/OpplysningDao.kt @@ -2,7 +2,7 @@ package no.nav.etterlatte.grunnlag import com.fasterxml.jackson.databind.JsonNode import com.fasterxml.jackson.module.kotlin.readValue -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.opplysningstyper.Opplysningstype import no.nav.etterlatte.libs.common.objectMapper @@ -235,7 +235,7 @@ class OpplysningDao( setBoolean(4, false) }.executeUpdate() .also { - checkInternFeil(it > 0) { "Kunne ikke oppdaterVersjonForBehandling for behandlingid $behandlingId sakid: $sakId" } + krev(it > 0) { "Kunne ikke oppdaterVersjonForBehandling for behandlingid $behandlingId sakid: $sakId" } } } diff --git a/apps/etterlatte-statistikk/src/main/kotlin/statistikk/domain/SoeknadStatistikk.kt b/apps/etterlatte-statistikk/src/main/kotlin/statistikk/domain/SoeknadStatistikk.kt index 8ddeeb3a7ee..0a742138ca7 100644 --- a/apps/etterlatte-statistikk/src/main/kotlin/statistikk/domain/SoeknadStatistikk.kt +++ b/apps/etterlatte-statistikk/src/main/kotlin/statistikk/domain/SoeknadStatistikk.kt @@ -1,7 +1,7 @@ package no.nav.etterlatte.statistikk.domain import no.nav.etterlatte.libs.common.behandling.SakType -import no.nav.etterlatte.libs.common.feilhaandtering.checkInternFeil +import no.nav.etterlatte.libs.common.feilhaandtering.krev data class SoeknadStatistikk( val soeknadId: Long, @@ -15,7 +15,7 @@ data class SoeknadStatistikk( true -> kriterierForIngenBehandling.isEmpty() false -> kriterierForIngenBehandling.isNotEmpty() } - checkInternFeil(erKonsekvent) { + krev(erKonsekvent) { "En søknad skal enten være gyldig for behandling med ingen kriterier for ingen behandling, " + "eller den er ikke gyldig for behandling på grunn av en eller flere kriterier " } diff --git a/apps/etterlatte-testdata/src/main/kotlin/no/nav/etterlatte/testdata/features/soeknad/OpprettSoeknad.kt b/apps/etterlatte-testdata/src/main/kotlin/no/nav/etterlatte/testdata/features/soeknad/OpprettSoeknad.kt index 7150091e08b..228b009a7ac 100644 --- a/apps/etterlatte-testdata/src/main/kotlin/no/nav/etterlatte/testdata/features/soeknad/OpprettSoeknad.kt +++ b/apps/etterlatte-testdata/src/main/kotlin/no/nav/etterlatte/testdata/features/soeknad/OpprettSoeknad.kt @@ -11,7 +11,7 @@ import io.ktor.server.routing.Route import io.ktor.server.routing.get import io.ktor.server.routing.post import no.nav.etterlatte.TestDataFeature -import no.nav.etterlatte.libs.common.feilhaandtering.checkInternFeil +import no.nav.etterlatte.libs.common.feilhaandtering.krev import no.nav.etterlatte.libs.common.innsendtsoeknad.common.SoeknadType import no.nav.etterlatte.libs.ktor.token.Claims import no.nav.etterlatte.libs.ktor.token.brukerTokenInfo @@ -104,7 +104,7 @@ private fun opprettSoeknadJson( behandlingssteg: Behandlingssteg, ): String { if (soeknadType == SoeknadType.BARNEPENSJON) { - checkInternFeil(barnFnr.isNotEmpty()) { + krev(barnFnr.isNotEmpty()) { "Kan ikke opprette barnepensjon uten fnr på barn!" } } diff --git a/apps/etterlatte-tilbakekreving/src/main/kotlin/tilbakekreving/TilbakekrevingHendelseRepository.kt b/apps/etterlatte-tilbakekreving/src/main/kotlin/tilbakekreving/TilbakekrevingHendelseRepository.kt index afb12b2bf46..e1c66014365 100644 --- a/apps/etterlatte-tilbakekreving/src/main/kotlin/tilbakekreving/TilbakekrevingHendelseRepository.kt +++ b/apps/etterlatte-tilbakekreving/src/main/kotlin/tilbakekreving/TilbakekrevingHendelseRepository.kt @@ -3,7 +3,7 @@ package no.nav.etterlatte.tilbakekreving import kotliquery.queryOf import kotliquery.sessionOf import kotliquery.using -import no.nav.etterlatte.libs.common.feilhaandtering.checkInternFeil +import no.nav.etterlatte.libs.common.feilhaandtering.krev import no.nav.etterlatte.libs.common.sak.SakId import no.nav.etterlatte.libs.common.tidspunkt.Tidspunkt import no.nav.etterlatte.libs.common.tidspunkt.toTimestamp @@ -71,7 +71,7 @@ class TilbakekrevingHendelseRepository( ), ).let { query -> session.update(query).also { - checkInternFeil(it == 1) { + krev(it == 1) { "Feil under lagring av hendelse for sak $sakId" } } @@ -130,7 +130,7 @@ class TilbakekrevingHendelseRepository( ), ).let { query -> session.update(query).also { - checkInternFeil(it == 1) { + krev(it == 1) { "Feil under oppdatering av hendelse for sak $sakId" } } diff --git a/apps/etterlatte-trygdetid/src/main/kotlin/trygdetid/TrygdetidService.kt b/apps/etterlatte-trygdetid/src/main/kotlin/trygdetid/TrygdetidService.kt index fb7a0024bdc..861a1cf4b47 100644 --- a/apps/etterlatte-trygdetid/src/main/kotlin/trygdetid/TrygdetidService.kt +++ b/apps/etterlatte-trygdetid/src/main/kotlin/trygdetid/TrygdetidService.kt @@ -18,7 +18,7 @@ import no.nav.etterlatte.libs.common.behandling.Revurderingaarsak import no.nav.etterlatte.libs.common.feilhaandtering.GenerellIkkeFunnetException 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 import no.nav.etterlatte.libs.common.grunnlag.Grunnlagsdata import no.nav.etterlatte.libs.common.grunnlag.Grunnlagsopplysning import no.nav.etterlatte.libs.common.grunnlag.hentDoedsdato @@ -888,7 +888,7 @@ class TrygdetidServiceImpl( val trygdetiderKilde = trygdetidRepository.hentTrygdetiderForBehandling(kildeBehandlingId) val trygdetiderMaal = trygdetidRepository.hentTrygdetiderForBehandling(behandlingId) - checkInternFeil(trygdetiderMaal.map { it.ident }.sorted() == trygdetiderKilde.map { it.ident }.sorted()) { + krev(trygdetiderMaal.map { it.ident }.sorted() == trygdetiderKilde.map { it.ident }.sorted()) { "Trygdetidene gjelder forskjellige avdøde" } diff --git a/apps/etterlatte-utbetaling/src/main/kotlin/no/nav/etterlatte/utbetaling/avstemming/AvstemmingDao.kt b/apps/etterlatte-utbetaling/src/main/kotlin/no/nav/etterlatte/utbetaling/avstemming/AvstemmingDao.kt index 69cb0011bc4..fa0df8e6419 100644 --- a/apps/etterlatte-utbetaling/src/main/kotlin/no/nav/etterlatte/utbetaling/avstemming/AvstemmingDao.kt +++ b/apps/etterlatte-utbetaling/src/main/kotlin/no/nav/etterlatte/utbetaling/avstemming/AvstemmingDao.kt @@ -5,7 +5,7 @@ import kotliquery.param import kotliquery.queryOf import kotliquery.sessionOf import kotliquery.using -import no.nav.etterlatte.libs.common.feilhaandtering.checkInternFeil +import no.nav.etterlatte.libs.common.feilhaandtering.krev import no.nav.etterlatte.libs.common.tidspunkt.Tidspunkt import no.nav.etterlatte.libs.common.tidspunkt.toTimestamp import no.nav.etterlatte.libs.common.toJson @@ -40,7 +40,7 @@ class AvstemmingDao( session.run( it.asUpdate, ) - }.also { checkInternFeil(it == 1) { "Kunne ikke opprette avstemming" } } + }.also { krev(it == 1) { "Kunne ikke opprette avstemming" } } } fun hentDatoOpprettetForSisteKonsistensavstemming(saktype: Saktype): Tidspunkt? = @@ -89,7 +89,7 @@ class AvstemmingDao( "saktype" to grensesnittavstemming.saktype.name.param(), ), ).let { session.run(it.asUpdate) }.also { - checkInternFeil(it == 1) { "Kunne ikke opprette avstemming" } + krev(it == 1) { "Kunne ikke opprette avstemming" } } } diff --git a/apps/etterlatte-utbetaling/src/main/kotlin/no/nav/etterlatte/utbetaling/avstemming/Grensesnittavstemming.kt b/apps/etterlatte-utbetaling/src/main/kotlin/no/nav/etterlatte/utbetaling/avstemming/Grensesnittavstemming.kt index 405f15caca6..7d69f33460f 100644 --- a/apps/etterlatte-utbetaling/src/main/kotlin/no/nav/etterlatte/utbetaling/avstemming/Grensesnittavstemming.kt +++ b/apps/etterlatte-utbetaling/src/main/kotlin/no/nav/etterlatte/utbetaling/avstemming/Grensesnittavstemming.kt @@ -1,6 +1,6 @@ package no.nav.etterlatte.utbetaling.avstemming -import no.nav.etterlatte.libs.common.feilhaandtering.checkInternFeil +import no.nav.etterlatte.libs.common.feilhaandtering.krev import no.nav.etterlatte.libs.common.tidspunkt.Tidspunkt import no.nav.etterlatte.utbetaling.iverksetting.utbetaling.Saktype import java.nio.ByteBuffer @@ -25,7 +25,7 @@ data class Avstemmingsperiode( val til: Tidspunkt, ) { init { - checkInternFeil(fraOgMed.isBefore(til)) { "fraOgMed-tidspunkt maa vaere foer til-tidspunkt" } + krev(fraOgMed.isBefore(til)) { "fraOgMed-tidspunkt maa vaere foer til-tidspunkt" } } } diff --git a/apps/etterlatte-utbetaling/src/main/kotlin/no/nav/etterlatte/utbetaling/iverksetting/utbetaling/UtbetalingDao.kt b/apps/etterlatte-utbetaling/src/main/kotlin/no/nav/etterlatte/utbetaling/iverksetting/utbetaling/UtbetalingDao.kt index 7cdf6ad9d07..96b259808f7 100644 --- a/apps/etterlatte-utbetaling/src/main/kotlin/no/nav/etterlatte/utbetaling/iverksetting/utbetaling/UtbetalingDao.kt +++ b/apps/etterlatte-utbetaling/src/main/kotlin/no/nav/etterlatte/utbetaling/iverksetting/utbetaling/UtbetalingDao.kt @@ -8,7 +8,7 @@ import kotliquery.queryOf import kotliquery.sessionOf import kotliquery.using 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.tidspunkt.Tidspunkt import no.nav.etterlatte.libs.common.tidspunkt.toTidspunkt @@ -344,7 +344,7 @@ class UtbetalingDao( ), ).let { tx.run(it.asUpdate) } .also { - checkInternFeil(it == 1) { + krev(it == 1) { "Kunne ikke oppdatere kvittering i utbetaling" } } diff --git a/apps/etterlatte-vedtaksvurdering/src/main/kotlin/no/nav/etterlatte/vedtaksvurdering/VedtakOgBeregningSammenligner.kt b/apps/etterlatte-vedtaksvurdering/src/main/kotlin/no/nav/etterlatte/vedtaksvurdering/VedtakOgBeregningSammenligner.kt index 9acb1a01e44..9c1d891d196 100644 --- a/apps/etterlatte-vedtaksvurdering/src/main/kotlin/no/nav/etterlatte/vedtaksvurdering/VedtakOgBeregningSammenligner.kt +++ b/apps/etterlatte-vedtaksvurdering/src/main/kotlin/no/nav/etterlatte/vedtaksvurdering/VedtakOgBeregningSammenligner.kt @@ -1,6 +1,6 @@ package no.nav.etterlatte.no.nav.etterlatte.vedtaksvurdering -import no.nav.etterlatte.libs.common.feilhaandtering.checkInternFeil +import no.nav.etterlatte.libs.common.feilhaandtering.krev import no.nav.etterlatte.libs.common.vedtak.Periode import no.nav.etterlatte.libs.common.vedtak.UtbetalingsperiodeType import no.nav.etterlatte.libs.common.vedtak.VedtakType @@ -42,7 +42,7 @@ object VedtakOgBeregningSammenligner { ?: beregning.beregning.beregningsperioder.sortedBy { it.datoFOM }.map { PeriodeMedBeloep(fom = it.datoFOM, tom = it.datoTOM, beloep = it.utbetaltBeloep) } - checkInternFeil(perioder.size == beregningsperioder.size) { + krev(perioder.size == beregningsperioder.size) { "Forventa like mange perioder i vedtak som i beregning for vedtak ${vedtak.id} i sak ${vedtak.sakId}. " + "Vedtak hadde ${perioder.size}, mens beregning hadde ${beregningsperioder.size}. " + "Alle perioder fra vedtak: ${perioder.map { "${it.periode}: ${it.beloep}" }}. " + @@ -54,10 +54,10 @@ object VedtakOgBeregningSammenligner { val periode = perioder[i] val beregningsperiode = beregningsperioder[i] - checkInternFeil(BigDecimal(beregningsperiode.beloep) == periode.beloep) { + krev(BigDecimal(beregningsperiode.beloep) == periode.beloep) { "Beløp for periode ${periode.periode} i vedtak ${vedtak.id} og behandling ${vedtak.behandlingId} var ${periode.beloep} i vedtak, men ${beregningsperiode.beloep} fra beregning og eventuell avkorting" } - checkInternFeil(Periode(beregningsperiode.fom, beregningsperiode.tom) == periode.periode) { + krev(Periode(beregningsperiode.fom, beregningsperiode.tom) == periode.periode) { "FOM og TOM for periode ${periode.periode} i vedtak ${vedtak.id} " + "og behandling ${vedtak.behandlingId} i vedtak, men ${Periode( beregningsperiode.fom, diff --git a/apps/etterlatte-vedtaksvurdering/src/main/kotlin/no/nav/etterlatte/vedtaksvurdering/VedtaksvurderingRepository.kt b/apps/etterlatte-vedtaksvurdering/src/main/kotlin/no/nav/etterlatte/vedtaksvurdering/VedtaksvurderingRepository.kt index d92a2e22d4d..126f7663b81 100644 --- a/apps/etterlatte-vedtaksvurdering/src/main/kotlin/no/nav/etterlatte/vedtaksvurdering/VedtaksvurderingRepository.kt +++ b/apps/etterlatte-vedtaksvurdering/src/main/kotlin/no/nav/etterlatte/vedtaksvurdering/VedtaksvurderingRepository.kt @@ -12,7 +12,7 @@ import no.nav.etterlatte.libs.common.behandling.SakType import no.nav.etterlatte.libs.common.beregning.AvkortetYtelseDto import no.nav.etterlatte.libs.common.beregning.AvkortingDto import no.nav.etterlatte.libs.common.deserialize -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.person.Folkeregisteridentifikator import no.nav.etterlatte.libs.common.sak.SakId @@ -407,7 +407,7 @@ class VedtaksvurderingRepository( "behandlingId" to behandlingId, ), loggtekst = "Fatter vedtak for behandling $behandlingId", - ).also { checkInternFeil(it == 1) { "Vedtak ble ikke oppdatert etter fatting behandlingid: $behandlingId" } } + ).also { krev(it == 1) { "Vedtak ble ikke oppdatert etter fatting behandlingid: $behandlingId" } } .let { hentVedtakNonNull(behandlingId, this) } } @@ -433,7 +433,7 @@ class VedtaksvurderingRepository( ), loggtekst = "Attesterer vedtak $behandlingId", ).also { - checkInternFeil(it == 1) { "Vedtak ble ikke oppdatert etter attestering behandlingid: $behandlingId" } + krev(it == 1) { "Vedtak ble ikke oppdatert etter attestering behandlingid: $behandlingId" } } opprett( @@ -465,7 +465,7 @@ class VedtaksvurderingRepository( params = mapOf("vedtakstatus" to VedtakStatus.RETURNERT.name, "behandlingId" to behandlingId), loggtekst = "Underkjenner vedtak for behandling $behandlingId", ).also { - checkInternFeil(it == 1) { "Vedtak ble ikke oppdatert etter underkjenning behandlingid: $behandlingId" } + krev(it == 1) { "Vedtak ble ikke oppdatert etter underkjenning behandlingid: $behandlingId" } } return@session hentVedtakNonNull(behandlingId, this) } @@ -480,7 +480,7 @@ class VedtaksvurderingRepository( params = mapOf("vedtakstatus" to VedtakStatus.TIL_SAMORDNING.name, "behandlingId" to behandlingId), loggtekst = "Lagrer til_samordning vedtak", ).also { - checkInternFeil(it == 1) { "Vedtak ble ikke oppdatert etter satt til samordning behandlingid: $behandlingId" } + krev(it == 1) { "Vedtak ble ikke oppdatert etter satt til samordning behandlingid: $behandlingId" } } return@session hentVedtakNonNull(behandlingId, this) } @@ -495,7 +495,7 @@ class VedtaksvurderingRepository( params = mapOf("vedtakstatus" to VedtakStatus.SAMORDNET.name, "behandlingId" to behandlingId), loggtekst = "Lagrer samordnet vedtak", ).also { - checkInternFeil(it == 1) { "Vedtak ble ikke oppdatert etter samordnet behandlingid: $behandlingId" } + krev(it == 1) { "Vedtak ble ikke oppdatert etter samordnet behandlingid: $behandlingId" } } return@session hentVedtakNonNull(behandlingId, this) } @@ -510,7 +510,7 @@ class VedtaksvurderingRepository( params = mapOf("vedtakstatus" to VedtakStatus.IVERKSATT.name, "behandlingId" to behandlingId), loggtekst = "Lagrer iverksatt vedtak", ).also { - checkInternFeil(it == 1) { "Vedtak ble ikke oppdatert etter iverksatt behandlingid: $behandlingId" } + krev(it == 1) { "Vedtak ble ikke oppdatert etter iverksatt behandlingid: $behandlingId" } } return@session hentVedtakNonNull(behandlingId, this) } @@ -617,7 +617,7 @@ class VedtaksvurderingRepository( ), loggtekst = "Returnerer vedtak $behandlingId", ).also { - checkInternFeil(it == 1) { "Vedtak ble ikke oppdatert returnert/tilbakestilt behandlingid: $behandlingId" } + krev(it == 1) { "Vedtak ble ikke oppdatert returnert/tilbakestilt behandlingid: $behandlingId" } } return@session hentVedtakNonNull(behandlingId, this) } diff --git a/apps/etterlatte-vedtaksvurdering/src/main/kotlin/no/nav/etterlatte/vedtaksvurdering/outbox/OutboxRepository.kt b/apps/etterlatte-vedtaksvurdering/src/main/kotlin/no/nav/etterlatte/vedtaksvurdering/outbox/OutboxRepository.kt index ff8a7c01f65..41f9a6d9f4e 100644 --- a/apps/etterlatte-vedtaksvurdering/src/main/kotlin/no/nav/etterlatte/vedtaksvurdering/outbox/OutboxRepository.kt +++ b/apps/etterlatte-vedtaksvurdering/src/main/kotlin/no/nav/etterlatte/vedtaksvurdering/outbox/OutboxRepository.kt @@ -2,7 +2,7 @@ package no.nav.etterlatte.vedtaksvurdering.outbox import kotliquery.Row import kotliquery.queryOf -import no.nav.etterlatte.libs.common.feilhaandtering.checkInternFeil +import no.nav.etterlatte.libs.common.feilhaandtering.krev import no.nav.etterlatte.libs.database.transaction import java.util.UUID import javax.sql.DataSource @@ -31,7 +31,7 @@ class OutboxRepository( """.trimIndent(), id, ).let { query -> tx.run(query.asUpdate) } - .also { checkInternFeil(it == 1) { "Fant ikke hendelse med id $id og status upublisert" } } + .also { krev(it == 1) { "Fant ikke hendelse med id $id og status upublisert" } } } } diff --git a/libs/etterlatte-behandling-model/src/main/kotlin/no/nav/etterlatte/libs/common/behandling/Klage.kt b/libs/etterlatte-behandling-model/src/main/kotlin/no/nav/etterlatte/libs/common/behandling/Klage.kt index 801423ba7c0..00909477b63 100644 --- a/libs/etterlatte-behandling-model/src/main/kotlin/no/nav/etterlatte/libs/common/behandling/Klage.kt +++ b/libs/etterlatte-behandling-model/src/main/kotlin/no/nav/etterlatte/libs/common/behandling/Klage.kt @@ -6,7 +6,7 @@ import com.fasterxml.jackson.annotation.JsonTypeInfo import com.fasterxml.jackson.annotation.JsonTypeName 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.innsendtsoeknad.common.PDFMal import no.nav.etterlatte.libs.common.klage.AarsakTilAvbrytelse @@ -209,7 +209,7 @@ data class Klage( is KlageUtfallMedData.AvvistMedOmgjoering -> null } hjemmel?.let { - checkInternFeil(it.kanBrukesForSaktype(this.sak.sakType)) { + krev(it.kanBrukesForSaktype(this.sak.sakType)) { "Hjemmelen $it er ikke gyldig for saktypen ${this.sak.sakType}" } } diff --git a/libs/etterlatte-behandling-model/src/main/kotlin/no/nav/etterlatte/libs/common/generellbehandling/GenerellBehandling.kt b/libs/etterlatte-behandling-model/src/main/kotlin/no/nav/etterlatte/libs/common/generellbehandling/GenerellBehandling.kt index 33384c3983e..5b1a00592c0 100644 --- a/libs/etterlatte-behandling-model/src/main/kotlin/no/nav/etterlatte/libs/common/generellbehandling/GenerellBehandling.kt +++ b/libs/etterlatte-behandling-model/src/main/kotlin/no/nav/etterlatte/libs/common/generellbehandling/GenerellBehandling.kt @@ -1,6 +1,6 @@ package no.nav.etterlatte.libs.common.generellbehandling -import no.nav.etterlatte.libs.common.feilhaandtering.checkInternFeil +import no.nav.etterlatte.libs.common.feilhaandtering.krev import no.nav.etterlatte.libs.common.sak.SakId import no.nav.etterlatte.libs.common.tidspunkt.Tidspunkt import java.util.UUID @@ -47,12 +47,12 @@ data class GenerellBehandling( if (innhold !== null) { when (type) { GenerellBehandlingType.ANNEN -> - checkInternFeil(innhold is Innhold.Annen) { + krev(innhold is Innhold.Annen) { "Type $type matcher " + "ikke innhold navn: ${innhold.javaClass.simpleName}" } GenerellBehandlingType.KRAVPAKKE_UTLAND -> - checkInternFeil(innhold is Innhold.KravpakkeUtland) { + krev(innhold is Innhold.KravpakkeUtland) { "Type $type matcher ikke innhold navn: ${innhold.javaClass.simpleName}" } } diff --git a/libs/etterlatte-beregning-model/src/main/kotlin/Prosent.kt b/libs/etterlatte-beregning-model/src/main/kotlin/Prosent.kt index c0943de49f0..3095c85e060 100644 --- a/libs/etterlatte-beregning-model/src/main/kotlin/Prosent.kt +++ b/libs/etterlatte-beregning-model/src/main/kotlin/Prosent.kt @@ -1,12 +1,12 @@ package no.nav.etterlatte.beregning.grunnlag -import no.nav.etterlatte.libs.common.feilhaandtering.checkInternFeil +import no.nav.etterlatte.libs.common.feilhaandtering.krev data class Prosent( val verdi: Int, ) { init { - checkInternFeil(verdi in 0..100) { "Ugyldig prosent verdi: $verdi" } + krev(verdi in 0..100) { "Ugyldig prosent verdi: $verdi" } } fun minus(verdi: Prosent?) = minus(verdi?.verdi ?: 0) diff --git a/libs/etterlatte-brev-model/src/main/kotlin/no/nav/etterlatte/brev/Brevkoder.kt b/libs/etterlatte-brev-model/src/main/kotlin/no/nav/etterlatte/brev/Brevkoder.kt index 2dbca81a056..e7568b3c23c 100644 --- a/libs/etterlatte-brev-model/src/main/kotlin/no/nav/etterlatte/brev/Brevkoder.kt +++ b/libs/etterlatte-brev-model/src/main/kotlin/no/nav/etterlatte/brev/Brevkoder.kt @@ -1,7 +1,7 @@ package no.nav.etterlatte.brev import no.nav.etterlatte.brev.model.Spraak -import no.nav.etterlatte.libs.common.feilhaandtering.checkInternFeil +import no.nav.etterlatte.libs.common.feilhaandtering.krev /* Breva våre er teknisk sett to brev som er satt sammen - den redigerbare delen, og delen som ligg fast. @@ -240,7 +240,7 @@ enum class Brevkoder( ; init { - checkInternFeil(redigering != ferdigstilling) { + krev(redigering != ferdigstilling) { "Bruk forskjellige maler for redigering og ferdigstilling. $redigering og $ferdigstilling er like" } } diff --git a/libs/etterlatte-database/src/main/kotlin/JdbcUtils.kt b/libs/etterlatte-database/src/main/kotlin/JdbcUtils.kt index 4f53bb7af09..d215c687078 100644 --- a/libs/etterlatte-database/src/main/kotlin/JdbcUtils.kt +++ b/libs/etterlatte-database/src/main/kotlin/JdbcUtils.kt @@ -1,6 +1,6 @@ package no.nav.etterlatte.libs.database -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.SakId import org.postgresql.util.PGobject @@ -10,18 +10,18 @@ import java.sql.ResultSet fun ResultSet.singleOrNull(block: ResultSet.() -> T): T? = if (next()) { block().also { - checkInternFeil(!next()) { "Skal være unik" } + krev(!next()) { "Skal være unik" } } } else { null } fun ResultSet.single(block: ResultSet.() -> T): T { - checkInternFeil(next()) { + krev(next()) { "Skal ha en verdi" } return block().also { - checkInternFeil(!next()) { "Skal være unik" } + krev(!next()) { "Skal være unik" } } } diff --git a/libs/etterlatte-kafka/src/main/kotlin/kafka/KafkaProdusent.kt b/libs/etterlatte-kafka/src/main/kotlin/kafka/KafkaProdusent.kt index 42f135b0e2d..45368d143ad 100644 --- a/libs/etterlatte-kafka/src/main/kotlin/kafka/KafkaProdusent.kt +++ b/libs/etterlatte-kafka/src/main/kotlin/kafka/KafkaProdusent.kt @@ -1,6 +1,6 @@ package no.nav.etterlatte.kafka -import no.nav.etterlatte.libs.common.feilhaandtering.checkInternFeil +import no.nav.etterlatte.libs.common.feilhaandtering.krev import org.apache.kafka.clients.producer.KafkaProducer import org.apache.kafka.clients.producer.ProducerRecord import org.apache.kafka.common.serialization.StringSerializer @@ -69,7 +69,7 @@ class TestProdusent : KafkaProdusent { verdi: V, headers: Map?, ): Pair { - checkInternFeil(!closed) { "Kafka produsent var lukket" } + krev(!closed) { "Kafka produsent var lukket" } return publiserteMeldinger.let { it.add(Record(noekkel, verdi, headers)) Pair(0, (it.size - 1).toLong()) diff --git a/libs/etterlatte-vedtaksvurdering-model/src/main/kotlin/VedtakDto.kt b/libs/etterlatte-vedtaksvurdering-model/src/main/kotlin/VedtakDto.kt index ca4e9a4441c..62a80ec698b 100644 --- a/libs/etterlatte-vedtaksvurdering-model/src/main/kotlin/VedtakDto.kt +++ b/libs/etterlatte-vedtaksvurdering-model/src/main/kotlin/VedtakDto.kt @@ -9,7 +9,7 @@ import no.nav.etterlatte.libs.common.behandling.BehandlingType import no.nav.etterlatte.libs.common.behandling.RevurderingInfo import no.nav.etterlatte.libs.common.behandling.Revurderingaarsak 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.Folkeregisteridentifikator import no.nav.etterlatte.libs.common.sak.SakId import no.nav.etterlatte.libs.common.sak.VedtakSak @@ -87,7 +87,7 @@ data class Periode( val tom: YearMonth?, ) { init { - checkInternFeil(isNull(tom) || fom == tom || fom.isBefore(tom)) { + krev(isNull(tom) || fom == tom || fom.isBefore(tom)) { "Fom må vera før eller lik tom, men fom er $fom og tom er $tom" } } diff --git a/libs/rapidsandrivers-extras/src/main/kotlin/rapidsandrivers/ListenerMedLoggingOgFeilhaandtering.kt b/libs/rapidsandrivers-extras/src/main/kotlin/rapidsandrivers/ListenerMedLoggingOgFeilhaandtering.kt index 58b06a3f9f7..922cd9d7508 100644 --- a/libs/rapidsandrivers-extras/src/main/kotlin/rapidsandrivers/ListenerMedLoggingOgFeilhaandtering.kt +++ b/libs/rapidsandrivers-extras/src/main/kotlin/rapidsandrivers/ListenerMedLoggingOgFeilhaandtering.kt @@ -1,7 +1,7 @@ package no.nav.etterlatte.rapidsandrivers import no.nav.etterlatte.libs.common.event.EventnameHendelseType -import no.nav.etterlatte.libs.common.feilhaandtering.checkInternFeil +import no.nav.etterlatte.libs.common.feilhaandtering.krev import no.nav.etterlatte.libs.common.logging.sikkerlogger import no.nav.etterlatte.libs.common.logging.withLogContext import no.nav.etterlatte.libs.common.rapidsandrivers.correlationId @@ -63,7 +63,7 @@ abstract class ListenerMedLoggingOgFeilhaandtering : River.PacketListener { block: River.() -> Unit = {}, ) { logger.info("Initialiserer river for ${this.javaClass.simpleName}") - checkInternFeil( + krev( kontekst() in setOf( Kontekst.MIGRERING, diff --git a/libs/saksbehandling-common/src/main/kotlin/Enhetsnummer.kt b/libs/saksbehandling-common/src/main/kotlin/Enhetsnummer.kt index b59a73af9c2..92aafa63e25 100644 --- a/libs/saksbehandling-common/src/main/kotlin/Enhetsnummer.kt +++ b/libs/saksbehandling-common/src/main/kotlin/Enhetsnummer.kt @@ -1,13 +1,13 @@ package no.nav.etterlatte.libs.common import com.fasterxml.jackson.annotation.JsonValue -import no.nav.etterlatte.libs.common.feilhaandtering.checkInternFeil +import no.nav.etterlatte.libs.common.feilhaandtering.krev data class Enhetsnummer( @JsonValue val enhetNr: String, ) { init { - checkInternFeil(enhetNr.length == 4 && enhetNr.toIntOrNull() != null) { + krev(enhetNr.length == 4 && enhetNr.toIntOrNull() != null) { "Enhetsnummer må være et firesifret tall, men var $enhetNr" } } diff --git a/libs/saksbehandling-common/src/main/kotlin/feilhaandtering/Preconditions.kt b/libs/saksbehandling-common/src/main/kotlin/feilhaandtering/Preconditions.kt new file mode 100644 index 00000000000..199763307dc --- /dev/null +++ b/libs/saksbehandling-common/src/main/kotlin/feilhaandtering/Preconditions.kt @@ -0,0 +1,63 @@ +package no.nav.etterlatte.libs.common.feilhaandtering + +/** + * Erstatter Kotlin sin precondition [require] + * + * @throws InternfeilException med [feilmelding] dersom [verdi] er false + **/ +inline fun krev( + verdi: Boolean, + feilmelding: () -> String, +) { + if (!verdi) { + throw InternfeilException(feilmelding()) + } +} + +/** + * Erstatter Kotlin sin precondition [requireNotNull] + * + * @throws InternfeilException med [feilmelding] dersom [verdi] er null + * + * @return [verdi] dersom den ikke er null + **/ +inline fun krevIkkeNull( + verdi: T?, + feilmelding: () -> String, +): T { + if (verdi == null) { + throw InternfeilException(feilmelding()) + } + return verdi +} + +/** + * Erstatter Kotlin sin precondition [check] + * + * @throws UgyldigForespoerselException med [feilmelding] dersom [verdi] er false + **/ +inline fun sjekk( + verdi: Boolean, + feilmelding: () -> String, +) { + if (!verdi) { + throw UgyldigForespoerselException(code = "TILSTANDSSJEKK_FEILET", detail = feilmelding()) + } +} + +/** + * Erstatter Kotlin sin precondition [checkNotNull] + * + * @throws UgyldigForespoerselException med [feilmelding] dersom [verdi] er null + * + * @return [verdi] dersom den ikke er null + **/ +inline fun sjekkIkkeNull( + verdi: T?, + feilmelding: () -> String, +): T { + if (verdi == null) { + throw UgyldigForespoerselException(code = "VERDI_ER_NULL", detail = feilmelding()) + } + return verdi +} diff --git a/libs/saksbehandling-common/src/main/kotlin/feilhaandtering/Utils.kt b/libs/saksbehandling-common/src/main/kotlin/feilhaandtering/Utils.kt deleted file mode 100644 index 54a9445b989..00000000000 --- a/libs/saksbehandling-common/src/main/kotlin/feilhaandtering/Utils.kt +++ /dev/null @@ -1,46 +0,0 @@ -package no.nav.etterlatte.libs.common.feilhaandtering - -fun checkNotNullOrThrowException( - value: T?, - throwable: () -> Throwable, -): T { - if (value == null) { - throw throwable() - } else { - return value - } -} - -// Todo endre til if(value) -fun checkInternFeil( - value: Boolean, - message: () -> String, -) { - if (!value) { - throw InternfeilException(message()) - } -} - -fun checkNotNullorThrowInternFeil( - value: T?, - message: () -> String, -): T { - if (value == null) { - throw InternfeilException(message()) - } else { - return value - } -} - -// Todo endre til if(value) -fun checkUgyldigForespoerselException( - value: Boolean, - code: String, - message: () -> String, -) { - if (!value) { - throw UgyldigForespoerselException(code = code, detail = message()) - } -} - -// TODO: må også bytte ut checknotnull og requirenotnull diff --git a/libs/saksbehandling-common/src/testFixtures/kotlin/etterlatte/soeknad/InnsendtSoeknadTestData.kt b/libs/saksbehandling-common/src/testFixtures/kotlin/etterlatte/soeknad/InnsendtSoeknadTestData.kt index cbe3b5a8121..26562456fde 100644 --- a/libs/saksbehandling-common/src/testFixtures/kotlin/etterlatte/soeknad/InnsendtSoeknadTestData.kt +++ b/libs/saksbehandling-common/src/testFixtures/kotlin/etterlatte/soeknad/InnsendtSoeknadTestData.kt @@ -75,7 +75,7 @@ object InnsendtSoeknadTestData { utenlandskBankNavn = null, utenlandskBankAdresse = null, iban = null, - swift = null + swift = null, ), ), soeker =