From fd5d6b6fd17d5451fef68a068da6b77bd23c7b15 Mon Sep 17 00:00:00 2001 From: Trine Linderud Date: Mon, 18 Nov 2024 10:56:35 +0100 Subject: [PATCH] =?UTF-8?q?Revert=20"=E2=9C=A8=20Legger=20med=20saksbehand?= =?UTF-8?q?lervurdering=20i=20begrunnelser=20i=20VedtakFattetMelder"?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit This reverts commit 05525cea04fd34c79ff044ba938d2b17578ad0bd. --- .../modell/vedtak/SaksbehandlerVurdering.kt | 14 ++++++----- .../helse/modell/vedtak/Sykepengevedtak.kt | 20 ++++++++++++---- .../vedtak/SykepengevedtakBuilderTest.kt | 4 ++-- .../main/kotlin/no/nav/helse/db/AvslagDao.kt | 5 +--- .../nav/helse/mediator/VedtakFattetMelder.kt | 23 ------------------- .../modell/vedtaksperiode/Vedtaksperiode.kt | 2 +- .../helse/mediator/VedtakFattetMelderTest.kt | 17 ++++---------- 7 files changed, 31 insertions(+), 54 deletions(-) diff --git a/spesialist-modell/src/main/kotlin/no/nav/helse/modell/vedtak/SaksbehandlerVurdering.kt b/spesialist-modell/src/main/kotlin/no/nav/helse/modell/vedtak/SaksbehandlerVurdering.kt index dace78a56..7609dcd26 100644 --- a/spesialist-modell/src/main/kotlin/no/nav/helse/modell/vedtak/SaksbehandlerVurdering.kt +++ b/spesialist-modell/src/main/kotlin/no/nav/helse/modell/vedtak/SaksbehandlerVurdering.kt @@ -2,7 +2,9 @@ package no.nav.helse.modell.vedtak import no.nav.helse.modell.vedtak.SaksbehandlerVurderingDto.VurderingDto -sealed class SaksbehandlerVurdering(private val vurdering: Vurdering) { +sealed class SaksbehandlerVurdering private constructor( + private val vurdering: Vurdering, +) { abstract val begrunnelse: String? class Innvilgelse(private var innvilgelsesbegrunnelse: String? = null) : SaksbehandlerVurdering(Vurdering.INNVILGELSE) { @@ -13,8 +15,8 @@ sealed class SaksbehandlerVurdering(private val vurdering: Vurdering) { override val begrunnelse: String get() = avslagsbegrunnelse } - class DelvisInnvilgelse(private var delvisInnvilgelsebegrunnelse: String) : SaksbehandlerVurdering(Vurdering.DELVIS_INNVILGELSE) { - override val begrunnelse: String get() = delvisInnvilgelsebegrunnelse + class DelvisAvslag(private var delvisAvslagsbegrunnelse: String) : SaksbehandlerVurdering(Vurdering.DELVIS_AVSLAG) { + override val begrunnelse: String get() = delvisAvslagsbegrunnelse } internal fun byggVedtak(vedtakBuilder: SykepengevedtakBuilder) { @@ -24,7 +26,7 @@ sealed class SaksbehandlerVurdering(private val vurdering: Vurdering) { fun toDto() = when (this) { is Avslag -> SaksbehandlerVurderingDto.Avslag(begrunnelse) - is DelvisInnvilgelse -> SaksbehandlerVurderingDto.DelvisInnvilgelse(begrunnelse) + is DelvisAvslag -> SaksbehandlerVurderingDto.DelvisAvslag(begrunnelse) is Innvilgelse -> SaksbehandlerVurderingDto.Innvilgelse(begrunnelse) } @@ -48,14 +50,14 @@ sealed class SaksbehandlerVurdering(private val vurdering: Vurdering) { enum class Vurdering { AVSLAG, - DELVIS_INNVILGELSE, + DELVIS_AVSLAG, INNVILGELSE, ; internal fun toDto() = when (this) { AVSLAG -> VurderingDto.AVSLAG - DELVIS_INNVILGELSE -> VurderingDto.DELVIS_INNVILGELSE + DELVIS_AVSLAG -> VurderingDto.DELVIS_AVSLAG INNVILGELSE -> VurderingDto.INNVILGELSE } } diff --git a/spesialist-modell/src/main/kotlin/no/nav/helse/modell/vedtak/Sykepengevedtak.kt b/spesialist-modell/src/main/kotlin/no/nav/helse/modell/vedtak/Sykepengevedtak.kt index 84501270a..d78d11f2f 100644 --- a/spesialist-modell/src/main/kotlin/no/nav/helse/modell/vedtak/Sykepengevedtak.kt +++ b/spesialist-modell/src/main/kotlin/no/nav/helse/modell/vedtak/Sykepengevedtak.kt @@ -183,6 +183,18 @@ enum class AvslagstypeDto { DELVIS_AVSLAG, } +// data class SaksbehandlerVurderingDto( +// val vurdering: VurderingDto, +// val begrunnelse: String?, +// ) +// { +// enum class VurderingDto { +// AVSLAG, +// DELVIS_AVSLAG, +// INNVILGELSE +// } +// } + sealed class SaksbehandlerVurderingDto(val vurdering: VurderingDto) { abstract val begrunnelse: String? @@ -190,10 +202,8 @@ sealed class SaksbehandlerVurderingDto(val vurdering: VurderingDto) { override val begrunnelse: String get() = avslagsbegrunnelse } - data class DelvisInnvilgelse(private val delvisInnvilgelsebegrunnelse: String) : SaksbehandlerVurderingDto( - VurderingDto.DELVIS_INNVILGELSE, - ) { - override val begrunnelse: String get() = delvisInnvilgelsebegrunnelse + data class DelvisAvslag(private val delvisAvslagsbegrunnelse: String) : SaksbehandlerVurderingDto(VurderingDto.DELVIS_AVSLAG) { + override val begrunnelse: String get() = delvisAvslagsbegrunnelse } data class Innvilgelse(private val innvilgelsesbegrunnelse: String? = null) : SaksbehandlerVurderingDto(VurderingDto.INNVILGELSE) { @@ -202,7 +212,7 @@ sealed class SaksbehandlerVurderingDto(val vurdering: VurderingDto) { enum class VurderingDto { AVSLAG, - DELVIS_INNVILGELSE, + DELVIS_AVSLAG, INNVILGELSE, } } diff --git a/spesialist-modell/src/test/kotlin/no/nav/helse/modell/vedtak/SykepengevedtakBuilderTest.kt b/spesialist-modell/src/test/kotlin/no/nav/helse/modell/vedtak/SykepengevedtakBuilderTest.kt index 1f06605f7..7075189f3 100644 --- a/spesialist-modell/src/test/kotlin/no/nav/helse/modell/vedtak/SykepengevedtakBuilderTest.kt +++ b/spesialist-modell/src/test/kotlin/no/nav/helse/modell/vedtak/SykepengevedtakBuilderTest.kt @@ -193,7 +193,7 @@ class SykepengevedtakBuilderTest { ) ) .saksbehandlerVurdering( - SaksbehandlerVurdering.DelvisInnvilgelse("En individuell begrunnelse for avslag") + SaksbehandlerVurdering.DelvisAvslag("En individuell begrunnelse for avslag") ) .tags(listOf("IngenNyArbeidsgiverperiode")) .build() @@ -221,7 +221,7 @@ class SykepengevedtakBuilderTest { skjønnsfastsettingopplysninger = null, tags = setOf("IngenNyArbeidsgiverperiode"), avslag = AvslagDto(AvslagstypeDto.DELVIS_AVSLAG, "En individuell begrunnelse for avslag"), - saksbehandlerVurdering = SaksbehandlerVurderingDto.DelvisInnvilgelse("En individuell begrunnelse for avslag"), + saksbehandlerVurdering = SaksbehandlerVurderingDto.DelvisAvslag("En individuell begrunnelse for avslag"), ), utkast, ) diff --git a/spesialist-selve/src/main/kotlin/no/nav/helse/db/AvslagDao.kt b/spesialist-selve/src/main/kotlin/no/nav/helse/db/AvslagDao.kt index ab85bc45a..806736435 100644 --- a/spesialist-selve/src/main/kotlin/no/nav/helse/db/AvslagDao.kt +++ b/spesialist-selve/src/main/kotlin/no/nav/helse/db/AvslagDao.kt @@ -119,10 +119,7 @@ class AvslagDao(queryRunner: QueryRunner) : QueryRunner by queryRunner { val begrunnelse = avslag.string("tekst") when (vurdering) { SaksbehandlerVurderingDto.VurderingDto.AVSLAG -> SaksbehandlerVurderingDto.Avslag(begrunnelse) - SaksbehandlerVurderingDto.VurderingDto.DELVIS_INNVILGELSE -> - SaksbehandlerVurderingDto.DelvisInnvilgelse( - begrunnelse, - ) + SaksbehandlerVurderingDto.VurderingDto.DELVIS_AVSLAG -> SaksbehandlerVurderingDto.DelvisAvslag(begrunnelse) SaksbehandlerVurderingDto.VurderingDto.INNVILGELSE -> throw RuntimeException( "Innvilgelse bør ikke forekomme før alt dette er skrevet om.", ) diff --git a/spesialist-selve/src/main/kotlin/no/nav/helse/mediator/VedtakFattetMelder.kt b/spesialist-selve/src/main/kotlin/no/nav/helse/mediator/VedtakFattetMelder.kt index c4bb8fd17..752d38256 100644 --- a/spesialist-selve/src/main/kotlin/no/nav/helse/mediator/VedtakFattetMelder.kt +++ b/spesialist-selve/src/main/kotlin/no/nav/helse/mediator/VedtakFattetMelder.kt @@ -3,7 +3,6 @@ package no.nav.helse.mediator import net.logstash.logback.argument.StructuredArguments.kv import no.nav.helse.modell.person.PersonObserver import no.nav.helse.modell.vedtak.AvslagstypeDto -import no.nav.helse.modell.vedtak.SaksbehandlerVurderingDto import no.nav.helse.modell.vedtak.Sykepengegrunnlagsfakta.Infotrygd import no.nav.helse.modell.vedtak.Sykepengegrunnlagsfakta.Spleis import no.nav.helse.modell.vedtak.Sykepengevedtak @@ -182,28 +181,6 @@ internal class VedtakFattetMelder( ) } - val saksbehandlerVurdering = sykepengevedtak.saksbehandlerVurdering - if (saksbehandlerVurdering != null) { - begrunnelser.addLast( - mapOf( - "type" to - when (saksbehandlerVurdering) { - is SaksbehandlerVurderingDto.Avslag -> "Avslag" - is SaksbehandlerVurderingDto.DelvisInnvilgelse -> "DelvisInnvilgelse" - is SaksbehandlerVurderingDto.Innvilgelse -> "Innvilgelse" - }, - "begrunnelse" to (saksbehandlerVurdering.begrunnelse ?: ""), - "perioder" to - listOf( - mapOf( - "fom" to "${sykepengevedtak.fom}", - "tom" to "${sykepengevedtak.tom}", - ), - ), - ), - ) - } - put("begrunnelser", begrunnelser) }, ) diff --git a/spesialist-selve/src/main/kotlin/no/nav/helse/modell/vedtaksperiode/Vedtaksperiode.kt b/spesialist-selve/src/main/kotlin/no/nav/helse/modell/vedtaksperiode/Vedtaksperiode.kt index e9ac0c020..606e5f273 100644 --- a/spesialist-selve/src/main/kotlin/no/nav/helse/modell/vedtaksperiode/Vedtaksperiode.kt +++ b/spesialist-selve/src/main/kotlin/no/nav/helse/modell/vedtaksperiode/Vedtaksperiode.kt @@ -232,7 +232,7 @@ internal class Vedtaksperiode private constructor( saksbehandlerVurdering?.let { when (it) { is SaksbehandlerVurderingDto.Avslag -> SaksbehandlerVurdering.Avslag(it.begrunnelse) - is SaksbehandlerVurderingDto.DelvisInnvilgelse -> SaksbehandlerVurdering.DelvisInnvilgelse(it.begrunnelse) + is SaksbehandlerVurderingDto.DelvisAvslag -> SaksbehandlerVurdering.DelvisAvslag(it.begrunnelse) is SaksbehandlerVurderingDto.Innvilgelse -> SaksbehandlerVurdering.Innvilgelse(it.begrunnelse) } }, diff --git a/spesialist-selve/src/test/kotlin/no/nav/helse/mediator/VedtakFattetMelderTest.kt b/spesialist-selve/src/test/kotlin/no/nav/helse/mediator/VedtakFattetMelderTest.kt index f7fbfadf5..b4426cb9f 100644 --- a/spesialist-selve/src/test/kotlin/no/nav/helse/mediator/VedtakFattetMelderTest.kt +++ b/spesialist-selve/src/test/kotlin/no/nav/helse/mediator/VedtakFattetMelderTest.kt @@ -244,7 +244,7 @@ internal class VedtakFattetMelderTest { } @Test - fun `vanlig vedtak sykepengegrunnlag fastsatt etter hovedregel med delvis innvilgelse`() { + fun `vanlig vedtak sykepengegrunnlag fastsatt etter hovedregel med delvis avslag`() { val spleis = Sykepengevedtak.Vedtak( fødselsnummer = FØDSELSNUMMER, @@ -282,7 +282,7 @@ internal class VedtakFattetMelderTest { skjønnsfastsettingopplysninger = null, tags = setOf("IngenNyArbeidsgiverperiode"), avslag = AvslagDto(AvslagstypeDto.DELVIS_AVSLAG, "En individuell begrunnelse"), - saksbehandlerVurdering = SaksbehandlerVurderingDto.DelvisInnvilgelse("En individuell begrunnelse"), + saksbehandlerVurdering = SaksbehandlerVurderingDto.DelvisAvslag("En individuell begrunnelse"), ) vedtakFattetMelder.vedtakFattet(spleis) vedtakFattetMelder.publiserUtgåendeMeldinger() @@ -328,13 +328,11 @@ internal class VedtakFattetMelderTest { objectMapper.convertValue(event["sykepengegrunnlagsfakta"]["arbeidsgivere"]), ) - assertEquals(2, event["begrunnelser"].size()) + assertEquals(1, event["begrunnelser"].size()) assertEquals(1, event["tags"].size()) assertEquals("IngenNyArbeidsgiverperiode", event["tags"].first().asText()) assertEquals("DelvisInnvilgelse", event["begrunnelser"][0]["type"].asText()) - assertEquals("DelvisInnvilgelse", event["begrunnelser"][1]["type"].asText()) assertEquals("En individuell begrunnelse", event["begrunnelser"][0]["begrunnelse"].asText()) - assertEquals("En individuell begrunnelse", event["begrunnelser"][1]["begrunnelse"].asText()) assertEquals( listOf(mapOf("fom" to fom, "tom" to tom)), objectMapper.convertValue>>(event["begrunnelser"][0]["perioder"]), @@ -560,7 +558,7 @@ internal class VedtakFattetMelderTest { ) assertEquals(13000.0, event["sykepengegrunnlagsfakta"]["skjønnsfastsatt"].asDouble()) - assertEquals(5, event["begrunnelser"].size()) + assertEquals(4, event["begrunnelser"].size()) assertEquals("SkjønnsfastsattSykepengegrunnlagMal", event["begrunnelser"][0]["type"].asText()) assertEquals("Mal", event["begrunnelser"][0]["begrunnelse"].asText()) @@ -590,13 +588,6 @@ internal class VedtakFattetMelderTest { objectMapper.convertValue>>(event["begrunnelser"][3]["perioder"]), ) - assertEquals("Avslag", event["begrunnelser"][4]["type"].asText()) - assertEquals("En individuell begrunnelse", event["begrunnelser"][4]["begrunnelse"].asText()) - assertEquals( - listOf(mapOf("fom" to fom, "tom" to tom)), - objectMapper.convertValue>>(event["begrunnelser"][4]["perioder"]), - ) - assertEquals(1, event["tags"].size()) assertEquals("IngenNyArbeidsgiverperiode", event["tags"].first().asText()) }