Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fjern toggle familie-ks-sak.allerede-utbetalt #1014

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Original file line number Diff line number Diff line change
Expand Up @@ -18,6 +18,5 @@ class FeatureToggleConfig {
const val BRUK_NY_LØYPE_FOR_GENERERING_AV_ANDELER = "familie-ks-sak.bruk-ny-loype-for-generering-av-andeler"

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"
}
}
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,6 @@ import no.nav.familie.ks.sak.common.util.MånedPeriode
import no.nav.familie.ks.sak.common.util.førsteDagIInneværendeMåned
import no.nav.familie.ks.sak.common.util.overlapperHeltEllerDelvisMed
import no.nav.familie.ks.sak.common.util.sisteDagIInneværendeMåned
import no.nav.familie.ks.sak.config.featureToggle.FeatureToggleConfig
import no.nav.familie.ks.sak.kjerne.behandling.steg.vilkårsvurdering.domene.VilkårsvurderingRepository
import no.nav.familie.ks.sak.kjerne.beregning.domene.AndelTilkjentYtelse
import no.nav.familie.ks.sak.kjerne.beregning.domene.AndelTilkjentYtelseRepository
Expand All @@ -15,7 +14,6 @@ import no.nav.familie.tidslinje.Periode
import no.nav.familie.tidslinje.Tidslinje
import no.nav.familie.tidslinje.tilTidslinje
import no.nav.familie.tidslinje.utvidelser.slåSammen
import no.nav.familie.unleash.UnleashService
import org.springframework.stereotype.Service
import org.springframework.transaction.annotation.Transactional
import java.math.BigDecimal
Expand All @@ -26,7 +24,6 @@ class AndelerTilkjentYtelseOgEndreteUtbetalingerService(
private val andelTilkjentYtelseRepository: AndelTilkjentYtelseRepository,
private val endretUtbetalingAndelRepository: EndretUtbetalingAndelRepository,
private val vilkårsvurderingRepository: VilkårsvurderingRepository,
private val unleashService: UnleashService,
) {
@Transactional
fun finnAndelerTilkjentYtelseMedEndreteUtbetalinger(behandlingId: Long): List<AndelTilkjentYtelseMedEndreteUtbetalinger> =
Expand All @@ -46,7 +43,6 @@ class AndelerTilkjentYtelseOgEndreteUtbetalingerService(
årsak = endretUtbetalingAndelMedAndelTilkjentYtelse.årsak,
endretUtbetalingAndel = endretUtbetalingAndelMedAndelTilkjentYtelse.endretUtbetaling,
vilkårsvurdering = vilkårsvurderingRepository.finnAktivForBehandling(behandlingId),
kanBrukeÅrsakAlleredeUtbetalt = unleashService.isEnabled(FeatureToggleConfig.ALLEREDE_UTBETALT_SOM_ENDRINGSÅRSAK),
)
}
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,6 @@ package no.nav.familie.ks.sak.kjerne.endretutbetaling
import no.nav.familie.ks.sak.api.dto.EndretUtbetalingAndelRequestDto
import no.nav.familie.ks.sak.api.dto.SanityBegrunnelseMedEndringsårsakResponseDto
import no.nav.familie.ks.sak.common.BehandlingId
import no.nav.familie.ks.sak.config.featureToggle.FeatureToggleConfig
import no.nav.familie.ks.sak.integrasjon.sanity.SanityService
import no.nav.familie.ks.sak.integrasjon.sanity.domene.SanityBegrunnelse
import no.nav.familie.ks.sak.kjerne.behandling.domene.Behandling
Expand All @@ -24,7 +23,6 @@ import no.nav.familie.ks.sak.kjerne.endretutbetaling.domene.EndretUtbetalingAnde
import no.nav.familie.ks.sak.kjerne.endretutbetaling.domene.EndretUtbetalingAndelRepository
import no.nav.familie.ks.sak.kjerne.endretutbetaling.domene.fraEndretUtbetalingAndelRequestDto
import no.nav.familie.ks.sak.kjerne.personopplysninggrunnlag.PersonopplysningGrunnlagService
import no.nav.familie.unleash.UnleashService
import org.springframework.stereotype.Service
import org.springframework.transaction.annotation.Transactional
import java.math.BigDecimal
Expand All @@ -38,7 +36,6 @@ class EndretUtbetalingAndelService(
private val vilkårsvurderingService: VilkårsvurderingService,
private val endretUtbetalingAndelOppdatertAbonnementer: List<EndretUtbetalingAndelerOppdatertAbonnent> = emptyList(),
private val sanityService: SanityService,
private val unleashService: UnleashService,
) {
fun hentEndredeUtbetalingAndeler(behandlingId: Long) = endretUtbetalingAndelRepository.hentEndretUtbetalingerForBehandling(behandlingId)

Expand Down Expand Up @@ -82,7 +79,6 @@ class EndretUtbetalingAndelService(
årsak = endretUtbetalingAndel.årsak,
endretUtbetalingAndel = endretUtbetalingAndel,
vilkårsvurdering = vilkårsvurderingService.hentAktivVilkårsvurderingForBehandling(behandlingId = behandling.id),
kanBrukeÅrsakAlleredeUtbetalt = unleashService.isEnabled(FeatureToggleConfig.ALLEREDE_UTBETALT_SOM_ENDRINGSÅRSAK),
)

validerUtbetalingMotÅrsak(
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -61,7 +61,6 @@ object EndretUtbetalingAndelValidator {
årsak: Årsak?,
endretUtbetalingAndel: EndretUtbetalingAndel,
vilkårsvurdering: Vilkårsvurdering?,
kanBrukeÅrsakAlleredeUtbetalt: Boolean = false,
) {
checkNotNull(årsak) { "Årsak kan ikke være null" }
when (årsak) {
Expand Down Expand Up @@ -90,7 +89,7 @@ object EndretUtbetalingAndelValidator {
)
}

Årsak.ALLEREDE_UTBETALT -> validerAlleredeUtbetalt(endretUtbetalingAndel = endretUtbetalingAndel, kanBrukeÅrsakAlleredeUtbetalt)
Årsak.ALLEREDE_UTBETALT -> validerAlleredeUtbetalt(endretUtbetalingAndel = endretUtbetalingAndel)

Årsak.ENDRE_MOTTAKER,
->
Expand All @@ -110,11 +109,7 @@ object EndretUtbetalingAndelValidator {

private fun validerAlleredeUtbetalt(
endretUtbetalingAndel: EndretUtbetalingAndel,
kanBrukeÅrsakAlleredeUtbetalt: Boolean,
) {
if (!kanBrukeÅrsakAlleredeUtbetalt) {
throw FunksjonellFeil("Årsak Allerede utbetalt er ikke implementert enda!!")
}
if (endretUtbetalingAndel.tom?.isAfter(YearMonth.now()) == true) {
throw FunksjonellFeil("Du har valgt årsaken allerede utbetalt. Du kan ikke velge denne årsaken og en til og med dato frem i tid. Ta kontakt med superbruker om du er usikker på hva du skal gjøre.")
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -75,7 +75,6 @@ class CucumberMock(
andelTilkjentYtelseRepository = andelTilkjentYtelseRepositoryMock,
endretUtbetalingAndelRepository = endretUtbetalingAndelRepositoryMock,
vilkårsvurderingRepository = vilkårsvurderingRepositoryMock,
unleashService = mockUnleashService(isEnabledDefault = false),
)

val personidentService =
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -21,11 +21,9 @@ import no.nav.familie.ks.sak.kjerne.beregning.domene.AndelTilkjentYtelseReposito
import no.nav.familie.ks.sak.kjerne.beregning.domene.maksBeløp
import no.nav.familie.ks.sak.kjerne.endretutbetaling.domene.EndretUtbetalingAndelRepository
import no.nav.familie.ks.sak.kjerne.personopplysninggrunnlag.domene.PersonType
import no.nav.familie.unleash.UnleashService
import org.junit.jupiter.api.Assertions.assertEquals
import org.junit.jupiter.api.Assertions.assertNotNull
import org.junit.jupiter.api.Assertions.assertTrue
import org.junit.jupiter.api.BeforeEach
import org.junit.jupiter.api.Test
import org.junit.jupiter.api.extension.ExtendWith
import java.math.BigDecimal
Expand All @@ -42,17 +40,9 @@ class AndelerTilkjentYtelseOgEndreteUtbetalingerServiceTest {
@MockK
private lateinit var vilkårsvurderingRepository: VilkårsvurderingRepository

@MockK
private lateinit var unleashService: UnleashService

@InjectMockKs
private lateinit var andelerTilkjentYtelseOgEndreteUtbetalingerService: AndelerTilkjentYtelseOgEndreteUtbetalingerService

@BeforeEach
fun setup() {
every { unleashService.isEnabled(any()) } returns false
}

val søker = randomAktør()
private val barn1 = randomAktør()

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -766,30 +766,6 @@ class EndretUtbetalingAndelValidatorTest {
)
}

@Test
fun `skal kaste feil når årsak er ALLEREDE_UTBETALT og unleash-togglen er skrudd av`() {
// Arrange
val endretUtbetalingAndel =
lagEndretUtbetalingAndel(
periodeFom = YearMonth.of(2024, 8),
periodeTom = YearMonth.of(2024, 8),
årsak = Årsak.ALLEREDE_UTBETALT,
)

// Act & assert
val exception =
assertThrows<FunksjonellFeil> {
EndretUtbetalingAndelValidator.validerÅrsak(
Årsak.ALLEREDE_UTBETALT,
endretUtbetalingAndel,
null,
kanBrukeÅrsakAlleredeUtbetalt = false,
)
}

assertThat(exception.message).isEqualTo("Årsak Allerede utbetalt er ikke implementert enda!!")
}

@Test
fun `skal ikke kaste feil når årsak er ALLEREDE_UTBETALT`() {
// Arrange
Expand All @@ -806,7 +782,6 @@ class EndretUtbetalingAndelValidatorTest {
Årsak.ALLEREDE_UTBETALT,
endretUtbetalingAndel,
null,
kanBrukeÅrsakAlleredeUtbetalt = true,
)
}
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,6 @@ import no.nav.familie.ks.sak.kjerne.endretutbetaling.domene.EndretUtbetalingAnde
import no.nav.familie.ks.sak.kjerne.endretutbetaling.domene.EndretUtbetalingAndelRepository
import no.nav.familie.ks.sak.kjerne.endretutbetaling.domene.Årsak
import no.nav.familie.ks.sak.kjerne.personopplysninggrunnlag.PersonopplysningGrunnlagService
import no.nav.familie.unleash.UnleashService
import org.hamcrest.CoreMatchers.nullValue
import org.hamcrest.MatcherAssert.assertThat
import org.junit.jupiter.api.Test
Expand Down Expand Up @@ -55,9 +54,6 @@ class EndretUtbetalingAndelServiceTest {
@MockK
private lateinit var sanityService: SanityService

@MockK
private lateinit var unleashService: UnleashService

@Test
fun `kopierEndretUtbetalingAndelFraForrigeBehandling - skal kopiere over endrete utbetaling i forrige behandling og lagre disse på ny`() {
val gammelBehandling = lagBehandling()
Expand Down
Loading