From e9b64d7f411ca21c42e05e12de5826a75e42f1f3 Mon Sep 17 00:00:00 2001 From: Thomas Alm <106926075+thoalm@users.noreply.github.com> Date: Mon, 9 Dec 2024 12:56:31 +0100 Subject: [PATCH] NAV-23413: Fjerner toggle for EnhetsnummerService (#1213) --- .../featureToggle/FeatureToggleConfig.kt | 1 - .../integrasjoner/AbstractOppgaveMapper.kt | 48 +------------------ .../integrasjoner/BarnetrygdOppgaveMapper.kt | 10 ---- .../Kontantst\303\270tteOppgaveMapper.kt" | 10 ---- .../mottak/integrasjoner/OppgaveMapperTest.kt | 16 ------- 5 files changed, 1 insertion(+), 84 deletions(-) diff --git a/src/main/kotlin/no/nav/familie/baks/mottak/config/featureToggle/FeatureToggleConfig.kt b/src/main/kotlin/no/nav/familie/baks/mottak/config/featureToggle/FeatureToggleConfig.kt index e7108564c..f5ae7715b 100644 --- a/src/main/kotlin/no/nav/familie/baks/mottak/config/featureToggle/FeatureToggleConfig.kt +++ b/src/main/kotlin/no/nav/familie/baks/mottak/config/featureToggle/FeatureToggleConfig.kt @@ -8,6 +8,5 @@ class FeatureToggleConfig { // Release const val AUTOMATISK_JOURNALFØRING_AV_KONTANTSTØTTE_SØKNADER = "familie-baks-mottak.automatisk-journalforing-av-ks-soknad" const val AUTOMATISK_JOURNALFØRING_AV_BARNETRYGD_SØKNADER = "familie-baks-mottak.automatisk-journalforing-av-ba-soknad" - const val BRUK_ENHETSNUMMERSERVICE = "familie-baks-mottak.bruk-enhetsnummerservice" } } diff --git a/src/main/kotlin/no/nav/familie/baks/mottak/integrasjoner/AbstractOppgaveMapper.kt b/src/main/kotlin/no/nav/familie/baks/mottak/integrasjoner/AbstractOppgaveMapper.kt index 7d246c204..b73291679 100644 --- a/src/main/kotlin/no/nav/familie/baks/mottak/integrasjoner/AbstractOppgaveMapper.kt +++ b/src/main/kotlin/no/nav/familie/baks/mottak/integrasjoner/AbstractOppgaveMapper.kt @@ -1,7 +1,5 @@ package no.nav.familie.baks.mottak.integrasjoner -import no.nav.familie.baks.mottak.config.featureToggle.FeatureToggleConfig -import no.nav.familie.baks.mottak.journalføring.JournalpostBrukerService import no.nav.familie.baks.mottak.util.erDnummer import no.nav.familie.baks.mottak.util.erOrgnr import no.nav.familie.baks.mottak.util.fristFerdigstillelse @@ -14,18 +12,11 @@ import no.nav.familie.kontrakter.felles.oppgave.IdentGruppe import no.nav.familie.kontrakter.felles.oppgave.OppgaveIdentV2 import no.nav.familie.kontrakter.felles.oppgave.Oppgavetype import no.nav.familie.kontrakter.felles.oppgave.OpprettOppgaveRequest -import no.nav.familie.unleash.UnleashService -import org.slf4j.LoggerFactory import org.springframework.stereotype.Service -import java.util.Locale abstract class AbstractOppgaveMapper( - private val hentEnhetClient: HentEnhetClient, - private val unleashService: UnleashService, private val enhetsnummerService: EnhetsnummerService, private val pdlClient: PdlClient, - private val arbeidsfordelingClient: ArbeidsfordelingClient, - private val journalpostBrukerService: JournalpostBrukerService, ) : IOppgaveMapper { override fun tilOpprettOppgaveRequest( oppgavetype: Oppgavetype, @@ -42,12 +33,7 @@ abstract class AbstractOppgaveMapper( oppgavetype = oppgavetype, fristFerdigstillelse = fristFerdigstillelse(), beskrivelse = tilBeskrivelse(journalpost, beskrivelse), - enhetsnummer = - if (unleashService.isEnabled(FeatureToggleConfig.BRUK_ENHETSNUMMERSERVICE)) { - enhetsnummerService.hentEnhetsnummer(journalpost) - } else { - utledEnhetsnummer(journalpost) - }, + enhetsnummer = enhetsnummerService.hentEnhetsnummer(journalpost), behandlingstema = hentBehandlingstemaVerdi(journalpost), behandlingstype = hentBehandlingstypeVerdi(journalpost), ) @@ -121,34 +107,6 @@ abstract class AbstractOppgaveMapper( return "${journalpost.hentHovedDokumentTittel().orEmpty()} $bindestrek ${beskrivelse.orEmpty()}".trim() } - private fun utledEnhetsnummer(journalpost: Journalpost): String? = - when { - journalpost.journalforendeEnhet == "2101" -> "4806" // Enhet 2101 er nedlagt. Rutes til 4806 - journalpost.journalforendeEnhet == "4847" -> "4817" // Enhet 4847 skal legges ned. Rutes til 4817 - journalpost.harDigitalBarnetrygdSøknad() || journalpost.harDigitalKontantstøtteSøknad() -> hentBehandlendeEnhetForPerson(journalpost) - journalpost.journalforendeEnhet.isNullOrBlank() -> null - hentEnhetClient.hentEnhet(journalpost.journalforendeEnhet!!).status.uppercase(Locale.getDefault()) == "NEDLAGT" -> null - hentEnhetClient.hentEnhet(journalpost.journalforendeEnhet!!).oppgavebehandler -> journalpost.journalforendeEnhet - else -> { - logger.warn("Enhet ${journalpost.journalforendeEnhet} kan ikke ta i mot oppgaver") - null - } - } - - private fun hentBehandlendeEnhetForPerson(journalpost: Journalpost): String? { - val journalpostBruker = journalpost.bruker - - return if (journalpostBruker != null) { - val personIdentPåJournalpost = journalpostBrukerService.tilPersonIdent(journalpostBruker, this.tema) - val behandlendeEnhetPåIdent = arbeidsfordelingClient.hentBehandlendeEnhetPåIdent(personIdentPåJournalpost, this.tema) - - behandlendeEnhetPåIdent.enhetId - } else { - logger.warn("Fant ikke bruker på journalpost ved forsøk på henting av behandlende enhet") - null - } - } - private fun hentAktørIdFraPdl( brukerId: String, tema: Tema, @@ -176,10 +134,6 @@ abstract class AbstractOppgaveMapper( private fun validerJournalpost(journalpost: Journalpost) { if (journalpost.dokumenter.isNullOrEmpty()) error("Journalpost ${journalpost.journalpostId} mangler dokumenter") } - - companion object { - private val logger = LoggerFactory.getLogger(this::class.java) - } } interface IOppgaveMapper { diff --git a/src/main/kotlin/no/nav/familie/baks/mottak/integrasjoner/BarnetrygdOppgaveMapper.kt b/src/main/kotlin/no/nav/familie/baks/mottak/integrasjoner/BarnetrygdOppgaveMapper.kt index ebfd7a7dc..807172eb6 100644 --- a/src/main/kotlin/no/nav/familie/baks/mottak/integrasjoner/BarnetrygdOppgaveMapper.kt +++ b/src/main/kotlin/no/nav/familie/baks/mottak/integrasjoner/BarnetrygdOppgaveMapper.kt @@ -1,31 +1,21 @@ package no.nav.familie.baks.mottak.integrasjoner -import no.nav.familie.baks.mottak.journalføring.JournalpostBrukerService import no.nav.familie.baks.mottak.søknad.barnetrygd.domene.SøknadRepository import no.nav.familie.baks.mottak.søknad.barnetrygd.domene.harEøsSteg import no.nav.familie.kontrakter.felles.Behandlingstema import no.nav.familie.kontrakter.felles.Tema import no.nav.familie.kontrakter.felles.journalpost.Journalpost import no.nav.familie.kontrakter.felles.oppgave.Behandlingstype -import no.nav.familie.unleash.UnleashService import org.springframework.stereotype.Service @Service class BarnetrygdOppgaveMapper( - hentEnhetClient: HentEnhetClient, - unleashService: UnleashService, enhetsnummerService: EnhetsnummerService, - arbeidsfordelingClient: ArbeidsfordelingClient, pdlClient: PdlClient, - journalpostBrukerService: JournalpostBrukerService, val søknadRepository: SøknadRepository, ) : AbstractOppgaveMapper( - hentEnhetClient = hentEnhetClient, - unleashService = unleashService, enhetsnummerService = enhetsnummerService, pdlClient = pdlClient, - arbeidsfordelingClient = arbeidsfordelingClient, - journalpostBrukerService = journalpostBrukerService, ) { override val tema: Tema = Tema.BAR diff --git "a/src/main/kotlin/no/nav/familie/baks/mottak/integrasjoner/Kontantst\303\270tteOppgaveMapper.kt" "b/src/main/kotlin/no/nav/familie/baks/mottak/integrasjoner/Kontantst\303\270tteOppgaveMapper.kt" index ee5130912..beb90fb57 100644 --- "a/src/main/kotlin/no/nav/familie/baks/mottak/integrasjoner/Kontantst\303\270tteOppgaveMapper.kt" +++ "b/src/main/kotlin/no/nav/familie/baks/mottak/integrasjoner/Kontantst\303\270tteOppgaveMapper.kt" @@ -1,31 +1,21 @@ package no.nav.familie.baks.mottak.integrasjoner -import no.nav.familie.baks.mottak.journalføring.JournalpostBrukerService import no.nav.familie.baks.mottak.søknad.kontantstøtte.domene.KontantstøtteSøknadRepository import no.nav.familie.baks.mottak.søknad.kontantstøtte.domene.harEøsSteg import no.nav.familie.kontrakter.felles.Behandlingstema import no.nav.familie.kontrakter.felles.Tema import no.nav.familie.kontrakter.felles.journalpost.Journalpost import no.nav.familie.kontrakter.felles.oppgave.Behandlingstype -import no.nav.familie.unleash.UnleashService import org.springframework.stereotype.Service @Service class KontantstøtteOppgaveMapper( - hentEnhetClient: HentEnhetClient, - unleashService: UnleashService, enhetsnummerService: EnhetsnummerService, - arbeidsfordelingClient: ArbeidsfordelingClient, pdlClient: PdlClient, - journalpostBrukerService: JournalpostBrukerService, val kontantstøtteSøknadRepository: KontantstøtteSøknadRepository, ) : AbstractOppgaveMapper( - hentEnhetClient = hentEnhetClient, - unleashService = unleashService, enhetsnummerService = enhetsnummerService, pdlClient = pdlClient, - arbeidsfordelingClient = arbeidsfordelingClient, - journalpostBrukerService = journalpostBrukerService, ) { override val tema: Tema = Tema.KON diff --git a/src/test/kotlin/no/nav/familie/baks/mottak/integrasjoner/OppgaveMapperTest.kt b/src/test/kotlin/no/nav/familie/baks/mottak/integrasjoner/OppgaveMapperTest.kt index d7ed6c8ce..595d9782d 100644 --- a/src/test/kotlin/no/nav/familie/baks/mottak/integrasjoner/OppgaveMapperTest.kt +++ b/src/test/kotlin/no/nav/familie/baks/mottak/integrasjoner/OppgaveMapperTest.kt @@ -3,8 +3,6 @@ package no.nav.familie.baks.mottak.integrasjoner import io.mockk.every import io.mockk.mockk import no.nav.familie.baks.mottak.DevLauncher -import no.nav.familie.baks.mottak.config.featureToggle.FeatureToggleConfig -import no.nav.familie.baks.mottak.journalføring.JournalpostBrukerService import no.nav.familie.baks.mottak.søknad.barnetrygd.domene.SøknadRepository import no.nav.familie.baks.mottak.søknad.kontantstøtte.domene.KontantstøtteSøknadRepository import no.nav.familie.kontrakter.felles.Behandlingstema @@ -16,7 +14,6 @@ import no.nav.familie.kontrakter.felles.oppgave.Behandlingstype import no.nav.familie.kontrakter.felles.oppgave.IdentGruppe import no.nav.familie.kontrakter.felles.oppgave.OppgaveIdentV2 import no.nav.familie.kontrakter.felles.oppgave.Oppgavetype -import no.nav.familie.unleash.UnleashService import org.assertj.core.api.Assertions.assertThat import org.junit.jupiter.api.Assertions import org.junit.jupiter.api.BeforeEach @@ -39,38 +36,25 @@ class OppgaveMapperTest( @Autowired private val kontantstøtteSøknadRepository: KontantstøtteSøknadRepository, ) { - private val mockUnleashService: UnleashService = mockk() - private val mockHentEnhetClient: HentEnhetClient = mockk(relaxed = true) private val mockEnhetsnummerService: EnhetsnummerService = mockk() - private val mockArbeidsfordelingClient: ArbeidsfordelingClient = mockk(relaxed = true) - private val mockJournalpostBrukerService: JournalpostBrukerService = mockk() private val barnetrygdOppgaveMapper: IOppgaveMapper = BarnetrygdOppgaveMapper( - hentEnhetClient = mockHentEnhetClient, - unleashService = mockUnleashService, enhetsnummerService = mockEnhetsnummerService, pdlClient = mockPdlClient, søknadRepository = barnetrygdSøknadRepository, - arbeidsfordelingClient = mockArbeidsfordelingClient, - journalpostBrukerService = mockJournalpostBrukerService, ) private val kontantstøtteOppgaveMapper: IOppgaveMapper = KontantstøtteOppgaveMapper( - hentEnhetClient = mockHentEnhetClient, - unleashService = mockUnleashService, enhetsnummerService = mockEnhetsnummerService, pdlClient = mockPdlClient, kontantstøtteSøknadRepository = kontantstøtteSøknadRepository, - arbeidsfordelingClient = mockArbeidsfordelingClient, - journalpostBrukerService = mockJournalpostBrukerService, ) @BeforeEach fun beforeEach() { every { mockEnhetsnummerService.hentEnhetsnummer(any()) } returns "1234" - every { mockUnleashService.isEnabled(FeatureToggleConfig.BRUK_ENHETSNUMMERSERVICE) } returns true } @Test