Legg inn aldersovergangservicetetss

* Og fiks standard oppsett for mocking av routes hvis man bruker runServer uten module i behandling
navikt · Nov 12, 2024 · 28aeabd · 28aeabd
1 parent 7675095
commit 28aeabd
Show file tree

Hide file tree

Showing 7 changed files with 107 additions and 95 deletions.
diff --git a/apps/etterlatte-behandling/src/main/kotlin/vilkaarsvurdering/AldersovergangRoutes.kt b/apps/etterlatte-behandling/src/main/kotlin/vilkaarsvurdering/AldersovergangRoutes.kt
@@ -6,6 +6,7 @@ import io.ktor.server.response.respond
 import io.ktor.server.routing.Route
 import io.ktor.server.routing.post
 import io.ktor.server.routing.route
+import no.nav.etterlatte.inTransaction
 import no.nav.etterlatte.libs.ktor.route.BEHANDLINGID_CALL_PARAMETER
 import no.nav.etterlatte.libs.ktor.route.behandlingId
 import no.nav.etterlatte.libs.ktor.route.routeLogger
@@ -24,11 +25,13 @@ fun Route.aldersovergang(aldersovergangService: AldersovergangService) {
 
             logger.info("Behandler aldersovergang [behandlingId=$behandlingId, løpende behandlingId=$loependeBehandlingId]")
             val vilkaarsvurdering =
-                aldersovergangService.behandleOpphoerAldersovergang(
-                    behandlingId = behandlingId,
-                    loependeBehandlingId = loependeBehandlingId,
-                    brukerTokenInfo = call.brukerTokenInfo,
-                )
+                inTransaction {
+                    aldersovergangService.behandleOpphoerAldersovergang(
+                        behandlingId = behandlingId,
+                        loependeBehandlingId = loependeBehandlingId,
+                        brukerTokenInfo = call.brukerTokenInfo,
+                    )
+                }
 
             call.respond(HttpStatusCode.OK, mapOf("vilkaarsvurderingid" to vilkaarsvurdering.id))
         }

diff --git a/.../etterlatte-behandling/src/main/kotlin/vilkaarsvurdering/service/AldersovergangService.kt b/.../etterlatte-behandling/src/main/kotlin/vilkaarsvurdering/service/AldersovergangService.kt
@@ -1,6 +1,5 @@
 package no.nav.etterlatte.vilkaarsvurdering.service
 
-import no.nav.etterlatte.inTransaction
 import no.nav.etterlatte.libs.common.vilkaarsvurdering.Utfall
 import no.nav.etterlatte.libs.common.vilkaarsvurdering.VilkaarType
 import no.nav.etterlatte.libs.common.vilkaarsvurdering.VilkaarTypeOgUtfall
