diff --git a/src/main/kotlin/no/nav/familie/ba/sak/config/featureToggle/FeatureToggleConfig.kt b/src/main/kotlin/no/nav/familie/ba/sak/config/featureToggle/FeatureToggleConfig.kt index 12e7d3d242a..f321c952447 100644 --- a/src/main/kotlin/no/nav/familie/ba/sak/config/featureToggle/FeatureToggleConfig.kt +++ b/src/main/kotlin/no/nav/familie/ba/sak/config/featureToggle/FeatureToggleConfig.kt @@ -19,9 +19,6 @@ class FeatureToggleConfig { // NAV-22311 const val OPPRETT_SAK_PÅ_RIKTIG_ENHET_OG_SAKSBEHANDLER = "familie-ba-ks-sak.opprett-sak-paa-riktig-enhet-og-saksbehandler" - // NAV-22361 - const val BRUK_NYTT_RETUR_OBJEKT_FOR_JOURNALPOSTER = "familie-ba-ks-sak.bruk-nytt-retur-objekt-for-journalposter" - // satsendring // Oppretter satsendring-tasker for de som ikke har fått ny task const val SATSENDRING_ENABLET: String = "familie-ba-sak.satsendring-enablet" diff --git "a/src/main/kotlin/no/nav/familie/ba/sak/integrasjoner/journalf\303\270ring/Journalf\303\270ringController.kt" "b/src/main/kotlin/no/nav/familie/ba/sak/integrasjoner/journalf\303\270ring/Journalf\303\270ringController.kt" index f58761eb9a7..e5c0f5457cc 100644 --- "a/src/main/kotlin/no/nav/familie/ba/sak/integrasjoner/journalf\303\270ring/Journalf\303\270ringController.kt" +++ "b/src/main/kotlin/no/nav/familie/ba/sak/integrasjoner/journalf\303\270ring/Journalf\303\270ringController.kt" @@ -3,13 +3,12 @@ package no.nav.familie.ba.sak.integrasjoner.journalføring import jakarta.validation.Valid import no.nav.familie.ba.sak.common.FunksjonellFeil import no.nav.familie.ba.sak.config.BehandlerRolle -import no.nav.familie.ba.sak.config.FeatureToggleConfig import no.nav.familie.ba.sak.ekstern.restDomene.RestJournalføring import no.nav.familie.ba.sak.sikkerhet.TilgangService import no.nav.familie.kontrakter.felles.PersonIdent import no.nav.familie.kontrakter.felles.Ressurs import no.nav.familie.kontrakter.felles.journalpost.Journalpost -import no.nav.familie.unleash.UnleashService +import no.nav.familie.kontrakter.felles.journalpost.TilgangsstyrtJournalpost import no.nav.security.token.support.core.api.ProtectedWithClaims import org.springframework.http.MediaType import org.springframework.http.ResponseEntity @@ -29,7 +28,6 @@ import org.springframework.web.bind.annotation.RestController class JournalføringController( private val innkommendeJournalføringService: InnkommendeJournalføringService, private val tilgangService: TilgangService, - private val unleashService: UnleashService, ) { @GetMapping(path = ["/{journalpostId}/hent"], produces = [MediaType.APPLICATION_JSON_VALUE]) fun hentJournalpost( @@ -39,19 +37,14 @@ class JournalføringController( @PostMapping(path = ["/for-bruker"]) fun hentJournalposterForBruker( @RequestBody personIdentBody: PersonIdent, - ): ResponseEntity>> { - val tilgangsstyrteJournalposter = - innkommendeJournalføringService.hentJournalposterForBruker( - personIdentBody.ident, - ) - val response = - if (unleashService.isEnabled(FeatureToggleConfig.BRUK_NYTT_RETUR_OBJEKT_FOR_JOURNALPOSTER, false)) { - tilgangsstyrteJournalposter - } else { - tilgangsstyrteJournalposter.map { it.journalpost } - } - return ResponseEntity.ok(Ressurs.success(response)) - } + ): ResponseEntity>> = + ResponseEntity.ok( + Ressurs.success( + innkommendeJournalføringService.hentJournalposterForBruker( + personIdentBody.ident, + ), + ), + ) @GetMapping("/{journalpostId}/hent/{dokumentInfoId}") fun hentDokument( diff --git "a/src/test/enhetstester/kotlin/no/nav/familie/ba/sak/integrasjoner/journalf\303\270ring/Journalf\303\270ringControllerTest.kt" "b/src/test/enhetstester/kotlin/no/nav/familie/ba/sak/integrasjoner/journalf\303\270ring/Journalf\303\270ringControllerTest.kt" index daa90545ef9..674d3dc6036 100644 --- "a/src/test/enhetstester/kotlin/no/nav/familie/ba/sak/integrasjoner/journalf\303\270ring/Journalf\303\270ringControllerTest.kt" +++ "b/src/test/enhetstester/kotlin/no/nav/familie/ba/sak/integrasjoner/journalf\303\270ring/Journalf\303\270ringControllerTest.kt" @@ -2,13 +2,9 @@ package no.nav.familie.ba.sak.integrasjoner.journalføring import io.mockk.every import io.mockk.mockk -import no.nav.familie.ba.sak.config.FeatureToggleConfig import no.nav.familie.ba.sak.integrasjoner.lagTilgangsstyrtJournalpost import no.nav.familie.ba.sak.sikkerhet.TilgangService import no.nav.familie.kontrakter.felles.PersonIdent -import no.nav.familie.kontrakter.felles.journalpost.Journalpost -import no.nav.familie.kontrakter.felles.journalpost.TilgangsstyrtJournalpost -import no.nav.familie.unleash.UnleashService import org.assertj.core.api.Assertions.assertThat import org.junit.jupiter.api.Nested import org.junit.jupiter.api.Test @@ -17,47 +13,21 @@ import org.springframework.http.HttpStatus class JournalføringControllerTest { private val innkommendeJournalføringService: InnkommendeJournalføringService = mockk() private val tilgangService: TilgangService = mockk() - private val unleashService: UnleashService = mockk() private val journalføringController: JournalføringController = JournalføringController( innkommendeJournalføringService = innkommendeJournalføringService, tilgangService = tilgangService, - unleashService = unleashService, ) @Nested inner class HentJournalposterForBruker { @Test - fun `skal returnere liste av journalposter når toggle er av`() { - // Arrange - val personIdent = PersonIdent("123") - val journalpostId = "1" - - every { innkommendeJournalføringService.hentJournalposterForBruker(personIdent.ident) } returns listOf(lagTilgangsstyrtJournalpost(personIdent.ident, journalpostId = journalpostId)) - every { unleashService.isEnabled(FeatureToggleConfig.BRUK_NYTT_RETUR_OBJEKT_FOR_JOURNALPOSTER, false) } returns false - - // Act - val responseEntity = journalføringController.hentJournalposterForBruker(personIdentBody = personIdent) - - // Assert - assertThat(responseEntity.statusCode).isEqualTo(HttpStatus.OK) - assertThat(responseEntity.body).isNotNull - assertThat(responseEntity.body!!.data).isNotNull - val journalposter = responseEntity.body!!.data!! - assertThat(journalposter).hasSize(1) - assertThat(journalposter.single()).isInstanceOf(Journalpost::class.java) - val journalpost = journalposter.single() as Journalpost - assertThat(journalpost.journalpostId).isEqualTo("1") - } - - @Test - fun `skal returnere liste av tilgangsstyrte journalposter når toggle er på`() { + fun `skal returnere liste av tilgangsstyrte journalposter`() { // Arrange val personIdent = PersonIdent("123") val journalpostId = "1" every { innkommendeJournalføringService.hentJournalposterForBruker(personIdent.ident) } returns listOf(lagTilgangsstyrtJournalpost(personIdent.ident, journalpostId = journalpostId, harTilgang = true)) - every { unleashService.isEnabled(FeatureToggleConfig.BRUK_NYTT_RETUR_OBJEKT_FOR_JOURNALPOSTER, false) } returns true // Act val responseEntity = journalføringController.hentJournalposterForBruker(personIdentBody = personIdent) @@ -68,8 +38,7 @@ class JournalføringControllerTest { assertThat(responseEntity.body!!.data).isNotNull val journalposter = responseEntity.body!!.data!! assertThat(journalposter).hasSize(1) - assertThat(journalposter.single()).isInstanceOf(TilgangsstyrtJournalpost::class.java) - val tilgangsstyrtJournalpost = journalposter.single() as TilgangsstyrtJournalpost + val tilgangsstyrtJournalpost = journalposter.single() assertThat(tilgangsstyrtJournalpost.journalpost.journalpostId).isEqualTo("1") assertThat(tilgangsstyrtJournalpost.harTilgang).isTrue }