From 8e51c9acf7c98dc57d5283c176d03c830f8c1abd Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?=C3=98yvind=20Haarberg?= Date: Thu, 12 Dec 2024 09:32:22 +0100 Subject: [PATCH] Fikser tester etter at lagreStatus(id, status, tidspunkt) er private --- .../kotlin/behandling/BehandlingDaoTest.kt | 2 +- .../behandling/BehandlingFactoryTest.kt | 8 ++-- .../behandling/BehandlingServiceImplTest.kt | 2 +- .../BehandlingStatusServiceImplTest.kt | 2 +- .../behandling/BehandlingStatusServiceTest.kt | 16 ++++---- .../ManuellRevurderingServiceTest.kt | 40 ++++++++----------- .../OmgjoeringKlageRevurderingServiceTest.kt | 9 +---- 7 files changed, 34 insertions(+), 45 deletions(-) diff --git a/apps/etterlatte-behandling/src/test/kotlin/behandling/BehandlingDaoTest.kt b/apps/etterlatte-behandling/src/test/kotlin/behandling/BehandlingDaoTest.kt index 5208db9a1e1..d392d1e216b 100644 --- a/apps/etterlatte-behandling/src/test/kotlin/behandling/BehandlingDaoTest.kt +++ b/apps/etterlatte-behandling/src/test/kotlin/behandling/BehandlingDaoTest.kt @@ -608,7 +608,7 @@ internal class BehandlingDaoTest( behandlingRepo.lagreNyttVirkningstidspunkt(it.id, it.virkningstidspunkt!!) behandlingRepo.lagreGyldighetsproeving(it.id, it.gyldighetsproeving()) kommerBarnetTilGodeDao.lagreKommerBarnetTilGode(it.kommerBarnetTilgode!!) - behandlingRepo.lagreStatus(it.id, it.status, it.sistEndret) + behandlingRepo.lagreStatus(it) } with(behandlingRepo.hentBehandling(foerstegangsbehandling.id) as Foerstegangsbehandling) { diff --git a/apps/etterlatte-behandling/src/test/kotlin/behandling/BehandlingFactoryTest.kt b/apps/etterlatte-behandling/src/test/kotlin/behandling/BehandlingFactoryTest.kt index 67bc346a465..e46732fb98d 100644 --- a/apps/etterlatte-behandling/src/test/kotlin/behandling/BehandlingFactoryTest.kt +++ b/apps/etterlatte-behandling/src/test/kotlin/behandling/BehandlingFactoryTest.kt @@ -204,7 +204,7 @@ internal class BehandlingFactoryTest { every { hendelseDaoMock.behandlingOpprettet(any()) } returns Unit every { behandlingDaoMock.lagreGyldighetsproeving(any(), any()) } returns Unit every { behandlingDaoMock.hentBehandlingerForSak(any()) } returns emptyList() - every { behandlingDaoMock.lagreStatus(any(), any(), any()) } returns Unit + every { behandlingDaoMock.lagreStatus(any()) } returns Unit every { behandlingHendelserKafkaProducerMock.sendMeldingForHendelseStatisitkk( any(), @@ -392,7 +392,7 @@ internal class BehandlingFactoryTest { every { behandlingDaoMock.hentBehandlingerForSak(any()) } returns emptyList() - every { behandlingDaoMock.lagreStatus(any(), any(), any()) } returns Unit + every { behandlingDaoMock.lagreStatus(any()) } returns Unit every { hendelseDaoMock.behandlingOpprettet(any()) } returns Unit every { behandlingHendelserKafkaProducerMock.sendMeldingForHendelseStatisitkk( @@ -765,7 +765,7 @@ internal class BehandlingFactoryTest { every { behandlingDaoMock.hentBehandlingerForSak(any()) } returns emptyList() - every { behandlingDaoMock.lagreStatus(any(), any(), any()) } returns Unit + every { behandlingDaoMock.lagreStatus(any()) } returns Unit every { hendelseDaoMock.behandlingOpprettet(any()) } returns Unit every { behandlingHendelserKafkaProducerMock.sendMeldingForHendelseStatisitkk( @@ -935,7 +935,7 @@ internal class BehandlingFactoryTest { every { behandlingDaoMock.hentBehandlingerForSak(any()) } returns emptyList() - every { behandlingDaoMock.lagreStatus(any(), any(), any()) } returns Unit + every { behandlingDaoMock.lagreStatus(any()) } returns Unit every { hendelseDaoMock.behandlingOpprettet(any()) } returns Unit every { behandlingHendelserKafkaProducerMock.sendMeldingForHendelseStatisitkk( diff --git a/apps/etterlatte-behandling/src/test/kotlin/behandling/BehandlingServiceImplTest.kt b/apps/etterlatte-behandling/src/test/kotlin/behandling/BehandlingServiceImplTest.kt index 642dba4e48f..30d05feda02 100644 --- a/apps/etterlatte-behandling/src/test/kotlin/behandling/BehandlingServiceImplTest.kt +++ b/apps/etterlatte-behandling/src/test/kotlin/behandling/BehandlingServiceImplTest.kt @@ -1009,7 +1009,7 @@ internal class BehandlingServiceImplTest { coEvery { grunnlagKlientMock.hentPersongalleri(behandling.id, TOKEN) } returns mockPersongalleri() coEvery { grunnlagServiceMock.leggTilNyeOpplysninger(behandling.id, any(), any()) } just runs coEvery { grunnlagServiceMock.oppdaterGrunnlag(behandling.id, behandling.sak.id, any()) } just runs - every { behandlingDaoMock.lagreStatus(any(), any(), any()) } just runs + every { behandlingDaoMock.lagreStatus(any()) } just runs every { behandlingDaoMock.lagreStatus(any()) } just runs val annenForelderInRequest = diff --git a/apps/etterlatte-behandling/src/test/kotlin/behandling/BehandlingStatusServiceImplTest.kt b/apps/etterlatte-behandling/src/test/kotlin/behandling/BehandlingStatusServiceImplTest.kt index 1ca953b56fb..eae076fa420 100644 --- a/apps/etterlatte-behandling/src/test/kotlin/behandling/BehandlingStatusServiceImplTest.kt +++ b/apps/etterlatte-behandling/src/test/kotlin/behandling/BehandlingStatusServiceImplTest.kt @@ -75,7 +75,7 @@ class BehandlingStatusServiceImplTest { vedtakType = VedtakType.ENDRING, ) - every { behandlingDao.lagreStatus(any(), any(), any()) } just runs + every { behandlingDao.lagreStatus(any()) } just runs every { behandlingService.registrerVedtakHendelse(any(), any(), any()) } just runs every { oppgaveService.tilUnderkjent(any(), any(), any()) } returns mockk() every { behandlingService.endreProsesstype(any(), any()) } just runs diff --git a/apps/etterlatte-behandling/src/test/kotlin/behandling/BehandlingStatusServiceTest.kt b/apps/etterlatte-behandling/src/test/kotlin/behandling/BehandlingStatusServiceTest.kt index 251c9828f31..1406c663cc4 100644 --- a/apps/etterlatte-behandling/src/test/kotlin/behandling/BehandlingStatusServiceTest.kt +++ b/apps/etterlatte-behandling/src/test/kotlin/behandling/BehandlingStatusServiceTest.kt @@ -141,7 +141,7 @@ internal class BehandlingStatusServiceTest { } verify { - behandlingdao.lagreStatus(behandlingId, BehandlingStatus.FATTET_VEDTAK, any()) + behandlingdao.lagreStatus(any()) behandlingService.registrerVedtakHendelse(behandlingId, vedtakHendelse, HendelseType.FATTET) oppgaveService.tilAttestering( behandlingId.toString(), @@ -166,7 +166,7 @@ internal class BehandlingStatusServiceTest { } verify { - behandlingdao.lagreStatus(behandlingId, BehandlingStatus.IVERKSATT, any()) + behandlingdao.lagreStatus(any()) behandlingService.hentBehandling(behandlingId) behandlingService.registrerVedtakHendelse(behandlingId, iverksettVedtak, HendelseType.IVERKSATT) behandlingInfoDao.hentBrevutfall(behandlingId) @@ -190,7 +190,7 @@ internal class BehandlingStatusServiceTest { } verify { - behandlingdao.lagreStatus(behandlingId, BehandlingStatus.RETURNERT, any()) + behandlingdao.lagreStatus(any()) behandlingService.registrerVedtakHendelse(behandlingId, vedtakHendelse, HendelseType.UNDERKJENT) oppgaveService.tilUnderkjent(behandlingId.toString(), OppgaveType.FOERSTEGANGSBEHANDLING, any()) } @@ -231,7 +231,7 @@ internal class BehandlingStatusServiceTest { } verify { - behandlingdao.lagreStatus(behandlingId, BehandlingStatus.IVERKSATT, any()) + behandlingdao.lagreStatus(any()) behandlingService.hentBehandling(behandlingId) behandlingService.registrerVedtakHendelse(behandlingId, iverksattVedtak, HendelseType.IVERKSATT) generellBehandlingService.opprettBehandling(any(), any()) @@ -276,7 +276,7 @@ internal class BehandlingStatusServiceTest { } verify { - behandlingdao.lagreStatus(behandlingId, BehandlingStatus.AVSLAG, any()) + behandlingdao.lagreStatus(any()) behandlingService.registrerVedtakHendelse(behandlingId, vedtakHendelse, HendelseType.ATTESTERT) generellBehandlingService.opprettBehandling(any(), any()) oppgaveService.ferdigStillOppgaveUnderBehandling( @@ -318,7 +318,7 @@ internal class BehandlingStatusServiceTest { } verify { - behandlingdao.lagreStatus(behandlingId, BehandlingStatus.ATTESTERT, any()) + behandlingdao.lagreStatus(any()) behandlingService.registrerVedtakHendelse(behandlingId, vedtakHendelse, HendelseType.ATTESTERT) oppgaveService.ferdigStillOppgaveUnderBehandling( behandlingId.toString(), @@ -358,7 +358,7 @@ internal class BehandlingStatusServiceTest { } verify { - behandlingdao.lagreStatus(behandlingId, BehandlingStatus.IVERKSATT, any()) + behandlingdao.lagreStatus(any()) behandlingService.hentBehandling(behandlingId) behandlingService.registrerVedtakHendelse(behandlingId, iverksettVedtak, HendelseType.IVERKSATT) behandlingInfoDao.hentBrevutfall(behandlingId) @@ -401,7 +401,7 @@ internal class BehandlingStatusServiceTest { } verify { - behandlingdao.lagreStatus(behandlingId, BehandlingStatus.IVERKSATT, any()) + behandlingdao.lagreStatus(any()) behandlingService.hentBehandling(behandlingId) behandlingService.registrerVedtakHendelse(behandlingId, iverksettVedtak, HendelseType.IVERKSATT) behandlingInfoDao.hentBrevutfall(behandlingId) 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 09e1238382e..1ea1d25478c 100644 --- a/apps/etterlatte-behandling/src/test/kotlin/behandling/revurdering/ManuellRevurderingServiceTest.kt +++ b/apps/etterlatte-behandling/src/test/kotlin/behandling/revurdering/ManuellRevurderingServiceTest.kt @@ -19,11 +19,13 @@ import no.nav.etterlatte.behandling.BehandlingsHendelserKafkaProducerImpl import no.nav.etterlatte.behandling.GrunnlagService import no.nav.etterlatte.behandling.aktivitetsplikt.AktivitetspliktDao import no.nav.etterlatte.behandling.aktivitetsplikt.AktivitetspliktKopierService +import no.nav.etterlatte.behandling.domain.AutomatiskRevurdering import no.nav.etterlatte.behandling.domain.Behandling import no.nav.etterlatte.behandling.domain.Foerstegangsbehandling import no.nav.etterlatte.behandling.domain.GrunnlagsendringStatus import no.nav.etterlatte.behandling.domain.GrunnlagsendringsType import no.nav.etterlatte.behandling.domain.Grunnlagsendringshendelse +import no.nav.etterlatte.behandling.domain.ManuellRevurdering import no.nav.etterlatte.behandling.domain.Revurdering import no.nav.etterlatte.behandling.domain.SamsvarMellomKildeOgGrunnlag import no.nav.etterlatte.behandling.domain.toStatistikkBehandling @@ -48,7 +50,6 @@ import no.nav.etterlatte.libs.common.oppgave.OppgaveType import no.nav.etterlatte.libs.common.oppgave.Status import no.nav.etterlatte.libs.common.sak.Sak import no.nav.etterlatte.libs.common.tidspunkt.Tidspunkt -import no.nav.etterlatte.libs.common.tidspunkt.toLocalDatetimeUTC import no.nav.etterlatte.nyKontekstMedBrukerOgDatabase import no.nav.etterlatte.oppgave.OppgaveService import no.nav.etterlatte.persongalleri @@ -243,11 +244,7 @@ class ManuellRevurderingServiceTest : BehandlingIntegrationTest() { } inTransaction { - applicationContext.behandlingDao.lagreStatus( - revurdering.id, - BehandlingStatus.IVERKSATT, - LocalDateTime.now(), - ) + iverksett(revurdering, "saksbehandler") } assertThrows { @@ -534,11 +531,7 @@ class ManuellRevurderingServiceTest : BehandlingIntegrationTest() { } inTransaction { - applicationContext.behandlingDao.lagreStatus( - behandling!!.id, - BehandlingStatus.IVERKSATT, - Tidspunkt.now().toLocalDatetimeUTC(), - ) + iverksett(behandling!!) ferdigstillOppgaver(behandling, saksbehandler) } val revurderingen = @@ -738,11 +731,7 @@ class ManuellRevurderingServiceTest : BehandlingIntegrationTest() { } inTransaction { - applicationContext.behandlingDao.lagreStatus( - nonNullBehandling.id, - BehandlingStatus.IVERKSATT, - Tidspunkt.now().toLocalDatetimeUTC(), - ) + applicationContext.behandlingDao.lagreStatus(nonNullBehandling.copy(status = BehandlingStatus.IVERKSATT)) ferdigstillOppgaver(behandling, saksbehandler) } @@ -908,13 +897,18 @@ class ManuellRevurderingServiceTest : BehandlingIntegrationTest() { } } - private fun iverksett(behandling: Behandling) { - applicationContext.behandlingDao.lagreStatus( - behandling.id, - BehandlingStatus.IVERKSATT, - Tidspunkt.now().toLocalDatetimeUTC(), - ) - ferdigstillOppgaver(behandling) + private fun iverksett( + behandling: Behandling, + saksbehandlerId: String = "sbh", + ) { + val iverksatt = + when (behandling) { + is Foerstegangsbehandling -> behandling.copy(status = BehandlingStatus.IVERKSATT) + is ManuellRevurdering -> behandling.copy(status = BehandlingStatus.IVERKSATT) + is AutomatiskRevurdering -> behandling.copy(status = BehandlingStatus.IVERKSATT) + } + applicationContext.behandlingDao.lagreStatus(iverksatt) + ferdigstillOppgaver(behandling, saksbehandlerId) } private fun ferdigstillOppgaver( diff --git a/apps/etterlatte-behandling/src/test/kotlin/behandling/revurdering/OmgjoeringKlageRevurderingServiceTest.kt b/apps/etterlatte-behandling/src/test/kotlin/behandling/revurdering/OmgjoeringKlageRevurderingServiceTest.kt index 399a3cc70a7..0853e1e6e5b 100644 --- a/apps/etterlatte-behandling/src/test/kotlin/behandling/revurdering/OmgjoeringKlageRevurderingServiceTest.kt +++ b/apps/etterlatte-behandling/src/test/kotlin/behandling/revurdering/OmgjoeringKlageRevurderingServiceTest.kt @@ -29,8 +29,6 @@ import no.nav.etterlatte.libs.common.behandling.VedtaketKlagenGjelder import no.nav.etterlatte.libs.common.oppgave.OppgaveType import no.nav.etterlatte.libs.common.oppgave.Status import no.nav.etterlatte.libs.common.sak.Sak -import no.nav.etterlatte.libs.common.tidspunkt.Tidspunkt -import no.nav.etterlatte.libs.common.tidspunkt.toLocalDatetimeUTC import no.nav.etterlatte.libs.common.vedtak.VedtakType import no.nav.etterlatte.libs.testdata.grunnlag.SOEKER_FOEDSELSNUMMER import no.nav.etterlatte.nyKontekstMedBrukerOgDatabase @@ -121,11 +119,8 @@ class OmgjoeringKlageRevurderingServiceTest : BehandlingIntegrationTest() { ) } inTransaction { - applicationContext.behandlingDao.lagreStatus( - behandling!!.id, - BehandlingStatus.IVERKSATT, - Tidspunkt.now().toLocalDatetimeUTC(), - ) + val iverksatt = behandling!!.copy(status = BehandlingStatus.IVERKSATT) + applicationContext.behandlingDao.lagreStatus(iverksatt) } val klage = inTransaction {