Nav 24098 tilpasser opphørsperiode for lovverk 2025 #1064
Conversation
…siste utbetalingsperiode pga. fremtidig opphør
Oppdaterer Testdata -> lagVilkårsvurdering til å bruke Set()
| @@ -127,18 +135,21 @@ private fun finnOpphørsperiodeEtterSisteUtbetalingsperiode( | |||
|
|
|||
| val erFramtidigOpphørPgaBarnehageplass = | |||
| vilkårsvurdering.personResultater.any { personResultat -> | |||
There was a problem hiding this comment.
Denne og på linje 142. Skal det fortsatt være any()? Nå vil den bare gi første og beste treff, men hva hvis det er flere barn med forskjellige fødselsdatoer? F.eks at barn1 treffer gammelt lovverk og barn2 treffer nytt lovverk.
There was a problem hiding this comment.
Den variabelen brukes til å sette begrunnelse, og den skal vel gjelde for alle lovverk
| emptyList(), | ||
| ) | ||
| @Test | ||
| fun `skal mappe opphørte perioder for forskjellige barn som treffer før og etter lovverk feb 2025`() { |
There was a problem hiding this comment.
Gir denne testen i det hele tatt mening? Ref. kommentar med any() så får man vel ikke testet dette på noen måte.
There was a problem hiding this comment.
Lovverk 2024 og lovverk 2025 gir forskjellige opphørsperioder, så denne testen vil feile uten logikken dere har lagt til, fordi man bare får én opphørsperiode
|
GardOS
deleted the
NAV_24098_tilpasser_opphørsperiode_for_lovverk_2025
branch
💰 Hva skal gjøres, og hvorfor?
Favro: https://favro.com/organization/98c34fb974ce445eac854de0/1844bbac3b6605eacc8f5543?card=NAV-24098
Oppdaterer
mapTilOpphørsperiodertil å ta høyde for ny lovverksendring.🔎️ Er det noe spesielt du ønsker tilbakemelding om?
Se kommentarer i PR.
✅ Checklist
Har du husket alle punktene i listen?
💬 Ønsker du en muntlig gjennomgang?