NAV-23413: Fjerner toggle for EnhetsnummerService (#1213)

navikt · Dec 9, 2024 · e9b64d7 · e9b64d7
1 parent 4e4fb67
commit e9b64d7
Show file tree

Hide file tree

Showing 5 changed files with 1 addition and 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
@@ -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
@@ -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
@@ -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øtteOppgaveMapper.kt b/src/main/kotlin/no/nav/familie/baks/mottak/integrasjoner/KontantstøtteOppgaveMapper.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
@@ -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