flytter funksjonalitet for å hente ut søknad på søknadID inn i eksist…

…erende servicer og klienter

src/main/kotlin/no/nav/familie/ef/søknad/ettersending/DokumentasjonsbehovController.kt

@@ -1,6 +1,6 @@
 package no.nav.familie.ef.søknad.ettersending
 
-import no.nav.familie.ef.søknad.søknad.SøknadClient
+import no.nav.familie.ef.søknad.søknad.MottakClient
 import no.nav.familie.kontrakter.ef.ettersending.SøknadMedDokumentasjonsbehovDto
 import no.nav.familie.sikkerhet.EksternBrukerUtils
 import no.nav.security.token.support.core.api.ProtectedWithClaims
@@ -13,7 +13,7 @@ import org.springframework.web.bind.annotation.RestController
 @RequestMapping(path = ["/api/dokumentasjonsbehov"])
 @ProtectedWithClaims(issuer = EksternBrukerUtils.ISSUER_TOKENX, claimMap = ["acr=Level4"])
 class DokumentasjonsbehovController(
-    private val søknadClient: SøknadClient,
+    private val søknadClient: MottakClient,
 ) {
     @GetMapping("/person")
     fun hentDokumentasjonsbehovForPerson(): ResponseEntity<List<SøknadMedDokumentasjonsbehovDto>> {

src/main/kotlin/no/nav/familie/ef/søknad/ettersending/EttersendingService.kt

@@ -1,7 +1,7 @@
 package no.nav.familie.ef.søknad.ettersending
 
 import no.nav.familie.ef.søknad.ettersending.mapper.EttersendingMapper
-import no.nav.familie.ef.søknad.søknad.SøknadClient
+import no.nav.familie.ef.søknad.søknad.MottakClient
 import no.nav.familie.ef.søknad.søknad.domain.Kvittering
 import no.nav.familie.ef.søknad.søknad.mapper.KvitteringMapper
 import no.nav.familie.kontrakter.ef.ettersending.EttersendelseDto
@@ -11,7 +11,7 @@ import java.time.LocalDateTime
 
 @Service
 class EttersendingService(
-    private val søknadClient: SøknadClient,
+    private val søknadClient: MottakClient,
 ) {
     fun sendInn(
         ettersending: EttersendelseDto,

src/main/kotlin/no/nav/familie/ef/søknad/infrastruktur/health/SøknadHealthIndicator.kt

@@ -1,10 +1,10 @@
 package no.nav.familie.ef.søknad.infrastruktur.health
 
-import no.nav.familie.ef.søknad.søknad.SøknadClient
+import no.nav.familie.ef.søknad.søknad.MottakClient
 import no.nav.familie.http.health.AbstractHealthIndicator
 import org.springframework.stereotype.Component
 
 @Component
 internal class SøknadHealthIndicator(
-    søknadClient: SøknadClient,
+    søknadClient: MottakClient,
 ) : AbstractHealthIndicator(søknadClient, "familie.ef.mottak")

src/main/kotlin/no/nav/familie/ef/søknad/søknad/SøknadClient.kt → src/main/kotlin/no/nav/familie/ef/søknad/søknad/MottakClient.kt

@@ -18,10 +18,11 @@ import org.springframework.beans.factory.annotation.Qualifier
 import org.springframework.http.HttpHeaders
 import org.springframework.stereotype.Service
 import org.springframework.web.client.RestOperations
+import org.springframework.web.util.UriComponentsBuilder
 import java.net.URI
 
 @Service
-class SøknadClient(
+class MottakClient(
     private val config: MottakConfig,
     @Qualifier("tokenExchange") operations: RestOperations,
 ) : AbstractPingableRestClient(operations, "søknad.innsending") {
@@ -60,6 +61,16 @@ class SøknadClient(
             HttpHeaders().medContentTypeJsonUTF8(),
         )
 
+    fun hentSøknadKvittering(søknadId: String): ByteArray =
+        getForEntity<ByteArray>(
+            UriComponentsBuilder
+                .fromUriString(
+                    "${config.hentSøknadKvitteringUri}/$søknadId",
+                ).build()
+                .toUri(),
+            HttpHeaders().medContentTypeJsonUTF8(),
+        )
+
     private fun HttpHeaders.medContentTypeJsonUTF8(): HttpHeaders {
         this.add("Content-Type", "application/json;charset=UTF-8")
         this.add("behandlingsnummer", Tema.ENF.behandlingsnummer)

src/main/kotlin/no/nav/familie/ef/søknad/søknad/SkjemaService.kt

@@ -8,7 +8,7 @@ import java.time.LocalDateTime
 
 @Service
 class SkjemaService(
-    val søknadClient: SøknadClient,
+    val søknadClient: MottakClient,
 ) {
     fun sendInn(
         arbeidssøker: Arbeidssøker,

src/main/kotlin/no/nav/familie/ef/søknad/søknad/SøknadKvitteringController.kt

@@ -28,7 +28,6 @@ import java.time.LocalDateTime
 @Validated
 class SøknadKvitteringController(
     val søknadService: SøknadService,
-    val søknadKvitteringService: SøknadKvitteringService,
 ) {
     @PostMapping("overgangsstonad")
     fun sendInn(
@@ -73,5 +72,5 @@ class SøknadKvitteringController(
     @GetMapping("{søknadId}")
     fun hentSøknad(
         @PathVariable søknadId: String,
-    ): ByteArray = søknadKvitteringService.hentSøknadPdf(søknadId)
+    ): ByteArray = søknadService.hentSøknadPdf(søknadId)
 }

src/main/kotlin/no/nav/familie/ef/søknad/søknad/SøknadService.kt

@@ -14,7 +14,7 @@ import java.time.LocalDateTime
 
 @Service
 class SøknadService(
-    private val søknadClient: SøknadClient,
+    private val mottakClient: MottakClient,
     private val overgangsstønadMapper: SøknadOvergangsstønadMapper,
     private val barnetilsynMapper: SøknadBarnetilsynMapper,
     private val skolepengerMapper: SøknadSkolepengerMapper,
@@ -24,7 +24,7 @@ class SøknadService(
         innsendingMottatt: LocalDateTime,
     ): Kvittering {
         val søknadRequestData = overgangsstønadMapper.mapTilIntern(søknad, innsendingMottatt)
-        val kvittering = søknadClient.sendInn(søknadRequestData)
+        val kvittering = mottakClient.sendInn(søknadRequestData)
         return KvitteringMapper.mapTilEkstern(kvittering, innsendingMottatt)
     }
 
@@ -33,7 +33,7 @@ class SøknadService(
         innsendingMottatt: LocalDateTime,
     ): Kvittering {
         val søknadRequestData = barnetilsynMapper.mapTilIntern(søknad, innsendingMottatt)
-        val kvittering = søknadClient.sendInnBarnetilsynsøknad(søknadRequestData)
+        val kvittering = mottakClient.sendInnBarnetilsynsøknad(søknadRequestData)
         return KvitteringMapper.mapTilEkstern(kvittering, innsendingMottatt)
     }
 
@@ -42,9 +42,11 @@ class SøknadService(
         innsendingMottatt: LocalDateTime,
     ): Kvittering {
         val søknadRequestData = skolepengerMapper.mapTilIntern(søknad, innsendingMottatt)
-        val kvittering = søknadClient.sendInnSkolepenger(søknadRequestData)
+        val kvittering = mottakClient.sendInnSkolepenger(søknadRequestData)
         return KvitteringMapper.mapTilEkstern(kvittering, innsendingMottatt)
     }
 
-    fun hentForrigeBarnetilsynSøknad(): SøknadBarnetilsynGjenbrukDto? = SøknadBarnetilsynMapper().mapTilDto(søknadClient.hentForrigeBarnetilsynSøknad())
+    fun hentSøknadPdf(søknadId: String): ByteArray = mottakClient.hentSøknadKvittering(søknadId)
+
+    fun hentForrigeBarnetilsynSøknad(): SøknadBarnetilsynGjenbrukDto? = SøknadBarnetilsynMapper().mapTilDto(mottakClient.hentForrigeBarnetilsynSøknad())
 }

src/test/kotlin/no/nav/familie/ef/søknad/ApplicationLocalLauncher.kt

@@ -23,7 +23,6 @@ fun main(args: Array<String>) {
             "mock-integrasjoner",
             "mock-saf",
             "mock-saksbehandling",
-            "mock-søknadkvittering",
         ).properties(mapOf("mock-oauth2-server.port" to mockOauth2ServerPort))
         .run(*args)
 }

src/test/kotlin/no/nav/familie/ef/søknad/ettersending/DokumentasjonsbehovControllerTest.kt

@@ -1,8 +1,6 @@
 package no.nav.familie.ef.søknad.ettersending
 
-import io.mockk.mockk
 import no.nav.familie.ef.søknad.infrastruktur.OppslagSpringRunnerTest
-import no.nav.familie.ef.søknad.søknad.SøknadClient
 import no.nav.familie.ef.søknad.søknad.SøknadClientUtil.filtrerVekkEldreDokumentasjonsbehov
 import no.nav.familie.kontrakter.ef.ettersending.SøknadMedDokumentasjonsbehovDto
 import no.nav.familie.kontrakter.ef.søknad.SøknadType
@@ -11,27 +9,10 @@ import no.nav.familie.kontrakter.felles.ef.StønadType
 import org.assertj.core.api.Assertions.assertThat
 import org.junit.jupiter.api.BeforeEach
 import org.junit.jupiter.api.Test
-import org.springframework.beans.factory.annotation.Autowired
-import org.springframework.context.annotation.Bean
-import org.springframework.context.annotation.Configuration
-import org.springframework.context.annotation.Primary
-import org.springframework.context.annotation.Profile
-import org.springframework.test.context.ActiveProfiles
 import java.time.LocalDate
 import java.util.UUID
 
-@Profile("dokumentasjonsbehov-test")
-@Configuration
-class DokumentasjonsbehovControllerTestConfiguration {
-    @Primary
-    @Bean
-    fun søknadClient(): SøknadClient = mockk()
-}
-
-@ActiveProfiles("dokumentasjonsbehov-test")
 internal class DokumentasjonsbehovControllerTest : OppslagSpringRunnerTest() {
-    @Autowired
-    lateinit var søknadClient: SøknadClient
 
     val tokenSubject = "12345678911"
 

src/test/kotlin/no/nav/familie/ef/søknad/infrastruktur/OppslagSpringRunnerTest.kt

@@ -32,7 +32,6 @@ import java.util.UUID
     "kodeverk-cache-test",
     "mock-saf",
     "mock-saksbehandling",
-    "mock-søknadkvittering",
 )
 @EnableMockOAuth2Server
 abstract class OppslagSpringRunnerTest {

src/test/kotlin/no/nav/familie/ef/søknad/mock/MottakClientMock.kt

@@ -2,7 +2,7 @@ package no.nav.familie.ef.søknad.mock
 
 import io.mockk.every
 import io.mockk.mockk
-import no.nav.familie.ef.søknad.søknad.SøknadClient
+import no.nav.familie.ef.søknad.søknad.MottakClient
 import no.nav.familie.ef.søknad.søknad.dto.KvitteringDto
 import no.nav.familie.kontrakter.ef.ettersending.EttersendelseDto
 import no.nav.familie.kontrakter.ef.ettersending.SøknadMedDokumentasjonsbehovDto
@@ -26,20 +26,21 @@ import java.util.UUID
 class MottakClientMock {
     @Bean
     @Primary
-    fun søknadClient(): SøknadClient {
-        val søknadClient: SøknadClient = mockk()
+    fun mottakClient(): MottakClient {
+        val mottakClient: MottakClient = mockk()
 
-        every { søknadClient.sendInn(any()) } returns KvitteringDto("OK MOCK")
-        every { søknadClient.sendInnArbeidsRegistreringsskjema(any()) } returns KvitteringDto("OK MOCK")
-        every { søknadClient.sendInnBarnetilsynsøknad(any()) } returns KvitteringDto("OK MOCK")
-        every { søknadClient.sendInnSkolepenger(any()) } returns KvitteringDto("OK MOCK")
-        every { søknadClient.sendInnEttersending(any()) } returns KvitteringDto("OK MOCK")
-        every { søknadClient.ping() } returns Unit
-        every { søknadClient.hentSøknaderMedDokumentasjonsbehov(any()) } returns søknaderMedDokumentasjonsbehov
-        every { søknadClient.hentEttersendingForPerson(any()) } returns listOf(ettersendingResponseData)
-        every { søknadClient.hentForrigeBarnetilsynSøknad() } returns null
+        every { mottakClient.sendInn(any()) } returns KvitteringDto("OK MOCK")
+        every { mottakClient.sendInnArbeidsRegistreringsskjema(any()) } returns KvitteringDto("OK MOCK")
+        every { mottakClient.sendInnBarnetilsynsøknad(any()) } returns KvitteringDto("OK MOCK")
+        every { mottakClient.sendInnSkolepenger(any()) } returns KvitteringDto("OK MOCK")
+        every { mottakClient.sendInnEttersending(any()) } returns KvitteringDto("OK MOCK")
+        every { mottakClient.ping() } returns Unit
+        every { mottakClient.hentSøknaderMedDokumentasjonsbehov(any()) } returns søknaderMedDokumentasjonsbehov
+        every { mottakClient.hentEttersendingForPerson(any()) } returns listOf(ettersendingResponseData)
+        every { mottakClient.hentForrigeBarnetilsynSøknad() } returns null
+        every { mottakClient.hentSøknadKvittering(any()) } returns "pdf".toByteArray()
 
-        return søknadClient
+        return mottakClient
     }
 
     private val dokumentasjonsbehovDto =

src/test/kotlin/no/nav/familie/ef/søknad/søknad/SøknadKvitteringControllerTest.kt

@@ -50,9 +50,6 @@ class SøknadKvitteringControllerTest {
         @Autowired
         lateinit var søknadService: SøknadService
 
-        @Autowired
-        lateinit var søknadKvitteringService: SøknadKvitteringService
-
         @Autowired
         lateinit var featureToggleService: FeatureToggleService
 
@@ -204,7 +201,7 @@ class SøknadKvitteringControllerTest {
 
         @Test
         fun `mottak returnerer pdf kvittering`() {
-            every { søknadKvitteringService.hentSøknadPdf("1") } returns "pdf".toByteArray()
+            every { søknadService.hentSøknadPdf("1") } returns "pdf".toByteArray()
 
             val response =
                 restTemplate.exchange<ByteArray>(