Revert "filtrer ut satsendringsfeil som feiler på vilkår og legg de i…

… satskjø…" (#4302) Reverts #4301 får UnexpectedRollbackException: Transaction silently rolled back because it has been marked as rollback-only
navikt · Jan 15, 2024 · 58843a4 · 58843a4
1 parent b79e6cb
commit 58843a4
Show file tree

Hide file tree

Showing 7 changed files with 22 additions and 67 deletions.
diff --git a/src/main/kotlin/no/nav/familie/ba/sak/common/Feil.kt b/src/main/kotlin/no/nav/familie/ba/sak/common/Feil.kt
@@ -23,22 +23,6 @@ open class FunksjonellFeil(
     override val cause: Throwable? = throwable,
 ) : RuntimeException(melding)
 
-class VilkårFeil(
-    melding: String,
-    frontendFeilmelding: String? = melding,
-    httpStatus: HttpStatus = HttpStatus.OK,
-    throwable: Throwable? = null,
-    cause: Throwable? = throwable,
-) : FunksjonellFeil(melding, frontendFeilmelding, httpStatus, throwable, cause)
-
-class SatsendringAndelFeil(
-    melding: String,
-    frontendFeilmelding: String? = melding,
-    httpStatus: HttpStatus = HttpStatus.OK,
-    throwable: Throwable? = null,
-    cause: Throwable? = throwable,
-) : FunksjonellFeil(melding, frontendFeilmelding, httpStatus, throwable, cause)
-
 class UtbetalingsikkerhetFeil(
     melding: String,
     override val frontendFeilmelding: String? = null,

diff --git a/...otlin/no/nav/familie/ba/sak/kjerne/autovedtak/satsendring/AutovedtakSatsendringService.kt b/...otlin/no/nav/familie/ba/sak/kjerne/autovedtak/satsendring/AutovedtakSatsendringService.kt
@@ -2,9 +2,7 @@ package no.nav.familie.ba.sak.kjerne.autovedtak.satsendring
 
 import io.micrometer.core.instrument.Metrics
 import no.nav.familie.ba.sak.common.Feil
-import no.nav.familie.ba.sak.common.SatsendringAndelFeil
 import no.nav.familie.ba.sak.common.UtbetalingsikkerhetFeil
-import no.nav.familie.ba.sak.common.VilkårFeil
 import no.nav.familie.ba.sak.common.secureLogger
 import no.nav.familie.ba.sak.config.TaskRepositoryWrapper
 import no.nav.familie.ba.sak.kjerne.autovedtak.AutovedtakService
@@ -103,30 +101,12 @@ class AutovedtakSatsendringService(
         }
 
         val behandlingEtterBehandlingsresultat =
-            try {
-                autovedtakService.opprettAutomatiskBehandlingOgKjørTilBehandlingsresultat(
-                    aktør = søkerAktør,
-                    behandlingType = BehandlingType.REVURDERING,
-                    behandlingÅrsak = BehandlingÅrsak.SATSENDRING,
-                    fagsakId = sisteVedtatteBehandling.fagsak.id,
-                )
-            } catch (e: VilkårFeil) {
-                val satsendringSvar = SatsendringSvar.BEHANDLING_HAR_FEIL_PÅ_VILKÅR
-
-                satskjøringForFagsak.feiltype = satsendringSvar.name
-                satskjøringRepository.save(satskjøringForFagsak)
-                logger.warn(satsendringSvar.melding)
-
-                return satsendringSvar
-            } catch (e: SatsendringAndelFeil) {
-                val satsendringSvar = SatsendringSvar.BEHANDLING_HAR_FEIL_PÅ_ANDELER
-
-                satskjøringForFagsak.feiltype = satsendringSvar.name
-                satskjøringRepository.save(satskjøringForFagsak)
-                logger.warn(satsendringSvar.melding)
-
-                return satsendringSvar
-            }
+            autovedtakService.opprettAutomatiskBehandlingOgKjørTilBehandlingsresultat(
+                aktør = søkerAktør,
+                behandlingType = BehandlingType.REVURDERING,
+                behandlingÅrsak = BehandlingÅrsak.SATSENDRING,
+                fagsakId = sisteVedtatteBehandling.fagsak.id,
+            )
 
         val opprettetVedtak =
             autovedtakService.opprettToTrinnskontrollOgVedtaksbrevForAutomatiskBehandling(
@@ -227,6 +207,4 @@ enum class SatsendringSvar(val melding: String) {
     ),
     BEHANDLING_KAN_SNIKES_FORBI("Behandling kan snikes forbi (toggle er slått av)"),
     BEHANDLING_KAN_IKKE_SETTES_PÅ_VENT("Behandlingen kan ikke settes på vent"),
-    BEHANDLING_HAR_FEIL_PÅ_VILKÅR("Behandlingen feiler på validering av vilkår."),
-    BEHANDLING_HAR_FEIL_PÅ_ANDELER("Behandlingen feiler på validering av andeler."),
 }
diff --git a/src/main/kotlin/no/nav/familie/ba/sak/kjerne/autovedtak/satsendring/StartSatsendring.kt b/src/main/kotlin/no/nav/familie/ba/sak/kjerne/autovedtak/satsendring/StartSatsendring.kt
@@ -160,14 +160,10 @@ class StartSatsendring(
 
             SatsendringSvar.BEHANDLING_ER_LÅST_SATSENDRING_TRIGGES_NESTE_VIRKEDAG,
             SatsendringSvar.BEHANDLING_KAN_IKKE_SETTES_PÅ_VENT,
-            -> throw FunksjonellFeil("Det finnes en åpen behandling på fagsaken som må avsluttes før satsendring kan gjennomføres.")
-
+            ->
+                throw FunksjonellFeil("Det finnes en åpen behandling på fagsaken som må avsluttes før satsendring kan gjennomføres.")
             SatsendringSvar.BEHANDLING_KAN_SNIKES_FORBI ->
                 throw FunksjonellFeil(resultatSatsendringBehandling.melding)
-
-            SatsendringSvar.BEHANDLING_HAR_FEIL_PÅ_VILKÅR,
-            SatsendringSvar.BEHANDLING_HAR_FEIL_PÅ_ANDELER,
-            -> throw FunksjonellFeil("Satsendring feiler." + resultatSatsendringBehandling.melding)
         }
     }
 

