Gjør exceptionnavn lettere å forstå

navikt · Nov 4, 2024 · 8b6279c · 8b6279c
1 parent f53ec02
commit 8b6279c
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 4 deletions.
diff --git a/src/main/kotlin/no/nav/helsearbeidsgiver/arbeidsgivernotifikasjon/Feil.kt b/src/main/kotlin/no/nav/helsearbeidsgiver/arbeidsgivernotifikasjon/Feil.kt
@@ -204,7 +204,7 @@ internal object Feil {
                     throw SakEllerOppgaveFinnesIkkeException(resultat.feilmelding)
 
                 is OppgavenErAlleredeUtfoertOppgaveEndrePaaminnelseByEksternId ->
-                    throw OppgaveUtfoertEndrePaaminnelseException(resultat.feilmelding)
+                    throw OppgaveAlleredeUtfoertException(resultat.feilmelding)
 
                 is UgyldigMerkelappOppgaveEndrePaaminnelseByEksternId -> resultat.feilmelding
                 is UgyldigPaaminnelseTidspunktOppgaveEndrePaaminnelseByEksternId -> resultat.feilmelding
@@ -238,9 +238,11 @@ class SakEllerOppgaveFinnesIkkeException(
     feilmelding: String,
 ) : Exception("Sak/oppgave finnes ikke. Trolig slettet pga. levetid. Feilmelding: '$feilmelding'.")
 
-class OppgaveUtfoertEndrePaaminnelseException(
+class OppgaveAlleredeUtfoertException(
     feilmelding: String,
-) : Exception("Oppgaven er utført, som gjør at det ikke er mulig å endre påminnelsene. Feilmelding: '$feilmelding'.")
+) : Exception(
+        "Oppgaven er allerede utført, som gjør at det ikke er mulig å endre påminnelsene. Feilmelding: '$feilmelding'.",
+    )
 
 class OpprettNySakException(
     grupperingsid: String,

diff --git a/...n/no/nav/helsearbeidsgiver/arbeidsgivernotifikasjon/ArbeidsgiverNotifikasjonKlientTest.kt b/...n/no/nav/helsearbeidsgiver/arbeidsgivernotifikasjon/ArbeidsgiverNotifikasjonKlientTest.kt
@@ -382,7 +382,7 @@ class ArbeidsgiverNotifikasjonKlientTest : FunSpec({
             val response = "responses/oppgaveEndrePaaminnelseByEksternId/oppgavenErAlleredeUtfoert.json".readResource()
             val arbeidsgiverNotifikasjonKlient = mockArbeidsgiverNotifikasjonKlient(response)
 
-            shouldThrowExactly<OppgaveUtfoertEndrePaaminnelseException> {
+            shouldThrowExactly<OppgaveAlleredeUtfoertException> {
                 arbeidsgiverNotifikasjonKlient.slettOppgavePaaminnelserByEksternId(
                     merkelapp = "mock merkelapp",
                     eksternId = "mock eksternId",