From 5b1a329f4e67998edffdb4f8e13f1149aa667db8 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Sebastian=20Oliver=20S=C3=B8berg?= <82504565+sebassonav@users.noreply.github.com> Date: Wed, 26 Feb 2025 10:44:36 +0100 Subject: [PATCH 1/7] Rydde opp litt grunnlagsting i behandling (#7127) MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit * Fjerne unødvendig brukertokeninfo i konstruktør * Fikse litt opp i funksjonskall grunnlag --- .../kotlin/behandling/BehandlingFactory.kt | 8 +- .../kotlin/behandling/BehandlingService.kt | 25 ++-- .../aktivitetsplikt/AktivitetspliktRoute.kt | 20 +-- .../aktivitetsplikt/AktivitetspliktService.kt | 65 ++++----- .../SjekkBrevMottakerService.kt | 14 +- .../revurdering/ManuellRevurderingService.kt | 3 +- .../OmgjoeringKlageRevurderingService.kt | 3 +- .../kotlin/brev/TilbakekrevingBrevService.kt | 9 +- .../kotlin/egenansatt/EgenAnsattService.kt | 3 +- .../main/kotlin/grunnlag/GrunnlagService.kt | 80 +++++------ .../GrunnlagsendringshendelseService.kt | 8 +- .../doedshendelse/DoedshendelseJobService.kt | 10 +- .../AarligInntektsjusteringJobbService.kt | 14 +- .../src/main/kotlin/sak/SakService.kt | 23 ++-- .../service/VilkaarsvurderingService.kt | 12 +- .../behandling/BehandlingFactoryTest.kt | 14 +- .../behandling/BehandlingServiceImplTest.kt | 24 ++-- .../AktivitetspliktServiceTest.kt | 27 ++-- .../GenerellBehandlingServiceTest.kt | 7 +- .../AutomatiskRevurderingServiceTest.kt | 2 +- .../ManuellRevurderingServiceTest.kt | 29 ++-- .../egenansatt/EgenAnsattServiceTest.kt | 6 +- .../BehandlingGrunnlagRoutesKtTest.kt | 16 ++- .../kotlin/grunnlag/GrunnlagServiceTest.kt | 60 +++------ .../test/kotlin/grunnlag/PersonRoutesTest.kt | 11 +- .../grunnlag/SakGrunnlagRoutesKtTest.kt | 10 +- .../GrunnlagsendringshendelseServiceTest.kt | 47 ++++--- .../DoedshendelseJobServiceTest.kt | 3 +- .../AarligInntektsjusteringJobbServiceTest.kt | 6 +- .../kotlin/integration/EksterneKlienter.kt | 124 ------------------ .../src/test/kotlin/sak/SakServiceTest.kt | 24 ++-- .../VilkaarsvurderingRoutesTest.kt | 6 +- .../VilkaarsvurderingServiceTest.kt | 31 +++-- 33 files changed, 269 insertions(+), 475 deletions(-) diff --git a/apps/etterlatte-behandling/src/main/kotlin/behandling/BehandlingFactory.kt b/apps/etterlatte-behandling/src/main/kotlin/behandling/BehandlingFactory.kt index afd954e6d8..7eff0baa95 100644 --- a/apps/etterlatte-behandling/src/main/kotlin/behandling/BehandlingFactory.kt +++ b/apps/etterlatte-behandling/src/main/kotlin/behandling/BehandlingFactory.kt @@ -370,7 +370,7 @@ class BehandlingFactory( OmgjoerBehandling(nyFoerstegangsbehandling, sisteAvslaatteBehandling, foerstegangsbehandlingViOmgjoerer) } - val persongalleri = runBlocking { grunnlagService.hentPersongalleri(sakId)!! } + val persongalleri = grunnlagService.hentPersongalleri(sakId)!! runBlocking { grunnlagService.opprettGrunnlag( @@ -462,10 +462,8 @@ class BehandlingFactory( "${persongalleri.soesken.size} søsken" } else if (sak.sakType == SakType.OMSTILLINGSSTOENAD) { val grunnlag: Grunnlag? = - runBlocking { - grunnlagService - .hentOpplysningsgrunnlag(behandlingId) - } + grunnlagService + .hentOpplysningsgrunnlag(behandlingId) val avdoede = grunnlag?.hentAvdoede()?.firstOrNull() val barn = avdoede?.hentAvdoedesbarn()?.verdi?.avdoedesBarn diff --git a/apps/etterlatte-behandling/src/main/kotlin/behandling/BehandlingService.kt b/apps/etterlatte-behandling/src/main/kotlin/behandling/BehandlingService.kt index c2ec34e688..99ed1e0284 100644 --- a/apps/etterlatte-behandling/src/main/kotlin/behandling/BehandlingService.kt +++ b/apps/etterlatte-behandling/src/main/kotlin/behandling/BehandlingService.kt @@ -407,7 +407,7 @@ internal class BehandlingServiceImpl( grunnlagsendringshendelseDao.kobleGrunnlagsendringshendelserFraBehandlingId(behandlingId) } - val persongalleri = runBlocking { grunnlagService.hentPersongalleri(behandlingId)!! } + val persongalleri = grunnlagService.hentPersongalleri(behandlingId)!! behandlingHendelser.sendMeldingForHendelseStatistikk( behandling.toStatistikkBehandling(persongalleri = persongalleri), @@ -434,9 +434,8 @@ internal class BehandlingServiceImpl( ): DetaljertBehandling? = hentBehandling(behandlingId)?.let { val persongalleri: Persongalleri = - runBlocking { - grunnlagService.hentPersongalleri(behandlingId) - } ?: throw NoSuchElementException("Persongalleri mangler for sak ${it.sak.id}") + grunnlagService.hentPersongalleri(behandlingId) + ?: throw NoSuchElementException("Persongalleri mangler for sak ${it.sak.id}") it.toDetaljertBehandlingWithPersongalleri(persongalleri) } @@ -487,7 +486,7 @@ internal class BehandlingServiceImpl( * https://lovdata.no/dokument/NL/lov/1997-02-28-19/KAPITTEL_8-2#%C2%A722-12 * https://lovdata.no/dokument/NL/lov/1997-02-28-19/KAPITTEL_8-2#%C2%A722-13 */ - private suspend fun erGyldigVirkningstidspunktFoerstegangsbehandling( + private fun erGyldigVirkningstidspunktFoerstegangsbehandling( request: VirkningstidspunktRequest, behandling: Behandling, ): Boolean { @@ -578,7 +577,7 @@ internal class BehandlingServiceImpl( } } - private suspend fun hentFoersteDoedsdato( + private fun hentFoersteDoedsdato( behandlingId: UUID, sakType: SakType, ): LocalDate? { @@ -970,7 +969,7 @@ internal class BehandlingServiceImpl( virkningstidspunkt != null && virkningstidspunkt.isAfter(opphoerFraOgMed) - private suspend fun endrePersongalleriOgOppdaterGrunnlag( + private fun endrePersongalleriOgOppdaterGrunnlag( behandlingId: UUID, brukerTokenInfo: BrukerTokenInfo, transform: (Persongalleri) -> Persongalleri, @@ -993,13 +992,11 @@ internal class BehandlingServiceImpl( periode = null, ), ) - runBlocking { - grunnlagService.lagreNyeSaksopplysninger( - behandling.sak.id, - behandlingId, - nyeOpplysinger, - ) - } + grunnlagService.lagreNyeSaksopplysninger( + behandling.sak.id, + behandlingId, + nyeOpplysinger, + ) runBlocking { grunnlagService.oppdaterGrunnlag( diff --git a/apps/etterlatte-behandling/src/main/kotlin/behandling/aktivitetsplikt/AktivitetspliktRoute.kt b/apps/etterlatte-behandling/src/main/kotlin/behandling/aktivitetsplikt/AktivitetspliktRoute.kt index 9235943d56..60b3979272 100644 --- a/apps/etterlatte-behandling/src/main/kotlin/behandling/aktivitetsplikt/AktivitetspliktRoute.kt +++ b/apps/etterlatte-behandling/src/main/kotlin/behandling/aktivitetsplikt/AktivitetspliktRoute.kt @@ -102,7 +102,7 @@ internal fun Route.aktivitetspliktRoutes( val aktiviteter = inTransaction { - aktivitetspliktService.slettAktivitet(aktivitetId, brukerTokenInfo, behandlingId) + aktivitetspliktService.slettAktivitet(aktivitetId, behandlingId) aktivitetspliktService.hentAktiviteter(behandlingId) } @@ -136,7 +136,7 @@ internal fun Route.aktivitetspliktRoutes( val aktiviteter = inTransaction { - aktivitetspliktService.slettAktivitet(aktivitetId, brukerTokenInfo, sakId = sakId) + aktivitetspliktService.slettAktivitet(aktivitetId, sakId = sakId) aktivitetspliktService.hentAktiviteter(sakId = sakId) } @@ -167,12 +167,10 @@ internal fun Route.aktivitetspliktRoutes( logger.info("Henter hendelser for sak $sakId") val dto = inTransaction { - runBlocking { - aktivitetspliktService.hentHendelser( - sakId = sakId, - behandlingId = behandlingId, - ) - } + aktivitetspliktService.hentHendelser( + sakId = sakId, + behandlingId = behandlingId, + ) } call.respond(dto) } @@ -197,7 +195,7 @@ internal fun Route.aktivitetspliktRoutes( val hendelser = inTransaction { - aktivitetspliktService.slettHendelse(hendelseId, brukerTokenInfo, sakId = sakId) + aktivitetspliktService.slettHendelse(hendelseId, sakId = sakId) aktivitetspliktService.hentHendelser(sakId = sakId) } @@ -369,7 +367,6 @@ internal fun Route.aktivitetspliktRoutes( oppgaveId = oppgaveId, aktivitetsgradId = aktivitetsgradId, sakId = sakId, - brukerTokenInfo = brukerTokenInfo, ) } call.respond(vurdering) @@ -421,7 +418,6 @@ internal fun Route.aktivitetspliktRoutes( oppgaveId = oppgaveId, sakId = sakId, unntakId = unntakId, - brukerTokenInfo = brukerTokenInfo, ) } call.respond(vurdering) @@ -481,7 +477,6 @@ internal fun Route.aktivitetspliktRoutes( behandlingId = behandlingId, aktivitetsgradId = aktivitetsgradId, sakId = sakId, - brukerTokenInfo = brukerTokenInfo, ) } call.respond(vurdering) @@ -514,7 +509,6 @@ internal fun Route.aktivitetspliktRoutes( behandlingId = behandlingId, sakId = sakId, unntakId = unntakId, - brukerTokenInfo = brukerTokenInfo, ) } call.respond(vurdering) 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 aaa0d1154e..7dff322580 100644 --- a/apps/etterlatte-behandling/src/main/kotlin/behandling/aktivitetsplikt/AktivitetspliktService.kt +++ b/apps/etterlatte-behandling/src/main/kotlin/behandling/aktivitetsplikt/AktivitetspliktService.kt @@ -1,6 +1,5 @@ package no.nav.etterlatte.behandling.aktivitetsplikt -import kotlinx.coroutines.runBlocking import no.nav.etterlatte.behandling.BehandlingHendelserKafkaProducer import no.nav.etterlatte.behandling.BehandlingService import no.nav.etterlatte.behandling.aktivitetsplikt.vurdering.AktivitetspliktAktivitetsgrad @@ -72,7 +71,7 @@ class AktivitetspliktService( fun hentAktivitetspliktOppfolging(behandlingId: UUID): AktivitetspliktOppfolging? = aktivitetspliktDao.finnSenesteAktivitetspliktOppfolging(behandlingId) - suspend fun hentAktivitetspliktDto( + fun hentAktivitetspliktDto( sakId: SakId, behandlingId: UUID?, ): AktivitetspliktDto { @@ -270,7 +269,7 @@ class AktivitetspliktService( } else { aktivitetspliktDao.opprettAktivitetForBehandling(behandlingId, aktivitet, kilde) } - runBlocking { sendDtoTilStatistikk(aktivitet.sakId, brukerTokenInfo, behandlingId) } + sendDtoTilStatistikk(aktivitet.sakId, behandlingId) } else if (sakId != null) { if (aktivitet.sakId != sakId) { throw SakidTilhoererIkkeBehandlingException() @@ -288,14 +287,13 @@ class AktivitetspliktService( fun slettAktivitet( aktivitetId: UUID, - brukerTokenInfo: BrukerTokenInfo, behandlingId: UUID? = null, sakId: SakId? = null, ) { if (behandlingId != null) { val behandling = hentBehandlingOgSjekkOmRedigerbar(behandlingId) aktivitetspliktDao.slettAktivitetForBehandling(aktivitetId, behandlingId) - runBlocking { sendDtoTilStatistikk(behandling.sak.id, brukerTokenInfo, behandlingId) } + sendDtoTilStatistikk(behandling.sak.id, behandlingId) } else if (sakId != null) { aktivitetspliktDao.slettAktivitetForSak(aktivitetId, sakId) } else { @@ -336,7 +334,7 @@ class AktivitetspliktService( } aktivitetspliktDao.upsertHendelse(behandlingId, hendelse, kilde) - runBlocking { sendDtoTilStatistikk(hendelse.sakId, brukerTokenInfo, behandlingId) } + sendDtoTilStatistikk(hendelse.sakId, behandlingId) } else if (sakId != null) { if (hendelse.sakId != sakId) { throw SakidTilhoererIkkeBehandlingException() @@ -349,7 +347,6 @@ class AktivitetspliktService( fun slettHendelse( hendelseId: UUID, - brukerTokenInfo: BrukerTokenInfo, sakId: SakId, ) { val hendelseForSak = @@ -365,7 +362,7 @@ class AktivitetspliktService( throw BehandlingKanIkkeEndres() } aktivitetspliktDao.slettHendelse(hendelseId) - runBlocking { sendDtoTilStatistikk(behandling.sak.id, brukerTokenInfo, hendelseForSak.behandlingId) } + sendDtoTilStatistikk(behandling.sak.id, hendelseForSak.behandlingId) } else { aktivitetspliktDao.slettHendelse(hendelseId) } @@ -387,8 +384,7 @@ class AktivitetspliktService( kilde, oppgaveId = oppgaveId, ) - - runBlocking { sendDtoTilStatistikk(sakId, brukerTokenInfo, null) } + sendDtoTilStatistikk(sakId) val vurdering = hentVurderingForOppgave(oppgaveId) if (vurdering.erTom()) { @@ -438,7 +434,7 @@ class AktivitetspliktService( oppgaveId = oppgaveId, ) - runBlocking { sendDtoTilStatistikk(sakId, brukerTokenInfo) } + sendDtoTilStatistikk(sakId) return hentVurderingForOppgave(oppgaveId) } @@ -475,7 +471,7 @@ class AktivitetspliktService( behandlingId = behandlingId, ) - runBlocking { sendDtoTilStatistikk(sakId, brukerTokenInfo, behandlingId) } + sendDtoTilStatistikk(sakId, behandlingId) val unntak = aktivitetspliktUnntakDao.hentUnntakForBehandling(behandlingId) val aktivitet = aktivitetspliktAktivitetsgradDao.hentAktivitetsgradForBehandling(behandlingId) @@ -496,12 +492,9 @@ class AktivitetspliktService( val kilde = Grunnlagsopplysning.Saksbehandler.create(brukerTokenInfo.ident()) aktivitetspliktUnntakDao.upsertUnntak(unntak, sakId, kilde, oppgaveId = oppgaveId) - runBlocking { - sendDtoTilStatistikk( - sakId = sakId, - brukerTokenInfo = brukerTokenInfo, - ) - } + sendDtoTilStatistikk( + sakId = sakId, + ) return hentVurderingForOppgave(oppgaveId) } @@ -510,14 +503,13 @@ class AktivitetspliktService( oppgaveId: UUID, aktivitetsgradId: UUID, sakId: SakId, - brukerTokenInfo: BrukerTokenInfo, ): AktivitetspliktVurdering { val oppgave = oppgaveService.hentOppgave(oppgaveId) sjekkOppgaveTilhoererSakOgErRedigerbar(oppgave, sakId) aktivitetspliktAktivitetsgradDao.slettAktivitetsgradForOppgave(aktivitetsgradId, oppgaveId) - runBlocking { sendDtoTilStatistikk(sakId, brukerTokenInfo) } + sendDtoTilStatistikk(sakId) return hentVurderingForOppgave(oppgaveId) } @@ -525,12 +517,11 @@ class AktivitetspliktService( oppgaveId: UUID, unntakId: UUID, sakId: SakId, - brukerTokenInfo: BrukerTokenInfo, ): AktivitetspliktVurdering { val oppgave = oppgaveService.hentOppgave(oppgaveId) sjekkOppgaveTilhoererSakOgErRedigerbar(oppgave, sakId) - runBlocking { sendDtoTilStatistikk(sakId, brukerTokenInfo, null) } + sendDtoTilStatistikk(sakId) aktivitetspliktUnntakDao.slettUnntakForOppgave(oppgaveId, unntakId) return hentVurderingForOppgave(oppgaveId) } @@ -565,7 +556,7 @@ class AktivitetspliktService( behandlingId = behandlingId, ) - runBlocking { sendDtoTilStatistikk(sakId, brukerTokenInfo, behandlingId = behandlingId) } + sendDtoTilStatistikk(sakId, behandlingId = behandlingId) return hentVurderingForBehandlingNy(behandlingId) } @@ -574,12 +565,11 @@ class AktivitetspliktService( behandlingId: UUID, aktivitetsgradId: UUID, sakId: SakId, - brukerTokenInfo: BrukerTokenInfo, ): AktivitetspliktVurdering { hentBehandlingOgSjekkOmRedigerbar(behandlingId) aktivitetspliktAktivitetsgradDao.slettAktivitetsgradForBehandling(aktivitetsgradId, behandlingId) - runBlocking { sendDtoTilStatistikk(sakId, brukerTokenInfo, behandlingId = behandlingId) } + sendDtoTilStatistikk(sakId, behandlingId = behandlingId) return hentVurderingForBehandlingNy(behandlingId) } @@ -587,11 +577,10 @@ class AktivitetspliktService( behandlingId: UUID, unntakId: UUID, sakId: SakId, - brukerTokenInfo: BrukerTokenInfo, ): AktivitetspliktVurdering { hentBehandlingOgSjekkOmRedigerbar(behandlingId) - runBlocking { sendDtoTilStatistikk(sakId, brukerTokenInfo, behandlingId = behandlingId) } + sendDtoTilStatistikk(sakId, behandlingId = behandlingId) aktivitetspliktUnntakDao.slettUnntakForBehandling(unntakId = unntakId, behandlingId = behandlingId) return hentVurderingForBehandlingNy(behandlingId) } @@ -609,12 +598,9 @@ class AktivitetspliktService( val kilde = Grunnlagsopplysning.Saksbehandler.create(brukerTokenInfo.ident()) aktivitetspliktUnntakDao.upsertUnntak(unntak, sakId, kilde, behandlingId = behandlingId) - runBlocking { - sendDtoTilStatistikk( - sakId = sakId, - brukerTokenInfo = brukerTokenInfo, - ) - } + sendDtoTilStatistikk( + sakId = sakId, + ) return hentVurderingForBehandlingNy(behandlingId) } @@ -664,12 +650,10 @@ class AktivitetspliktService( "Fant ikke forrige behandling i sak ${request.sakId}sakId" } val persongalleri = - runBlocking { - krevIkkeNull( - grunnlagService.hentPersongalleri(forrigeBehandling.id), - ) { - "Fant ikke persongalleri for behandling ${forrigeBehandling.id}" - } + krevIkkeNull( + grunnlagService.hentPersongalleri(forrigeBehandling.id), + ) { + "Fant ikke persongalleri for behandling ${forrigeBehandling.id}" } val aktivitetspliktDato = request.behandlingsmaaned.atDay(1).plusMonths(1) @@ -779,9 +763,8 @@ class AktivitetspliktService( } } - private suspend fun sendDtoTilStatistikk( + private fun sendDtoTilStatistikk( sakId: SakId, - brukerTokenInfo: BrukerTokenInfo, behandlingId: UUID? = null, ) { try { diff --git a/apps/etterlatte-behandling/src/main/kotlin/behandling/brevutsendelse/SjekkBrevMottakerService.kt b/apps/etterlatte-behandling/src/main/kotlin/behandling/brevutsendelse/SjekkBrevMottakerService.kt index 34da6c7b4d..b129ea56b5 100644 --- a/apps/etterlatte-behandling/src/main/kotlin/behandling/brevutsendelse/SjekkBrevMottakerService.kt +++ b/apps/etterlatte-behandling/src/main/kotlin/behandling/brevutsendelse/SjekkBrevMottakerService.kt @@ -111,14 +111,12 @@ class SjekkBrevMottakerService( sak: Sak, behandling: Behandling, ): Person = - runBlocking { - grunnlagService - .hentPersonopplysninger( - behandling.id, - sak.sakType, - ).soeker - ?.opplysning ?: throw InternfeilException("Fant ikke opplysninger for sak=${sak.id}") - } + grunnlagService + .hentPersonopplysninger( + behandling.id, + sak.sakType, + ).soeker + ?.opplysning ?: throw InternfeilException("Fant ikke opplysninger for sak=${sak.id}") private fun erLikeVergemaal( vergerEn: List?, diff --git a/apps/etterlatte-behandling/src/main/kotlin/behandling/revurdering/ManuellRevurderingService.kt b/apps/etterlatte-behandling/src/main/kotlin/behandling/revurdering/ManuellRevurderingService.kt index 322eb5157e..fed9966ccd 100644 --- a/apps/etterlatte-behandling/src/main/kotlin/behandling/revurdering/ManuellRevurderingService.kt +++ b/apps/etterlatte-behandling/src/main/kotlin/behandling/revurdering/ManuellRevurderingService.kt @@ -1,7 +1,6 @@ package no.nav.etterlatte.behandling.revurdering import io.ktor.server.plugins.BadRequestException -import kotlinx.coroutines.runBlocking import no.nav.etterlatte.behandling.BehandlingService import no.nav.etterlatte.behandling.domain.Behandling import no.nav.etterlatte.behandling.domain.Revurdering @@ -109,7 +108,7 @@ class ManuellRevurderingService( saksbehandler: Saksbehandler, ): Revurdering = forrigeBehandling.let { - val persongalleri = runBlocking { grunnlagService.hentPersongalleri(sakId) } + val persongalleri = grunnlagService.hentPersongalleri(sakId) val triggendeOppgave = paaGrunnAvOppgave?.let { oppgaveService.hentOppgave(it) } revurderingService diff --git a/apps/etterlatte-behandling/src/main/kotlin/behandling/revurdering/OmgjoeringKlageRevurderingService.kt b/apps/etterlatte-behandling/src/main/kotlin/behandling/revurdering/OmgjoeringKlageRevurderingService.kt index 54c00501a3..868bcb5d6c 100644 --- a/apps/etterlatte-behandling/src/main/kotlin/behandling/revurdering/OmgjoeringKlageRevurderingService.kt +++ b/apps/etterlatte-behandling/src/main/kotlin/behandling/revurdering/OmgjoeringKlageRevurderingService.kt @@ -1,6 +1,5 @@ package no.nav.etterlatte.behandling.revurdering -import kotlinx.coroutines.runBlocking import no.nav.etterlatte.behandling.BehandlingDao import no.nav.etterlatte.behandling.domain.Revurdering import no.nav.etterlatte.behandling.klage.KlageService @@ -65,7 +64,7 @@ class OmgjoeringKlageRevurderingService( behandlingDao.hentBehandling(behandlingSomOmgjoeresId) ?: throw FeilIOmgjoering.ManglerBehandlingForOmgjoering(klagenViOmgjoerPaaGrunnAv) - val persongalleri = runBlocking { grunnlagService.hentPersongalleri(sakId) } + val persongalleri = grunnlagService.hentPersongalleri(sakId) return revurderingService .opprettRevurdering( sakId = sakId, diff --git a/apps/etterlatte-behandling/src/main/kotlin/brev/TilbakekrevingBrevService.kt b/apps/etterlatte-behandling/src/main/kotlin/brev/TilbakekrevingBrevService.kt index 166176b486..87a0d44b6b 100644 --- a/apps/etterlatte-behandling/src/main/kotlin/brev/TilbakekrevingBrevService.kt +++ b/apps/etterlatte-behandling/src/main/kotlin/brev/TilbakekrevingBrevService.kt @@ -119,13 +119,8 @@ class TilbakekrevingBrevService( async { vedtakKlient.hentVedtak(behandlingId, bruker) } - val grunnlagDef = - async { - grunnlagService.hentOpplysningsgrunnlagForSak(sak.id)!! - } - - // TODO trenger vi egt async? - val grunnlag = grunnlagDef.await() + val grunnlag = + grunnlagService.hentOpplysningsgrunnlagForSak(sak.id) ?: throw InternfeilException("Fant ikke grunnlag med sakId=$sakId") val vedtak = vedtakDeferred.await() diff --git a/apps/etterlatte-behandling/src/main/kotlin/egenansatt/EgenAnsattService.kt b/apps/etterlatte-behandling/src/main/kotlin/egenansatt/EgenAnsattService.kt index 1799359e3a..ed63acf3f2 100644 --- a/apps/etterlatte-behandling/src/main/kotlin/egenansatt/EgenAnsattService.kt +++ b/apps/etterlatte-behandling/src/main/kotlin/egenansatt/EgenAnsattService.kt @@ -1,6 +1,5 @@ package no.nav.etterlatte.egenansatt -import kotlinx.coroutines.runBlocking import no.nav.etterlatte.grunnlag.GrunnlagService import no.nav.etterlatte.libs.common.behandling.Persongalleri import no.nav.etterlatte.libs.common.sak.SakId @@ -26,7 +25,7 @@ class EgenAnsattService( val saker = sakService.finnSaker(skjermetHendelse.fnr) saker .map { - val pg = runBlocking { grunnlagService.hentPersongalleri(it.id)!! } + val pg = grunnlagService.hentPersongalleri(it.id)!! SakIdOgPersongalleri(sakId = it.id, persongalleri = pg) }.forEach { oppdaterTilgangService.haandtergraderingOgEgenAnsatt( diff --git a/apps/etterlatte-behandling/src/main/kotlin/grunnlag/GrunnlagService.kt b/apps/etterlatte-behandling/src/main/kotlin/grunnlag/GrunnlagService.kt index 8f790c3146..b9509d932b 100644 --- a/apps/etterlatte-behandling/src/main/kotlin/grunnlag/GrunnlagService.kt +++ b/apps/etterlatte-behandling/src/main/kotlin/grunnlag/GrunnlagService.kt @@ -18,7 +18,6 @@ import no.nav.etterlatte.libs.common.grunnlag.Grunnlagsopplysning import no.nav.etterlatte.libs.common.grunnlag.OppdaterGrunnlagRequest import no.nav.etterlatte.libs.common.grunnlag.Opplysningsbehov import no.nav.etterlatte.libs.common.grunnlag.hentFoedselsnummer -import no.nav.etterlatte.libs.common.grunnlag.hentNavn import no.nav.etterlatte.libs.common.grunnlag.opplysningstyper.Opplysningstype import no.nav.etterlatte.libs.common.grunnlag.opplysningstyper.Opplysningstype.AVDOED_PDL_V1 import no.nav.etterlatte.libs.common.grunnlag.opplysningstyper.Opplysningstype.GJENLEVENDE_FORELDER_PDL_V1 @@ -42,51 +41,49 @@ import org.slf4j.LoggerFactory import java.util.UUID interface GrunnlagService { - suspend fun grunnlagFinnesForSak(sakId: SakId): Boolean + fun grunnlagFinnesForSak(sakId: SakId): Boolean - suspend fun hentGrunnlagAvType( + fun hentGrunnlagAvType( behandlingId: UUID, opplysningstype: Opplysningstype, ): Grunnlagsopplysning? - suspend fun lagreNyeSaksopplysninger( + fun lagreNyeSaksopplysninger( sakId: SakId, behandlingId: UUID, nyeOpplysninger: List>, ) - suspend fun lagreNyeSaksopplysningerBareSak( + fun lagreNyeSaksopplysningerBareSak( sakId: SakId, nyeOpplysninger: List>, ) - suspend fun lagreNyePersonopplysninger( + fun lagreNyePersonopplysninger( sakId: SakId, behandlingId: UUID, fnr: Folkeregisteridentifikator, nyeOpplysninger: List>, ) - suspend fun hentOpplysningsgrunnlagForSak(sakId: SakId): Grunnlag? + fun hentOpplysningsgrunnlagForSak(sakId: SakId): Grunnlag? - suspend fun hentPersongalleri(sakId: SakId): Persongalleri? + fun hentPersongalleri(sakId: SakId): Persongalleri? - suspend fun hentPersongalleri(behandlingId: UUID): Persongalleri? + fun hentPersongalleri(behandlingId: UUID): Persongalleri? - suspend fun hentOpplysningsgrunnlag(behandlingId: UUID): Grunnlag? + fun hentOpplysningsgrunnlag(behandlingId: UUID): Grunnlag? - suspend fun hentPersonopplysninger( + fun hentPersonopplysninger( behandlingId: UUID, sakstype: SakType, ): PersonopplysningerResponse - suspend fun hentSakerOgRoller(fnr: Folkeregisteridentifikator): PersonMedSakerOgRoller + fun hentSakerOgRoller(fnr: Folkeregisteridentifikator): PersonMedSakerOgRoller - suspend fun laasVersjonForBehandling(behandlingId: UUID) + fun laasVersjonForBehandling(behandlingId: UUID) - suspend fun hentAlleSakerForFnr(fnr: Folkeregisteridentifikator): Set - - suspend fun hentPersonerISak(sakId: SakId): Map? + fun hentAlleSakerForFnr(fnr: Folkeregisteridentifikator): Set suspend fun opprettGrunnlag( behandlingId: UUID, @@ -108,9 +105,9 @@ interface GrunnlagService { suspend fun hentHistoriskForeldreansvar(behandlingId: UUID): Grunnlagsopplysning? - suspend fun hentPersongalleriSamsvar(behandlingId: UUID): PersongalleriSamsvar + fun hentPersongalleriSamsvar(behandlingId: UUID): PersongalleriSamsvar - suspend fun laasTilVersjonForBehandling( + fun laasTilVersjonForBehandling( skalLaasesId: UUID, idLaasesTil: UUID, ): BehandlingGrunnlagVersjon @@ -123,7 +120,7 @@ class GrunnlagServiceImpl( ) : GrunnlagService { private val logger = LoggerFactory.getLogger(this::class.java) - override suspend fun grunnlagFinnesForSak(sakId: SakId): Boolean { + override fun grunnlagFinnesForSak(sakId: SakId): Boolean { logger.info("Sjekker om det finnes grunnlag for sak=$sakId") return opplysningDao @@ -131,7 +128,7 @@ class GrunnlagServiceImpl( .also { logger.info("Grunnlag finnes '$it' for sak=$sakId") } } - override suspend fun hentOpplysningsgrunnlagForSak(sakId: SakId): Grunnlag? { + override fun hentOpplysningsgrunnlagForSak(sakId: SakId): Grunnlag? { val persongalleriJsonNode = opplysningDao.finnNyesteGrunnlagForSak(sakId, PERSONGALLERI_V1)?.opplysning @@ -145,7 +142,7 @@ class GrunnlagServiceImpl( return OpplysningsgrunnlagMapper(grunnlag, persongalleri).hentGrunnlag() } - override suspend fun hentPersongalleri(sakId: SakId): Persongalleri? { + override fun hentPersongalleri(sakId: SakId): Persongalleri? { val persongalleriJsonNode = opplysningDao.finnNyesteGrunnlagForSak(sakId, PERSONGALLERI_V1)?.opplysning @@ -156,7 +153,7 @@ class GrunnlagServiceImpl( return objectMapper.readValue(persongalleriJsonNode.opplysning.toJson(), Persongalleri::class.java) } - override suspend fun hentPersongalleri(behandlingId: UUID): Persongalleri? { + override fun hentPersongalleri(behandlingId: UUID): Persongalleri? { val persongalleriJsonNode = opplysningDao.finnNyesteGrunnlagForBehandling(behandlingId, PERSONGALLERI_V1)?.opplysning @@ -168,7 +165,7 @@ class GrunnlagServiceImpl( return objectMapper.readValue(persongalleriJsonNode.opplysning.toJson(), Persongalleri::class.java) } - override suspend fun hentOpplysningsgrunnlag(behandlingId: UUID): Grunnlag? { + override fun hentOpplysningsgrunnlag(behandlingId: UUID): Grunnlag? { val persongalleriJsonNode = opplysningDao.finnNyesteGrunnlagForBehandling(behandlingId, PERSONGALLERI_V1)?.opplysning @@ -182,7 +179,7 @@ class GrunnlagServiceImpl( return OpplysningsgrunnlagMapper(grunnlag, persongalleri).hentGrunnlag() } - override suspend fun hentPersonopplysninger( + override fun hentPersonopplysninger( behandlingId: UUID, sakstype: SakType, ): PersonopplysningerResponse { @@ -244,7 +241,7 @@ class GrunnlagServiceImpl( else -> false } - override suspend fun hentSakerOgRoller(fnr: Folkeregisteridentifikator): PersonMedSakerOgRoller { + override fun hentSakerOgRoller(fnr: Folkeregisteridentifikator): PersonMedSakerOgRoller { val result = opplysningDao .finnAllePersongalleriHvorPersonFinnes(fnr) @@ -258,24 +255,7 @@ class GrunnlagServiceImpl( return result } - override suspend fun hentAlleSakerForFnr(fnr: Folkeregisteridentifikator): Set = opplysningDao.finnAlleSakerForPerson(fnr) - - override suspend fun hentPersonerISak(sakId: SakId): Map? { - val grunnlag = hentOpplysningsgrunnlagForSak(sakId) ?: return null - - val personer = listOf(grunnlag.soeker) + grunnlag.familie - return personer - .mapNotNull { - val navn = it.hentNavn()?.verdi ?: return@mapNotNull null - val fnr = it.hentFoedselsnummer()?.verdi ?: return@mapNotNull null - PersonMedNavn( - fnr = fnr, - fornavn = navn.fornavn, - etternavn = navn.etternavn, - mellomnavn = navn.mellomnavn, - ) - }.associateBy { it.fnr } - } + override fun hentAlleSakerForFnr(fnr: Folkeregisteridentifikator): Set = opplysningDao.finnAlleSakerForPerson(fnr) override suspend fun opprettGrunnlag( behandlingId: UUID, @@ -346,7 +326,7 @@ class GrunnlagServiceImpl( return historiskForeldreansvar } - override suspend fun hentPersongalleriSamsvar(behandlingId: UUID): PersongalleriSamsvar { + override fun hentPersongalleriSamsvar(behandlingId: UUID): PersongalleriSamsvar { val grunnlag = opplysningDao.hentGrunnlagAvTypeForBehandling(behandlingId, PERSONGALLERI_PDL_V1, PERSONGALLERI_V1) val opplysningPersongalleriSak = @@ -387,7 +367,7 @@ class GrunnlagServiceImpl( ) } - override suspend fun laasTilVersjonForBehandling( + override fun laasTilVersjonForBehandling( skalLaasesId: UUID, idLaasesTil: UUID, ): BehandlingGrunnlagVersjon { @@ -473,12 +453,12 @@ class GrunnlagServiceImpl( else -> Saksrolle.UKJENT } - override suspend fun hentGrunnlagAvType( + override fun hentGrunnlagAvType( behandlingId: UUID, opplysningstype: Opplysningstype, ): Grunnlagsopplysning? = opplysningDao.finnNyesteGrunnlagForBehandling(behandlingId, opplysningstype)?.opplysning - override suspend fun lagreNyePersonopplysninger( + override fun lagreNyePersonopplysninger( sakId: SakId, behandlingId: UUID, fnr: Folkeregisteridentifikator, @@ -488,7 +468,7 @@ class GrunnlagServiceImpl( oppdaterGrunnlagOgVersjon(sakId, behandlingId, fnr, nyeOpplysninger) } - override suspend fun lagreNyeSaksopplysninger( + override fun lagreNyeSaksopplysninger( sakId: SakId, behandlingId: UUID, nyeOpplysninger: List>, @@ -497,7 +477,7 @@ class GrunnlagServiceImpl( oppdaterGrunnlagOgVersjon(sakId, behandlingId, fnr = null, nyeOpplysninger) } - override suspend fun lagreNyeSaksopplysningerBareSak( + override fun lagreNyeSaksopplysningerBareSak( sakId: SakId, nyeOpplysninger: List>, ) { @@ -599,7 +579,7 @@ class GrunnlagServiceImpl( } } - override suspend fun laasVersjonForBehandling(behandlingId: UUID) { + override fun laasVersjonForBehandling(behandlingId: UUID) { logger.info("Låser grunnlagsversjon for behandling (id=$behandlingId)") opplysningDao.laasGrunnlagVersjonForBehandling(behandlingId) } diff --git a/apps/etterlatte-behandling/src/main/kotlin/grunnlagsendring/GrunnlagsendringshendelseService.kt b/apps/etterlatte-behandling/src/main/kotlin/grunnlagsendring/GrunnlagsendringshendelseService.kt index 6ae93c062b..cd2b7791d6 100644 --- a/apps/etterlatte-behandling/src/main/kotlin/grunnlagsendring/GrunnlagsendringshendelseService.kt +++ b/apps/etterlatte-behandling/src/main/kotlin/grunnlagsendring/GrunnlagsendringshendelseService.kt @@ -178,7 +178,7 @@ class GrunnlagsendringshendelseService( ) } - suspend fun oppdaterAdressebeskyttelseHendelse(adressebeskyttelse: Adressebeskyttelse) { + fun oppdaterAdressebeskyttelseHendelse(adressebeskyttelse: Adressebeskyttelse) { val sakIder = grunnlagService.hentAlleSakerForFnr(Folkeregisteridentifikator.of(adressebeskyttelse.fnr)) if (sakIder.isEmpty()) { logger.info("Forkaster hendelse da vi ikke fant noen saker på den.") @@ -209,7 +209,7 @@ class GrunnlagsendringshendelseService( samsvar: SamsvarMellomKildeOgGrunnlag, ): List { val personMedSakerOgRoller = - runBlocking { grunnlagService.hentSakerOgRoller(Folkeregisteridentifikator.of(fnr)).sakiderOgRoller } + grunnlagService.hentSakerOgRoller(Folkeregisteridentifikator.of(fnr)).sakiderOgRoller return inTransaction { personMedSakerOgRoller @@ -281,7 +281,7 @@ class GrunnlagsendringshendelseService( ): List { val tidspunktForMottakAvHendelse = now().toLocalDatetimeUTC() val sakerOgRoller = - runBlocking { grunnlagService.hentSakerOgRoller(Folkeregisteridentifikator.of(fnr)).sakiderOgRoller } + grunnlagService.hentSakerOgRoller(Folkeregisteridentifikator.of(fnr)).sakiderOgRoller val sakerOgRollerGruppert = sakerOgRoller.distinct() @@ -378,7 +378,7 @@ class GrunnlagsendringshendelseService( personRolle, sak.sakType, ) - val grunnlag = runBlocking { grunnlagService.hentOpplysningsgrunnlagForSak(sak.id) } + val grunnlag = grunnlagService.hentOpplysningsgrunnlagForSak(sak.id) try { val samsvarMellomPdlOgGrunnlag = finnSamsvarForHendelse(grunnlagsendringshendelse, pdlData, grunnlag, personRolle, sak.sakType) diff --git a/apps/etterlatte-behandling/src/main/kotlin/grunnlagsendring/doedshendelse/DoedshendelseJobService.kt b/apps/etterlatte-behandling/src/main/kotlin/grunnlagsendring/doedshendelse/DoedshendelseJobService.kt index 1d5bbd48bf..76986ae460 100644 --- a/apps/etterlatte-behandling/src/main/kotlin/grunnlagsendring/doedshendelse/DoedshendelseJobService.kt +++ b/apps/etterlatte-behandling/src/main/kotlin/grunnlagsendring/doedshendelse/DoedshendelseJobService.kt @@ -196,12 +196,10 @@ class DoedshendelseJobService( val spraak = hentSpraak(doedshendelse) val spraakOpplysning = lagOpplysning(Opplysningstype.SPRAAK, kilde, spraak.verdi.toJsonNode()) - runBlocking { - grunnlagService.lagreNyeSaksopplysningerBareSak( - sakId = opprettetSak.id, - nyeOpplysninger = listOf(spraakOpplysning), - ) - } + grunnlagService.lagreNyeSaksopplysningerBareSak( + sakId = opprettetSak.id, + nyeOpplysninger = listOf(spraakOpplysning), + ) return opprettetSak } diff --git a/apps/etterlatte-behandling/src/main/kotlin/inntektsjustering/AarligInntektsjusteringJobbService.kt b/apps/etterlatte-behandling/src/main/kotlin/inntektsjustering/AarligInntektsjusteringJobbService.kt index e0166c4653..605901e70f 100644 --- a/apps/etterlatte-behandling/src/main/kotlin/inntektsjustering/AarligInntektsjusteringJobbService.kt +++ b/apps/etterlatte-behandling/src/main/kotlin/inntektsjustering/AarligInntektsjusteringJobbService.kt @@ -325,7 +325,7 @@ class AarligInntektsjusteringJobbService( loependeFom: YearMonth, begrunnelse: String, ): Revurdering { - val persongalleri = runBlocking { grunnlagService.hentPersongalleri(sakId) } + val persongalleri = grunnlagService.hentPersongalleri(sakId) val revurdering = revurderingService @@ -416,13 +416,11 @@ class AarligInntektsjusteringJobbService( sak: Sak, sisteBehandlingId: UUID, ): Personopplysning = - runBlocking { - grunnlagService - .hentPersonopplysninger( - sisteBehandlingId, - sak.sakType, - ).soeker ?: throw InternfeilException("Fant ikke opplysninger for behandling=$sisteBehandlingId") - } + grunnlagService + .hentPersonopplysninger( + sisteBehandlingId, + sak.sakType, + ).soeker ?: throw InternfeilException("Fant ikke opplysninger for behandling=$sisteBehandlingId") private fun manuellBehandlingSkruddPaa(): Boolean = featureToggleService.isEnabled(ManuellBehandlingToggle.MANUELL_BEHANDLING, defaultValue = false) diff --git a/apps/etterlatte-behandling/src/main/kotlin/sak/SakService.kt b/apps/etterlatte-behandling/src/main/kotlin/sak/SakService.kt index d5be1b24c7..a308aaf7ea 100644 --- a/apps/etterlatte-behandling/src/main/kotlin/sak/SakService.kt +++ b/apps/etterlatte-behandling/src/main/kotlin/sak/SakService.kt @@ -213,7 +213,7 @@ class SakServiceImpl( override fun finnSakerOmsOgHvisAvdoed(ident: String): List { val saker = finnSakerForPerson(ident, SakType.OMSTILLINGSSTOENAD).filterForEnheter() val sakerOgRollerForPerson = - runBlocking { grunnlagService.hentSakerOgRoller(Folkeregisteridentifikator.of(ident)) } + grunnlagService.hentSakerOgRoller(Folkeregisteridentifikator.of(ident)) val sakerOgRollerGruppert = sakerOgRollerForPerson.sakiderOgRoller.distinct() val avdoedSak = sakerOgRollerGruppert.filter { it.rolle == Saksrolle.AVDOED } val sakerForAvdoed = avdoedSak.map { it.sakId } @@ -262,13 +262,13 @@ class SakServiceImpl( } private fun leggTilGrunnlag(sak: Sak) { - runBlocking { - val harGrunnlag = grunnlagService.grunnlagFinnesForSak(sak.id) + val harGrunnlag = grunnlagService.grunnlagFinnesForSak(sak.id) - if (harGrunnlag) { - logger.info("Finnes allerede grunnlag på sak=${sak.id}") - return@runBlocking - } + if (harGrunnlag) { + logger.info("Finnes allerede grunnlag på sak=${sak.id}") + return + } + runBlocking { logger.info("Fant ingen grunnlag på sak=${sak.id} - oppretter grunnlag") val kilde = Grunnlagsopplysning.Gjenny(Fagsaksystem.EY.navn, Tidspunkt.now()) @@ -307,13 +307,12 @@ class SakServiceImpl( ?: throw InternfeilException("Sak ${sak.id} har flere eller ingen gyldige identer samtidig. Kan ikke oppdatere ident.") dao.oppdaterIdent(sak.id, gjeldendeIdent) + val oppdatertPersongalleri = + grunnlagService + .hentPersongalleri(sak.id)!! + .copy(soeker = gjeldendeIdent.value) runBlocking { - val oppdatertPersongalleri = - grunnlagService - .hentPersongalleri(sak.id)!! - .copy(soeker = gjeldendeIdent.value) - grunnlagService.opprettEllerOppdaterGrunnlagForSak(sak.id, opplysningsbehov(sak, oppdatertPersongalleri)) } diff --git a/apps/etterlatte-behandling/src/main/kotlin/vilkaarsvurdering/service/VilkaarsvurderingService.kt b/apps/etterlatte-behandling/src/main/kotlin/vilkaarsvurdering/service/VilkaarsvurderingService.kt index 42026ac4f9..d41e40500f 100644 --- a/apps/etterlatte-behandling/src/main/kotlin/vilkaarsvurdering/service/VilkaarsvurderingService.kt +++ b/apps/etterlatte-behandling/src/main/kotlin/vilkaarsvurdering/service/VilkaarsvurderingService.kt @@ -1,6 +1,5 @@ package no.nav.etterlatte.vilkaarsvurdering.service -import kotlinx.coroutines.runBlocking import no.nav.etterlatte.behandling.BehandlingService import no.nav.etterlatte.behandling.BehandlingStatusService import no.nav.etterlatte.behandling.domain.Behandling @@ -411,7 +410,7 @@ class VilkaarsvurderingService( private fun hentDataForVilkaarsvurdering(behandlingId: UUID): Pair { val behandling = behandlingService.hentBehandling(behandlingId)!! - val grunnlag = runBlocking { grunnlagService.hentOpplysningsgrunnlag(behandlingId)!! } + val grunnlag = grunnlagService.hentOpplysningsgrunnlag(behandlingId)!! return Pair(behandling, grunnlag) } @@ -451,7 +450,8 @@ class VilkaarsvurderingService( } val avdoedeForGjeldendeBehandling: List = - runBlocking { grunnlagService.hentPersongalleri(gjeldendeBehandling.sak.id)!! } + grunnlagService + .hentPersongalleri(gjeldendeBehandling.sak.id)!! .avdoed .map { Folkeregisteridentifikator.of(it) } @@ -479,7 +479,8 @@ class VilkaarsvurderingService( val kandidatSakerForAvdoede: Set = avdoedeForGjeldendeBehandling .flatMap { avdoed -> - runBlocking { grunnlagService.hentSakerOgRoller(Folkeregisteridentifikator.of(avdoed.value)) } + grunnlagService + .hentSakerOgRoller(Folkeregisteridentifikator.of(avdoed.value)) .sakiderOgRoller .filter { it.sakId != gjeldendeSak && it.rolle == Saksrolle.AVDOED } .map { it.sakId } @@ -495,7 +496,8 @@ class VilkaarsvurderingService( val aktuelleBehandlinger = kandidatSakerForAvdoede.mapNotNull { sakId -> val avdoedeForKandidatSak = - runBlocking { grunnlagService.hentPersongalleri(sakId)!! } + grunnlagService + .hentPersongalleri(sakId)!! .avdoed .map { Folkeregisteridentifikator.of(it) } .toSet() diff --git a/apps/etterlatte-behandling/src/test/kotlin/behandling/BehandlingFactoryTest.kt b/apps/etterlatte-behandling/src/test/kotlin/behandling/BehandlingFactoryTest.kt index 239e5e5cd2..c37b83f476 100644 --- a/apps/etterlatte-behandling/src/test/kotlin/behandling/BehandlingFactoryTest.kt +++ b/apps/etterlatte-behandling/src/test/kotlin/behandling/BehandlingFactoryTest.kt @@ -662,7 +662,7 @@ internal class BehandlingFactoryTest { val opprettBehandlingSlot = slot() every { behandlingDaoMock.opprettBehandling(capture(opprettBehandlingSlot)) } just runs - coEvery { grunnlagService.hentPersongalleri(sak.id) } returns Persongalleri(sak.ident) + every { grunnlagService.hentPersongalleri(sak.id) } returns Persongalleri(sak.ident) coEvery { grunnlagService.opprettGrunnlag(any(), any()) } just runs every { oppgaveService.opprettOppgave(any(), any(), any(), any(), any(), frist = any()) @@ -714,9 +714,9 @@ internal class BehandlingFactoryTest { hendelseDaoMock.behandlingOpprettet(any()) behandlingHendelserKafkaProducerMock.sendMeldingForHendelseStatistikk(any(), any()) vilkaarsvurderingService.kopierVilkaarsvurdering(any(), any(), any()) + grunnlagService.hentPersongalleri(sak.id) } coVerify { - grunnlagService.hentPersongalleri(sak.id) grunnlagService.opprettGrunnlag(any(), any()) } } @@ -765,7 +765,7 @@ internal class BehandlingFactoryTest { val opprettBehandlingSlot = slot() every { behandlingDaoMock.opprettBehandling(capture(opprettBehandlingSlot)) } just runs - coEvery { grunnlagService.hentPersongalleri(sak.id) } returns Persongalleri(sak.ident) + every { grunnlagService.hentPersongalleri(sak.id) } returns Persongalleri(sak.ident) coEvery { grunnlagService.opprettGrunnlag(any(), any()) } just runs every { oppgaveService.opprettOppgave(any(), any(), any(), any(), any(), frist = any()) } returns oppgaveInternMock every { oppgaveService.hentOppgaverForSak(any(), any()) } returns listOf(oppgaveInternMock) @@ -795,9 +795,9 @@ internal class BehandlingFactoryTest { behandlingHendelserKafkaProducerMock.sendMeldingForHendelseStatistikk(any(), any()) behandlingDaoMock.lagreGyldighetsproeving(opprettetBehandling.id, any()) oppgaveService.ferdigstillOppgave(omgjoeringsOppgaveId, saksbehandler) + grunnlagService.hentPersongalleri(sak.id) } coVerify { - grunnlagService.hentPersongalleri(sak.id) grunnlagService.opprettGrunnlag(any(), any()) } } @@ -825,7 +825,7 @@ internal class BehandlingFactoryTest { val opprettBehandlingSlot = slot() every { behandlingDaoMock.opprettBehandling(capture(opprettBehandlingSlot)) } just runs - coEvery { grunnlagService.hentPersongalleri(sak.id) } returns Persongalleri(sak.ident) + every { grunnlagService.hentPersongalleri(sak.id) } returns Persongalleri(sak.ident) coEvery { grunnlagService.opprettGrunnlag(any(), any()) } just runs every { oppgaveService.opprettOppgave(any(), any(), any(), any(), any(), frist = any()) @@ -869,9 +869,9 @@ internal class BehandlingFactoryTest { hendelseDaoMock.behandlingOpprettet(any()) behandlingHendelserKafkaProducerMock.sendMeldingForHendelseStatistikk(any(), any()) behandlingDaoMock.lagreGyldighetsproeving(opprettetBehandling.id, any()) + grunnlagService.hentPersongalleri(sak.id) } coVerify { - grunnlagService.hentPersongalleri(sak.id) grunnlagService.opprettGrunnlag(any(), any()) } } @@ -1352,11 +1352,11 @@ internal class BehandlingFactoryTest { } coVerify { grunnlagService.opprettGrunnlag(any(), any()) - grunnlagService.lagreNyeSaksopplysninger(any(), any(), any()) } verify(exactly = 1) { sakServiceMock.oppdaterAdressebeskyttelse(any(), any()) sakServiceMock.hentGraderingForSak(any(), any()) + grunnlagService.lagreNyeSaksopplysninger(any(), any(), any()) } } diff --git a/apps/etterlatte-behandling/src/test/kotlin/behandling/BehandlingServiceImplTest.kt b/apps/etterlatte-behandling/src/test/kotlin/behandling/BehandlingServiceImplTest.kt index 60d5771a1a..5a37bee52d 100644 --- a/apps/etterlatte-behandling/src/test/kotlin/behandling/BehandlingServiceImplTest.kt +++ b/apps/etterlatte-behandling/src/test/kotlin/behandling/BehandlingServiceImplTest.kt @@ -248,7 +248,7 @@ internal class BehandlingServiceImplTest { every { oppgaveServiceMock.hentOppgaverForSak(any(), any()) } returns listOf(oppgaveKlage) every { oppgaveServiceMock.opprettOppgave(any(), any(), any(), any(), any(), any(), any()) } returns oppgaveKlage - coEvery { grunnlagService.hentPersongalleri(any()) } returns mockPersongalleri() + every { grunnlagService.hentPersongalleri(any()) } returns mockPersongalleri() behandlingService.avbrytBehandling(revurderingbehandling.id, simpleSaksbehandler(), AarsakTilAvbrytelse.ANNET, "") @@ -288,7 +288,7 @@ internal class BehandlingServiceImplTest { every { oppgaveServiceMock.avbrytAapneOppgaverMedReferanse(any(), any()) } just runs every { oppgaveServiceMock.hentOppgaverForSak(any(), any()) } returns listOf(oppgaveKlage) every { oppgaveServiceMock.opprettOppgave(any(), any(), any(), any(), any(), any(), any()) } returns oppgaveKlage - coEvery { grunnlagService.hentPersongalleri(any()) } returns mockPersongalleri() + every { grunnlagService.hentPersongalleri(any()) } returns mockPersongalleri() behandlingService.avbrytBehandling(nyFoerstegangsbehandling.id, simpleSaksbehandler(), AarsakTilAvbrytelse.ANNET, "") @@ -328,7 +328,7 @@ internal class BehandlingServiceImplTest { every { oppgaveServiceMock.avbrytAapneOppgaverMedReferanse(any(), any()) } just runs - coEvery { grunnlagService.hentPersongalleri(any()) } returns mockPersongalleri() + every { grunnlagService.hentPersongalleri(any()) } returns mockPersongalleri() val saksbehandler = simpleSaksbehandler() assertThrows { @@ -360,7 +360,7 @@ internal class BehandlingServiceImplTest { every { grunnlagsendringshendelseDaoMock.kobleGrunnlagsendringshendelserFraBehandlingId(any()) } just runs every { grunnlagsendringshendelseDaoMock.hentGrunnlagsendringshendelseSomErTattMedIBehandling(any()) } returns emptyList() every { oppgaveServiceMock.avbrytAapneOppgaverMedReferanse(any(), any()) } just runs - coEvery { grunnlagService.hentPersongalleri(any()) } returns mockPersongalleri() + every { grunnlagService.hentPersongalleri(any()) } returns mockPersongalleri() behandlingService.avbrytBehandling(nyFoerstegangsbehandling.id, simpleSaksbehandler(), AarsakTilAvbrytelse.ANNET, "test") @@ -434,7 +434,7 @@ internal class BehandlingServiceImplTest { every { grunnlagsendringshendelseDaoMock.kobleGrunnlagsendringshendelserFraBehandlingId(any()) } just runs every { grunnlagsendringshendelseDaoMock.hentGrunnlagsendringshendelseSomErTattMedIBehandling(any()) } returns emptyList() every { oppgaveServiceMock.avbrytAapneOppgaverMedReferanse(any(), any()) } just runs - coEvery { grunnlagService.hentPersongalleri(any()) } returns mockPersongalleri() + every { grunnlagService.hentPersongalleri(any()) } returns mockPersongalleri() behandlingService.avbrytBehandling(nyFoerstegangsbehandling.id, simpleSaksbehandler(), AarsakTilAvbrytelse.ANNET, "") @@ -464,7 +464,7 @@ internal class BehandlingServiceImplTest { every { grunnlagsendringshendelseDaoMock.kobleGrunnlagsendringshendelserFraBehandlingId(any()) } just runs every { grunnlagsendringshendelseDaoMock.hentGrunnlagsendringshendelseSomErTattMedIBehandling(any()) } returns emptyList() every { oppgaveServiceMock.avbrytAapneOppgaverMedReferanse(any(), any()) } just runs - coEvery { grunnlagService.hentPersongalleri(any()) } returns mockPersongalleri() + every { grunnlagService.hentPersongalleri(any()) } returns mockPersongalleri() behandlingService.avbrytBehandling(nyFoerstegangsbehandling.id, simpleSaksbehandler()) verify(exactly = 1) { @@ -1090,8 +1090,8 @@ internal class BehandlingServiceImplTest { nyKontekstMedBruker(mockSaksbehandler()) val behandling = foerstegangsbehandling(sakId = randomSakId(), id = UUID.randomUUID()) every { behandlingDaoMock.hentBehandling(behandling.id) } returns behandling - coEvery { grunnlagService.hentPersongalleri(behandling.id) } returns mockPersongalleri() - coEvery { grunnlagService.lagreNyeSaksopplysninger(any(), behandling.id, any()) } just runs + every { grunnlagService.hentPersongalleri(behandling.id) } returns mockPersongalleri() + every { grunnlagService.lagreNyeSaksopplysninger(any(), behandling.id, any()) } just runs coEvery { grunnlagService.oppdaterGrunnlag(behandling.id, behandling.sak.id, any()) } just runs every { behandlingDaoMock.lagreStatus(any()) } just runs every { behandlingDaoMock.lagreStatus(any()) } just runs @@ -1110,9 +1110,11 @@ internal class BehandlingServiceImplTest { } val slot = slot>>() coVerify { - grunnlagService.lagreNyeSaksopplysninger(any(), behandling.id, capture(slot)) grunnlagService.oppdaterGrunnlag(behandling.id, behandling.sak.id, any()) } + verify { + grunnlagService.lagreNyeSaksopplysninger(any(), behandling.id, capture(slot)) + } val grunnlagsopplysning = slot.captured.single() assertEquals(Opplysningstype.PERSONGALLERI_V1, grunnlagsopplysning.opplysningType) @@ -1187,7 +1189,7 @@ internal class BehandlingServiceImplTest { ) } - coEvery { + every { grunnlagService.hentPersonopplysninger(behandling.id, sakType) } returns PersonopplysningerResponse( @@ -1205,7 +1207,7 @@ internal class BehandlingServiceImplTest { soeker = null, annenForelder = null, ) - coEvery { grunnlagService.hentPersongalleri(behandling.id) } answers { callOriginal() } + every { grunnlagService.hentPersongalleri(behandling.id) } answers { callOriginal() } every { behandlingDaoMock.hentBehandling(BEHANDLINGS_ID) } returns behandling every { behandlingDaoMock.hentBehandlingerForSak(any()) } returns tidligereBehandlinger // TODO fjern? diff --git a/apps/etterlatte-behandling/src/test/kotlin/behandling/aktivitetsplikt/AktivitetspliktServiceTest.kt b/apps/etterlatte-behandling/src/test/kotlin/behandling/aktivitetsplikt/AktivitetspliktServiceTest.kt index ec64024c79..55b1792d22 100644 --- a/apps/etterlatte-behandling/src/test/kotlin/behandling/aktivitetsplikt/AktivitetspliktServiceTest.kt +++ b/apps/etterlatte-behandling/src/test/kotlin/behandling/aktivitetsplikt/AktivitetspliktServiceTest.kt @@ -4,7 +4,6 @@ import io.kotest.matchers.shouldBe import io.kotest.matchers.shouldNotBe import io.mockk.Called import io.mockk.Runs -import io.mockk.coEvery import io.mockk.coVerify import io.mockk.every import io.mockk.just @@ -390,7 +389,7 @@ class AktivitetspliktServiceTest { every { status } returns BehandlingStatus.VILKAARSVURDERT } - service.slettAktivitet(aktivitetId, brukerTokenInfo, behandling.id) + service.slettAktivitet(aktivitetId, behandling.id) coVerify { aktivitetspliktDao.slettAktivitetForBehandling(aktivitetId, behandling.id) } } @@ -403,7 +402,7 @@ class AktivitetspliktServiceTest { } assertThrows { - service.slettAktivitet(aktivitetId, brukerTokenInfo, behandling.id) + service.slettAktivitet(aktivitetId, behandling.id) } } } @@ -482,8 +481,8 @@ class AktivitetspliktServiceTest { val vurdering = service.hentVurderingForOppgave(oppgaveId) vurdering shouldNotBe null - vurdering?.aktivitet?.isEmpty() shouldNotBe true - vurdering?.unntak shouldBe emptyList() + vurdering.aktivitet.isEmpty() shouldNotBe true + vurdering.unntak shouldBe emptyList() verify { aktivitetspliktAktivitetsgradDao.hentAktivitetsgradForOppgave(oppgaveId) } verify { aktivitetspliktUnntakDao.hentUnntakForOppgave(oppgaveId) } @@ -555,8 +554,8 @@ class AktivitetspliktServiceTest { val vurdering = service.hentVurderingForOppgave(oppgaveId) vurdering shouldNotBe null - vurdering?.aktivitet shouldBe emptyList() - vurdering?.unntak?.isEmpty() shouldBe false + vurdering.aktivitet shouldBe emptyList() + vurdering.unntak.isEmpty() shouldBe false verify { aktivitetspliktAktivitetsgradDao.hentAktivitetsgradForOppgave(oppgaveId) } verify { aktivitetspliktUnntakDao.hentUnntakForOppgave(oppgaveId) } @@ -743,7 +742,7 @@ class AktivitetspliktServiceTest { every { aktivitetspliktUnntakDao.hentNyesteUnntak(sakId) } returns emptyList() every { behandlingService.hentSisteIverksatte(sakId) } returns forrigeBehandling every { behandlingService.hentBehandlingerForSak(sakId) } returns listOf(forrigeBehandling) - coEvery { grunnlagService.hentPersongalleri(forrigeBehandling.id) } returns persongalleri + every { grunnlagService.hentPersongalleri(forrigeBehandling.id) } returns persongalleri every { revurderingService.opprettRevurdering( sakId = sakId, @@ -796,7 +795,7 @@ class AktivitetspliktServiceTest { every { aktivitetspliktUnntakDao.hentNyesteUnntak(sakId) } returns emptyList() every { behandlingService.hentSisteIverksatte(sakId) } returns forrigeBehandling every { behandlingService.hentBehandlingerForSak(sakId) } returns listOf(forrigeBehandling) - coEvery { grunnlagService.hentPersongalleri(forrigeBehandling.id) } returns persongalleri + every { grunnlagService.hentPersongalleri(forrigeBehandling.id) } returns persongalleri every { revurderingService.opprettRevurdering( sakId = sakId, @@ -852,7 +851,7 @@ class AktivitetspliktServiceTest { every { aktivitetspliktUnntakDao.hentNyesteUnntak(sakId) } returns emptyList() every { behandlingService.hentSisteIverksatte(sakId) } returns forrigeBehandling every { behandlingService.hentBehandlingerForSak(sakId) } returns listOf(forrigeBehandling) - coEvery { grunnlagService.hentPersongalleri(forrigeBehandling.id) } returns persongalleri + every { grunnlagService.hentPersongalleri(forrigeBehandling.id) } returns persongalleri every { revurderingService.opprettRevurdering( sakId = sakId, @@ -924,7 +923,7 @@ class AktivitetspliktServiceTest { every { aktivitetspliktUnntakDao.hentNyesteUnntak(sakId) } returns emptyList() every { behandlingService.hentSisteIverksatte(sakId) } returns forrigeBehandling every { behandlingService.hentBehandlingerForSak(sakId) } returns listOf(forrigeBehandling) - coEvery { grunnlagService.hentPersongalleri(forrigeBehandling.id) } returns persongalleri + every { grunnlagService.hentPersongalleri(forrigeBehandling.id) } returns persongalleri every { revurderingService.opprettRevurdering( sakId = sakId, @@ -996,7 +995,7 @@ class AktivitetspliktServiceTest { every { aktivitetspliktUnntakDao.hentNyesteUnntak(sakId) } returns emptyList() every { behandlingService.hentSisteIverksatte(sakId) } returns forrigeBehandling every { behandlingService.hentBehandlingerForSak(sakId) } returns listOf(forrigeBehandling) - coEvery { grunnlagService.hentPersongalleri(forrigeBehandling.id) } returns persongalleri + every { grunnlagService.hentPersongalleri(forrigeBehandling.id) } returns persongalleri every { revurderingService.opprettRevurdering( sakId = sakId, @@ -1065,7 +1064,7 @@ class AktivitetspliktServiceTest { every { aktivitetspliktUnntakDao.hentNyesteUnntak(sakId) } returns emptyList() every { behandlingService.hentSisteIverksatte(sakId) } returns forrigeBehandling every { behandlingService.hentBehandlingerForSak(sakId) } returns listOf(forrigeBehandling, aapenBehandling) - coEvery { grunnlagService.hentPersongalleri(forrigeBehandling.id) } returns persongalleri + every { grunnlagService.hentPersongalleri(forrigeBehandling.id) } returns persongalleri every { oppgaveService.opprettOppgave( sakId = sakId, @@ -1104,7 +1103,7 @@ class AktivitetspliktServiceTest { ) every { aktivitetspliktUnntakDao.hentNyesteUnntak(sakId) } returns emptyList() every { behandlingService.hentSisteIverksatte(sakId) } returns forrigeBehandling - coEvery { grunnlagService.hentPersongalleri(forrigeBehandling.id) } returns persongalleri + every { grunnlagService.hentPersongalleri(forrigeBehandling.id) } returns persongalleri val resultat = service.opprettRevurderingHvisKravIkkeOppfylt(request6mnd) diff --git a/apps/etterlatte-behandling/src/test/kotlin/behandling/generellbehandling/GenerellBehandlingServiceTest.kt b/apps/etterlatte-behandling/src/test/kotlin/behandling/generellbehandling/GenerellBehandlingServiceTest.kt index c4da4d8e5d..7672f43661 100644 --- a/apps/etterlatte-behandling/src/test/kotlin/behandling/generellbehandling/GenerellBehandlingServiceTest.kt +++ b/apps/etterlatte-behandling/src/test/kotlin/behandling/generellbehandling/GenerellBehandlingServiceTest.kt @@ -4,8 +4,6 @@ import io.kotest.inspectors.forExactly import io.kotest.matchers.shouldBe import io.kotest.matchers.string.shouldContain import io.mockk.clearAllMocks -import io.mockk.coEvery -import io.mockk.coVerify import io.mockk.confirmVerified import io.mockk.every import io.mockk.mockk @@ -178,12 +176,11 @@ internal class GenerellBehandlingServiceTest( ) val foerstegangsbehandlingHentet = behandlingRepo.hentBehandling(foerstegangsbehandling.id) as Foerstegangsbehandling - val brukerTokenInfo = simpleSaksbehandler() every { behandlingService.hentBehandlingerForSak(sak.id) } returns listOf(foerstegangsbehandlingHentet) val doedsdato = LocalDate.parse("2016-12-30") val personopplysning = personOpplysning(doedsdato = doedsdato) val grunnlagsopplysningMedPersonopplysning = grunnlagsOpplysningMedPersonopplysning(personopplysning) - coEvery { + every { grunnlagKlient.hentGrunnlagAvType( foerstegangsbehandling.id, Opplysningstype.AVDOED_PDL_V1, @@ -201,7 +198,7 @@ internal class GenerellBehandlingServiceTest( val kravpakkeMedArbeidetUtlandet = runBlocking { service.hentKravpakkeForSak(sak.id) } assertEquals(opprettBehandlingGenerell.id, kravpakkeMedArbeidetUtlandet.kravpakke.id) assertEquals(foerstegangsbehandling.id, kravpakkeMedArbeidetUtlandet.kravpakke.tilknyttetBehandling) - coVerify { grunnlagKlient.hentGrunnlagAvType(any(), any()) } + verify { grunnlagKlient.hentGrunnlagAvType(any(), any()) } verify { behandlingService.hentBehandlingerForSak(any()) } } diff --git a/apps/etterlatte-behandling/src/test/kotlin/behandling/revurdering/AutomatiskRevurderingServiceTest.kt b/apps/etterlatte-behandling/src/test/kotlin/behandling/revurdering/AutomatiskRevurderingServiceTest.kt index 07bfd97ed7..34a81731b0 100644 --- a/apps/etterlatte-behandling/src/test/kotlin/behandling/revurdering/AutomatiskRevurderingServiceTest.kt +++ b/apps/etterlatte-behandling/src/test/kotlin/behandling/revurdering/AutomatiskRevurderingServiceTest.kt @@ -74,7 +74,7 @@ class AutomatiskRevurderingServiceTest { }, grunnlagService = mockk { - coEvery { hentPersongalleri(any()) } returns mockk() + every { hentPersongalleri(any()) } returns mockk() }, vedtakKlient = vedtakKlient, beregningKlient = beregningKlient, diff --git a/apps/etterlatte-behandling/src/test/kotlin/behandling/revurdering/ManuellRevurderingServiceTest.kt b/apps/etterlatte-behandling/src/test/kotlin/behandling/revurdering/ManuellRevurderingServiceTest.kt index 7dec33b1cd..20bc89b3b4 100644 --- a/apps/etterlatte-behandling/src/test/kotlin/behandling/revurdering/ManuellRevurderingServiceTest.kt +++ b/apps/etterlatte-behandling/src/test/kotlin/behandling/revurdering/ManuellRevurderingServiceTest.kt @@ -165,11 +165,11 @@ class ManuellRevurderingServiceTest : BehandlingIntegrationTest() { coVerify { grunnlagService.opprettGrunnlag(any(), any()) + } + verify { grunnlagService.hentPersongalleri(sak.id) grunnlagService.lagreNyePersonopplysninger(sak.id, revurdering.id, any(), any()) grunnlagService.lagreNyeSaksopplysninger(sak.id, revurdering.id, any()) - } - verify { oppgaveService.opprettOppgave( referanse = revurdering.id.toString(), sakId = sak.id, @@ -227,11 +227,12 @@ class ManuellRevurderingServiceTest : BehandlingIntegrationTest() { } val revurderingInfo = RevurderingInfo.Soeskenjustering(BarnepensjonSoeskenjusteringGrunn.SOESKEN_DOER) + val saksbehandler = "saksbehandler" inTransaction { revurderingService.lagreRevurderingInfo( revurdering.id, RevurderingInfoMedBegrunnelse(revurderingInfo, "abc"), - "saksbehandler", + saksbehandler, ) } @@ -249,7 +250,7 @@ class ManuellRevurderingServiceTest : BehandlingIntegrationTest() { revurderingService.lagreRevurderingInfo( revurdering.id, RevurderingInfoMedBegrunnelse(nyRevurderingInfo, null), - "saksbehandler", + saksbehandler, ) } inTransaction { @@ -258,7 +259,7 @@ class ManuellRevurderingServiceTest : BehandlingIntegrationTest() { } inTransaction { - iverksett(revurdering, "saksbehandler") + iverksett(revurdering, saksbehandler) } assertThrows { @@ -266,7 +267,7 @@ class ManuellRevurderingServiceTest : BehandlingIntegrationTest() { revurderingService.lagreRevurderingInfo( revurdering.id, RevurderingInfoMedBegrunnelse(revurderingInfo, null), - "saksbehandler", + saksbehandler, ) } } @@ -286,14 +287,14 @@ class ManuellRevurderingServiceTest : BehandlingIntegrationTest() { merknad = revurdering.revurderingsaarsak?.lesbar(), gruppeId = defaultPersongalleriGydligeFnr.avdoed.first(), ) - oppgaveService.tildelSaksbehandler(any(), "saksbehandler") - } - coVerify { - grunnlagService.opprettGrunnlag(any(), any()) + oppgaveService.tildelSaksbehandler(any(), saksbehandler) grunnlagService.hentPersongalleri(sak.id) grunnlagService.lagreNyePersonopplysninger(sak.id, revurdering.id, any(), any()) grunnlagService.lagreNyeSaksopplysninger(sak.id, revurdering.id, any()) } + coVerify { + grunnlagService.opprettGrunnlag(any(), any()) + } confirmVerified(hendelser, grunnlagService, oppgaveService) } } @@ -403,12 +404,12 @@ class ManuellRevurderingServiceTest : BehandlingIntegrationTest() { assertEquals(revurdering.id, grunnlaghendelse?.behandlingId) coVerify { grunnlagService.opprettGrunnlag(any(), any()) + } + verify { grunnlagService.hentPersongalleri(sak.id) grunnlagService.lagreNyePersonopplysninger(sak.id, behandling!!.id, any(), any()) grunnlagService.lagreNyeSaksopplysninger(sak.id, behandling.id, any()) grunnlagService.laasTilVersjonForBehandling(revurdering.id, behandling.id) - } - verify { oppgaveService.hentOppgaverForSak(sak.id) oppgaveService.avbrytAapneOppgaverMedReferanse(behandling!!.id.toString()) oppgaveService.hentOppgave(any()) @@ -502,11 +503,11 @@ class ManuellRevurderingServiceTest : BehandlingIntegrationTest() { coVerify { grunnlagService.opprettGrunnlag(any(), any()) + } + verify { grunnlagService.hentPersongalleri(sak.id) grunnlagService.lagreNyePersonopplysninger(sak.id, revurdering.id, any(), any()) grunnlagService.lagreNyeSaksopplysninger(sak.id, revurdering.id, any()) - } - verify { oppgaveService.opprettOppgave( referanse = revurdering.id.toString(), sakId = sak.id, diff --git a/apps/etterlatte-behandling/src/test/kotlin/egenansatt/EgenAnsattServiceTest.kt b/apps/etterlatte-behandling/src/test/kotlin/egenansatt/EgenAnsattServiceTest.kt index cede9f3340..ca29e2cc1a 100644 --- a/apps/etterlatte-behandling/src/test/kotlin/egenansatt/EgenAnsattServiceTest.kt +++ b/apps/etterlatte-behandling/src/test/kotlin/egenansatt/EgenAnsattServiceTest.kt @@ -92,8 +92,8 @@ internal class EgenAnsattServiceTest( val norg2Klient = mockk() coEvery { grunnlagService.opprettGrunnlag(any(), any()) } just runs - coEvery { grunnlagService.lagreNyeSaksopplysningerBareSak(any(), any()) } just runs - coEvery { grunnlagService.grunnlagFinnesForSak(any()) } returns false + every { grunnlagService.lagreNyeSaksopplysningerBareSak(any(), any()) } just runs + every { grunnlagService.grunnlagFinnesForSak(any()) } returns false val featureToggleService = mockk() val skjermingKlient = mockk() @@ -174,7 +174,7 @@ internal class EgenAnsattServiceTest( val bruktSak = sakService.finnEllerOpprettSakMedGrunnlag(fnr, SakType.BARNEPENSJON, Enheter.EGNE_ANSATTE.enhetNr) sakService.finnEllerOpprettSakMedGrunnlag(fnr2, SakType.BARNEPENSJON, Enheter.EGNE_ANSATTE.enhetNr) - coEvery { grunnlagService.hentPersongalleri(bruktSak.id) } returns persongalleri + every { grunnlagService.hentPersongalleri(bruktSak.id) } returns persongalleri every { oppdaterTilgangService.haandtergraderingOgEgenAnsatt(bruktSak.id, any()) } just Runs assertNotNull(sakService.finnSak(fnr, SakType.BARNEPENSJON)) diff --git a/apps/etterlatte-behandling/src/test/kotlin/grunnlag/BehandlingGrunnlagRoutesKtTest.kt b/apps/etterlatte-behandling/src/test/kotlin/grunnlag/BehandlingGrunnlagRoutesKtTest.kt index b36602d424..256cf16bfb 100644 --- a/apps/etterlatte-behandling/src/test/kotlin/grunnlag/BehandlingGrunnlagRoutesKtTest.kt +++ b/apps/etterlatte-behandling/src/test/kotlin/grunnlag/BehandlingGrunnlagRoutesKtTest.kt @@ -18,9 +18,11 @@ import io.mockk.clearAllMocks import io.mockk.coEvery import io.mockk.coVerify import io.mockk.confirmVerified +import io.mockk.every import io.mockk.just import io.mockk.mockk import io.mockk.slot +import io.mockk.verify import no.nav.etterlatte.behandling.randomSakId import no.nav.etterlatte.ktor.runServer import no.nav.etterlatte.ktor.startRandomPort @@ -84,7 +86,7 @@ internal class BehandlingGrunnlagRoutesKtTest { fun `returnerer 404 hvis grunnlag ikke finnes`() { val behandlingId = UUID.randomUUID() - coEvery { grunnlagService.hentOpplysningsgrunnlag(any()) } returns null + every { grunnlagService.hentOpplysningsgrunnlag(any()) } returns null testApplication { runServer(mockOAuth2Server, "api/grunnlag") { @@ -104,7 +106,7 @@ internal class BehandlingGrunnlagRoutesKtTest { assertEquals(HttpStatusCode.NotFound, response.status) } - coVerify(exactly = 1) { grunnlagService.hentOpplysningsgrunnlag(any()) } + verify(exactly = 1) { grunnlagService.hentOpplysningsgrunnlag(any()) } } @Test @@ -112,7 +114,7 @@ internal class BehandlingGrunnlagRoutesKtTest { val behandlingId = UUID.randomUUID() val testData = GrunnlagTestData().hentOpplysningsgrunnlag() - coEvery { grunnlagService.hentOpplysningsgrunnlag(any()) } returns testData + every { grunnlagService.hentOpplysningsgrunnlag(any()) } returns testData testApplication { val response = @@ -127,7 +129,7 @@ internal class BehandlingGrunnlagRoutesKtTest { assertEquals(serialize(testData), response.body()) } - coVerify(exactly = 1) { grunnlagService.hentOpplysningsgrunnlag(any()) } + verify(exactly = 1) { grunnlagService.hentOpplysningsgrunnlag(any()) } } @ParameterizedTest @@ -144,7 +146,7 @@ internal class BehandlingGrunnlagRoutesKtTest { opplysning = """{}""".toJsonNode(), ) - coEvery { grunnlagService.hentGrunnlagAvType(any(), any()) } returns opplysning + every { grunnlagService.hentGrunnlagAvType(any(), any()) } returns opplysning testApplication { val response = @@ -159,7 +161,7 @@ internal class BehandlingGrunnlagRoutesKtTest { assertEquals(serialize(opplysning), response.body()) } - coVerify(exactly = 1) { grunnlagService.hentGrunnlagAvType(behandlingId, type) } + verify(exactly = 1) { grunnlagService.hentGrunnlagAvType(behandlingId, type) } } @Test @@ -208,7 +210,7 @@ internal class BehandlingGrunnlagRoutesKtTest { ), ) - coEvery { grunnlagService.lagreNyeSaksopplysninger(any(), any(), any()) } just Runs + every { grunnlagService.lagreNyeSaksopplysninger(any(), any(), any()) } just Runs testApplication { val actualResponse = diff --git a/apps/etterlatte-behandling/src/test/kotlin/grunnlag/GrunnlagServiceTest.kt b/apps/etterlatte-behandling/src/test/kotlin/grunnlag/GrunnlagServiceTest.kt index 6cdafe191b..f59a01292b 100644 --- a/apps/etterlatte-behandling/src/test/kotlin/grunnlag/GrunnlagServiceTest.kt +++ b/apps/etterlatte-behandling/src/test/kotlin/grunnlag/GrunnlagServiceTest.kt @@ -189,7 +189,7 @@ internal class GrunnlagServiceTest { every { opplysningDaoMock.hentAlleGrunnlagForSak(sakId1) } returns grunnlagshendelser - val actual = runBlocking { grunnlagService.hentOpplysningsgrunnlagForSak(sakId1)!! } + val actual = grunnlagService.hentOpplysningsgrunnlagForSak(sakId1)!! val expected = mapOf( NAVN to Opplysning.Konstant(statiskUuid, kilde, nyttNavn.toJsonNode()), @@ -257,10 +257,7 @@ internal class GrunnlagServiceTest { ) } returns grunnlagshendelser.first() - val grunnlag = - runBlocking { - grunnlagService.hentOpplysningsgrunnlagForSak(sakId) - } + val grunnlag = grunnlagService.hentOpplysningsgrunnlagForSak(sakId) assertNotNull(grunnlag) if (grunnlag != null) { val hentetGalleriPaaSoeker = @@ -286,10 +283,7 @@ internal class GrunnlagServiceTest { every { opplysningDaoMock.hentAlleGrunnlagForSak(sakId1) } returns grunnlagshendelser - val actual = - runBlocking { - grunnlagService.hentOpplysningsgrunnlagForSak(sakId1) - }!! + val actual = grunnlagService.hentOpplysningsgrunnlagForSak(sakId1)!! val expected = mapOf( NAVN to Opplysning.Konstant(statiskUuid, kilde, nyttNavn.toJsonNode()), @@ -308,10 +302,7 @@ internal class GrunnlagServiceTest { every { opplysningDaoMock.hentAlleGrunnlagForSak(sakId1) } returns grunnlagshendelser - val actual = - runBlocking { - grunnlagService.hentOpplysningsgrunnlagForSak(sakId1)!! - } + val actual = grunnlagService.hentOpplysningsgrunnlagForSak(sakId1)!! val expected = mapOf( NAVN to Opplysning.Konstant(statiskUuid, kilde, nyttNavn.toJsonNode()), @@ -329,10 +320,7 @@ internal class GrunnlagServiceTest { every { opplysningDaoMock.hentAlleGrunnlagForSak(sakId1) } returns grunnlagshendelser - val actual = - runBlocking { - grunnlagService.hentOpplysningsgrunnlagForSak(sakId1)!! - } + val actual = grunnlagService.hentOpplysningsgrunnlagForSak(sakId1)!! val expected = mapOf( NAVN to Opplysning.Konstant(statiskUuid, kilde, nyttNavn.toJsonNode()), @@ -374,10 +362,8 @@ internal class GrunnlagServiceTest { every { opplysningDaoMock.hentAlleGrunnlagForSak(sakId1) } returns grunnlagshendelser val opplysning = - runBlocking { - grunnlagService - .hentOpplysningsgrunnlagForSak(sakId1)!! - } + grunnlagService + .hentOpplysningsgrunnlagForSak(sakId1)!! assertEquals( 1, @@ -388,10 +374,7 @@ internal class GrunnlagServiceTest { @Test fun `tar alltid seneste versjon av samme opplysning`() { every { opplysningDaoMock.hentAlleGrunnlagForSak(sakId1) } returns grunnlagshendelser - val opplysningsgrunnlag = - runBlocking { - grunnlagService.hentOpplysningsgrunnlagForSak(sakId1)!! - } + val opplysningsgrunnlag = grunnlagService.hentOpplysningsgrunnlagForSak(sakId1)!! assertEquals( 2, @@ -439,10 +422,7 @@ internal class GrunnlagServiceTest { every { opplysningDaoMock.hentAlleGrunnlagForSak(sakId1) } returns grunnlagshendelser - val actual = - runBlocking { - grunnlagService.hentOpplysningsgrunnlagForSak(sakId1)!! - } + val actual = grunnlagService.hentOpplysningsgrunnlagForSak(sakId1)!! val expected = Opplysning.Konstant( uuid1, @@ -499,10 +479,7 @@ internal class GrunnlagServiceTest { ), ) - val opplysningsgrunnlag = - runBlocking { - grunnlagService.hentOpplysningsgrunnlagForSak(sakId1)!! - } + val opplysningsgrunnlag = grunnlagService.hentOpplysningsgrunnlagForSak(sakId1)!! assertEquals(1, opplysningsgrunnlag.sak.size) assertEquals(2, opplysningsgrunnlag.soeker.size) @@ -577,7 +554,7 @@ internal class GrunnlagServiceTest { /* * Barn 1 søker barnepensjon og har rolle søsken i annen sak */ - val barn1 = runBlocking { grunnlagService.hentSakerOgRoller(barnepensjonSoeker1) } + val barn1 = grunnlagService.hentSakerOgRoller(barnepensjonSoeker1) assertEquals(2, barn1.sakiderOgRoller.size) assertEquals(barnepensjonSoeker1.value, barn1.fnr) @@ -590,7 +567,7 @@ internal class GrunnlagServiceTest { /* * Barn 2 søker barnepensjon og har rolle søsken i annen sak */ - val barn2 = runBlocking { grunnlagService.hentSakerOgRoller(barnepensjonSoeker2) } + val barn2 = grunnlagService.hentSakerOgRoller(barnepensjonSoeker2) assertEquals(2, barn2.sakiderOgRoller.size) assertEquals(barnepensjonSoeker2.value, barn2.fnr) @@ -613,7 +590,7 @@ internal class GrunnlagServiceTest { grunnlaghendelse3, ) - val gjenlevende = runBlocking { grunnlagService.hentSakerOgRoller(gjenlevendeFnr) } + val gjenlevende = grunnlagService.hentSakerOgRoller(gjenlevendeFnr) assertEquals(3, gjenlevende.sakiderOgRoller.size) assertEquals(gjenlevendeFnr.value, gjenlevende.fnr) @@ -636,7 +613,7 @@ internal class GrunnlagServiceTest { grunnlaghendelse2, ) - val soesken = runBlocking { grunnlagService.hentSakerOgRoller(soeskenFnr) } + val soesken = grunnlagService.hentSakerOgRoller(soeskenFnr) assertEquals(2, soesken.sakiderOgRoller.size) assertTrue(soesken.sakiderOgRoller.all { it.rolle == Saksrolle.SOESKEN }) @@ -692,8 +669,7 @@ internal class GrunnlagServiceTest { persongalleri, ) - val opplysningsgrunnlag = - runBlocking { grunnlagService.hentOpplysningsgrunnlagForSak(sakId1)!! } + val opplysningsgrunnlag = grunnlagService.hentOpplysningsgrunnlagForSak(sakId1)!! assertEquals(1, opplysningsgrunnlag.sak.size) assertEquals(1, opplysningsgrunnlag.familie.size) @@ -940,8 +916,7 @@ internal class GrunnlagServiceTest { ), ) - val resultat = - runBlocking { grunnlagService.hentPersonopplysninger(behandlingsid, SakType.BARNEPENSJON) } + val resultat = grunnlagService.hentPersonopplysninger(behandlingsid, SakType.BARNEPENSJON) resultat.innsender?.opplysning?.foedselsnummer shouldBe SOEKER_FOEDSELSNUMMER resultat.soeker?.opplysning?.foedselsnummer shouldBe SOEKER_FOEDSELSNUMMER @@ -1018,8 +993,7 @@ internal class GrunnlagServiceTest { kilde = kilde, ), ) - val resultat = - runBlocking { grunnlagService.hentPersonopplysninger(behandlingsid, SakType.BARNEPENSJON) } + val resultat = grunnlagService.hentPersonopplysninger(behandlingsid, SakType.BARNEPENSJON) resultat.innsender?.opplysning?.foedselsnummer shouldBe GJENLEVENDE_FOEDSELSNUMMER resultat.soeker?.opplysning?.foedselsnummer shouldBe GJENLEVENDE_FOEDSELSNUMMER diff --git a/apps/etterlatte-behandling/src/test/kotlin/grunnlag/PersonRoutesTest.kt b/apps/etterlatte-behandling/src/test/kotlin/grunnlag/PersonRoutesTest.kt index e3dc7cfb70..3916c80629 100644 --- a/apps/etterlatte-behandling/src/test/kotlin/grunnlag/PersonRoutesTest.kt +++ b/apps/etterlatte-behandling/src/test/kotlin/grunnlag/PersonRoutesTest.kt @@ -13,10 +13,11 @@ import io.ktor.server.testing.ApplicationTestBuilder import io.ktor.server.testing.testApplication import io.mockk.Called import io.mockk.clearAllMocks -import io.mockk.coEvery import io.mockk.coVerify import io.mockk.confirmVerified +import io.mockk.every import io.mockk.mockk +import io.mockk.verify import no.nav.etterlatte.behandling.sakId1 import no.nav.etterlatte.behandling.sakId2 import no.nav.etterlatte.behandling.sakId3 @@ -85,7 +86,7 @@ internal class PersonRoutesTest { SakidOgRolle(sakId1, Saksrolle.SOEKER), ), ) - coEvery { grunnlagService.hentSakerOgRoller(any()) } returns response + every { grunnlagService.hentSakerOgRoller(any()) } returns response testApplication { val httpClient = createHttpClient() @@ -102,14 +103,14 @@ internal class PersonRoutesTest { assertEquals(serialize(response), actualResponse.body()) } - coVerify(exactly = 1) { grunnlagService.hentSakerOgRoller(any()) } + verify(exactly = 1) { grunnlagService.hentSakerOgRoller(any()) } coVerify { tilgangsservice wasNot Called } } @Test fun `Hent alle saker tilknyttet person`() { val response: Set = setOf(sakId1, sakId2, sakId3) - coEvery { grunnlagService.hentAlleSakerForFnr(any()) } returns response + every { grunnlagService.hentAlleSakerForFnr(any()) } returns response testApplication { val httpClient = createHttpClient() @@ -126,7 +127,7 @@ internal class PersonRoutesTest { assertEquals(serialize(response), actualResponse.body()) } - coVerify(exactly = 1) { grunnlagService.hentAlleSakerForFnr(any()) } + verify(exactly = 1) { grunnlagService.hentAlleSakerForFnr(any()) } coVerify { tilgangsservice wasNot Called } } diff --git a/apps/etterlatte-behandling/src/test/kotlin/grunnlag/SakGrunnlagRoutesKtTest.kt b/apps/etterlatte-behandling/src/test/kotlin/grunnlag/SakGrunnlagRoutesKtTest.kt index e7a637f1f5..f750abd743 100644 --- a/apps/etterlatte-behandling/src/test/kotlin/grunnlag/SakGrunnlagRoutesKtTest.kt +++ b/apps/etterlatte-behandling/src/test/kotlin/grunnlag/SakGrunnlagRoutesKtTest.kt @@ -14,7 +14,9 @@ import io.mockk.clearAllMocks import io.mockk.coEvery import io.mockk.coVerify import io.mockk.confirmVerified +import io.mockk.every import io.mockk.mockk +import io.mockk.verify import no.nav.etterlatte.behandling.randomSakId import no.nav.etterlatte.ktor.runServer import no.nav.etterlatte.ktor.startRandomPort @@ -69,7 +71,7 @@ internal class SakGrunnlagRoutesKtTest { fun `returnerer 404 hvis grunnlag ikke finnes`() { val sakId = randomSakId() - coEvery { grunnlagService.hentOpplysningsgrunnlagForSak(any()) } returns null + every { grunnlagService.hentOpplysningsgrunnlagForSak(any()) } returns null coEvery { tilgangsservice.harTilgangTilSak(any(), any()) } returns true testApplication { @@ -84,7 +86,7 @@ internal class SakGrunnlagRoutesKtTest { assertEquals(HttpStatusCode.NotFound, response.status) } - coVerify(exactly = 1) { grunnlagService.hentOpplysningsgrunnlagForSak(any()) } + verify(exactly = 1) { grunnlagService.hentOpplysningsgrunnlagForSak(any()) } } @Test @@ -92,7 +94,7 @@ internal class SakGrunnlagRoutesKtTest { val sakId = randomSakId() val testData = GrunnlagTestData().hentOpplysningsgrunnlag() - coEvery { grunnlagService.hentOpplysningsgrunnlagForSak(any()) } returns testData + every { grunnlagService.hentOpplysningsgrunnlagForSak(any()) } returns testData coEvery { tilgangsservice.harTilgangTilSak(any(), any()) } returns true testApplication { @@ -108,7 +110,7 @@ internal class SakGrunnlagRoutesKtTest { assertEquals(serialize(testData), response.body()) } - coVerify(exactly = 1) { grunnlagService.hentOpplysningsgrunnlagForSak(sakId) } + verify(exactly = 1) { grunnlagService.hentOpplysningsgrunnlagForSak(sakId) } } private fun ApplicationTestBuilder.createHttpClient(): HttpClient = diff --git a/apps/etterlatte-behandling/src/test/kotlin/grunnlagsendring/GrunnlagsendringshendelseServiceTest.kt b/apps/etterlatte-behandling/src/test/kotlin/grunnlagsendring/GrunnlagsendringshendelseServiceTest.kt index 09de72f734..456105237a 100644 --- a/apps/etterlatte-behandling/src/test/kotlin/grunnlagsendring/GrunnlagsendringshendelseServiceTest.kt +++ b/apps/etterlatte-behandling/src/test/kotlin/grunnlagsendring/GrunnlagsendringshendelseServiceTest.kt @@ -327,9 +327,9 @@ internal class GrunnlagsendringshendelseServiceTest { every { grunnlagshendelsesDao.hentGrunnlagsendringshendelserMedStatuserISak(any(), any()) } returns emptyList() - coEvery { grunnlagService.hentOpplysningsgrunnlagForSak(any()) } returns Grunnlag.empty() + every { grunnlagService.hentOpplysningsgrunnlagForSak(any()) } returns Grunnlag.empty() val sak2 = sakId2 - coEvery { grunnlagService.hentSakerOgRoller(any()) } + every { grunnlagService.hentSakerOgRoller(any()) } .returns( PersonMedSakerOgRoller( fnr, @@ -381,7 +381,7 @@ internal class GrunnlagsendringshendelseServiceTest { fun `Gyldige hendelser for saktype BP`(grltype: GrunnlagsendringsType) { val soekerFnr = KONTANT_FOT.value val sakId = sakId1 - coEvery { grunnlagService.hentSakerOgRoller(any()) } + every { grunnlagService.hentSakerOgRoller(any()) } .returns( PersonMedSakerOgRoller( soekerFnr, @@ -395,7 +395,7 @@ internal class GrunnlagsendringshendelseServiceTest { } returns sak every { pdlService.hentPdlModellForSaktype(soekerFnr, any(), sak.sakType) } returns mockPerson() - coEvery { grunnlagService.hentOpplysningsgrunnlagForSak(any()) } returns Grunnlag.empty() + every { grunnlagService.hentOpplysningsgrunnlagForSak(any()) } returns Grunnlag.empty() every { grunnlagshendelsesDao.hentGrunnlagsendringshendelserMedStatuserISak( sakId, @@ -438,7 +438,7 @@ internal class GrunnlagsendringshendelseServiceTest { fun `Gyldige hendelser for saktype OMS`(grltype: GrunnlagsendringsType) { val soekerFnr = KONTANT_FOT.value val sakId = sakId1 - coEvery { grunnlagService.hentSakerOgRoller(any()) } + every { grunnlagService.hentSakerOgRoller(any()) } .returns( PersonMedSakerOgRoller( soekerFnr, @@ -452,7 +452,7 @@ internal class GrunnlagsendringshendelseServiceTest { } returns sak every { pdlService.hentPdlModellForSaktype(soekerFnr, any(), sak.sakType) } returns mockPerson() - coEvery { grunnlagService.hentOpplysningsgrunnlagForSak(any()) } returns Grunnlag.empty() + every { grunnlagService.hentOpplysningsgrunnlagForSak(any()) } returns Grunnlag.empty() every { grunnlagshendelsesDao.hentGrunnlagsendringshendelserMedStatuserISak( sakId, @@ -490,7 +490,7 @@ internal class GrunnlagsendringshendelseServiceTest { fun `Skal filtrere bort sivilstandshendelser for BP saker men ikke andre, OMS skal få de`() { val soekerFnr = KONTANT_FOT.value val sakId = sakId1 - coEvery { grunnlagService.hentSakerOgRoller(any()) } + every { grunnlagService.hentSakerOgRoller(any()) } .returns( PersonMedSakerOgRoller( soekerFnr, @@ -515,7 +515,7 @@ internal class GrunnlagsendringshendelseServiceTest { every { pdlService.hentPdlModellForSaktype(soekerFnr, any(), SakType.OMSTILLINGSSTOENAD) } returns mockPerson() - coEvery { grunnlagService.hentOpplysningsgrunnlagForSak(any()) } returns Grunnlag.empty() + every { grunnlagService.hentOpplysningsgrunnlagForSak(any()) } returns Grunnlag.empty() every { grunnlagshendelsesDao.hentGrunnlagsendringshendelserMedStatuserISak( sakId, @@ -559,12 +559,12 @@ internal class GrunnlagsendringshendelseServiceTest { every { pdlService.hentPdlModellForSaktype(gjenlevendeFnr, any(), SakType.BARNEPENSJON) } returns mockPerson() .copy(bostedsadresse = listOf(OpplysningDTO(bostedAdresse, "adresse"))) - coEvery { grunnlagService.hentOpplysningsgrunnlagForSak(any()) } returns Grunnlag.empty() + every { grunnlagService.hentOpplysningsgrunnlagForSak(any()) } returns Grunnlag.empty() every { behandlingService.hentBehandlingerForSak(sakId) } returns emptyList() every { sakService.finnSak(sakId) } returns Sak(soekerFnr, SakType.BARNEPENSJON, sakId, Enheter.defaultEnhet.enhetNr) - coEvery { grunnlagService.hentSakerOgRoller(any()) } + every { grunnlagService.hentSakerOgRoller(any()) } .returns( PersonMedSakerOgRoller( soekerFnr, @@ -658,13 +658,13 @@ internal class GrunnlagsendringshendelseServiceTest { val adressebeskyttelse = Adressebeskyttelse("1", Endringstype.OPPRETTET, fnr) - coEvery { grunnlagService.hentAlleSakerForFnr(any()) } returns emptySet() + every { grunnlagService.hentAlleSakerForFnr(any()) } returns emptySet() runBlocking { grunnlagsendringshendelseService.oppdaterAdressebeskyttelseHendelse(adressebeskyttelse) } - coVerify(exactly = 1) { + verify(exactly = 1) { grunnlagService.hentAlleSakerForFnr(Folkeregisteridentifikator.of(adressebeskyttelse.fnr)) } } @@ -676,18 +676,18 @@ internal class GrunnlagsendringshendelseServiceTest { Adressebeskyttelse("1", Endringstype.OPPRETTET, fnr) val sakId = SakId(1L) - coEvery { grunnlagService.hentAlleSakerForFnr(any()) } returns setOf(sakId) + every { grunnlagService.hentAlleSakerForFnr(any()) } returns setOf(sakId) val persongalleri = persongalleri() - coEvery { grunnlagService.hentPersongalleri(sakId) } returns persongalleri + every { grunnlagService.hentPersongalleri(sakId) } returns persongalleri coEvery { oppdaterTilgangService.haandtergraderingOgEgenAnsatt(sakId, persongalleri) } just Runs runBlocking { grunnlagsendringshendelseService.oppdaterAdressebeskyttelseHendelse(adressebeskyttelse) } coVerify(exactly = 1) { - grunnlagService.hentAlleSakerForFnr(Folkeregisteridentifikator.of(adressebeskyttelse.fnr)) oppdaterTilgangService.haandtergraderingOgEgenAnsatt(sakId, persongalleri) } + verify { grunnlagService.hentAlleSakerForFnr(Folkeregisteridentifikator.of(adressebeskyttelse.fnr)) } } @Test @@ -724,7 +724,7 @@ internal class GrunnlagsendringshendelseServiceTest { every { pdlService.hentPdlModellForSaktype(any(), any(), SakType.BARNEPENSJON) } returns mockPerson() every { behandlingService.hentBehandlingerForSak(any()) } returns emptyList() - coEvery { grunnlagService.hentOpplysningsgrunnlagForSak(sakId) } returns Grunnlag.empty() + every { grunnlagService.hentOpplysningsgrunnlagForSak(sakId) } returns Grunnlag.empty() grunnlagsendringshendelseService.opprettEndretGrunnbeloepHendelse(sakId) @@ -747,7 +747,7 @@ internal class GrunnlagsendringshendelseServiceTest { every { sakService.finnSak(sakId) } returns Sak(fnr, SakType.BARNEPENSJON, sakId, Enheter.defaultEnhet.enhetNr) - coEvery { grunnlagService.hentSakerOgRoller(any()) } + every { grunnlagService.hentSakerOgRoller(any()) } .returns( PersonMedSakerOgRoller( KONTANT_FOT.value, @@ -794,7 +794,7 @@ internal class GrunnlagsendringshendelseServiceTest { every { sakService.finnSak(sakId) } returns Sak(fnr, SakType.BARNEPENSJON, sakId, Enheter.defaultEnhet.enhetNr) - coEvery { grunnlagService.hentSakerOgRoller(any()) } + every { grunnlagService.hentSakerOgRoller(any()) } .returns( PersonMedSakerOgRoller( KONTANT_FOT.value, @@ -829,7 +829,7 @@ internal class GrunnlagsendringshendelseServiceTest { every { sakService.finnSak(sakId) } returns null - coEvery { grunnlagService.hentSakerOgRoller(any()) } + every { grunnlagService.hentSakerOgRoller(any()) } .returns( PersonMedSakerOgRoller( KONTANT_FOT.value, @@ -878,7 +878,7 @@ internal class GrunnlagsendringshendelseServiceTest { every { sakService.finnSaker(any()) } returns listOf(sak) every { pdlService.hentPdlModellForSaktype(any(), any(), SakType.OMSTILLINGSSTOENAD) } returns mockPerson() - coEvery { grunnlagService.hentOpplysningsgrunnlagForSak(any()) } returns Grunnlag.empty() + every { grunnlagService.hentOpplysningsgrunnlagForSak(any()) } returns Grunnlag.empty() every { grunnlagshendelsesDao.hentGrunnlagsendringshendelserMedStatuserISak(any(), any()) } returns emptyList() @@ -912,9 +912,8 @@ internal class GrunnlagsendringshendelseServiceTest { (this.samsvarMellomKildeOgGrunnlag as SamsvarMellomKildeOgGrunnlag.Folkeregisteridentifikatorsamsvar).fraPdl!!.value, ) } - - coVerify { grunnlagService.hentOpplysningsgrunnlagForSak(sak.id) } verify { + grunnlagService.hentOpplysningsgrunnlagForSak(sak.id) sakService.finnSaker(hendelse.fnr) pdlService.hentPdlModellForSaktype(sak.ident, PersonRolle.GJENLEVENDE, sak.sakType) grunnlagshendelsesDao.hentGrunnlagsendringshendelserMedStatuserISak(sak.id, any()) @@ -943,7 +942,7 @@ internal class GrunnlagsendringshendelseServiceTest { every { sakService.finnSaker(any()) } returns listOf(sak) every { pdlService.hentPdlModellForSaktype(any(), any(), SakType.BARNEPENSJON) } returns mockPerson() - coEvery { grunnlagService.hentOpplysningsgrunnlagForSak(any()) } returns Grunnlag.empty() + every { grunnlagService.hentOpplysningsgrunnlagForSak(any()) } returns Grunnlag.empty() every { grunnlagshendelsesDao.hentGrunnlagsendringshendelserMedStatuserISak(any(), any()) } returns emptyList() @@ -978,8 +977,8 @@ internal class GrunnlagsendringshendelseServiceTest { ) } - coVerify { grunnlagService.hentOpplysningsgrunnlagForSak(sak.id) } verify { + grunnlagService.hentOpplysningsgrunnlagForSak(sak.id) sakService.finnSaker(hendelse.fnr) pdlService.hentPdlModellForSaktype(sak.ident, PersonRolle.BARN, sak.sakType) grunnlagshendelsesDao.hentGrunnlagsendringshendelserMedStatuserISak(sak.id, any()) diff --git a/apps/etterlatte-behandling/src/test/kotlin/grunnlagsendring/doedshendelse/DoedshendelseJobServiceTest.kt b/apps/etterlatte-behandling/src/test/kotlin/grunnlagsendring/doedshendelse/DoedshendelseJobServiceTest.kt index fe1261e8ef..ce01c9dea9 100644 --- a/apps/etterlatte-behandling/src/test/kotlin/grunnlagsendring/doedshendelse/DoedshendelseJobServiceTest.kt +++ b/apps/etterlatte-behandling/src/test/kotlin/grunnlagsendring/doedshendelse/DoedshendelseJobServiceTest.kt @@ -5,6 +5,7 @@ import io.mockk.coEvery import io.mockk.coJustRun import io.mockk.every import io.mockk.just +import io.mockk.justRun import io.mockk.mockk import io.mockk.runs import io.mockk.slot @@ -71,7 +72,7 @@ class DoedshendelseJobServiceTest { private val grunnlagService = mockk { coJustRun { opprettGrunnlag(any(), any()) } - coJustRun { lagreNyeSaksopplysningerBareSak(any(), any()) } + justRun { lagreNyeSaksopplysningerBareSak(any(), any()) } coJustRun { opprettEllerOppdaterGrunnlagForSak(any(), any()) } } diff --git a/apps/etterlatte-behandling/src/test/kotlin/inntektsjustering/AarligInntektsjusteringJobbServiceTest.kt b/apps/etterlatte-behandling/src/test/kotlin/inntektsjustering/AarligInntektsjusteringJobbServiceTest.kt index 412e25afda..56c7fffc52 100644 --- a/apps/etterlatte-behandling/src/test/kotlin/inntektsjustering/AarligInntektsjusteringJobbServiceTest.kt +++ b/apps/etterlatte-behandling/src/test/kotlin/inntektsjustering/AarligInntektsjusteringJobbServiceTest.kt @@ -130,7 +130,7 @@ class AarligInntektsjusteringJobbServiceTest { every { boddEllerArbeidetUtlandet } returns mockk() every { opphoerFraOgMed } returns null } - coEvery { grunnlagService.hentPersonopplysninger(any(), any()) } returns + every { grunnlagService.hentPersonopplysninger(any(), any()) } returns mockk { every { soeker } returns mockk { @@ -138,7 +138,7 @@ class AarligInntektsjusteringJobbServiceTest { } } - coEvery { grunnlagService.hentPersongalleri(any()) } returns persongalleri + every { grunnlagService.hentPersongalleri(any()) } returns persongalleri coEvery { revurderingService.opprettRevurdering( any(), @@ -605,7 +605,7 @@ class AarligInntektsjusteringJobbServiceTest { vergeEllerFullmektig = VergeEllerFullmektig(null, null, null, null, null), ) - coEvery { grunnlagService.hentPersonopplysninger(any(), any()) } returns + every { grunnlagService.hentPersonopplysninger(any(), any()) } returns mockk { every { soeker } returns mockk { diff --git a/apps/etterlatte-behandling/src/test/kotlin/integration/EksterneKlienter.kt b/apps/etterlatte-behandling/src/test/kotlin/integration/EksterneKlienter.kt index 0df4e597e8..c8efa93ff8 100644 --- a/apps/etterlatte-behandling/src/test/kotlin/integration/EksterneKlienter.kt +++ b/apps/etterlatte-behandling/src/test/kotlin/integration/EksterneKlienter.kt @@ -1,6 +1,5 @@ package no.nav.etterlatte -import com.fasterxml.jackson.databind.JsonNode import io.mockk.every import io.mockk.mockk import no.nav.etterlatte.behandling.domain.ArbeidsFordelingEnhet @@ -18,7 +17,6 @@ import no.nav.etterlatte.behandling.klienter.SaksbehandlerInfo import no.nav.etterlatte.behandling.klienter.TilbakekrevingKlient import no.nav.etterlatte.behandling.klienter.VedtakKlient import no.nav.etterlatte.behandling.randomSakId -import no.nav.etterlatte.behandling.sakId1 import no.nav.etterlatte.brev.BrevKlient import no.nav.etterlatte.brev.BrevParametre import no.nav.etterlatte.brev.BrevPayload @@ -40,11 +38,6 @@ import no.nav.etterlatte.common.klienter.PdlTjenesterKlient import no.nav.etterlatte.common.klienter.PesysKlient import no.nav.etterlatte.common.klienter.SakSammendragResponse import no.nav.etterlatte.common.klienter.SkjermingKlient -import no.nav.etterlatte.grunnlag.BehandlingGrunnlagVersjon -import no.nav.etterlatte.grunnlag.GrunnlagService -import no.nav.etterlatte.grunnlag.PersonMedNavn -import no.nav.etterlatte.grunnlag.PersongalleriSamsvar -import no.nav.etterlatte.grunnlag.PersonopplysningerResponse import no.nav.etterlatte.kodeverk.Beskrivelse import no.nav.etterlatte.kodeverk.Betydning import no.nav.etterlatte.kodeverk.KodeverkKlient @@ -52,20 +45,12 @@ import no.nav.etterlatte.kodeverk.KodeverkNavn import no.nav.etterlatte.kodeverk.KodeverkResponse import no.nav.etterlatte.libs.common.Enhetsnummer import no.nav.etterlatte.libs.common.behandling.Klage -import no.nav.etterlatte.libs.common.behandling.PersonMedSakerOgRoller import no.nav.etterlatte.libs.common.behandling.Persongalleri import no.nav.etterlatte.libs.common.behandling.SakType -import no.nav.etterlatte.libs.common.behandling.SakidOgRolle -import no.nav.etterlatte.libs.common.behandling.Saksrolle import no.nav.etterlatte.libs.common.beregning.AvkortingDto import no.nav.etterlatte.libs.common.beregning.InntektsjusteringAvkortingInfoResponse import no.nav.etterlatte.libs.common.brev.BestillingsIdDto import no.nav.etterlatte.libs.common.brev.JournalpostIdDto -import no.nav.etterlatte.libs.common.grunnlag.Grunnlag -import no.nav.etterlatte.libs.common.grunnlag.Grunnlagsopplysning -import no.nav.etterlatte.libs.common.grunnlag.OppdaterGrunnlagRequest -import no.nav.etterlatte.libs.common.grunnlag.Opplysningsbehov -import no.nav.etterlatte.libs.common.grunnlag.opplysningstyper.Opplysningstype import no.nav.etterlatte.libs.common.pdl.PersonDTO import no.nav.etterlatte.libs.common.person.AdressebeskyttelseGradering import no.nav.etterlatte.libs.common.person.Folkeregisteridentifikator @@ -90,7 +75,6 @@ import no.nav.etterlatte.libs.ktor.ServiceStatus import no.nav.etterlatte.libs.ktor.route.SakTilgangsSjekk import no.nav.etterlatte.libs.ktor.token.BrukerTokenInfo import no.nav.etterlatte.libs.ktor.token.Saksbehandler -import no.nav.etterlatte.libs.testdata.grunnlag.GrunnlagTestData import no.nav.etterlatte.libs.testdata.grunnlag.soeker import no.nav.etterlatte.oppgaveGosys.EndreStatusRequest import no.nav.etterlatte.oppgaveGosys.GosysApiOppgave @@ -103,114 +87,6 @@ import no.nav.etterlatte.saksbehandler.SaksbehandlerEnhet import java.time.LocalDate import java.time.YearMonth import java.util.UUID -import kotlin.random.Random - -class GrunnlagServiceTest : GrunnlagService { - override suspend fun grunnlagFinnesForSak(sakId: SakId): Boolean = false - - override suspend fun hentGrunnlagAvType( - behandlingId: UUID, - opplysningstype: Opplysningstype, - ): Grunnlagsopplysning? { - val personopplysning = personOpplysning(doedsdato = LocalDate.parse("2022-01-01")) - return grunnlagsOpplysningMedPersonopplysning(personopplysning) - } - - override suspend fun lagreNyeSaksopplysninger( - sakId: SakId, - behandlingId: UUID, - nyeOpplysninger: List>, - ) { - // do nothing - } - - override suspend fun lagreNyeSaksopplysningerBareSak( - sakId: SakId, - nyeOpplysninger: List>, - ) { - // do nothing - } - - override suspend fun lagreNyePersonopplysninger( - sakId: SakId, - behandlingId: UUID, - fnr: Folkeregisteridentifikator, - nyeOpplysninger: List>, - ) { - // do nothing - } - - override suspend fun hentPersongalleri(sakId: SakId): Persongalleri = defaultPersongalleriGydligeFnr - - override suspend fun hentPersongalleri(behandlingId: UUID): Persongalleri = defaultPersongalleriGydligeFnr - - override suspend fun hentOpplysningsgrunnlagForSak(sakId: SakId): Grunnlag? = GrunnlagTestData().hentOpplysningsgrunnlag() - - override suspend fun hentOpplysningsgrunnlag(behandlingId: UUID): Grunnlag = GrunnlagTestData().hentOpplysningsgrunnlag() - - override suspend fun hentPersonopplysninger( - behandlingId: UUID, - sakstype: SakType, - ): PersonopplysningerResponse = GrunnlagTestData().hentPersonopplysninger() - - override suspend fun hentAlleSakerForFnr(fnr: Folkeregisteridentifikator): Set = setOf(sakId1) - - override suspend fun hentPersonerISak(sakId: SakId): Map? { - TODO() - } - - override suspend fun opprettGrunnlag( - behandlingId: UUID, - opplysningsbehov: Opplysningsbehov, - ) { - // do nothing - } - - override suspend fun oppdaterGrunnlagForSak(oppdaterGrunnlagRequest: OppdaterGrunnlagRequest) { - // do nothing - } - - override suspend fun opprettEllerOppdaterGrunnlagForSak( - sakId: SakId, - opplysningsbehov: Opplysningsbehov, - ) { - // do nothing - } - - override suspend fun oppdaterGrunnlag( - behandlingId: UUID, - sakId: SakId, - sakType: SakType, - ) { - // do nothing - } - - override suspend fun hentHistoriskForeldreansvar(behandlingId: UUID): Grunnlagsopplysning? { - TODO("Not yet implemented") - } - - override suspend fun hentPersongalleriSamsvar(behandlingId: UUID): PersongalleriSamsvar { - TODO("Not yet implemented") - } - - override suspend fun laasTilVersjonForBehandling( - skalLaasesId: UUID, - idLaasesTil: UUID, - ): BehandlingGrunnlagVersjon = BehandlingGrunnlagVersjon(UUID.randomUUID(), sakId1, Random.nextLong(), true) - - override suspend fun hentSakerOgRoller(fnr: Folkeregisteridentifikator): PersonMedSakerOgRoller = - PersonMedSakerOgRoller("08071272487", listOf(SakidOgRolle(sakId1, Saksrolle.SOEKER))) - - override suspend fun laasVersjonForBehandling(behandlingId: UUID) { - TODO("Not yet implemented") - } - - suspend fun aldersovergangMaaned( - sakId: SakId, - sakType: SakType, - brukerTokenInfo: BrukerTokenInfo, - ) = YearMonth.now() -} class BeregningKlientTest : BeregningKlient, diff --git a/apps/etterlatte-behandling/src/test/kotlin/sak/SakServiceTest.kt b/apps/etterlatte-behandling/src/test/kotlin/sak/SakServiceTest.kt index ae13a3ce11..beaffb8cb5 100644 --- a/apps/etterlatte-behandling/src/test/kotlin/sak/SakServiceTest.kt +++ b/apps/etterlatte-behandling/src/test/kotlin/sak/SakServiceTest.kt @@ -106,7 +106,7 @@ internal class SakServiceTest { Gjelder ikke for oppdaterIdentForSak() */ coEvery { grunnlagservice.opprettGrunnlag(any(), any()) } just runs - coEvery { grunnlagservice.lagreNyeSaksopplysningerBareSak(any(), any()) } just runs + every { grunnlagservice.lagreNyeSaksopplysningerBareSak(any(), any()) } just runs coEvery { krrKlient.hentDigitalKontaktinformasjon(any()) } returns DigitalKontaktinformasjon( @@ -293,7 +293,7 @@ internal class SakServiceTest { val sakId = sakId1 coEvery { pdlTjenesterKlient.hentPdlFolkeregisterIdenter(any()) } returns dummyPdlResponse(KONTANT_FOT.value) - coEvery { grunnlagservice.hentSakerOgRoller(KONTANT_FOT) } returns + every { grunnlagservice.hentSakerOgRoller(KONTANT_FOT) } returns PersonMedSakerOgRoller( KONTANT_FOT.value, emptyList(), @@ -312,7 +312,7 @@ internal class SakServiceTest { finnSakerOmsOgHvisAvdoed shouldContainExactly listOf(sakId) - coVerify { grunnlagservice.hentSakerOgRoller(KONTANT_FOT) } + verify { grunnlagservice.hentSakerOgRoller(KONTANT_FOT) } coVerify(exactly = 1) { pdlTjenesterKlient.hentPdlFolkeregisterIdenter(KONTANT_FOT.value) } verify(exactly = 1) { sakLesDao.finnSaker(KONTANT_FOT.value, SakType.OMSTILLINGSSTOENAD) } } @@ -323,7 +323,7 @@ internal class SakServiceTest { val sakId = sakId1 coEvery { pdlTjenesterKlient.hentPdlFolkeregisterIdenter(any()) } returns dummyPdlResponse(KONTANT_FOT.value) - coEvery { grunnlagservice.hentSakerOgRoller(KONTANT_FOT) } returns + every { grunnlagservice.hentSakerOgRoller(KONTANT_FOT) } returns PersonMedSakerOgRoller( KONTANT_FOT.value, listOf( @@ -336,9 +336,11 @@ internal class SakServiceTest { finnSakerOmsOgHvisAvdoed shouldContainExactly listOf(sakId) - coVerify { grunnlagservice.hentSakerOgRoller(KONTANT_FOT) } coVerify(exactly = 1) { pdlTjenesterKlient.hentPdlFolkeregisterIdenter(KONTANT_FOT.value) } - verify(exactly = 1) { sakLesDao.finnSaker(KONTANT_FOT.value, SakType.OMSTILLINGSSTOENAD) } + verify(exactly = 1) { + sakLesDao.finnSaker(KONTANT_FOT.value, SakType.OMSTILLINGSSTOENAD) + grunnlagservice.hentSakerOgRoller(KONTANT_FOT) + } } @Test @@ -419,7 +421,7 @@ internal class SakServiceTest { coEvery { skjermingKlient.personErSkjermet(KONTANT_FOT.value) } returns false every { sakSkrivDao.oppdaterSkjerming(any(), any()) } just runs every { sakSkrivDao.oppdaterAdresseBeskyttelse(sakId1, AdressebeskyttelseGradering.UGRADERT) } just runs - coEvery { grunnlagservice.grunnlagFinnesForSak(any()) } returns true + every { grunnlagservice.grunnlagFinnesForSak(any()) } returns true every { norg2Klient.hentArbeidsfordelingForOmraadeOgTema( ArbeidsFordelingRequest( @@ -471,7 +473,7 @@ internal class SakServiceTest { coEvery { pdlTjenesterKlient.hentPdlFolkeregisterIdenter(any()) } returns dummyPdlResponse(KONTANT_FOT.value) every { sakLesDao.finnSaker(KONTANT_FOT.value, SakType.BARNEPENSJON) } returns emptyList() coEvery { skjermingKlient.personErSkjermet(KONTANT_FOT.value) } returns false - coEvery { grunnlagservice.grunnlagFinnesForSak(any()) } returns true + every { grunnlagservice.grunnlagFinnesForSak(any()) } returns true every { sakSkrivDao.oppdaterSkjerming(any(), any()) } just runs val sak1 = Sak( @@ -567,7 +569,7 @@ internal class SakServiceTest { every { sakSkrivDao.oppdaterEnhet(any()) } just runs every { sakSkrivDao.oppdaterAdresseBeskyttelse(sakId1, AdressebeskyttelseGradering.UGRADERT) } just runs every { sakSkrivDao.oppdaterSkjerming(any(), any()) } just runs - coEvery { grunnlagservice.grunnlagFinnesForSak(any()) } returns true + every { grunnlagservice.grunnlagFinnesForSak(any()) } returns true every { norg2Klient.hentArbeidsfordelingForOmraadeOgTema( @@ -896,8 +898,8 @@ internal class SakServiceTest { ) justRun { sakSkrivDao.oppdaterIdent(any(), any()) } every { sakLesDao.hentSak(any()) } returns sak.copy(ident = JOVIAL_LAMA.value) - coEvery { grunnlagservice.hentPersongalleri(any()) } returns persongalleri - coJustRun { grunnlagservice.lagreNyeSaksopplysningerBareSak(any(), any()) } + every { grunnlagservice.hentPersongalleri(any()) } returns persongalleri + justRun { grunnlagservice.lagreNyeSaksopplysningerBareSak(any(), any()) } coJustRun { grunnlagservice.opprettEllerOppdaterGrunnlagForSak(any(), any()) } val oppdatertSak = service.oppdaterIdentForSak(sak, saksbehandler) diff --git a/apps/etterlatte-behandling/src/test/kotlin/vilkaarsvurdering/VilkaarsvurderingRoutesTest.kt b/apps/etterlatte-behandling/src/test/kotlin/vilkaarsvurdering/VilkaarsvurderingRoutesTest.kt index 3b234a5a63..c5f3610bae 100644 --- a/apps/etterlatte-behandling/src/test/kotlin/vilkaarsvurdering/VilkaarsvurderingRoutesTest.kt +++ b/apps/etterlatte-behandling/src/test/kotlin/vilkaarsvurdering/VilkaarsvurderingRoutesTest.kt @@ -114,11 +114,11 @@ internal class VilkaarsvurderingRoutesTest( // TODO: må se på mock strukturen her kontra alle steder man gjør alt dette x**y ganger @BeforeEach fun beforeEach() { - coEvery { behandlingService.hentBehandling(any()) } returns behandling() + every { behandlingService.hentBehandling(any()) } returns behandling() every { behandlingStatus.settVilkaarsvurdert(any(), any()) } just Runs every { behandlingStatus.settOpprettet(any(), any(), any()) } just Runs val grunnlagMock = grunnlagMedVersjon(grunnlagVersjon) - coEvery { grunnlagService.hentOpplysningsgrunnlag(any()) } returns grunnlagMock + every { grunnlagService.hentOpplysningsgrunnlag(any()) } returns grunnlagMock } @AfterEach @@ -183,7 +183,7 @@ internal class VilkaarsvurderingRoutesTest( opprettVilkaarsvurdering(vilkaarsvurderingServiceImpl) - coEvery { grunnlagService.hentOpplysningsgrunnlag(behandlingId) } returns + every { grunnlagService.hentOpplysningsgrunnlag(behandlingId) } returns grunnlagMedVersjon(nyGrunnlagVersjon) val response = diff --git a/apps/etterlatte-behandling/src/test/kotlin/vilkaarsvurdering/VilkaarsvurderingServiceTest.kt b/apps/etterlatte-behandling/src/test/kotlin/vilkaarsvurdering/VilkaarsvurderingServiceTest.kt index c76ee0a75d..f92f6a1032 100644 --- a/apps/etterlatte-behandling/src/test/kotlin/vilkaarsvurdering/VilkaarsvurderingServiceTest.kt +++ b/apps/etterlatte-behandling/src/test/kotlin/vilkaarsvurdering/VilkaarsvurderingServiceTest.kt @@ -10,7 +10,6 @@ import io.kotest.matchers.shouldNotBe import io.mockk.Runs import io.mockk.clearAllMocks import io.mockk.coEvery -import io.mockk.coVerify import io.mockk.every import io.mockk.just import io.mockk.mockk @@ -620,7 +619,7 @@ internal class VilkaarsvurderingServiceTest( vilkaarsvurderingMedKopierteOppdaterteVilkaar.resultat shouldBe null - coVerify { behandlingService.hentBehandling(any()) } + verify { behandlingService.hentBehandling(any()) } verify(exactly = 1) { behandlingStatus.settVilkaarsvurdert(any(), brukerTokenInfo, true) } verify(exactly = 0) { behandlingStatus.settVilkaarsvurdert(any(), brukerTokenInfo, false) } @@ -812,9 +811,9 @@ internal class VilkaarsvurderingServiceTest( coEvery { behandlingService.hentBehandling(behandling2.id) } returns behandling2 every { behandlingService.hentBehandlingerForSak(any()) } returns listOf(behandling1, behandling2) - coEvery { grunnlagService.hentPersongalleri(sakId1) } returns mockk { every { avdoed } returns listOf(avdoed1.value) } - coEvery { grunnlagService.hentPersongalleri(sakId2) } returns mockk { every { avdoed } returns listOf(avdoed1.value) } - coEvery { grunnlagService.hentSakerOgRoller(avdoed1) } returns + every { grunnlagService.hentPersongalleri(sakId1) } returns mockk { every { avdoed } returns listOf(avdoed1.value) } + every { grunnlagService.hentPersongalleri(sakId2) } returns mockk { every { avdoed } returns listOf(avdoed1.value) } + every { grunnlagService.hentSakerOgRoller(avdoed1) } returns PersonMedSakerOgRoller( avdoed1.value, listOf( @@ -863,9 +862,9 @@ internal class VilkaarsvurderingServiceTest( coEvery { behandlingService.hentBehandling(behandling2.id) } returns behandling2 every { behandlingService.hentBehandlingerForSak(any()) } returns listOf(behandling1, behandling2) - coEvery { grunnlagService.hentPersongalleri(sakId1) } returns mockk { every { avdoed } returns listOf(avdoed1.value) } - coEvery { grunnlagService.hentPersongalleri(sakId2) } returns mockk { every { avdoed } returns listOf(avdoed1.value) } - coEvery { grunnlagService.hentSakerOgRoller(avdoed1) } returns + every { grunnlagService.hentPersongalleri(sakId1) } returns mockk { every { avdoed } returns listOf(avdoed1.value) } + every { grunnlagService.hentPersongalleri(sakId2) } returns mockk { every { avdoed } returns listOf(avdoed1.value) } + every { grunnlagService.hentSakerOgRoller(avdoed1) } returns PersonMedSakerOgRoller( avdoed1.value, listOf( @@ -907,11 +906,11 @@ internal class VilkaarsvurderingServiceTest( coEvery { behandlingService.hentBehandling(behandling2.id) } returns behandling2 every { behandlingService.hentBehandlingerForSak(any()) } returns listOf(behandling1, behandling2) - coEvery { grunnlagService.hentPersongalleri(sakId1) } returns + every { grunnlagService.hentPersongalleri(sakId1) } returns mockk { every { avdoed } returns listOf(avdoed1.value, avdoed2.value) } - coEvery { grunnlagService.hentPersongalleri(sakId2) } returns + every { grunnlagService.hentPersongalleri(sakId2) } returns mockk { every { avdoed } returns listOf(avdoed1.value, avdoed2.value) } - coEvery { grunnlagService.hentSakerOgRoller(avdoed1) } returns + every { grunnlagService.hentSakerOgRoller(avdoed1) } returns PersonMedSakerOgRoller( avdoed1.value, listOf( @@ -919,7 +918,7 @@ internal class VilkaarsvurderingServiceTest( SakidOgRolle(sakId2, Saksrolle.AVDOED), ), ) - coEvery { grunnlagService.hentSakerOgRoller(avdoed2) } returns + every { grunnlagService.hentSakerOgRoller(avdoed2) } returns PersonMedSakerOgRoller( avdoed2.value, listOf( @@ -955,11 +954,11 @@ internal class VilkaarsvurderingServiceTest( coEvery { behandlingService.hentBehandling(behandling2.id) } returns behandling2 every { behandlingService.hentBehandlingerForSak(any()) } returns listOf(behandling1, behandling2) - coEvery { grunnlagService.hentPersongalleri(sakId1) } returns + every { grunnlagService.hentPersongalleri(sakId1) } returns mockk { every { avdoed } returns listOf(avdoed1.value, avdoed2.value) } - coEvery { grunnlagService.hentPersongalleri(sakId2) } returns mockk { every { avdoed } returns listOf(avdoed1.value) } + every { grunnlagService.hentPersongalleri(sakId2) } returns mockk { every { avdoed } returns listOf(avdoed1.value) } - coEvery { grunnlagService.hentSakerOgRoller(avdoed1) } returns + every { grunnlagService.hentSakerOgRoller(avdoed1) } returns PersonMedSakerOgRoller( avdoed1.value, listOf( @@ -968,7 +967,7 @@ internal class VilkaarsvurderingServiceTest( ), ) - coEvery { grunnlagService.hentSakerOgRoller(avdoed2) } returns + every { grunnlagService.hentSakerOgRoller(avdoed2) } returns PersonMedSakerOgRoller( avdoed2.value, listOf( From c92726bf0a6f989209cffa82475f75f33dd03d72 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Sebastian=20Oliver=20S=C3=B8berg?= <82504565+sebassonav@users.noreply.github.com> Date: Wed, 26 Feb 2025 11:00:15 +0100 Subject: [PATCH 2/7] SLeng inn servicebruker dev (#7133) --- .../src/main/resources/db/dev/V241__lag_servicerolledev.sql | 1 + 1 file changed, 1 insertion(+) create mode 100644 apps/etterlatte-behandling/src/main/resources/db/dev/V241__lag_servicerolledev.sql diff --git a/apps/etterlatte-behandling/src/main/resources/db/dev/V241__lag_servicerolledev.sql b/apps/etterlatte-behandling/src/main/resources/db/dev/V241__lag_servicerolledev.sql new file mode 100644 index 0000000000..c1c002ebf5 --- /dev/null +++ b/apps/etterlatte-behandling/src/main/resources/db/dev/V241__lag_servicerolledev.sql @@ -0,0 +1 @@ +create role "cloudsqliamserviceaccount"; \ No newline at end of file From 94079612e516e0f228f987c82270bc69d7424e32 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Sebastian=20Oliver=20S=C3=B8berg?= <82504565+sebassonav@users.noreply.github.com> Date: Wed, 26 Feb 2025 11:46:14 +0100 Subject: [PATCH 3/7] Rollen fantes allikevel (#7134) --- .../src/main/resources/db/dev/V241__lag_servicerolledev.sql | 1 - 1 file changed, 1 deletion(-) delete mode 100644 apps/etterlatte-behandling/src/main/resources/db/dev/V241__lag_servicerolledev.sql diff --git a/apps/etterlatte-behandling/src/main/resources/db/dev/V241__lag_servicerolledev.sql b/apps/etterlatte-behandling/src/main/resources/db/dev/V241__lag_servicerolledev.sql deleted file mode 100644 index c1c002ebf5..0000000000 --- a/apps/etterlatte-behandling/src/main/resources/db/dev/V241__lag_servicerolledev.sql +++ /dev/null @@ -1 +0,0 @@ -create role "cloudsqliamserviceaccount"; \ No newline at end of file From f0214402e3476ec3e5c926605edcb23ab95ebac1 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Sebastian=20Oliver=20S=C3=B8berg?= <82504565+sebassonav@users.noreply.github.com> Date: Wed, 26 Feb 2025 11:56:06 +0100 Subject: [PATCH 4/7] =?UTF-8?q?Ikke=20kunne=20lagre=20p=C3=A5=20nytt=20f?= =?UTF-8?q?=C3=B8r=20siden=20har=20blitt=20refreshet=20(#7132)?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- .../familieforhold/RedigerFamilieforholdModal.tsx | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/apps/etterlatte-saksbehandling-ui/client/src/components/behandling/soeknadsoversikt/familieforhold/RedigerFamilieforholdModal.tsx b/apps/etterlatte-saksbehandling-ui/client/src/components/behandling/soeknadsoversikt/familieforhold/RedigerFamilieforholdModal.tsx index e6dd67d874..313ca6cb32 100644 --- a/apps/etterlatte-saksbehandling-ui/client/src/components/behandling/soeknadsoversikt/familieforhold/RedigerFamilieforholdModal.tsx +++ b/apps/etterlatte-saksbehandling-ui/client/src/components/behandling/soeknadsoversikt/familieforhold/RedigerFamilieforholdModal.tsx @@ -3,7 +3,7 @@ import React, { useState } from 'react' import { PencilIcon, PlusIcon, XMarkIcon } from '@navikt/aksel-icons' import { useApiCall } from '~shared/hooks/useApiCall' import { redigerFamilieforhold } from '~shared/api/behandling' -import { isPending, mapResult } from '~shared/api/apiUtils' +import { isPending, isSuccess, mapResult } from '~shared/api/apiUtils' import { Personopplysninger } from '~shared/types/grunnlag' import { IDetaljertBehandling } from '~shared/types/IDetaljertBehandling' import { useFieldArray, useForm } from 'react-hook-form' @@ -176,7 +176,7 @@ export const RedigerFamilieforholdModal = ({ behandling, personopplysninger }: P - From b0aac8591c88c42469deabdea7302d399d7c8155 Mon Sep 17 00:00:00 2001 From: Andreas Balevik Date: Wed, 26 Feb 2025 13:29:09 +0100 Subject: [PATCH 5/7] =?UTF-8?q?FAGSYSTEM-371203=20ikke=20kast=20exception?= =?UTF-8?q?=20hvis=20fors=C3=B8ker=20=C3=A5=20slette=20brev=20men=20?= =?UTF-8?q?=E2=80=A6=20(#7135)?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit * FAGSYSTEM-371203 ikke kast exception hvis forsøker å slette brev men det er allerede slettet * FAGSYSTEM-371203 fikset test * FAGSYSTEM-371203 rette tilbakemeldinger * FAGSYSTEM-371203 bedre håndtering av vedtaksbrev ka nikke slettes * FAGSYSTEM-371203 rette try catch * Revert "FAGSYSTEM-371203 rette try catch" This reverts commit 2640adc1e8e6d4c10c527794a5ccd67dfd48a98d. * FAGSYSTEM-371203 la til ekstra kommentar --- .../no/nav/etterlatte/brev/BrevService.kt | 25 +++++++++++++------ .../no/nav/etterlatte/brev/BrevServiceTest.kt | 5 ++-- 2 files changed, 21 insertions(+), 9 deletions(-) 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 32114b450c..c3a42542f8 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 @@ -435,16 +435,26 @@ class BrevService( id: BrevID, bruker: BrukerTokenInfo, ) { - logger.info("Sjekker om brev med id=$id kan slettes") + try { + logger.info("Sjekker om brev med id=$id kan slettes") + val brev = sjekkOmBrevKanEndres(id) - val brev = sjekkOmBrevKanEndres(id) + sjekk(brev.behandlingId == null) { + "Brev med id=$id er et vedtaksbrev og kan ikke slettes" + } - sjekk(brev.behandlingId == null) { - "Brev med id=$id er et vedtaksbrev og kan ikke slettes" + val result = db.settBrevSlettet(id, bruker) + logger.info("Brev med id=$id slettet=$result") + } catch (e: UgyldigForespoerselException) { + throw e // hvis brevet er et vedtaksbrev skal vi kaste feil + } catch (e: BrevKanIkkeEndres) { + if (e.meta?.get("status") == Status.SLETTET) { + // skal egentlig ikke kunne slette noe som allerede er slettet + logger.warn("Brev ble forsøkt slettet, men brevid=$id har allerede status=SLETTET.") + } else { + throw e + } } - - val result = db.settBrevSlettet(id, bruker) - logger.info("Brev med id=$id slettet=$result") } fun markerSomUtgaatt( @@ -517,6 +527,7 @@ class BrevKanIkkeEndres( mapOf( "brevId" to brev.id, "status" to brev.status, + "behandlingId" to brev.behandlingId.toString(), ), ) diff --git a/apps/etterlatte-brev-api/src/test/kotlin/no/nav/etterlatte/brev/BrevServiceTest.kt b/apps/etterlatte-brev-api/src/test/kotlin/no/nav/etterlatte/brev/BrevServiceTest.kt index 588c5a7ab5..6845157697 100644 --- a/apps/etterlatte-brev-api/src/test/kotlin/no/nav/etterlatte/brev/BrevServiceTest.kt +++ b/apps/etterlatte-brev-api/src/test/kotlin/no/nav/etterlatte/brev/BrevServiceTest.kt @@ -486,9 +486,10 @@ internal class BrevServiceTest { } } + // Hvis noe har status SLETTET, så trenger vi ikke kaste exception @ParameterizedTest - @EnumSource(Status::class, names = ["OPPRETTET", "OPPDATERT"], mode = EnumSource.Mode.EXCLUDE) - fun `Brev som ikke lenger er under arbeid skal IKKE kunne slettes`(status: Status) { + @EnumSource(Status::class, names = ["OPPRETTET", "OPPDATERT", "SLETTET"], mode = EnumSource.Mode.EXCLUDE) + fun `Brev som ikke lenger kan endres skal IKKE kunne slettes`(status: Status) { val brev = opprettBrev(status, BrevProsessType.MANUELL) every { db.hentBrev(any()) } returns brev From 9a84b4ab0c999c089b26f3998a5817ecc5f3810a Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Sebastian=20Oliver=20S=C3=B8berg?= <82504565+sebassonav@users.noreply.github.com> Date: Wed, 26 Feb 2025 13:29:18 +0100 Subject: [PATCH 6/7] Merge grunnlagklient i behandlingklient i brev og fjerne config for grunnalg (#7130) --- .../no/nav/etterlatte/ApplicationContext.kt | 6 +- .../behandling/BehandlingKlient.kt | 112 +++++++++++++++ .../grunnlag/GrunnlagKlient.kt | 136 ------------------ .../grunnlag/GrunnlagService.kt | 3 +- .../brev/varselbrev/VarselbrevService.kt | 4 +- .../src/main/resources/application.conf | 3 - .../brev/varselbrev/VarselbrevTest.kt | 8 +- 7 files changed, 122 insertions(+), 150 deletions(-) delete mode 100644 apps/etterlatte-brev-api/src/main/kotlin/no/nav/etterlatte/brev/hentinformasjon/grunnlag/GrunnlagKlient.kt diff --git a/apps/etterlatte-brev-api/src/main/kotlin/no/nav/etterlatte/ApplicationContext.kt b/apps/etterlatte-brev-api/src/main/kotlin/no/nav/etterlatte/ApplicationContext.kt index 5d8553110f..1c6dee404b 100644 --- a/apps/etterlatte-brev-api/src/main/kotlin/no/nav/etterlatte/ApplicationContext.kt +++ b/apps/etterlatte-brev-api/src/main/kotlin/no/nav/etterlatte/ApplicationContext.kt @@ -44,7 +44,6 @@ import no.nav.etterlatte.brev.hentinformasjon.BrevdataFacade import no.nav.etterlatte.brev.hentinformasjon.behandling.BehandlingService import no.nav.etterlatte.brev.hentinformasjon.beregning.BeregningKlient import no.nav.etterlatte.brev.hentinformasjon.beregning.BeregningService -import no.nav.etterlatte.brev.hentinformasjon.grunnlag.GrunnlagKlient import no.nav.etterlatte.brev.hentinformasjon.grunnlag.GrunnlagService import no.nav.etterlatte.brev.hentinformasjon.trygdetid.TrygdetidKlient import no.nav.etterlatte.brev.hentinformasjon.trygdetid.TrygdetidService @@ -97,7 +96,6 @@ internal class ApplicationContext { val regoppslagKlient = RegoppslagKlient(httpClient(REGOPPSLAG_SCOPE), env.requireEnvValue(REGOPPSLAG_URL)) val saksbehandlerKlient = SaksbehandlerKlient(config, httpClient()) - val grunnlagKlient = GrunnlagKlient(config, httpClient()) val vedtakKlient = VedtaksvurderingKlient(config, httpClient()) val beregningKlient = BeregningKlient(config, httpClient()) val pdltjenesterKlient = PdlTjenesterKlient(config, httpClient()) @@ -114,7 +112,7 @@ internal class ApplicationContext { val norg2Klient = Norg2Klient(env.requireEnvValue(NORG2_URL), httpClient()) val adresseService = AdresseService(norg2Klient, saksbehandlerKlient, regoppslagKlient, pdltjenesterKlient) - val grunnlagService = GrunnlagService(grunnlagKlient, adresseService) + val grunnlagService = GrunnlagService(behandlingKlient, adresseService) val vedtaksvurderingService = VedtaksvurderingService(vedtakKlient) val vilkaarsvurderingService = VilkaarsvurderingService(vilkaarsvurderingKlient) @@ -198,7 +196,7 @@ internal class ApplicationContext { behandlingService, pdfGenerator, brevDataMapperFerdigstillVarsel, - grunnlagKlient, + behandlingKlient, ) val journalfoerBrevService = JournalfoerBrevService(db, behandlingService, dokarkivService, vedtaksbrevService) diff --git a/apps/etterlatte-brev-api/src/main/kotlin/no/nav/etterlatte/brev/hentinformasjon/behandling/BehandlingKlient.kt b/apps/etterlatte-brev-api/src/main/kotlin/no/nav/etterlatte/brev/hentinformasjon/behandling/BehandlingKlient.kt index 959ddbf7eb..7f459ea0c4 100644 --- a/apps/etterlatte-brev-api/src/main/kotlin/no/nav/etterlatte/brev/hentinformasjon/behandling/BehandlingKlient.kt +++ b/apps/etterlatte-brev-api/src/main/kotlin/no/nav/etterlatte/brev/hentinformasjon/behandling/BehandlingKlient.kt @@ -3,6 +3,7 @@ package no.nav.etterlatte.brev.behandlingklient import com.github.michaelbull.result.mapBoth import com.typesafe.config.Config import io.ktor.client.HttpClient +import io.ktor.client.plugins.ResponseException import no.nav.etterlatte.brev.model.EtterbetalingDTO import no.nav.etterlatte.libs.common.RetryResult import no.nav.etterlatte.libs.common.behandling.BrevutfallDto @@ -11,6 +12,9 @@ import no.nav.etterlatte.libs.common.behandling.Klage import no.nav.etterlatte.libs.common.behandling.SisteIverksatteBehandling import no.nav.etterlatte.libs.common.behandling.TidligereFamiliepleier import no.nav.etterlatte.libs.common.deserialize +import no.nav.etterlatte.libs.common.feilhaandtering.ForespoerselException +import no.nav.etterlatte.libs.common.grunnlag.Grunnlag +import no.nav.etterlatte.libs.common.grunnlag.OppdaterGrunnlagRequest import no.nav.etterlatte.libs.common.kodeverk.LandDto import no.nav.etterlatte.libs.common.retry import no.nav.etterlatte.libs.common.sak.Sak @@ -19,12 +23,15 @@ import no.nav.etterlatte.libs.ktor.ktor.ktorobo.AzureAdClient import no.nav.etterlatte.libs.ktor.ktor.ktorobo.DownstreamResourceClient import no.nav.etterlatte.libs.ktor.ktor.ktorobo.Resource import no.nav.etterlatte.libs.ktor.token.BrukerTokenInfo +import org.slf4j.LoggerFactory import java.util.UUID class BehandlingKlient( config: Config, httpClient: HttpClient, ) { + private val logger = LoggerFactory.getLogger(this::class.java) + private val azureAdClient = AzureAdClient(config) private val downstreamResourceClient = DownstreamResourceClient(azureAdClient, httpClient) @@ -154,6 +161,111 @@ class BehandlingKlient( errorMessage = { "Kunne ikke hente land fra kodeverk" }, brukerTokenInfo = brukerTokenInfo, ) + + internal suspend fun hentGrunnlagForSak( + sakid: SakId, + brukerTokenInfo: BrukerTokenInfo, + ): Grunnlag { + try { + logger.info("Henter grunnlag for sak med sakId=$sakid") + + return downstreamResourceClient + .get( + Resource(clientId, "$resourceUrl/api/grunnlag/sak/${sakid.sakId}"), + brukerTokenInfo, + ).mapBoth( + success = { deserialize(it.response.toString()) }, + failure = { throwableErrorMessage -> throw throwableErrorMessage }, + ) + } catch (e: ResponseException) { + logger.error("Henting av grunnlag for sak med sakId=$sakid feilet", e) + + throw ForespoerselException( + status = e.response.status.value, + code = "UKJENT_FEIL_HENT_GRUNNLAG", + detail = "Henting av grunnlag for sak feilet", + ) + } + } + + internal suspend fun hentGrunnlag( + behandlingId: UUID, + brukerTokenInfo: BrukerTokenInfo, + ): Grunnlag { + try { + logger.info("Henter grunnlag for behandling med id=$behandlingId") + + return downstreamResourceClient + .get( + Resource(clientId, "$resourceUrl/api/grunnlag/behandling/$behandlingId"), + brukerTokenInfo, + ).mapBoth( + success = { deserialize(it.response.toString()) }, + failure = { throwableErrorMessage -> throw throwableErrorMessage }, + ) + } catch (e: ResponseException) { + logger.error("Henting av grunnlag for behandling med id=$behandlingId feilet", e) + + throw ForespoerselException( + status = e.response.status.value, + code = "UKJENT_FEIL_HENT_GRUNNLAG", + detail = "Henting av grunnlag for behandling feilet", + ) + } + } + + internal suspend fun oppdaterGrunnlagForSak( + sak: Sak, + brukerTokenInfo: BrukerTokenInfo, + ): Boolean { + try { + logger.info("Oppdaterer grunnlag for sak med id=${sak.id}") + + return downstreamResourceClient + .post( + Resource(clientId, "$resourceUrl/api/grunnlag/sak/${sak.id}/oppdater-grunnlag"), + brukerTokenInfo, + OppdaterGrunnlagRequest(sak.id, sak.sakType), + ).mapBoth( + success = { true }, + failure = { throwableErrorMessage -> throw throwableErrorMessage }, + ) + } catch (e: ResponseException) { + logger.error("Oppdatering av grunnlag for sak med id=${sak.id} feilet", e) + + throw ForespoerselException( + status = e.response.status.value, + code = "UKJENT_FEIL_OPPDATER_GRUNNLAG", + detail = "Oppdatering av grunnlag for sak feilet id: ${sak.id}", + ) + } + } + + internal suspend fun finnesGrunnlagForSak( + sakId: SakId, + bruker: BrukerTokenInfo, + ): Boolean { + try { + logger.info("Oppdaterer grunnlag for sak med id=$sakId") + + return downstreamResourceClient + .get( + Resource(clientId, "$resourceUrl/api/grunnlag/sak/$sakId/grunnlag-finnes"), + bruker, + ).mapBoth( + success = { deserialize(it.response!!.toString()) }, + failure = { throw it }, + ) + } catch (e: ResponseException) { + logger.error("Sjekk på om grunnlag finnes feilet (sakId=$sakId)", e) + + throw ForespoerselException( + status = e.response.status.value, + code = "UKJENT_FEIL_FINNES_GRUNNLAG", + detail = "Sjekk på om grunnlag finnes feilet", + ) + } + } } class BehandlingKlientException( diff --git a/apps/etterlatte-brev-api/src/main/kotlin/no/nav/etterlatte/brev/hentinformasjon/grunnlag/GrunnlagKlient.kt b/apps/etterlatte-brev-api/src/main/kotlin/no/nav/etterlatte/brev/hentinformasjon/grunnlag/GrunnlagKlient.kt deleted file mode 100644 index 30ae55021d..0000000000 --- a/apps/etterlatte-brev-api/src/main/kotlin/no/nav/etterlatte/brev/hentinformasjon/grunnlag/GrunnlagKlient.kt +++ /dev/null @@ -1,136 +0,0 @@ -package no.nav.etterlatte.brev.hentinformasjon.grunnlag - -import com.github.michaelbull.result.mapBoth -import com.typesafe.config.Config -import io.ktor.client.HttpClient -import io.ktor.client.plugins.ResponseException -import no.nav.etterlatte.libs.common.deserialize -import no.nav.etterlatte.libs.common.feilhaandtering.ForespoerselException -import no.nav.etterlatte.libs.common.grunnlag.Grunnlag -import no.nav.etterlatte.libs.common.grunnlag.OppdaterGrunnlagRequest -import no.nav.etterlatte.libs.common.sak.Sak -import no.nav.etterlatte.libs.common.sak.SakId -import no.nav.etterlatte.libs.ktor.ktor.ktorobo.AzureAdClient -import no.nav.etterlatte.libs.ktor.ktor.ktorobo.DownstreamResourceClient -import no.nav.etterlatte.libs.ktor.ktor.ktorobo.Resource -import no.nav.etterlatte.libs.ktor.token.BrukerTokenInfo -import org.slf4j.LoggerFactory -import java.util.UUID - -class GrunnlagKlient( - config: Config, - httpClient: HttpClient, -) { - private val logger = LoggerFactory.getLogger(GrunnlagKlient::class.java) - - private val azureAdClient = AzureAdClient(config) - private val downstreamResourceClient = DownstreamResourceClient(azureAdClient, httpClient) - - private val clientId = config.getString("behandling.client.id") - private val baseUrl = config.getString("behandling.resource.url") - - internal suspend fun hentGrunnlagForSak( - sakid: SakId, - brukerTokenInfo: BrukerTokenInfo, - ): Grunnlag { - try { - logger.info("Henter grunnlag for sak med sakId=$sakid") - - return downstreamResourceClient - .get( - Resource(clientId, "$baseUrl/api/grunnlag/sak/${sakid.sakId}"), - brukerTokenInfo, - ).mapBoth( - success = { deserialize(it.response.toString()) }, - failure = { throwableErrorMessage -> throw throwableErrorMessage }, - ) - } catch (e: ResponseException) { - logger.error("Henting av grunnlag for sak med sakId=$sakid feilet", e) - - throw ForespoerselException( - status = e.response.status.value, - code = "UKJENT_FEIL_HENT_GRUNNLAG", - detail = "Henting av grunnlag for sak feilet", - ) - } - } - - internal suspend fun hentGrunnlag( - behandlingId: UUID, - brukerTokenInfo: BrukerTokenInfo, - ): Grunnlag { - try { - logger.info("Henter grunnlag for behandling med id=$behandlingId") - - return downstreamResourceClient - .get( - Resource(clientId, "$baseUrl/api/grunnlag/behandling/$behandlingId"), - brukerTokenInfo, - ).mapBoth( - success = { deserialize(it.response.toString()) }, - failure = { throwableErrorMessage -> throw throwableErrorMessage }, - ) - } catch (e: ResponseException) { - logger.error("Henting av grunnlag for behandling med id=$behandlingId feilet", e) - - throw ForespoerselException( - status = e.response.status.value, - code = "UKJENT_FEIL_HENT_GRUNNLAG", - detail = "Henting av grunnlag for behandling feilet", - ) - } - } - - internal suspend fun oppdaterGrunnlagForSak( - sak: Sak, - brukerTokenInfo: BrukerTokenInfo, - ): Boolean { - try { - logger.info("Oppdaterer grunnlag for sak med id=${sak.id}") - - return downstreamResourceClient - .post( - Resource(clientId, "$baseUrl/api/grunnlag/sak/${sak.id}/oppdater-grunnlag"), - brukerTokenInfo, - OppdaterGrunnlagRequest(sak.id, sak.sakType), - ).mapBoth( - success = { true }, - failure = { throwableErrorMessage -> throw throwableErrorMessage }, - ) - } catch (e: ResponseException) { - logger.error("Oppdatering av grunnlag for sak med id=${sak.id} feilet", e) - - throw ForespoerselException( - status = e.response.status.value, - code = "UKJENT_FEIL_OPPDATER_GRUNNLAG", - detail = "Oppdatering av grunnlag for sak feilet id: ${sak.id}", - ) - } - } - - internal suspend fun finnesGrunnlagForSak( - sakId: SakId, - bruker: BrukerTokenInfo, - ): Boolean { - try { - logger.info("Oppdaterer grunnlag for sak med id=$sakId") - - return downstreamResourceClient - .get( - Resource(clientId, "$baseUrl/api/grunnlag/sak/$sakId/grunnlag-finnes"), - bruker, - ).mapBoth( - success = { deserialize(it.response!!.toString()) }, - failure = { throw it }, - ) - } catch (e: ResponseException) { - logger.error("Sjekk på om grunnlag finnes feilet (sakId=$sakId)", e) - - throw ForespoerselException( - status = e.response.status.value, - code = "UKJENT_FEIL_FINNES_GRUNNLAG", - detail = "Sjekk på om grunnlag finnes feilet", - ) - } - } -} diff --git a/apps/etterlatte-brev-api/src/main/kotlin/no/nav/etterlatte/brev/hentinformasjon/grunnlag/GrunnlagService.kt b/apps/etterlatte-brev-api/src/main/kotlin/no/nav/etterlatte/brev/hentinformasjon/grunnlag/GrunnlagService.kt index 4563078744..767d29ba56 100644 --- a/apps/etterlatte-brev-api/src/main/kotlin/no/nav/etterlatte/brev/hentinformasjon/grunnlag/GrunnlagService.kt +++ b/apps/etterlatte-brev-api/src/main/kotlin/no/nav/etterlatte/brev/hentinformasjon/grunnlag/GrunnlagService.kt @@ -3,6 +3,7 @@ package no.nav.etterlatte.brev.hentinformasjon.grunnlag import no.nav.etterlatte.brev.adresse.AdresseService import no.nav.etterlatte.brev.behandling.erOver18 import no.nav.etterlatte.brev.behandling.hentForelderVerge +import no.nav.etterlatte.brev.behandlingklient.BehandlingKlient import no.nav.etterlatte.libs.common.behandling.BrevutfallDto import no.nav.etterlatte.libs.common.behandling.SakType import no.nav.etterlatte.libs.common.feilhaandtering.InternfeilException @@ -22,7 +23,7 @@ import no.nav.etterlatte.sikkerLogg import java.util.UUID class GrunnlagService( - private val klient: GrunnlagKlient, + private val klient: BehandlingKlient, private val adresseService: AdresseService, ) { suspend fun hentGrunnlag( diff --git a/apps/etterlatte-brev-api/src/main/kotlin/no/nav/etterlatte/brev/varselbrev/VarselbrevService.kt b/apps/etterlatte-brev-api/src/main/kotlin/no/nav/etterlatte/brev/varselbrev/VarselbrevService.kt index 25b13b099c..8d526441a1 100644 --- a/apps/etterlatte-brev-api/src/main/kotlin/no/nav/etterlatte/brev/varselbrev/VarselbrevService.kt +++ b/apps/etterlatte-brev-api/src/main/kotlin/no/nav/etterlatte/brev/varselbrev/VarselbrevService.kt @@ -7,9 +7,9 @@ import no.nav.etterlatte.brev.Brevoppretter import no.nav.etterlatte.brev.Brevtype import no.nav.etterlatte.brev.behandling.ForenkletVedtak import no.nav.etterlatte.brev.behandling.opprettAvsenderRequest +import no.nav.etterlatte.brev.behandlingklient.BehandlingKlient import no.nav.etterlatte.brev.db.BrevRepository import no.nav.etterlatte.brev.hentinformasjon.behandling.BehandlingService -import no.nav.etterlatte.brev.hentinformasjon.grunnlag.GrunnlagKlient import no.nav.etterlatte.brev.model.Brev import no.nav.etterlatte.brev.model.BrevID import no.nav.etterlatte.brev.pdf.PDFGenerator @@ -26,7 +26,7 @@ internal class VarselbrevService( private val behandlingService: BehandlingService, private val pdfGenerator: PDFGenerator, private val brevDataMapperFerdigstillVarsel: BrevDataMapperFerdigstillVarsel, - private val grunnlagKlient: GrunnlagKlient, + private val grunnlagKlient: BehandlingKlient, ) { fun hentVarselbrev(behandlingId: UUID) = db.hentBrevForBehandling(behandlingId, Brevtype.VARSEL) diff --git a/apps/etterlatte-brev-api/src/main/resources/application.conf b/apps/etterlatte-brev-api/src/main/resources/application.conf index 32c75f0664..bbda245b8d 100644 --- a/apps/etterlatte-brev-api/src/main/resources/application.conf +++ b/apps/etterlatte-brev-api/src/main/resources/application.conf @@ -24,9 +24,6 @@ dokdist.resource.url=${?DOKDIST_URL} dokdistkanal.client.id=${?DOKDISTKANAL_CLIENT_ID} dokdistkanal.resource.url=${?DOKDISTKANAL_URL} -grunnlag.client.id = ${?ETTERLATTE_GRUNNLAG_CLIENT_ID} -grunnlag.resource.url = ${?ETTERLATTE_GRUNNLAG_URL} - beregning.client.id = ${?ETTERLATTE_BEREGNING_CLIENT_ID} beregning.resource.url = ${?ETTERLATTE_BEREGNING_URL} diff --git a/apps/etterlatte-brev-api/src/test/kotlin/no/nav/etterlatte/brev/varselbrev/VarselbrevTest.kt b/apps/etterlatte-brev-api/src/test/kotlin/no/nav/etterlatte/brev/varselbrev/VarselbrevTest.kt index 6b58f7ce72..42b2503cea 100644 --- a/apps/etterlatte-brev-api/src/test/kotlin/no/nav/etterlatte/brev/varselbrev/VarselbrevTest.kt +++ b/apps/etterlatte-brev-api/src/test/kotlin/no/nav/etterlatte/brev/varselbrev/VarselbrevTest.kt @@ -15,11 +15,11 @@ import no.nav.etterlatte.brev.adresse.Avsender import no.nav.etterlatte.brev.behandling.GenerellBrevData import no.nav.etterlatte.brev.behandling.PersonerISak import no.nav.etterlatte.brev.behandling.Soeker +import no.nav.etterlatte.brev.behandlingklient.BehandlingKlient import no.nav.etterlatte.brev.brevbaker.BrevbakerService import no.nav.etterlatte.brev.db.BrevRepository import no.nav.etterlatte.brev.hentinformasjon.BrevdataFacade import no.nav.etterlatte.brev.hentinformasjon.behandling.BehandlingService -import no.nav.etterlatte.brev.hentinformasjon.grunnlag.GrunnlagKlient import no.nav.etterlatte.brev.model.Adresse import no.nav.etterlatte.brev.model.Brev import no.nav.etterlatte.brev.model.Mottaker @@ -154,8 +154,8 @@ class VarselbrevTest( } } - val grunnlagKlient = - mockk().also { + val behandlingklient = + mockk().also { coEvery { it.hentGrunnlag(any(), any()) } returns mockk() } @@ -169,7 +169,7 @@ class VarselbrevTest( innholdTilRedigerbartBrevHenter, ) val pdfGenerator = mockk() - service = VarselbrevService(brevRepository, brevoppretter, behandlingService, pdfGenerator, mockk(), grunnlagKlient) + service = VarselbrevService(brevRepository, brevoppretter, behandlingService, pdfGenerator, mockk(), behandlingklient) } @Test From 0210aace214375d3f3c5d1caa8f0efa9583e61db Mon Sep 17 00:00:00 2001 From: Henrik Gundersen Date: Wed, 26 Feb 2025 13:31:31 +0100 Subject: [PATCH 7/7] =?UTF-8?q?FAGSYSTEM-371156:=20Legger=20p=C3=A5=20simu?= =?UTF-8?q?lering=20ved=20opph=C3=B8r=20(#7137)?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- .../src/components/behandling/beregne/BeregneBP.tsx | 1 + .../src/components/behandling/beregne/BeregneOMS.tsx | 1 + .../components/behandling/beregne/SimulerUtbetaling.tsx | 9 ++++++++- 3 files changed, 10 insertions(+), 1 deletion(-) diff --git a/apps/etterlatte-saksbehandling-ui/client/src/components/behandling/beregne/BeregneBP.tsx b/apps/etterlatte-saksbehandling-ui/client/src/components/behandling/beregne/BeregneBP.tsx index 3ab9090110..63daf15332 100644 --- a/apps/etterlatte-saksbehandling-ui/client/src/components/behandling/beregne/BeregneBP.tsx +++ b/apps/etterlatte-saksbehandling-ui/client/src/components/behandling/beregne/BeregneBP.tsx @@ -82,6 +82,7 @@ export const BeregneBP = (props: { behandling: IBehandlingReducer }) => { <> {erOpphoer ? ( + setManglerbrevutfall(false)} /> ) : ( diff --git a/apps/etterlatte-saksbehandling-ui/client/src/components/behandling/beregne/BeregneOMS.tsx b/apps/etterlatte-saksbehandling-ui/client/src/components/behandling/beregne/BeregneOMS.tsx index cc5b0489e8..dfa89f04fc 100644 --- a/apps/etterlatte-saksbehandling-ui/client/src/components/behandling/beregne/BeregneOMS.tsx +++ b/apps/etterlatte-saksbehandling-ui/client/src/components/behandling/beregne/BeregneOMS.tsx @@ -114,6 +114,7 @@ export const BeregneOMS = () => { <> {erOpphoer ? ( + setManglerbrevutfall(false)} /> ) : ( diff --git a/apps/etterlatte-saksbehandling-ui/client/src/components/behandling/beregne/SimulerUtbetaling.tsx b/apps/etterlatte-saksbehandling-ui/client/src/components/behandling/beregne/SimulerUtbetaling.tsx index b721c146e5..374b3ee3a6 100644 --- a/apps/etterlatte-saksbehandling-ui/client/src/components/behandling/beregne/SimulerUtbetaling.tsx +++ b/apps/etterlatte-saksbehandling-ui/client/src/components/behandling/beregne/SimulerUtbetaling.tsx @@ -15,6 +15,7 @@ import { erFerdigBehandlet } from '~components/behandling/felles/utils' import { useAppSelector } from '~store/Store' import { compareDesc } from 'date-fns' import { SakType } from '~shared/types/sak' +import { VilkaarsvurderingResultat } from '~shared/api/vilkaarsvurdering' export const SimulerUtbetaling = (props: { behandling: IBehandlingReducer }) => { const { behandling } = props @@ -25,6 +26,8 @@ export const SimulerUtbetaling = (props: { behandling: IBehandlingReducer }) => // For OMS, lytte etter oppdatert beregning/avkorting const avkorting = useAppSelector((state) => state.behandlingReducer.behandling?.avkorting) + const erOpphoer = behandling.vilkaarsvurdering?.resultat?.utfall == VilkaarsvurderingResultat.IKKE_OPPFYLT + function behandlingStatusFerdigEllerVedtakFattet() { return erFerdigBehandlet(behandling.status) || behandling.status === IBehandlingStatus.FATTET_VEDTAK } @@ -42,7 +45,11 @@ export const SimulerUtbetaling = (props: { behandling: IBehandlingReducer }) => }, [behandling.status, avkorting]) const simuler = () => { - if (behandling.status === IBehandlingStatus.BEREGNET || behandling.status === IBehandlingStatus.AVKORTET) { + if ( + behandling.status === IBehandlingStatus.BEREGNET || + behandling.status === IBehandlingStatus.AVKORTET || + erOpphoer + ) { simulerUtbetalingRequest(behandling.id) } }