diff --git a/src/main/kotlin/no/nav/familie/ba/sak/kjerne/beregning/TilkjentYtelseValidering.kt b/src/main/kotlin/no/nav/familie/ba/sak/kjerne/beregning/TilkjentYtelseValidering.kt
@@ -3,7 +3,6 @@ package no.nav.familie.ba.sak.kjerne.beregning
 import no.nav.familie.ba.sak.common.Feil
 import no.nav.familie.ba.sak.common.KONTAKT_TEAMET_SUFFIX
 import no.nav.familie.ba.sak.common.MånedPeriode
-import no.nav.familie.ba.sak.common.SatsendringAndelFeil
 import no.nav.familie.ba.sak.common.UtbetalingsikkerhetFeil
 import no.nav.familie.ba.sak.common.Utils
 import no.nav.familie.ba.sak.common.tilKortString
@@ -71,16 +70,16 @@ object TilkjentYtelseValidering {
         andelerGruppert.outerJoin(forrigeAndelerGruppert) { nåværendeAndel, forrigeAndel ->
             when {
                 forrigeAndel == null && nåværendeAndel != null ->
-                    throw SatsendringAndelFeil("Satsendring kan ikke legge til en andel som ikke var der i forrige behandling")
+                    throw Feil("Satsendring kan ikke legge til en andel som ikke var der i forrige behandling")
 
                 forrigeAndel != null && nåværendeAndel == null ->
-                    throw SatsendringAndelFeil("Satsendring kan ikke fjerne en andel som fantes i forrige behandling")
+                    throw Feil("Satsendring kan ikke fjerne en andel som fantes i forrige behandling")
 
                 forrigeAndel != null && forrigeAndel.prosent != nåværendeAndel?.prosent ->
-                    throw SatsendringAndelFeil("Satsendring kan ikke endre på prosenten til en andel")
+                    throw Feil("Satsendring kan ikke endre på prosenten til en andel")
 
                 forrigeAndel != null && forrigeAndel.type != nåværendeAndel?.type ->
-                    throw SatsendringAndelFeil("Satsendring kan ikke endre YtelseType til en andel")
+                    throw Feil("Satsendring kan ikke endre YtelseType til en andel")
 
                 else -> false
             }

diff --git a/...n/kotlin/no/nav/familie/ba/sak/kjerne/endretutbetaling/EndretUtbetalingAndelValidering.kt b/...n/kotlin/no/nav/familie/ba/sak/kjerne/endretutbetaling/EndretUtbetalingAndelValidering.kt
@@ -5,7 +5,6 @@ import no.nav.familie.ba.sak.common.FunksjonellFeil
 import no.nav.familie.ba.sak.common.MånedPeriode
 import no.nav.familie.ba.sak.common.Periode
 import no.nav.familie.ba.sak.common.TIDENES_ENDE
-import no.nav.familie.ba.sak.common.VilkårFeil
 import no.nav.familie.ba.sak.common.erBack2BackIMånedsskifte
 import no.nav.familie.ba.sak.common.erDagenFør
 import no.nav.familie.ba.sak.common.erMellom
@@ -380,7 +379,7 @@ fun validerBarnasVilkår(
     }
 
     if (listeAvFeil.isNotEmpty()) {
-        throw VilkårFeil(listeAvFeil.joinToString(separator = "\n"))
+        throw FunksjonellFeil(listeAvFeil.joinToString(separator = "\n"))
     }
 }
 

diff --git a/...er/kotlin/no/nav/familie/ba/sak/kjerne/behandlingsresultat/BehandlingsresultatStegTest.kt b/...er/kotlin/no/nav/familie/ba/sak/kjerne/behandlingsresultat/BehandlingsresultatStegTest.kt
@@ -6,7 +6,6 @@ import io.mockk.mockk
 import io.mockk.runs
 import no.nav.familie.ba.sak.common.Feil
 import no.nav.familie.ba.sak.common.FunksjonellFeil
-import no.nav.familie.ba.sak.common.SatsendringAndelFeil
 import no.nav.familie.ba.sak.common.førsteDagIInneværendeMåned
 import no.nav.familie.ba.sak.common.lagAndelTilkjentYtelse
 import no.nav.familie.ba.sak.common.lagBehandling
@@ -390,7 +389,7 @@ class BehandlingsresultatStegTest {
             barn = listOf(barn),
         )
 
-        assertThatThrownBy { behandlingsresultatSteg.preValiderSteg(behandling) }.isInstanceOf(SatsendringAndelFeil::class.java)
+        assertThatThrownBy { behandlingsresultatSteg.preValiderSteg(behandling) }.isInstanceOf(Feil::class.java)
             .hasMessage("Satsendring kan ikke endre på prosenten til en andel")
     }
 