@@ -22,59 +21,58 @@ class AldersovergangService(
         loependeBehandlingId: UUID,
         brukerTokenInfo: BrukerTokenInfo,
         tidspunkt: () -> LocalDateTime = { LocalDateTime.now() },
-    ): Vilkaarsvurdering =
-        inTransaction {
-            val vilkaarsvurdering =
-                vilkaarsvurderingService.hentVilkaarsvurdering(behandlingId)
-                    ?: vilkaarsvurderingService
-                        .kopierVilkaarsvurdering(
-                            behandlingId = behandlingId,
-                            kopierFraBehandling = loependeBehandlingId,
-                            brukerTokenInfo = brukerTokenInfo,
-                            kopierResultat = false,
-                        ).vilkaarsvurdering
+    ): Vilkaarsvurdering {
+        val vilkaarsvurdering =
+            vilkaarsvurderingService.hentVilkaarsvurdering(behandlingId)
+                ?: vilkaarsvurderingService
+                    .kopierVilkaarsvurdering(
+                        behandlingId = behandlingId,
+                        kopierFraBehandling = loependeBehandlingId,
+                        brukerTokenInfo = brukerTokenInfo,
+                        kopierResultat = false,
+                    ).vilkaarsvurdering
 
-            val aldersvilkaar =
-                vilkaarsvurdering.vilkaar.single {
-                    it.hovedvilkaar.type in
-                        listOf(
-                            VilkaarType.BP_ALDER_BARN_2024,
-                            VilkaarType.OMS_OVERLAPPENDE_YTELSER,
-                        )
-                }
+        val aldersvilkaar =
+            vilkaarsvurdering.vilkaar.single {
+                it.hovedvilkaar.type in
+                    listOf(
+                        VilkaarType.BP_ALDER_BARN_2024,
+                        VilkaarType.OMS_OVERLAPPENDE_YTELSER,
+                    )
+            }
 
-            // Aldersvilkår => ikke oppfylt
-            vilkaarsvurderingService.oppdaterVurderingPaaVilkaar(
-                behandlingId = behandlingId,
-                brukerTokenInfo = brukerTokenInfo,
-                vurdertVilkaar =
-                    VurdertVilkaar(
-                        vilkaarId = aldersvilkaar.id,
-                        hovedvilkaar =
-                            VilkaarTypeOgUtfall(
-                                type = aldersvilkaar.hovedvilkaar.type,
-                                resultat = Utfall.IKKE_OPPFYLT,
-                            ),
-                        vurdering =
-                            VilkaarVurderingData(
-                                kommentar = "Aldersgrensen er passert",
-                                tidspunkt = tidspunkt(),
-                                saksbehandler = Fagsaksystem.EY.navn,
-                            ),
-                    ),
-            )
+        // Aldersvilkår => ikke oppfylt
+        vilkaarsvurderingService.oppdaterVurderingPaaVilkaar(
+            behandlingId = behandlingId,
+            brukerTokenInfo = brukerTokenInfo,
+            vurdertVilkaar =
+                VurdertVilkaar(
+                    vilkaarId = aldersvilkaar.id,
+                    hovedvilkaar =
+                        VilkaarTypeOgUtfall(
+                            type = aldersvilkaar.hovedvilkaar.type,
+                            resultat = Utfall.IKKE_OPPFYLT,
+                        ),
+                    vurdering =
+                        VilkaarVurderingData(
+                            kommentar = "Aldersgrensen er passert",
+                            tidspunkt = tidspunkt(),
+                            saksbehandler = Fagsaksystem.EY.navn,
+                        ),
+                ),
+        )
 
-            // Resultat på hele vurderingen => ikke oppfylt
-            vilkaarsvurderingService
-                .oppdaterTotalVurdering(
-                    behandlingId,
-                    brukerTokenInfo,
-                    VilkaarsvurderingResultat(
-                        utfall = VilkaarsvurderingUtfall.IKKE_OPPFYLT,
-                        kommentar = "Automatisk aldersovergang",
-                        tidspunkt = tidspunkt(),
-                        saksbehandler = Fagsaksystem.EY.navn,
-                    ),
-                ).vilkaarsvurdering
-        }
+        // Resultat på hele vurderingen => ikke oppfylt
+        return vilkaarsvurderingService
+            .oppdaterTotalVurdering(
+                behandlingId,
+                brukerTokenInfo,
+                VilkaarsvurderingResultat(
+                    utfall = VilkaarsvurderingUtfall.IKKE_OPPFYLT,
+                    kommentar = "Automatisk aldersovergang",
+                    tidspunkt = tidspunkt(),
+                    saksbehandler = Fagsaksystem.EY.navn,
+                ),
+            ).vilkaarsvurdering
+    }
 }
diff --git a/apps/etterlatte-behandling/src/test/kotlin/TestHelper.kt b/apps/etterlatte-behandling/src/test/kotlin/TestHelper.kt
@@ -156,6 +156,25 @@ fun Route.attachMockContextWithDb(
     }
 }
 
