Revertet løsning for å hente fagsakytelsetype fra bunkerepo

navikt · Oct 2, 2023 · b69dd22 · b69dd22
1 parent 2a22fc4
commit b69dd22
Show file tree

Hide file tree

Showing 6 changed files with 26 additions and 28 deletions.
diff --git a/src/main/kotlin/no/nav/k9punsj/domenetjenester/SoknadService.kt b/src/main/kotlin/no/nav/k9punsj/domenetjenester/SoknadService.kt
@@ -49,8 +49,7 @@ class SoknadService(
     private val k9SakService: K9SakService,
     private val sakClient: SakClient,
     private val pdlService: PdlService,
-    private val dokarkivGateway: DokarkivGateway,
-    private val bunkeRepository: BunkeRepository
+    private val dokarkivGateway: DokarkivGateway
 ) {
 
     internal suspend fun sendSøknad(
@@ -66,7 +65,7 @@ class SoknadService(
 
         val søkerFnr = søknad.søker.personIdent.verdi
         val k9YtelseType = Søknadstype.fraBrevkode(brevkode).k9YtelseType
-        val fagsakYtelseType = no.nav.k9punsj.felles.FagsakYtelseType.fromKode(k9YtelseType)
+        val fagsakYtelseType = FagsakYtelseType.fromKode(k9YtelseType)
 
         if (!journalposterKanSendesInn) {
             return HttpStatus.CONFLICT to "En eller alle journalpostene $journalpostIder har blitt sendt inn fra før"
@@ -109,7 +108,10 @@ class SoknadService(
             }
             fagsakIder.first().second
         } else {
-            val k9Respons = k9SakService.hentEllerOpprettSaksnummer(søknad.søknadId.toString())
+            val k9Respons = k9SakService.hentEllerOpprettSaksnummer(
+                    søknadEntitet = hentSøknad(søknad.søknadId.id)!!,
+                    fagsakYtelseType = fagsakYtelseType
+                )
             require(k9Respons.second.isNullOrBlank()) { "Feil ved henting av saksnummer: ${k9Respons.second}" }
             logger.info("Fick saksnummer (${k9Respons.second} av K9Sak for Journalpost ${journalpostIder.first()}")
             k9Respons.first
@@ -235,12 +237,6 @@ class SoknadService(
         return søknadRepository.hentSøknad(søknadId)?.endret_av!!.replace("\"", "")
     }
 
-    suspend fun henteYtelsetypeForSøknad(søknadId: String): FagsakYtelseType? {
-        return hentSøknad(søknadId)?.bunkeId?.let { bunkeId ->
-            bunkeRepository.hentYtelseTypeForBunke(bunkeId)
-        }
-    }
-
     private suspend fun leggerVedPayload(
         søknad: Søknad,
         journalpostIder: MutableSet<String>

diff --git a/src/main/kotlin/no/nav/k9punsj/integrasjoner/k9sak/K9SakService.kt b/src/main/kotlin/no/nav/k9punsj/integrasjoner/k9sak/K9SakService.kt
@@ -6,6 +6,7 @@ import no.nav.k9punsj.felles.FagsakYtelseType
 import no.nav.k9punsj.felles.dto.ArbeidsgiverMedArbeidsforholdId
 import no.nav.k9punsj.felles.dto.PeriodeDto
 import no.nav.k9punsj.felles.dto.SaksnummerDto
+import no.nav.k9punsj.felles.dto.SøknadEntitet
 
 interface K9SakService {
 
@@ -34,7 +35,10 @@ interface K9SakService {
      * Henter saksnummer fra K9Sak, hvis det ikke finnes oppretter vi en ny fagsak.
      * Periode hentes fra soknaden å defaulter til 1/1 og 31/12 hvis fom eller tom ikke er satt.
      */
-    suspend fun hentEllerOpprettSaksnummer(søknadId: String): Pair<String?, String?>
+    suspend fun hentEllerOpprettSaksnummer(
+        søknadEntitet: SøknadEntitet,
+        fagsakYtelseType: FagsakYtelseType
+    ): Pair<String?, String?>
 
     suspend fun hentSisteSaksnummerForPeriode(
         fagsakYtelseType: FagsakYtelseType,

diff --git a/src/main/kotlin/no/nav/k9punsj/integrasjoner/k9sak/K9SakServiceImpl.kt b/src/main/kotlin/no/nav/k9punsj/integrasjoner/k9sak/K9SakServiceImpl.kt
@@ -54,9 +54,7 @@ class K9SakServiceImpl(
     @Value("\${no.nav.k9sak.base_url}") private val baseUrl: URI,
     @Value("\${no.nav.k9sak.scope}") private val k9sakScope: Set<String>,
     @Qualifier("sts") private val accessTokenClient: AccessTokenClient,
-    private val personService: PersonService,
-    private val journalpostService: JournalpostService,
-    private val soknadService: SoknadService,
+    private val personService: PersonService
 ) : K9SakService {
 
     private val cachedAccessTokenClient = CachedAccessTokenClient(accessTokenClient)
@@ -195,13 +193,10 @@ class K9SakServiceImpl(
     }
 
     override suspend fun hentEllerOpprettSaksnummer(
-        søknadId: String
+        søknadEntitet: SøknadEntitet,
+        fagsakYtelseType: no.nav.k9punsj.felles.FagsakYtelseType
     ): Pair<String?, String?> {
-        val søknad = soknadService.hentSøknad(søknadId)
-            ?: return Pair(null, "Fant ikke søknad")
-        val fagsakYtelseType = soknadService.henteYtelsetypeForSøknad(søknad!!.søknadId)
-            ?: return Pair(null, "Fant ikke fagsakytelsetype")
-        val k9SaksnummerGrunnlag = søknad.tilK9saksnummerGrunnlag(fagsakYtelseType)
+        val k9SaksnummerGrunnlag = søknadEntitet.tilK9saksnummerGrunnlag(fagsakYtelseType)
 
         val søkerAktørId = personService.finnEllerOpprettPersonVedNorskIdent(k9SaksnummerGrunnlag.søker).aktørId
         val pleietrengendeAktørId =

diff --git a/src/test/kotlin/no/nav/k9punsj/domenetjenester/SoknadServiceTest.kt b/src/test/kotlin/no/nav/k9punsj/domenetjenester/SoknadServiceTest.kt
@@ -79,9 +79,6 @@ internal class SoknadServiceTest {
 
     private lateinit var soknadService: SoknadService
 
-    @MockK
-    private lateinit var bunkeRepository: BunkeRepository
-
     @BeforeAll
     fun setup() {
         MockKAnnotations.init(this)
@@ -95,7 +92,6 @@ internal class SoknadServiceTest {
             )
         )
         coEvery { mockJournalpostService.hent(any()) }.returns(PunsjJournalpost(UUID.randomUUID(), "1", aktørId = "1"))
-        coEvery { bunkeRepository.hentYtelseTypeForBunke(any()) }.returns(FagsakYtelseType.OMSORGSPENGER)
         soknadService = SoknadService(
             journalpostService = mockJournalpostService,
             søknadRepository = mockSøknadRepository,
@@ -104,8 +100,7 @@ internal class SoknadServiceTest {
             k9SakService = k9SakService,
             sakClient = sakClient,
             pdlService = pdlService,
-            dokarkivGateway = dokarkivGateway,
-            bunkeRepository = bunkeRepository,
+            dokarkivGateway = dokarkivGateway
         )
     }
 

diff --git a/src/test/kotlin/no/nav/k9punsj/rest/eksternt/k9sak/LokalK9SakService.kt b/src/test/kotlin/no/nav/k9punsj/rest/eksternt/k9sak/LokalK9SakService.kt
@@ -7,6 +7,7 @@ import no.nav.k9punsj.felles.FagsakYtelseType
 import no.nav.k9punsj.felles.dto.ArbeidsgiverMedArbeidsforholdId
 import no.nav.k9punsj.felles.dto.PeriodeDto
 import no.nav.k9punsj.felles.dto.SaksnummerDto
+import no.nav.k9punsj.felles.dto.SøknadEntitet
 import no.nav.k9punsj.integrasjoner.k9sak.Fagsak
 import no.nav.k9punsj.integrasjoner.k9sak.HentK9SaksnummerGrunnlag
 import no.nav.k9punsj.integrasjoner.k9sak.K9SakService
@@ -72,7 +73,10 @@ class LokalK9SakService : K9SakService {
         second = null
     )
 
-    override suspend fun hentEllerOpprettSaksnummer(søknadId: String): Pair<String?, String?> {
+    override suspend fun hentEllerOpprettSaksnummer(
+        søknadEntitet: SøknadEntitet,
+        fagsakYtelseType: no.nav.k9punsj.felles.FagsakYtelseType
+    ): Pair<String?, String?> {
         return Pair("ABC123", null)
     }
 

diff --git a/src/test/kotlin/no/nav/k9punsj/rest/eksternt/k9sak/TestK9SakService.kt b/src/test/kotlin/no/nav/k9punsj/rest/eksternt/k9sak/TestK9SakService.kt
@@ -7,6 +7,7 @@ import no.nav.k9punsj.felles.FagsakYtelseType
 import no.nav.k9punsj.felles.dto.ArbeidsgiverMedArbeidsforholdId
 import no.nav.k9punsj.felles.dto.PeriodeDto
 import no.nav.k9punsj.felles.dto.SaksnummerDto
+import no.nav.k9punsj.felles.dto.SøknadEntitet
 import no.nav.k9punsj.integrasjoner.k9sak.Fagsak
 import no.nav.k9punsj.integrasjoner.k9sak.HentK9SaksnummerGrunnlag
 import no.nav.k9punsj.integrasjoner.k9sak.K9SakService
@@ -86,7 +87,10 @@ internal class TestK9SakService : K9SakService {
         second = null
     )
 
-    override suspend fun hentEllerOpprettSaksnummer(søknadId: String): Pair<String?, String?> {
+    override suspend fun hentEllerOpprettSaksnummer(
+        søknadEntitet: SøknadEntitet,
+        fagsakYtelseType: no.nav.k9punsj.felles.FagsakYtelseType
+    ): Pair<String?, String?> {
         return Pair("ABC123", null)
     }