diff --git a/...nhetstester/kotlin/no/nav/familie/ba/sak/kjerne/beregning/TilkjentYtelseValideringTest.kt b/...nhetstester/kotlin/no/nav/familie/ba/sak/kjerne/beregning/TilkjentYtelseValideringTest.kt
@@ -1,7 +1,7 @@
 package no.nav.familie.ba.sak.kjerne.beregning
 
+import no.nav.familie.ba.sak.common.Feil
 import no.nav.familie.ba.sak.common.MånedPeriode
-import no.nav.familie.ba.sak.common.SatsendringAndelFeil
 import no.nav.familie.ba.sak.common.inneværendeMåned
 import no.nav.familie.ba.sak.common.lagAndelTilkjentYtelse
 import no.nav.familie.ba.sak.common.lagPerson
@@ -374,7 +374,7 @@ class TilkjentYtelseValideringTest {
                     andelerFraForrigeBehandling = forrigeAndeler,
                     andelerTilkjentYtelse = nåværendeAndeler,
                 )
-            }.isInstanceOf(SatsendringAndelFeil::class.java)
+            }.isInstanceOf(Feil::class.java)
                 .hasMessageContaining("Satsendring kan ikke legge til en andel som ikke var der i forrige behandling")
         }
 
@@ -408,7 +408,7 @@ class TilkjentYtelseValideringTest {
                     andelerFraForrigeBehandling = forrigeAndeler,
                     andelerTilkjentYtelse = nåværendeAndeler,
                 )
-            }.isInstanceOf(SatsendringAndelFeil::class.java)
+            }.isInstanceOf(Feil::class.java)
                 .hasMessageContaining("Satsendring kan ikke fjerne en andel som fantes i forrige behandling")
         }
 
@@ -444,7 +444,7 @@ class TilkjentYtelseValideringTest {
                     andelerFraForrigeBehandling = forrigeAndeler,
                     andelerTilkjentYtelse = nåværendeAndeler,
                 )
-            }.isInstanceOf(SatsendringAndelFeil::class.java)
+            }.isInstanceOf(Feil::class.java)
                 .hasMessageContaining("Satsendring kan ikke endre på prosenten til en andel")
         }
 
@@ -516,7 +516,7 @@ class TilkjentYtelseValideringTest {
                     andelerFraForrigeBehandling = forrigeAndeler,
                     andelerTilkjentYtelse = nåværendeAndeler,
                 )
-            }.isInstanceOf(SatsendringAndelFeil::class.java)
+            }.isInstanceOf(Feil::class.java)
                 .hasMessageContaining("Satsendring kan ikke legge til en andel som ikke var der i forrige behandling")
         }
 
@@ -549,7 +549,7 @@ class TilkjentYtelseValideringTest {
                     andelerFraForrigeBehandling = forrigeAndeler,
                     andelerTilkjentYtelse = nåværendeAndeler,
                 )
-            }.isInstanceOf(SatsendringAndelFeil::class.java)
+            }.isInstanceOf(Feil::class.java)
                 .hasMessageContaining("Satsendring kan ikke fjerne en andel som fantes i forrige behandling")
         }
 
@@ -580,7 +580,7 @@ class TilkjentYtelseValideringTest {
                     ),
                 )
 
-            assertThrows<SatsendringAndelFeil> {
+            assertThrows<Feil> {
                 TilkjentYtelseValidering.validerAtSatsendringKunOppdatererSatsPåEksisterendePerioder(
                     andelerFraForrigeBehandling = forrigeAndeler,
                     andelerTilkjentYtelse = nåværendeAndeler,
@@ -615,7 +615,7 @@ class TilkjentYtelseValideringTest {
                     ),
                 )
 
-            assertThrows<SatsendringAndelFeil> {
+            assertThrows<Feil> {
                 TilkjentYtelseValidering.validerAtSatsendringKunOppdatererSatsPåEksisterendePerioder(
                     andelerFraForrigeBehandling = forrigeAndeler,
                     andelerTilkjentYtelse = nåværendeAndeler,