+fun Route.attachMockContextWithDbAndSakdao(
+    testUser: User,
+    sakTilgangDao: SakTilgangDao,
+) {
+    intercept(ApplicationCallPipeline.Call) {
+        val context1 = lagContext(testUser, sakTilgangDao = sakTilgangDao)
+
+        withContext(
+            Dispatchers.Default +
+                Kontekst.asContextElement(
+                    value = context1,
+                ),
+        ) {
+            proceed()
+        }
+        Kontekst.remove()
+    }
+}
+
 fun Route.attachMockContext(testUser: User? = null) {
     intercept(ApplicationCallPipeline.Call) {
         val context1 = lagContext(testUser ?: user)

diff --git a/apps/etterlatte-behandling/src/test/kotlin/tilgangsstyring/TilgangsstyringTest.kt b/apps/etterlatte-behandling/src/test/kotlin/tilgangsstyring/TilgangsstyringTest.kt
@@ -6,26 +6,21 @@ import io.ktor.client.request.header
 import io.ktor.http.ContentType
 import io.ktor.http.HttpHeaders
 import io.ktor.http.contentType
-import io.ktor.server.application.ApplicationCallPipeline
 import io.ktor.server.application.call
 import io.ktor.server.response.respond
 import io.ktor.server.routing.get
 import io.ktor.server.routing.route
 import io.ktor.server.testing.testApplication
 import io.mockk.every
 import io.mockk.mockk
-import kotlinx.coroutines.Dispatchers
-import kotlinx.coroutines.asContextElement
-import kotlinx.coroutines.withContext
-import no.nav.etterlatte.Kontekst
 import no.nav.etterlatte.SaksbehandlerMedEnheterOgRoller
+import no.nav.etterlatte.attachMockContextWithDbAndSakdao
 import no.nav.etterlatte.behandling.sakId1
 import no.nav.etterlatte.behandling.sakId2
 import no.nav.etterlatte.common.Enheter
 import no.nav.etterlatte.ktor.runServer
 import no.nav.etterlatte.ktor.startRandomPort
 import no.nav.etterlatte.ktor.token.issueSaksbehandlerToken
-import no.nav.etterlatte.lagContext
 import no.nav.etterlatte.libs.common.sak.SakMedGraderingOgSkjermet
 import no.nav.etterlatte.libs.ktor.route.BEHANDLINGID_CALL_PARAMETER
 import no.nav.etterlatte.libs.ktor.route.KLAGEID_CALL_PARAMETER
@@ -276,20 +271,7 @@ class TilgangsstyringTest {
         testApplication {
             val client =
                 runServer(mockOAuth2Server) {
-                    intercept(ApplicationCallPipeline.Call) {
-                        val context =
-                            lagContext(user.also { every { it.name() } returns this::class.java.simpleName }, sakTilgangDao = sakTilgangDao)
-
-                        withContext(
-                            Dispatchers.Default +
-                                Kontekst.asContextElement(
-                                    value = context,
-                                ),
-                        ) {
-                            proceed()
-                        }
-                        Kontekst.remove()
-                    }
+                    attachMockContextWithDbAndSakdao(user, sakTilgangDao)
 
                     route("/api") {
                         route("/sak") {

diff --git a/apps/etterlatte-behandling/src/test/kotlin/vilkaarsvurdering/AldersovergangRoutesTest.kt b/apps/etterlatte-behandling/src/test/kotlin/vilkaarsvurdering/AldersovergangRoutesTest.kt
@@ -12,10 +12,16 @@ import io.mockk.coVerify
 import io.mockk.every
 import io.mockk.mockk
 import io.mockk.verify
+import no.nav.etterlatte.Context
+import no.nav.etterlatte.Kontekst
+import no.nav.etterlatte.attachMockContext
+import no.nav.etterlatte.common.DatabaseContext
 import no.nav.etterlatte.ktor.runServer
 import no.nav.etterlatte.ktor.startRandomPort
 import no.nav.etterlatte.ktor.token.issueSystembrukerToken
 import no.nav.etterlatte.libs.common.vilkaarsvurdering.Vilkaarsvurdering
+import no.nav.etterlatte.mockSaksbehandler
+import no.nav.etterlatte.mockedSakTilgangDao
 import no.nav.etterlatte.vilkaarsvurdering.service.AldersovergangService
 import no.nav.security.mock.oauth2.MockOAuth2Server
 import org.junit.jupiter.api.AfterAll
@@ -35,6 +41,7 @@ internal class AldersovergangRoutesTest {
     @BeforeAll
     fun before() {
         mockOAuth2Server.startRandomPort()
+        Kontekst.set(Context(saksbehandler, DatabaseContext(mockk()), mockedSakTilgangDao(), null))
     }
 
     @AfterAll
@@ -43,11 +50,13 @@ internal class AldersovergangRoutesTest {
     }
 
     private val token: String by lazy { mockOAuth2Server.issueSystembrukerToken() }
+    private val saksbehandler = mockSaksbehandler("User")
 
     @Test
     fun `skal delegere til aldersovergangservice`() {
         testApplication {
             runServer(mockOAuth2Server) {
+                attachMockContext(saksbehandler)
                 aldersovergang(aldersovergangService)
             }
 

diff --git a/apps/etterlatte-behandling/src/test/kotlin/vilkaarsvurdering/AldersovergangServiceTest.kt b/apps/etterlatte-behandling/src/test/kotlin/vilkaarsvurdering/AldersovergangServiceTest.kt
@@ -1,26 +1,26 @@
-/*
 package no.nav.etterlatte.vilkaarsvurdering
 
 import io.kotest.matchers.shouldBe
 import io.mockk.coEvery
 import io.mockk.coVerify
 import io.mockk.mockk
-import kotlinx.coroutines.runBlocking
 import no.nav.etterlatte.libs.common.vilkaarsvurdering.Delvilkaar
 import no.nav.etterlatte.libs.common.vilkaarsvurdering.Lovreferanse
 import no.nav.etterlatte.libs.common.vilkaarsvurdering.Utfall
 import no.nav.etterlatte.libs.common.vilkaarsvurdering.Vilkaar
 import no.nav.etterlatte.libs.common.vilkaarsvurdering.VilkaarType
+import no.nav.etterlatte.libs.common.vilkaarsvurdering.VilkaarTypeOgUtfall
 import no.nav.etterlatte.libs.common.vilkaarsvurdering.VilkaarVurderingData
+import no.nav.etterlatte.libs.common.vilkaarsvurdering.Vilkaarsvurdering
+import no.nav.etterlatte.libs.common.vilkaarsvurdering.VilkaarsvurderingMedBehandlingGrunnlagsversjon
 import no.nav.etterlatte.libs.common.vilkaarsvurdering.VilkaarsvurderingResultat
 import no.nav.etterlatte.libs.common.vilkaarsvurdering.VilkaarsvurderingUtfall
+import no.nav.etterlatte.libs.common.vilkaarsvurdering.VurdertVilkaar
 import no.nav.etterlatte.libs.ktor.token.BrukerTokenInfo
 import no.nav.etterlatte.libs.ktor.token.Fagsaksystem
+import no.nav.etterlatte.vilkaarsvurdering.service.AldersovergangService
+import no.nav.etterlatte.vilkaarsvurdering.service.VilkaarsvurderingService
 import org.junit.jupiter.api.Test
-import vilkaarsvurdering.VilkaarTypeOgUtfall
-import vilkaarsvurdering.Vilkaarsvurdering
-import vilkaarsvurdering.VilkaarsvurderingMedBehandlingGrunnlagsversjon
-import vilkaarsvurdering.VurdertVilkaar
 import java.time.LocalDateTime
 import java.time.Month
 import java.time.YearMonth
@@ -107,17 +107,15 @@ class AldersovergangServiceTest {
             vilkaarsvurderingService.oppdaterTotalVurdering(behandlingId, brukerTokenInfo, vilkaarsvurderingResultat)
         } returns VilkaarsvurderingMedBehandlingGrunnlagsversjon(vilkaarsvurdering, 1L)
 
-        runBlocking {
-            val res =
-                aldersAldersovergangService.behandleOpphoerAldersovergang(
-                    behandlingId = behandlingId,
-                    loependeBehandlingId = loependeBehandlingId,
-                    brukerTokenInfo = brukerTokenInfo,
-                    tidspunkt = { tidspunkt },
-                )
+        val res =
+            aldersAldersovergangService.behandleOpphoerAldersovergang(
+                behandlingId = behandlingId,
+                loependeBehandlingId = loependeBehandlingId,
+                brukerTokenInfo = brukerTokenInfo,
+                tidspunkt = { tidspunkt },
+            )
 
-            res shouldBe vilkaarsvurdering
-        }
+        res shouldBe vilkaarsvurdering
 
         coVerify { vilkaarsvurderingService.hentVilkaarsvurdering(behandlingId) }
         coVerify {
@@ -144,4 +142,3 @@ class AldersovergangServiceTest {
         }
     }
 }
-*/
diff --git a/libs/etterlatte-ktor/src/testFixtures/kotlin/no/nav/etterlatte/ktor/RunServer.kt b/libs/etterlatte-ktor/src/testFixtures/kotlin/no/nav/etterlatte/ktor/RunServer.kt
@@ -30,6 +30,10 @@ fun ApplicationTestBuilder.runServerWithModule(
     }
 }
 
+/*
+Obs krever at du intercepter og legger på egen context hvis du ikke bruker Application.module i behandling som kaller settOppApplikasjonen()
+Se TestHelper.kt i behandling
+ */
 fun ApplicationTestBuilder.runServer(
     server: MockOAuth2Server,
     routePrefix: String = "",