Satt sluttdato på tilleggsorba i SatsService og skrevet tester #5173
Conversation
…lseHenleggelseTest skal fungere etter at TILLEGGS_ORBA er deaktivert
| val maksUtvidet = utvidet.maxBy { it.beløp }.beløp | ||
|
|
||
| return if (fagsakType == FagsakType.BARN_ENSLIG_MINDREÅRIG) { | ||
| maksOrdinærMedTillegg + maksUtvidet | ||
| } else { | ||
| when (personType) { | ||
| PersonType.BARN -> maksOrdinærMedTillegg | ||
| PersonType.BARN -> maxOf(maksOrdinær, maksOrdinærMedTillegg) |
There was a problem hiding this comment.
Denne returnerte alltid maksOrdinærMedTilleg før, for man antok at den var størst
| ORDINÆR_BARNETRYGD -> { | ||
| val sisteSatsdatoForTilleggsOrba = SatsService.finnSisteSatsFor(SatsType.TILLEGG_ORBA).gyldigTom | ||
| if (ytelseDato.isAfter(sisteSatsdatoForTilleggsOrba)) { | ||
| SatsType.ORBA |
There was a problem hiding this comment.
Alltid returner ORBA etter at siste dato for TILLEGG_ORBA
| assertPeriode(TestKrPeriode(beløp = 1054, fom = "2019-12", tom = "2020-08"), ordinærePerioder[0]) | ||
| assertPeriode(TestKrPeriode(beløp = 1354, fom = "2020-09", tom = "2021-08"), ordinærePerioder[1]) | ||
| assertPeriode(TestKrPeriode(beløp = 1654, fom = "2021-09", tom = "2021-12"), ordinærePerioder[2]) | ||
| assertPeriode(TestKrPeriode(beløp = 1676, fom = "2022-01", tom = "2023-02"), ordinærePerioder[3]) | ||
| assertPeriode(TestKrPeriode(beløp = 1723, fom = "2023-03", tom = "2023-06"), ordinærePerioder[4]) | ||
| assertPeriode(TestKrPeriode(beløp = 1766, fom = "2023-07", tom = null), ordinærePerioder[5]) |
There was a problem hiding this comment.
Hadde vært digg om vi her også kunne validert at rett SatsType ligger bak beløpet nå når siste sats for TILLEGS_ORBA og ORBA er den samme (1766), men ser at det ikke nødvendigvis er så lett å få til uten å skrive om lagOrdinærTidslinje og det er jeg ikke så sikker på at vi vil 😅 Så bare anse denne kommentaren som en observasjon 😛
There was a problem hiding this comment.
Jeg har egentlig alltid irritert meg over at sats i aty er et beløp i kr og ikke satstypen, men det er ikke så lett å endre
.../familie/ba/sak/kjerne/verdikjedetester/JournalførOgBehandleFørstegangssøknadNasjonalTest.kt
Show resolved
Hide resolved
...ester/kotlin/no/nav/familie/ba/sak/kjerne/verdikjedetester/FødselshendelseHenleggelseTest.kt
Show resolved
Hide resolved
|
github-merge-queue
bot
removed this pull request from the merge queue due to failed status checks
💰 Hva skal gjøres, og hvorfor?
NAV-22043
Satt riktig sluttdato på SatsType TILLEGG_ORBA. Oppdatert kode og tester til å ta hensyn til dette.
Verden ble veldig merkelig hvor satstidspunktet ble rart mocket i kombinasjon med utvidelsen i AndelTilkjentYtelse. Som var et $%#&!! å finne ut av. Løsningen var frustrende enkel å spole frem satstidspunket til siste satsendring.
🔎️ Er det noe spesielt du ønsker tilbakemelding om?
Noe jeg kan ha glemt?
✅ Checklist
Har du husket alle punktene i listen?
Jeg har ikke skrevet tester fordi:
💬 Ønsker du en muntlig gjennomgang?