Skip to content

Commit

Permalink
Slett toggle for å fylle inn sb navn i godkjenne vedtak-oppgave (#1052)
Browse files Browse the repository at this point in the history
### 💰 Hva skal gjøres, og hvorfor?
Sletter toggle for å fylle inn sb navn i godkjenne vedtak-oppgave. Har
kjørt fint i prod en uke.

### 🔎️ Er det noe spesielt du ønsker tilbakemelding om?
Nei

### ✅ Checklist
_Har du husket alle punktene i listen?_
- [x] Jeg har testet mine endringer i henhold til akseptansekriteriene
🕵️
- [ ] Jeg har config- eller sql-endringer. I så fall, husk manuell
deploy til miljø for å verifisere endringene.
- [ ] Jeg har skrevet tester. Hvis du ikke har skrevet tester, beskriv
hvorfor under 👇

_Jeg har ikke skrevet tester fordi:_
Gir ikke mening

### 💬 Ønsker du en muntlig gjennomgang?
- [ ] Ja
- [x] Nei
  • Loading branch information
idaame authored Jan 28, 2025
1 parent f439be9 commit a39cc28
Show file tree
Hide file tree
Showing 3 changed files with 1 addition and 11 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -16,6 +16,5 @@ class FeatureToggleConfig {

const val BRUK_OMSKRIVING_AV_HJEMLER_I_BREV = "familie-ks-sak.bruk_omskriving_av_hjemler_i_brev"
const val ALLEREDE_UTBETALT_SOM_ENDRINGSÅRSAK = "familie-ks-sak.allerede-utbetalt"
const val FYLL_INN_SB_NAVN_I_GODKJENNE_VEDTAK_OPPGAVE = "familie-ks-sak.sb-navn-i-godkjenne-vedtak-oppgave"
}
}
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,6 @@ package no.nav.familie.ks.sak.kjerne.behandling.steg.vedtak

import no.nav.familie.kontrakter.felles.oppgave.Oppgavetype
import no.nav.familie.ks.sak.common.exception.Feil
import no.nav.familie.ks.sak.config.featureToggle.FeatureToggleConfig.Companion.FYLL_INN_SB_NAVN_I_GODKJENNE_VEDTAK_OPPGAVE
import no.nav.familie.ks.sak.integrasjon.oppgave.FerdigstillOppgaverTask
import no.nav.familie.ks.sak.integrasjon.oppgave.OppgaveService
import no.nav.familie.ks.sak.integrasjon.oppgave.OpprettOppgaveTask
Expand All @@ -19,7 +18,6 @@ import no.nav.familie.ks.sak.kjerne.logg.LoggService
import no.nav.familie.ks.sak.kjerne.totrinnskontroll.TotrinnskontrollService
import no.nav.familie.ks.sak.sikkerhet.SikkerhetContext
import no.nav.familie.prosessering.internal.TaskService
import no.nav.familie.unleash.UnleashService
import org.slf4j.Logger
import org.slf4j.LoggerFactory
import org.springframework.stereotype.Service
Expand All @@ -35,7 +33,6 @@ class VedtakSteg(
private val oppgaveService: OppgaveService,
private val vedtakService: VedtakService,
private val vedtaksperiodeService: VedtaksperiodeService,
private val unleashService: UnleashService,
) : IBehandlingSteg {
override fun getBehandlingssteg(): BehandlingSteg = BehandlingSteg.VEDTAK

Expand All @@ -58,7 +55,7 @@ class VedtakSteg(
behandlingId = behandling.id,
oppgavetype = Oppgavetype.GodkjenneVedtak,
fristForFerdigstillelse = LocalDate.now(),
beskrivelse = if (unleashService.isEnabled(FYLL_INN_SB_NAVN_I_GODKJENNE_VEDTAK_OPPGAVE)) SikkerhetContext.hentSaksbehandlerNavn() else null,
beskrivelse = SikkerhetContext.hentSaksbehandlerNavn(),
)

taskService.save(godkjenneVedtakTask)
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,6 @@ import io.mockk.mockk
import io.mockk.runs
import io.mockk.verify
import no.nav.familie.ks.sak.common.exception.Feil
import no.nav.familie.ks.sak.config.featureToggle.FeatureToggleConfig
import no.nav.familie.ks.sak.data.lagBehandling
import no.nav.familie.ks.sak.integrasjon.oppgave.OppgaveService
import no.nav.familie.ks.sak.kjerne.behandling.BehandlingService
Expand All @@ -21,7 +20,6 @@ import no.nav.familie.ks.sak.kjerne.behandling.steg.vedtak.vedtaksperiode.Vedtak
import no.nav.familie.ks.sak.kjerne.logg.LoggService
import no.nav.familie.ks.sak.kjerne.totrinnskontroll.TotrinnskontrollService
import no.nav.familie.prosessering.internal.TaskService
import no.nav.familie.unleash.UnleashService
import org.hamcrest.MatcherAssert.assertThat
import org.junit.jupiter.api.Test
import org.junit.jupiter.api.assertThrows
Expand Down Expand Up @@ -53,9 +51,6 @@ class VedtakStegTest {
@MockK
private lateinit var vedtaksperiodeService: VedtaksperiodeService

@MockK
private lateinit var unleashService: UnleashService

@InjectMockKs
private lateinit var vedtakSteg: VedtakSteg

Expand Down Expand Up @@ -109,7 +104,6 @@ class VedtakStegTest {
every { vedtakService.hentAktivVedtakForBehandling(any()) } returns mockk(relaxed = true)
every { vedtaksperiodeService.hentUtvidetVedtaksperioderMedBegrunnelser(any()) } returns mockk(relaxed = true)
every { vedtakService.oppdaterVedtakMedDatoOgStønadsbrev(any()) } returns mockk()
every { unleashService.isEnabled(FeatureToggleConfig.FYLL_INN_SB_NAVN_I_GODKJENNE_VEDTAK_OPPGAVE) } returns true

vedtakSteg.utførSteg(200)

Expand Down

0 comments on commit a39cc28

Please sign in to comment.