diff --git "a/src/main/kotlin/no/nav/familie/ef/s\303\270knad/ettersending/DokumentasjonsbehovController.kt" "b/src/main/kotlin/no/nav/familie/ef/s\303\270knad/ettersending/DokumentasjonsbehovController.kt" index 4ddfe3942..7df614b06 100644 --- "a/src/main/kotlin/no/nav/familie/ef/s\303\270knad/ettersending/DokumentasjonsbehovController.kt" +++ "b/src/main/kotlin/no/nav/familie/ef/s\303\270knad/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> { diff --git "a/src/main/kotlin/no/nav/familie/ef/s\303\270knad/ettersending/EttersendingService.kt" "b/src/main/kotlin/no/nav/familie/ef/s\303\270knad/ettersending/EttersendingService.kt" index 63cd637d8..f22f699fb 100644 --- "a/src/main/kotlin/no/nav/familie/ef/s\303\270knad/ettersending/EttersendingService.kt" +++ "b/src/main/kotlin/no/nav/familie/ef/s\303\270knad/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, diff --git "a/src/main/kotlin/no/nav/familie/ef/s\303\270knad/infrastruktur/health/S\303\270knadHealthIndicator.kt" "b/src/main/kotlin/no/nav/familie/ef/s\303\270knad/infrastruktur/health/S\303\270knadHealthIndicator.kt" index f736348ca..4b142b414 100644 --- "a/src/main/kotlin/no/nav/familie/ef/s\303\270knad/infrastruktur/health/S\303\270knadHealthIndicator.kt" +++ "b/src/main/kotlin/no/nav/familie/ef/s\303\270knad/infrastruktur/health/S\303\270knadHealthIndicator.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") diff --git "a/src/main/kotlin/no/nav/familie/ef/s\303\270knad/s\303\270knad/S\303\270knadClient.kt" "b/src/main/kotlin/no/nav/familie/ef/s\303\270knad/s\303\270knad/MottakClient.kt" similarity index 89% rename from "src/main/kotlin/no/nav/familie/ef/s\303\270knad/s\303\270knad/S\303\270knadClient.kt" rename to "src/main/kotlin/no/nav/familie/ef/s\303\270knad/s\303\270knad/MottakClient.kt" index 5895081a8..d66784884 100644 --- "a/src/main/kotlin/no/nav/familie/ef/s\303\270knad/s\303\270knad/S\303\270knadClient.kt" +++ "b/src/main/kotlin/no/nav/familie/ef/s\303\270knad/s\303\270knad/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( + 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) diff --git "a/src/main/kotlin/no/nav/familie/ef/s\303\270knad/s\303\270knad/SkjemaService.kt" "b/src/main/kotlin/no/nav/familie/ef/s\303\270knad/s\303\270knad/SkjemaService.kt" index 9a07b1817..c596ab93f 100644 --- "a/src/main/kotlin/no/nav/familie/ef/s\303\270knad/s\303\270knad/SkjemaService.kt" +++ "b/src/main/kotlin/no/nav/familie/ef/s\303\270knad/s\303\270knad/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, diff --git "a/src/main/kotlin/no/nav/familie/ef/s\303\270knad/s\303\270knad/S\303\270knadKvitteringClient.kt" "b/src/main/kotlin/no/nav/familie/ef/s\303\270knad/s\303\270knad/S\303\270knadKvitteringClient.kt" deleted file mode 100644 index bc14ed032..000000000 --- "a/src/main/kotlin/no/nav/familie/ef/s\303\270knad/s\303\270knad/S\303\270knadKvitteringClient.kt" +++ /dev/null @@ -1,37 +0,0 @@ -package no.nav.familie.ef.søknad.søknad - -import no.nav.familie.ef.søknad.infrastruktur.config.MottakConfig -import no.nav.familie.http.client.AbstractPingableRestClient -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øknadKvitteringClient( - private val config: MottakConfig, - @Qualifier("tokenExchange") operations: RestOperations, -) : AbstractPingableRestClient( - operations, - "søknad.henting", - ) { - override val pingUri: URI = config.pingUri - - fun hentSøknadKvittering(søknadId: String): ByteArray = - getForEntity( - UriComponentsBuilder - .fromUriString( - "${config.hentSøknadKvitteringUri}/$søknadId", - ).build() - .toUri(), - HttpHeaders().medContentTypeJsonUTF8(), - ) -} - -fun HttpHeaders.medContentTypeJsonUTF8(): HttpHeaders { - this.add("Content-Type", "application/json;charset=UTF-8") - this.acceptCharset = listOf(Charsets.UTF_8) - return this -} diff --git "a/src/main/kotlin/no/nav/familie/ef/s\303\270knad/s\303\270knad/S\303\270knadKvitteringController.kt" "b/src/main/kotlin/no/nav/familie/ef/s\303\270knad/s\303\270knad/S\303\270knadKvitteringController.kt" index 5b661e5f1..ed73308c0 100644 --- "a/src/main/kotlin/no/nav/familie/ef/s\303\270knad/s\303\270knad/S\303\270knadKvitteringController.kt" +++ "b/src/main/kotlin/no/nav/familie/ef/s\303\270knad/s\303\270knad/S\303\270knadKvitteringController.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) } diff --git "a/src/main/kotlin/no/nav/familie/ef/s\303\270knad/s\303\270knad/S\303\270knadKvitteringService.kt" "b/src/main/kotlin/no/nav/familie/ef/s\303\270knad/s\303\270knad/S\303\270knadKvitteringService.kt" deleted file mode 100644 index c3719d718..000000000 --- "a/src/main/kotlin/no/nav/familie/ef/s\303\270knad/s\303\270knad/S\303\270knadKvitteringService.kt" +++ /dev/null @@ -1,10 +0,0 @@ -package no.nav.familie.ef.søknad.søknad - -import org.springframework.stereotype.Service - -@Service -class SøknadKvitteringService( - private val søknadKvitteringClient: SøknadKvitteringClient, -) { - fun hentSøknadPdf(søknadId: String): ByteArray = søknadKvitteringClient.hentSøknadKvittering(søknadId) -} diff --git "a/src/main/kotlin/no/nav/familie/ef/s\303\270knad/s\303\270knad/S\303\270knadService.kt" "b/src/main/kotlin/no/nav/familie/ef/s\303\270knad/s\303\270knad/S\303\270knadService.kt" index e1ebcc98a..a635374a7 100644 --- "a/src/main/kotlin/no/nav/familie/ef/s\303\270knad/s\303\270knad/S\303\270knadService.kt" +++ "b/src/main/kotlin/no/nav/familie/ef/s\303\270knad/s\303\270knad/S\303\270knadService.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()) } diff --git "a/src/test/kotlin/no/nav/familie/ef/s\303\270knad/ApplicationLocalLauncher.kt" "b/src/test/kotlin/no/nav/familie/ef/s\303\270knad/ApplicationLocalLauncher.kt" index 95639e4d5..8cd67d1cb 100644 --- "a/src/test/kotlin/no/nav/familie/ef/s\303\270knad/ApplicationLocalLauncher.kt" +++ "b/src/test/kotlin/no/nav/familie/ef/s\303\270knad/ApplicationLocalLauncher.kt" @@ -23,7 +23,6 @@ fun main(args: Array) { "mock-integrasjoner", "mock-saf", "mock-saksbehandling", - "mock-søknadkvittering", ).properties(mapOf("mock-oauth2-server.port" to mockOauth2ServerPort)) .run(*args) } diff --git "a/src/test/kotlin/no/nav/familie/ef/s\303\270knad/ettersending/DokumentasjonsbehovControllerTest.kt" "b/src/test/kotlin/no/nav/familie/ef/s\303\270knad/ettersending/DokumentasjonsbehovControllerTest.kt" index f07b92462..3a2640bce 100644 --- "a/src/test/kotlin/no/nav/familie/ef/s\303\270knad/ettersending/DokumentasjonsbehovControllerTest.kt" +++ "b/src/test/kotlin/no/nav/familie/ef/s\303\270knad/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" diff --git "a/src/test/kotlin/no/nav/familie/ef/s\303\270knad/infrastruktur/OppslagSpringRunnerTest.kt" "b/src/test/kotlin/no/nav/familie/ef/s\303\270knad/infrastruktur/OppslagSpringRunnerTest.kt" index a46777e29..75164f072 100644 --- "a/src/test/kotlin/no/nav/familie/ef/s\303\270knad/infrastruktur/OppslagSpringRunnerTest.kt" +++ "b/src/test/kotlin/no/nav/familie/ef/s\303\270knad/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 { diff --git "a/src/test/kotlin/no/nav/familie/ef/s\303\270knad/mock/MottakClientMock.kt" "b/src/test/kotlin/no/nav/familie/ef/s\303\270knad/mock/MottakClientMock.kt" index 16209fb94..db07fd290 100644 --- "a/src/test/kotlin/no/nav/familie/ef/s\303\270knad/mock/MottakClientMock.kt" +++ "b/src/test/kotlin/no/nav/familie/ef/s\303\270knad/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 = diff --git "a/src/test/kotlin/no/nav/familie/ef/s\303\270knad/mock/S\303\270knadKvitteringClientMock.kt" "b/src/test/kotlin/no/nav/familie/ef/s\303\270knad/mock/S\303\270knadKvitteringClientMock.kt" deleted file mode 100644 index 6469344ee..000000000 --- "a/src/test/kotlin/no/nav/familie/ef/s\303\270knad/mock/S\303\270knadKvitteringClientMock.kt" +++ /dev/null @@ -1,22 +0,0 @@ -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øknadKvitteringClient -import org.springframework.context.annotation.Bean -import org.springframework.context.annotation.Configuration -import org.springframework.context.annotation.Primary -import org.springframework.context.annotation.Profile - -@Configuration -@Profile("mock-søknadkvittering") -class SøknadKvitteringClientMock { - @Bean - @Primary - fun søknadKvitteringClient(): SøknadKvitteringClient { - val søknadKvitteringClient = mockk() - - every { søknadKvitteringClient.hentSøknadKvittering(any()) } returns "pdf".toByteArray() - return søknadKvitteringClient - } -} diff --git "a/src/test/kotlin/no/nav/familie/ef/s\303\270knad/s\303\270knad/S\303\270knadKvitteringControllerTest.kt" "b/src/test/kotlin/no/nav/familie/ef/s\303\270knad/s\303\270knad/S\303\270knadKvitteringControllerTest.kt" index efee2b863..45410ca82 100644 --- "a/src/test/kotlin/no/nav/familie/ef/s\303\270knad/s\303\270knad/S\303\270knadKvitteringControllerTest.kt" +++ "b/src/test/kotlin/no/nav/familie/ef/s\303\270knad/s\303\270knad/S\303\270knadKvitteringControllerTest.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(