TSFF-2229 Fix validering kontrollperiodeer #887
Conversation
| private static void validerPerioderForRapporterteInntekter(LocalDateTimeline<BigDecimal> rapportertInntektTidslinje, LocalDateTimeline<YearMonth> månedstidslinjeForYtelse) { | ||
| final var rapporterteInntekterSomIkkeMatcherYtelsesperiode = rapportertInntektTidslinje.stream().filter(s -> harIkkeMatchendeYtelseMåned(s, månedstidslinjeForYtelse)).toList(); | ||
|
|
||
| Set<YearMonth> rapporterteMåneder1 = lagMånedsTidslinje(rapportertInntektTidslinje) |
There was a problem hiding this comment.
Kvifor er dette nødvendig? rapportertInntektTidslinje skal vel allerede vere splittet på måned
There was a problem hiding this comment.
pga flytting av startdato fremover så blir hele fagsak/programperioden flyttet tilsvarende. Da blir det diff mellom det som er kontrollert og programperioden i siste periode. Tidligere krevde valideringen at periodene matchet 1-1. Det funker ikke for siste periode. Det funket før fordi siste periode var utelatt.
Vi var ikke helt sikre på hva som er formålet med denne valideringen. Er det for å ikke få kontrollerte perioder utenfor fagsakperioden? I så fall så sjekker vi at månedene for rapportert inntekt matcher med programperiodemånedene, ikke at periodene er identisk.
I tillegg til feilen i Jira så får man også feil i en/flere av testene til EndretUngdomsprogramTest i verdikjede hvor programperioden er over 2 måneder mener jeg å huske.
Behov / Bakgrunn
Løsning
Andre endringer
Skjermbilder (hvis relevant)