Skip to content

Commit

Permalink
Slett toggle etterbetaling 3 måneder (#4859)
Browse files Browse the repository at this point in the history
### 💰 Hva skal gjøres, og hvorfor?

Feature toggle ble skrudd på 1. oktober. Har vært skrudd på uten
problemer siden.

### ✅ Checklist
_Har du husket alle punktene i listen?_
- [ ] 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:_
Endrer ikke funksjonalitet fra 1. oktober

### 💬 Ønsker du en muntlig gjennomgang?
- [ ] Ja
- [x] Nei
  • Loading branch information
MagnusTonnessen authored Nov 4, 2024
1 parent 21401ae commit 26c9f0a
Show file tree
Hide file tree
Showing 6 changed files with 1 addition and 26 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -2,9 +2,6 @@ package no.nav.familie.ba.sak.config

class FeatureToggleConfig {
companion object {
// Release
const val ETTERBETALING_3_MND = "familie-ba-sak.etterbetaling-3-mnd"

// Operasjonelle
const val KAN_MANUELT_KORRIGERE_MED_VEDTAKSBREV = "familie-ba-sak.behandling.korreksjon-vedtaksbrev"
const val TEKNISK_VEDLIKEHOLD_HENLEGGELSE = "familie-ba-sak.teknisk-vedlikehold-henleggelse.tilgangsstyring"
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -2,8 +2,6 @@ package no.nav.familie.ba.sak.kjerne.beregning

import no.nav.familie.ba.sak.common.Feil
import no.nav.familie.ba.sak.common.secureLogger
import no.nav.familie.ba.sak.config.FeatureToggleConfig.Companion.ETTERBETALING_3_MND
import no.nav.familie.ba.sak.config.featureToggle.UnleashNextMedContextService
import no.nav.familie.ba.sak.kjerne.behandling.BehandlingHentOgPersisterService
import no.nav.familie.ba.sak.kjerne.behandling.domene.Behandling
import no.nav.familie.ba.sak.kjerne.beregning.TilkjentYtelseValidering.finnAktørIderMedUgyldigEtterbetalingsperiode
Expand All @@ -22,7 +20,6 @@ class TilkjentYtelseValideringService(
private val beregningService: BeregningService,
private val persongrunnlagService: PersongrunnlagService,
private val behandlingHentOgPersisterService: BehandlingHentOgPersisterService,
private val unleashNextMedContextService: UnleashNextMedContextService,
) {
fun validerAtIngenUtbetalingerOverstiger100Prosent(behandling: Behandling) {
if (behandling.erMigrering() || behandling.erTekniskEndring() || behandling.erSatsendring()) return
Expand Down Expand Up @@ -102,11 +99,7 @@ class TilkjentYtelseValideringService(
?.let { beregningService.hentOptionalTilkjentYtelseForBehandling(behandlingId = it.id) }
?.andelerTilkjentYtelse

val gyldigEtterbetalingFom =
when (unleashNextMedContextService.isEnabled(ETTERBETALING_3_MND)) {
true -> hentGyldigEtterbetaling3MndFom(behandling.opprettetTidspunkt.toLocalDate())
false -> hentGyldigEtterbetaling3ÅrFom(behandling.opprettetTidspunkt.toLocalDate())
}
val gyldigEtterbetalingFom = hentGyldigEtterbetaling3MndFom(behandling.opprettetTidspunkt.toLocalDate())

return finnAktørIderMedUgyldigEtterbetalingsperiode(
forrigeAndelerTilkjentYtelse = forrigeAndelerTilkjentYtelse ?: emptyList(),
Expand Down
Original file line number Diff line number Diff line change
@@ -1,8 +1,6 @@
package no.nav.familie.ba.sak.kjerne.endretutbetaling

import no.nav.familie.ba.sak.common.Feil
import no.nav.familie.ba.sak.config.FeatureToggleConfig.Companion.ETTERBETALING_3_MND
import no.nav.familie.ba.sak.config.featureToggle.UnleashNextMedContextService
import no.nav.familie.ba.sak.ekstern.restDomene.RestEndretUtbetalingAndel
import no.nav.familie.ba.sak.kjerne.behandling.domene.Behandling
import no.nav.familie.ba.sak.kjerne.beregning.BeregningService
Expand All @@ -13,7 +11,6 @@ import no.nav.familie.ba.sak.kjerne.endretutbetaling.EndretUtbetalingAndelValide
import no.nav.familie.ba.sak.kjerne.endretutbetaling.domene.EndretUtbetalingAndel
import no.nav.familie.ba.sak.kjerne.endretutbetaling.domene.EndretUtbetalingAndelRepository
import no.nav.familie.ba.sak.kjerne.endretutbetaling.domene.fraRestEndretUtbetalingAndel
import no.nav.familie.ba.sak.kjerne.endretutbetaling.domene.Årsak
import no.nav.familie.ba.sak.kjerne.eøs.felles.BehandlingId
import no.nav.familie.ba.sak.kjerne.grunnlag.personopplysninger.PersongrunnlagService
import no.nav.familie.ba.sak.kjerne.grunnlag.personopplysninger.PersonopplysningGrunnlagRepository
Expand All @@ -32,7 +29,6 @@ class EndretUtbetalingAndelService(
private val vilkårsvurderingService: VilkårsvurderingService,
private val endretUtbetalingAndelOppdatertAbonnementer: List<EndretUtbetalingAndelerOppdatertAbonnent> = emptyList(),
private val endretUtbetalingAndelHentOgPersisterService: EndretUtbetalingAndelHentOgPersisterService,
private val unleashMedContextService: UnleashNextMedContextService,
) {
@Transactional
fun oppdaterEndretUtbetalingAndelOgOppdaterTilkjentYtelse(
Expand All @@ -54,10 +50,6 @@ class EndretUtbetalingAndelService(

endretUtbetalingAndel.fraRestEndretUtbetalingAndel(restEndretUtbetalingAndel, person)

if (endretUtbetalingAndel.årsak == Årsak.ETTERBETALING_3MND && !unleashMedContextService.isEnabled(ETTERBETALING_3_MND)) {
throw Feil("Feature toggle for å opprette endret utbetaling med årsak etterbetaling 3 mnd er ikke skrudd på")
}

val andreEndredeAndelerPåBehandling =
endretUtbetalingAndelHentOgPersisterService
.hentForBehandling(behandling.id)
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,6 @@ import no.nav.familie.ba.sak.common.lagAndelTilkjentYtelse
import no.nav.familie.ba.sak.common.lagBehandling
import no.nav.familie.ba.sak.common.lagPerson
import no.nav.familie.ba.sak.common.tilfeldigPerson
import no.nav.familie.ba.sak.config.featureToggle.UnleashNextMedContextService
import no.nav.familie.ba.sak.kjerne.behandling.BehandlingHentOgPersisterService
import no.nav.familie.ba.sak.kjerne.beregning.domene.TilkjentYtelse
import no.nav.familie.ba.sak.kjerne.grunnlag.personopplysninger.PersonType
Expand All @@ -23,7 +22,6 @@ class TilkjentYtelseValideringServiceTest {
private val beregningServiceMock = mockk<BeregningService>()
private val totrinnskontrollServiceMock = mockk<TotrinnskontrollService>()
private val persongrunnlagServiceMock = mockk<PersongrunnlagService>()
private val unleashNextMedContextService = mockk<UnleashNextMedContextService>()

private lateinit var tilkjentYtelseValideringService: TilkjentYtelseValideringService

Expand All @@ -35,7 +33,6 @@ class TilkjentYtelseValideringServiceTest {
totrinnskontrollService = totrinnskontrollServiceMock,
persongrunnlagService = persongrunnlagServiceMock,
behandlingHentOgPersisterService = behandlingHentOgPersisterService,
unleashNextMedContextService = unleashNextMedContextService,
)

every {
Expand Down Expand Up @@ -142,7 +139,6 @@ class TilkjentYtelseValideringServiceTest {
every { behandlingHentOgPersisterService.hent(behandlingId = behandling.id) } answers { behandling }
every { behandlingHentOgPersisterService.hentForrigeBehandlingSomErIverksatt(behandling = behandling) } answers { forrigeBehandling }
every { beregningServiceMock.hentOptionalTilkjentYtelseForBehandling(behandlingId = forrigeBehandling.id) } answers { forrigeTilkjentYtelse }
every { unleashNextMedContextService.isEnabled(any()) } returns true

Assertions.assertTrue(tilkjentYtelseValideringService.finnAktørerMedUgyldigEtterbetalingsperiode(behandlingId = behandling.id).size == 1)
Assertions.assertEquals(
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -52,7 +52,6 @@ class EndretUtbetalingAndelServiceTest {
andelTilkjentYtelseRepository = mockAndelTilkjentYtelseRepository,
vilkårsvurderingService = mockVilkårsvurderingService,
endretUtbetalingAndelHentOgPersisterService = mockEndretUtbetalingAndelHentOgPersisterService,
unleashMedContextService = mockk(),
)
}

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -363,7 +363,6 @@ class CucumberMock(
totrinnskontrollService = totrinnskontrollService,
persongrunnlagService = persongrunnlagService,
behandlingHentOgPersisterService = behandlingHentOgPersisterService,
unleashNextMedContextService = unleashNextMedContextService,
)

val utbetalingsoppdragGeneratorService =
Expand Down Expand Up @@ -438,7 +437,6 @@ class CucumberMock(
vilkårsvurderingService = vilkårsvurderingService,
endretUtbetalingAndelOppdatertAbonnementer = emptyList(),
endretUtbetalingAndelHentOgPersisterService = endretUtbetalingAndelHentOgPersisterService,
unleashMedContextService = unleashNextMedContextService,
)

val vilkårsvurderingForNyBehandlingService =
Expand Down

0 comments on commit 26c9f0a

Please sign in to comment.