Merge pull request #2805 from navikt/fix/journfore-samvarsavtale-fagsakId

Skal sende med fagsakId på journalføring for samværsberegning eller k…

Author: EndreHM

src/main/kotlin/no/nav/familie/ef/sak/samværsavtale/SamværsavtaleService.kt

@@ -10,6 +10,7 @@ import no.nav.familie.ef.sak.brev.BrevsignaturService
 import no.nav.familie.ef.sak.brev.dto.Avsnitt
 import no.nav.familie.ef.sak.brev.dto.FritekstBrevRequestDto
 import no.nav.familie.ef.sak.brev.dto.FritekstBrevRequestMedSignatur
+import no.nav.familie.ef.sak.fagsak.FagsakService
 import no.nav.familie.ef.sak.infrastruktur.exception.brukerfeilHvis
 import no.nav.familie.ef.sak.infrastruktur.sikkerhet.SikkerhetContext
 import no.nav.familie.ef.sak.journalføring.JournalpostClient
@@ -36,6 +37,7 @@ import java.util.UUID
 class SamværsavtaleService(
     val samværsavtaleRepository: SamværsavtaleRepository,
     val behandlingService: BehandlingService,
+    val fagsakService: FagsakService,
     val tilordnetRessursService: TilordnetRessursService,
     val barnService: BarnService,
     val personopplysningerService: PersonopplysningerService,
@@ -97,12 +99,14 @@ class SamværsavtaleService(
     }
 
     fun journalførBeregnetSamvær(request: JournalførBeregnetSamværRequest): String {
-        validerjournalføringRequest(request)
+        val eksternFagsakId = fagsakService.finnFagsaker(setOf(request.personIdent)).firstOrNull()?.eksternId
+        validerjournalføringRequest(request, eksternFagsakId)
+
         val fritekstBrevRequest = lagFritekstBrevRequestMedSignatur(request)
         val dokument = brevClient.genererFritekstBrev(fritekstBrevRequest)
 
         val saksbehandler = SikkerhetContext.hentSaksbehandler()
-        val arkiverDokumentRequest = lagArkiverDokumentRequest(dokument, request.personIdent)
+        val arkiverDokumentRequest = lagArkiverDokumentRequest(dokument, request.personIdent, eksternFagsakId)
         val respons = journalpostClient.arkiverDokument(arkiverDokumentRequest, saksbehandler)
 
         return respons.journalpostId
@@ -130,6 +134,7 @@ class SamværsavtaleService(
     private fun lagArkiverDokumentRequest(
         pdf: ByteArray,
         personIdent: String,
+        eksternFagsakId: Long?,
     ): ArkiverDokumentRequest {
         val dokument =
             Dokument(
@@ -146,6 +151,7 @@ class SamværsavtaleService(
             hoveddokumentvarianter = listOf(dokument),
             vedleggsdokumenter = listOf(),
             journalførendeEnhet = journalførendeEnhet,
+            fagsakId = eksternFagsakId.toString(),
         )
     }
 
@@ -185,12 +191,18 @@ class SamværsavtaleService(
         }
     }
 
-    private fun validerjournalføringRequest(request: JournalførBeregnetSamværRequest) {
+    private fun validerjournalføringRequest(
+        request: JournalførBeregnetSamværRequest,
+        eksternFagsakId: Long?,
+    ) {
         brukerfeilHvis(request.notat.isEmpty()) {
             "Kan ikke journalføre samværsavtale uten notat"
         }
         brukerfeilHvis(request.oppsummering.isEmpty()) {
             "Kan ikke journalføre samværsavtale uten oppsumering"
         }
+        brukerfeilHvis(eksternFagsakId == null) {
+            "Kan ikke journalføre samværsavtale på person uten fagsak"
+        }
     }
 }

src/test/kotlin/no/nav/familie/ef/sak/samværsavtale/SamværsavtaleServiceTest.kt

@@ -14,13 +14,15 @@ import no.nav.familie.ef.sak.behandling.domain.BehandlingStatus.UTREDES
 import no.nav.familie.ef.sak.brev.BrevClient
 import no.nav.familie.ef.sak.brev.BrevsignaturService
 import no.nav.familie.ef.sak.brev.dto.SignaturDto
+import no.nav.familie.ef.sak.fagsak.FagsakService
 import no.nav.familie.ef.sak.felles.util.BrukerContextUtil
 import no.nav.familie.ef.sak.infrastruktur.exception.ApiFeil
 import no.nav.familie.ef.sak.journalføring.JournalpostClient
 import no.nav.familie.ef.sak.oppgave.TilordnetRessursService
 import no.nav.familie.ef.sak.opplysninger.personopplysninger.PersonopplysningerService
 import no.nav.familie.ef.sak.repository.behandling
 import no.nav.familie.ef.sak.repository.behandlingBarn
+import no.nav.familie.ef.sak.repository.fagsak
 import no.nav.familie.ef.sak.repository.hovedregelMetadata
 import no.nav.familie.ef.sak.repository.samværsavtale
 import no.nav.familie.ef.sak.repository.samværsuke
@@ -48,6 +50,7 @@ import java.util.UUID
 internal class SamværsavtaleServiceTest {
     private val samværsavtaleRepository: SamværsavtaleRepository = mockk()
     private val behandlingService: BehandlingService = mockk()
+    private val fagsakService: FagsakService = mockk()
     private val tilordnetRessursService: TilordnetRessursService = mockk()
     private val barnService: BarnService = mockk()
     private val personopplysningerService: PersonopplysningerService = mockk()
@@ -60,6 +63,7 @@ internal class SamværsavtaleServiceTest {
         SamværsavtaleService(
             samværsavtaleRepository = samværsavtaleRepository,
             behandlingService = behandlingService,
+            fagsakService = fagsakService,
             tilordnetRessursService = tilordnetRessursService,
             barnService = barnService,
             personopplysningerService = personopplysningerService,
@@ -473,6 +477,7 @@ internal class SamværsavtaleServiceTest {
             BrukerContextUtil.mockBrukerContext()
             every { brevClient.genererFritekstBrev(any()) } returns "1".toByteArray()
             every { arbeidsfordelingService.hentNavEnhetIdEllerBrukMaskinellEnhetHvisNull("123") } returns "4489"
+            every { fagsakService.finnFagsaker(setOf("123")) } returns listOf(fagsak(eksternId = 1))
             every { personopplysningerService.hentGjeldeneNavn(any()) } returns mapOf("123" to "")
             every { brevsignaturService.lagSaksbehandlerSignatur(any(), any()) } returns SignaturDto(navn = "saksbehandler", enhet = "4489", skjulBeslutter = false)
             every { journalpostClient.arkiverDokument(capture(arkivRequestSlot), any()) } returns ArkiverDokumentResponse("", true)
@@ -493,16 +498,18 @@ internal class SamværsavtaleServiceTest {
             assertThat(arkivRequest.hoveddokumentvarianter.first().tittel).isEqualTo("Samværsberegning")
             assertThat(arkivRequest.hoveddokumentvarianter.first().dokumenttype).isEqualTo(Dokumenttype.BEREGNET_SAMVÆR_NOTAT)
             assertThat(arkivRequest.journalførendeEnhet).isEqualTo("4489")
+            assertThat(arkivRequest.fagsakId).isEqualTo("1")
         }
 
         @Test
-        internal fun `Skal kaste feil hvis notat string er tom`() {
+        internal fun `Skal kaste feil ved journalføring av samværsberegning uten notat`() {
             val arkivRequestSlot = slot<ArkiverDokumentRequest>()
             val journalførRequestTomNotat = JournalførBeregnetSamværRequest("123", listOf(samværsuke(listOf(KVELD_NATT, MORGEN, BARNEHAGE_SKOLE))), "", "oppsumering")
 
             BrukerContextUtil.mockBrukerContext()
             every { brevClient.genererFritekstBrev(any()) } returns "1".toByteArray()
             every { arbeidsfordelingService.hentNavEnhetIdEllerBrukMaskinellEnhetHvisNull("123") } returns "4489"
+            every { fagsakService.finnFagsaker(any()) } returns listOf(fagsak(eksternId = 1L))
             every { personopplysningerService.hentGjeldeneNavn(any()) } returns mapOf("123" to "")
             every { brevsignaturService.lagSaksbehandlerSignatur(any(), any()) } returns SignaturDto(navn = "saksbehandler", enhet = "4489", skjulBeslutter = false)
             every { journalpostClient.arkiverDokument(capture(arkivRequestSlot), any()) } returns ArkiverDokumentResponse("", true)
@@ -513,20 +520,39 @@ internal class SamværsavtaleServiceTest {
         }
 
         @Test
-        internal fun `Skal kaste feil hvis oppsumering string er tom`() {
+        internal fun `Skal kaste feil ved journalføring av samværsberegning uten oppsummering`() {
             val arkivRequestSlot = slot<ArkiverDokumentRequest>()
             val journalførRequestTomOppsumering = JournalførBeregnetSamværRequest("123", listOf(samværsuke(listOf(KVELD_NATT, MORGEN, BARNEHAGE_SKOLE))), "Notat", "")
 
             BrukerContextUtil.mockBrukerContext()
             every { brevClient.genererFritekstBrev(any()) } returns "1".toByteArray()
             every { arbeidsfordelingService.hentNavEnhetIdEllerBrukMaskinellEnhetHvisNull("123") } returns "4489"
             every { personopplysningerService.hentGjeldeneNavn(any()) } returns mapOf("123" to "")
+            every { fagsakService.finnFagsaker(any()) } returns listOf(fagsak(eksternId = 1))
             every { brevsignaturService.lagSaksbehandlerSignatur(any(), any()) } returns SignaturDto(navn = "saksbehandler", enhet = "4489", skjulBeslutter = false)
             every { journalpostClient.arkiverDokument(capture(arkivRequestSlot), any()) } returns ArkiverDokumentResponse("", true)
 
             val feil = assertThrows<ApiFeil> { samværsavtaleService.journalførBeregnetSamvær(journalførRequestTomOppsumering) }
 
             assertThat(feil.message).isEqualTo("Kan ikke journalføre samværsavtale uten oppsumering")
         }
+
+        @Test
+        internal fun `Skal kaste feil ved journalføring av samværsberegning på person uten fagsak`() {
+            val arkivRequestSlot = slot<ArkiverDokumentRequest>()
+            val journalførRequestTomOppsumering = JournalførBeregnetSamværRequest("123", listOf(samværsuke(listOf(KVELD_NATT, MORGEN, BARNEHAGE_SKOLE))), "Notat", "Oppsummering")
+
+            BrukerContextUtil.mockBrukerContext()
+            every { brevClient.genererFritekstBrev(any()) } returns "1".toByteArray()
+            every { arbeidsfordelingService.hentNavEnhetIdEllerBrukMaskinellEnhetHvisNull("123") } returns "4489"
+            every { personopplysningerService.hentGjeldeneNavn(any()) } returns mapOf("123" to "")
+            every { fagsakService.finnFagsaker(any()) } returns listOf()
+            every { brevsignaturService.lagSaksbehandlerSignatur(any(), any()) } returns SignaturDto(navn = "saksbehandler", enhet = "4489", skjulBeslutter = false)
+            every { journalpostClient.arkiverDokument(capture(arkivRequestSlot), any()) } returns ArkiverDokumentResponse("", true)
+
+            val feil = assertThrows<ApiFeil> { samværsavtaleService.journalførBeregnetSamvær(journalførRequestTomOppsumering) }
+
+            assertThat(feil.message).isEqualTo("Kan ikke journalføre samværsavtale på person uten fagsak")
+        }
     }
 }