Skip to content

Commit

Permalink
Chore: optional andel tilkjent ytelse for å unngå mange logginnslag i…
Browse files Browse the repository at this point in the history
… sentry (#4200)

### 💰 Hva skal gjøres, og hvorfor?
Får mange av
[denne](https://sentry.gc.nav.no/organizations/nav/issues/520499/?project=112)
feilen i Sentry. Skyldes at saksbehandler kommer inn på
behandlingsresultat-siden når det ikke finnes tilkjent ytelse - feks ved
henlagt behandling. Sørger for at tilkjent ytelse kan være optional.
Hvis det ikke finnes tilkjent ytelse så sender vi bare inn en tom liste
med andeler.

### 🔎️ Er det noe spesielt du ønsker tilbakemelding om?
Nei

### ✅ 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:_
Tenker det er unødvendig

### 💬 Ønsker du en muntlig gjennomgang?
- [ ] Ja
- [x] Nei
  • Loading branch information
idaame authored Nov 21, 2023
1 parent 5dda244 commit 0bda63c
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 5 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -86,20 +86,22 @@ class TilkjentYtelseValideringService(
fun finnAktørerMedUgyldigEtterbetalingsperiode(
behandlingId: Long,
): List<Aktør> {
val tilkjentYtelse = beregningService.hentTilkjentYtelseForBehandling(behandlingId = behandlingId)
val tilkjentYtelse = beregningService.hentOptionalTilkjentYtelseForBehandling(behandlingId = behandlingId)

val behandling = behandlingHentOgPersisterService.hent(behandlingId)

val forrigeBehandling =
behandlingHentOgPersisterService.hentForrigeBehandlingSomErIverksatt(
behandling = behandlingHentOgPersisterService.hent(behandlingId),
behandling = behandling,
)
val forrigeAndelerTilkjentYtelse =
forrigeBehandling?.let { beregningService.hentOptionalTilkjentYtelseForBehandling(behandlingId = it.id) }
?.andelerTilkjentYtelse

return finnAktørIderMedUgyldigEtterbetalingsperiode(
forrigeAndelerTilkjentYtelse = forrigeAndelerTilkjentYtelse ?: emptyList(),
andelerTilkjentYtelse = tilkjentYtelse.andelerTilkjentYtelse.toList(),
kravDato = tilkjentYtelse.behandling.opprettetTidspunkt,
andelerTilkjentYtelse = tilkjentYtelse?.andelerTilkjentYtelse?.toList() ?: emptyList(),
kravDato = behandling.opprettetTidspunkt,
)
}

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -135,7 +135,7 @@ class TilkjentYtelseValideringServiceTest {
),
)

every { beregningServiceMock.hentTilkjentYtelseForBehandling(behandlingId = behandling.id) } answers { tilkjentYtelse }
every { beregningServiceMock.hentOptionalTilkjentYtelseForBehandling(behandlingId = behandling.id) } answers { tilkjentYtelse }
every { behandlingHentOgPersisterService.hent(behandlingId = behandling.id) } answers { behandling }
every { behandlingHentOgPersisterService.hentForrigeBehandlingSomErIverksatt(behandling = behandling) } answers { forrigeBehandling }
every { beregningServiceMock.hentOptionalTilkjentYtelseForBehandling(behandlingId = forrigeBehandling.id) } answers { forrigeTilkjentYtelse }
Expand Down

0 comments on commit 0bda63c

Please sign in to comment.