Skip to content

Commit

Permalink
Fjern revurderingsinfo fra behandling i vedtaket og gjør det som ligg…
Browse files Browse the repository at this point in the history
…er i vedtak til ustrukturert (#6823)

* Ikke lagre ned ting i vedtak som kan forandre eg

* Fiks bad tests

* Renaming

* Fiks

* Fiske test

* Stupid

* Fjern revurderingsinfo fra forenkletbehandling fra vedtak da den aldri brukes til noe

* Glemte å fjerne den fra selve typen :D

* Fjerne settingen av den fjernede revurderingsinfo i forenkletbehandling

* IKke døpe om klassenavn da det lagres i ubetalingbasen

* Bruk behandling som variabelnavn da det trengs for å bevare husfreden
  • Loading branch information
sebassonav authored Jan 23, 2025
1 parent 61dd138 commit 32591b1
Show file tree
Hide file tree
Showing 15 changed files with 19 additions and 32 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,6 @@ package no.nav.etterlatte.brev.behandling

import no.nav.etterlatte.libs.common.Enhetsnummer
import no.nav.etterlatte.libs.common.behandling.Klage
import no.nav.etterlatte.libs.common.behandling.RevurderingInfo
import no.nav.etterlatte.libs.common.tilbakekreving.Tilbakekreving
import no.nav.etterlatte.libs.common.vedtak.VedtakStatus
import no.nav.etterlatte.libs.common.vedtak.VedtakType
Expand All @@ -18,7 +17,6 @@ data class ForenkletVedtak(
val attestantIdent: String?,
val vedtaksdato: LocalDate?,
val virkningstidspunkt: YearMonth? = null,
val revurderingInfo: RevurderingInfo? = null,
val tilbakekreving: Tilbakekreving? = null,
val klage: Klage? = null,
)
Original file line number Diff line number Diff line change
Expand Up @@ -137,7 +137,6 @@ class BrevdataFacade(
attestantIdent,
vedtak.vedtakFattet?.tidspunkt?.toNorskLocalDate(),
virkningstidspunkt = vedtakInnhold.virkningstidspunkt,
revurderingInfo = vedtakInnhold.behandling.revurderingInfo,
klage =
if (vedtakInnhold.behandling.revurderingsaarsak == Revurderingaarsak.OMGJOERING_ETTER_KLAGE) {
val klageId = UUID.fromString(relatertKlageId)
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -804,7 +804,6 @@ internal class VedtaksbrevServiceTest {
attestantIdent = null,
vedtaksdato = null,
virkningstidspunkt = YearMonth.now(),
revurderingInfo = null,
),
spraak = Spraak.NB,
systemkilde = systemkilde,
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -203,7 +203,6 @@ internal class BrevdataFacadeImplTest {
every { behandling.id } returns BEHANDLING_ID
every { virkningstidspunkt } returns YearMonth.now()
every { behandling.revurderingsaarsak } returns null
every { behandling.revurderingInfo } returns null
every { behandling.type } returns BehandlingType.FØRSTEGANGSBEHANDLING
}
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -62,7 +62,6 @@ internal class DistribuerBrevRiverTest {
every { behandling.id } returns behandlingId
every { virkningstidspunkt } returns YearMonth.now()
every { behandling.revurderingsaarsak } returns null
every { behandling.revurderingInfo } returns null
every { behandling.type } returns BehandlingType.FØRSTEGANGSBEHANDLING
every { utbetalingsperioder } returns emptyList()
every { opphoerFraOgMed } returns mockk()
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -143,8 +143,6 @@ internal class OpprettJournalfoerOgDistribuer {
Behandling(
type = BehandlingType.FØRSTEGANGSBEHANDLING,
id = behandlingId,
revurderingsaarsak = null,
revurderingInfo = null,
),
utbetalingsperioder = listOf(),
opphoerFraOgMed = null,
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -34,7 +34,6 @@ data class Utbetalingsvedtak(
type = innhold.behandling.type,
id = innhold.behandling.id,
revurderingsaarsak = innhold.behandling.revurderingsaarsak,
revurderingInfo = innhold.behandling.revurderingInfo,
),
pensjonTilUtbetaling =
innhold.utbetalingsperioder.map {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -45,7 +45,7 @@ class AutomatiskBehandlingService(
brukerTokenInfo: BrukerTokenInfo,
): VedtakOgRapid {
logger.info("Håndterer behandling $behandlingId")
val vedtak = service.hentVedtakForBehandling(behandlingId, brukerTokenInfo)
val vedtak = service.hentVedtakForBehandling(behandlingId)
if (vedtak != null) {
logger.warn(
"Skal opprette og fatte vedtak, men har allerede et vedtak for behandlingen" +
Expand Down
Original file line number Diff line number Diff line change
@@ -1,8 +1,8 @@
package no.nav.etterlatte.vedtaksvurdering

import com.fasterxml.jackson.databind.JsonNode
import com.fasterxml.jackson.databind.node.ObjectNode
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.IkkeTillattException
Expand Down Expand Up @@ -63,7 +63,6 @@ data class Vedtak(
innhold.behandlingType,
behandlingId,
innhold.revurderingAarsak,
innhold.revurderingInfo,
),
utbetalingsperioder = innhold.utbetalingsperioder,
opphoerFraOgMed = innhold.opphoerFraOgMed,
Expand All @@ -89,12 +88,12 @@ sealed interface VedtakInnhold {
data class Behandling(
val behandlingType: BehandlingType,
val revurderingAarsak: Revurderingaarsak?,
val revurderingInfo: JsonNode? = null,
val virkningstidspunkt: YearMonth,
val beregning: ObjectNode?,
val avkorting: ObjectNode?,
val vilkaarsvurdering: ObjectNode?,
val utbetalingsperioder: List<Utbetalingsperiode>,
val revurderingInfo: RevurderingInfo? = null,
val opphoerFraOgMed: YearMonth? = null,
) : VedtakInnhold

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -19,6 +19,7 @@ import no.nav.etterlatte.libs.common.rapidsandrivers.REVURDERING_AARSAK
import no.nav.etterlatte.libs.common.rapidsandrivers.SKAL_SENDE_BREV
import no.nav.etterlatte.libs.common.sak.SakId
import no.nav.etterlatte.libs.common.tidspunkt.Tidspunkt
import no.nav.etterlatte.libs.common.toJsonNode
import no.nav.etterlatte.libs.common.toObjectNode
import no.nav.etterlatte.libs.common.vedtak.Attestasjon
import no.nav.etterlatte.libs.common.vedtak.Periode
Expand Down Expand Up @@ -524,7 +525,7 @@ class VedtakBehandlingService(
opphoerFraOgMed = opphoerFraOgMed,
),
revurderingAarsak = behandling.revurderingsaarsak,
revurderingInfo = behandling.revurderingInfo,
revurderingInfo = behandling.revurderingInfo?.toJsonNode(),
opphoerFraOgMed = opphoerFraOgMed,
),
)
Expand Down Expand Up @@ -557,7 +558,7 @@ class VedtakBehandlingService(
sakType = behandling.sakType,
opphoerFraOgMed = opphoerFraOgMed,
),
revurderingInfo = behandling.revurderingInfo,
revurderingInfo = behandling.revurderingInfo?.toJsonNode(),
opphoerFraOgMed = opphoerFraOgMed,
),
)
Expand Down Expand Up @@ -753,10 +754,7 @@ class VedtakBehandlingService(
this.innhold is VedtakInnhold.Behandling &&
Revurderingaarsak.REGULERING == this.innhold.revurderingAarsak

fun hentVedtakForBehandling(
behandlingId: UUID,
info: BrukerTokenInfo,
): Vedtak? = repository.hentVedtak(behandlingId)
fun hentVedtakForBehandling(behandlingId: UUID): Vedtak? = repository.hentVedtak(behandlingId)
}

class VedtakTilstandException(
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -64,7 +64,6 @@ private fun Vedtak.toSamordningsvedtakDto(avkortetYtelsePerioder: List<AvkortetY
innhold.behandlingType,
behandlingId,
innhold.revurderingAarsak,
innhold.revurderingInfo,
),
virkningstidspunkt = innhold.virkningstidspunkt,
beregning = innhold.beregning,
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,6 @@ import kotliquery.queryOf
import no.nav.etterlatte.libs.common.Enhetsnummer
import no.nav.etterlatte.libs.common.Regelverk
import no.nav.etterlatte.libs.common.behandling.BehandlingType
import no.nav.etterlatte.libs.common.behandling.Revurderingaarsak
import no.nav.etterlatte.libs.common.behandling.SakType
import no.nav.etterlatte.libs.common.beregning.AvkortetYtelseDto
import no.nav.etterlatte.libs.common.beregning.AvkortingDto
Expand Down Expand Up @@ -95,7 +94,7 @@ class VedtaksvurderingRepository(
"beregningsresultat" to it.beregning?.toJson(),
"avkorting" to it.avkorting?.toJson(),
"vilkaarsresultat" to it.vilkaarsvurdering?.toJson(),
"revurderingsaarsak" to it.revurderingAarsak?.name,
"revurderingsaarsak" to it.revurderingAarsak?.toJson(),
"revurderinginfo" to it.revurderingInfo?.toJson(),
"opphoer_fom" to it.opphoerFraOgMed?.atDay(1),
)
Expand Down Expand Up @@ -571,7 +570,7 @@ class VedtaksvurderingRepository(
beregning = stringOrNull("beregningsresultat")?.let { objectMapper.readValue(it) },
avkorting = stringOrNull("avkorting")?.let { objectMapper.readValue(it) },
utbetalingsperioder = utbetalingsperioder,
revurderingAarsak = stringOrNull("revurderingsaarsak")?.let { Revurderingaarsak.valueOf(it) },
revurderingAarsak = stringOrNull("revurderingsaarsak")?.let { objectMapper.readValue(it) },
revurderingInfo = stringOrNull("revurderinginfo")?.let { objectMapper.readValue(it) },
opphoerFraOgMed = sqlDateOrNull("opphoer_fom")?.toLocalDate()?.let { YearMonth.from(it) },
)
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -99,7 +99,7 @@ internal class AutomatiskBehandlingRoutesKtTest {
behandlingId,
),
)
coEvery { vedtakService.hentVedtakForBehandling(any(), any()) } returns null
coEvery { vedtakService.hentVedtakForBehandling(any()) } returns null
coEvery { behandlingKlient.hentOppgaverForSak(any(), any()) } returns
listOf(lagOppgave(behandlingId, Status.ATTESTERING))
coEvery { behandlingKlient.tildelSaksbehandler(any(), any()) } returns true
Expand Down Expand Up @@ -143,7 +143,7 @@ internal class AutomatiskBehandlingRoutesKtTest {
assertEquals(respons.rapidInfo1.vedtakhendelse, VedtakKafkaHendelseHendelseType.FATTET)
assertEquals(respons.rapidInfo2!!.vedtakhendelse, VedtakKafkaHendelseHendelseType.ATTESTERT)
coVerify(atMost = 1) {
vedtakService.hentVedtakForBehandling(any(), any())
vedtakService.hentVedtakForBehandling(any())
}
coVerify(exactly = 1) {
vedtakService.opprettEllerOppdaterVedtak(behandlingId, any())
Expand All @@ -166,7 +166,7 @@ internal class AutomatiskBehandlingRoutesKtTest {
testApplication {
val opprettetVedtak = vedtak()
val behandlingId = UUID.randomUUID()
coEvery { vedtakService.hentVedtakForBehandling(any(), any()) } returns null
coEvery { vedtakService.hentVedtakForBehandling(any()) } returns null
coEvery { vedtakService.opprettEllerOppdaterVedtak(any(), any()) } returns
opprettetVedtak
coEvery { vedtakService.fattVedtak(behandlingId, any(), any()) } returns
Expand Down Expand Up @@ -224,7 +224,7 @@ internal class AutomatiskBehandlingRoutesKtTest {
assertEquals(respons.rapidInfo1.vedtakhendelse, VedtakKafkaHendelseHendelseType.FATTET)
assertEquals(respons.rapidInfo2!!.vedtakhendelse, VedtakKafkaHendelseHendelseType.ATTESTERT)
coVerify(exactly = 1) {
vedtakService.hentVedtakForBehandling(behandlingId, any())
vedtakService.hentVedtakForBehandling(behandlingId)
vedtakService.opprettEllerOppdaterVedtak(behandlingId, any())
behandlingKlient.hentOppgaverForSak(sakId1, any())
vedtakService.fattVedtak(behandlingId, any(), Fagsaksystem.EY.navn)
Expand All @@ -242,7 +242,7 @@ internal class AutomatiskBehandlingRoutesKtTest {
testApplication {
val opprettetVedtak = vedtak()
val behandlingId = UUID.randomUUID()
coEvery { vedtakService.hentVedtakForBehandling(any(), any()) } returns null
coEvery { vedtakService.hentVedtakForBehandling(any()) } returns null
coEvery { runBlocking { vedtakService.opprettEllerOppdaterVedtak(any(), any()) } } returns
opprettetVedtak
coEvery { runBlocking { vedtakService.fattVedtak(behandlingId, any(), any()) } } returns
Expand Down Expand Up @@ -279,7 +279,7 @@ internal class AutomatiskBehandlingRoutesKtTest {

assertEquals(respons.vedtak.id, opprettetVedtak.id)
coVerify(atMost = 1) {
vedtakService.hentVedtakForBehandling(any(), any())
vedtakService.hentVedtakForBehandling(any())
}
coVerify(exactly = 1) {
vedtakService.opprettEllerOppdaterVedtak(behandlingId, any())
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,6 @@ import com.fasterxml.jackson.databind.node.ObjectNode
import no.nav.etterlatte.libs.common.Enhetsnummer
import no.nav.etterlatte.libs.common.Regelverk
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.krev
Expand Down Expand Up @@ -79,7 +78,6 @@ data class Behandling(
val type: BehandlingType,
val id: UUID,
val revurderingsaarsak: Revurderingaarsak? = null,
val revurderingInfo: RevurderingInfo? = null,
)

data class Periode(
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -29,7 +29,10 @@ private sealed class KanBrukesIMiljoe {
}
}

// Disse må ha en oversettelse i frontend Revurderingaarsak.ts
/*
Disse må ha en oversettelse i frontend Revurderingaarsak.ts
Endring av enumnavnet her må også hensynta at det ligger lagret i vedtaksbasen og parses med objectmapper der til denne klassen.
*/
enum class Revurderingaarsak(
private val gyldigFor: List<SakType>,
private val miljoe: KanBrukesIMiljoe,
Expand Down

0 comments on commit 32591b1

Please sign in to comment.