Skip to content

Commit

Permalink
Merge pull request #395 from navikt/feature/rund-av-barnetillegg
Browse files Browse the repository at this point in the history
Lagt til avrunding av beløp for barnetillegg. Har også fjernet overst…
  • Loading branch information
rinnan17 authored Dec 6, 2024
2 parents 45fd7b4 + 1f1eb6d commit 35744b8
Show file tree
Hide file tree
Showing 8 changed files with 11 additions and 54 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -11,6 +11,7 @@ import no.nav.bidrag.grunnlag.util.GrunnlagUtil.Companion.evaluerFeilmelding
import no.nav.bidrag.grunnlag.util.GrunnlagUtil.Companion.evaluerFeiltype
import no.nav.bidrag.transport.behandling.grunnlag.response.BarnetilleggGrunnlagDto
import no.nav.bidrag.transport.behandling.grunnlag.response.FeilrapporteringDto
import java.math.RoundingMode

class HentBarnetilleggService(private val pensjonConsumer: PensjonConsumer) {

Expand Down Expand Up @@ -73,7 +74,7 @@ class HentBarnetilleggService(private val pensjonConsumer: PensjonConsumer) {
barnetilleggType = Barnetilleggstype.PENSJON.toString(),
periodeFra = it.fom,
periodeTil = it.tom.plusMonths(1).withDayOfMonth(1) ?: null,
beløpBrutto = it.beloep,
beløpBrutto = it.beloep.setScale(0, RoundingMode.HALF_UP),
barnType = if (it.erFellesbarn) BarnType.FELLES.toString() else BarnType.SÆRKULL.toString(),
),
)
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -41,7 +41,6 @@ import org.springframework.stereotype.Service
import java.math.BigDecimal
import java.time.LocalDate
import java.time.LocalDateTime
import java.time.Period

@Service
class HentGrunnlagService(
Expand All @@ -60,15 +59,6 @@ class HentGrunnlagService(
companion object {
@JvmStatic
val LOGGER: Logger = LoggerFactory.getLogger(HentGrunnlagService::class.java)

fun sjekkOgJusterDato(dato: LocalDate): LocalDate {
// Hvis dato er mer enn 5 år tilbake i tid, settes den til dagens dato minus 5 år
return if (Period.between(dato, LocalDate.now()).years >= 5) {
LocalDate.now().minusYears(5)
} else {
dato
}
}
}

suspend fun hentGrunnlag(hentGrunnlagRequestDto: HentGrunnlagRequestDto): HentGrunnlagDto {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,6 @@ import no.nav.bidrag.grunnlag.consumer.familiebasak.FamilieBaSakConsumer
import no.nav.bidrag.grunnlag.consumer.familiebasak.api.FamilieBaSakRequest
import no.nav.bidrag.grunnlag.consumer.familiebasak.api.FamilieBaSakResponse
import no.nav.bidrag.grunnlag.exception.RestResponse
import no.nav.bidrag.grunnlag.service.HentGrunnlagService.Companion.sjekkOgJusterDato
import no.nav.bidrag.grunnlag.util.GrunnlagUtil.Companion.evaluerFeilmelding
import no.nav.bidrag.grunnlag.util.GrunnlagUtil.Companion.evaluerFeiltype
import no.nav.bidrag.transport.behandling.grunnlag.response.FeilrapporteringDto
Expand All @@ -19,10 +18,9 @@ class HentUtvidetBarnetrygdOgSmåbarnstilleggService(private val familieBaSakCon
val feilrapporteringListe = mutableListOf<FeilrapporteringDto>()

ubstRequestListe.forEach {
// fradato sjekkes og justeres hvis den er lenger enn 5 år tilbake i tid. Settes da til dagens dato minus 5 år.
val hentUbstRequest = FamilieBaSakRequest(
personIdent = it.personId,
fraDato = sjekkOgJusterDato(it.periodeFra),
fraDato = it.periodeFra,
)

when (
Expand Down
4 changes: 2 additions & 2 deletions src/test/kotlin/no/nav/bidrag/grunnlag/TestUtil.kt
Original file line number Diff line number Diff line change
Expand Up @@ -888,14 +888,14 @@ class TestUtil {
fun byggHentBarnetilleggPensjonResponse() = listOf(
BarnetilleggPensjon(
barn = "barnIdent",
beloep = BigDecimal.valueOf(1000.11),
beloep = BigDecimal.valueOf(1001.45),
fom = LocalDate.parse("2021-01-01"),
tom = LocalDate.parse("2021-12-31"),
erFellesbarn = true,
),
BarnetilleggPensjon(
barn = "barnIdent",
beloep = BigDecimal.valueOf(2000.22),
beloep = BigDecimal.valueOf(2002.50),
fom = LocalDate.parse("2022-01-01"),
tom = LocalDate.parse("2022-12-31"),
erFellesbarn = true,
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -6,8 +6,6 @@ import no.nav.bidrag.grunnlag.consumer.familiebasak.FamilieBaSakConsumer
import no.nav.bidrag.grunnlag.consumer.familiebasak.api.BisysStønadstype
import no.nav.bidrag.grunnlag.consumer.familiebasak.api.FamilieBaSakResponse
import no.nav.bidrag.grunnlag.exception.RestResponse
import no.nav.bidrag.grunnlag.service.HentGrunnlagService
import no.nav.bidrag.grunnlag.service.PersonIdOgPeriodeRequest
import org.assertj.core.api.Assertions.assertThat
import org.assertj.core.api.Assertions.fail
import org.junit.jupiter.api.Assertions.assertAll
Expand All @@ -28,7 +26,6 @@ import org.springframework.http.HttpStatus
import org.springframework.http.MediaType
import org.springframework.http.ResponseEntity
import org.springframework.web.client.HttpClientErrorException
import java.time.LocalDate
import java.time.YearMonth

@ExtendWith(MockitoExtension::class)
Expand Down Expand Up @@ -114,35 +111,6 @@ internal class FamilieBaSakConsumerTest {
return HttpEntity(body, httpHeaders)
}

@Test
fun `Skal justere fradato til dagens dato minus 5 år`() {
// Sjekker først at eldre dato blir justert
var ubstRequest = PersonIdOgPeriodeRequest(
personId = "personident",
periodeFra = LocalDate.parse("2013-01-01"),
periodeTil = LocalDate.parse("2024-01-01"),
)

val justertDato = HentGrunnlagService.sjekkOgJusterDato(ubstRequest.periodeFra)

assertAll(
{ assertThat(justertDato).isEqualTo(LocalDate.now().minusYears(5)) },
)

// Sjekker at nyere dato ikke blir justert
ubstRequest = PersonIdOgPeriodeRequest(
personId = "personident",
periodeFra = LocalDate.parse("2033-01-01"),
periodeTil = LocalDate.parse("2024-01-01"),
)

val justertDato2 = HentGrunnlagService.sjekkOgJusterDato(ubstRequest.periodeFra)

assertAll(
{ assertThat(justertDato2).isEqualTo(ubstRequest.periodeFra) },
)
}

companion object {
private const val FAMILIEBASAK_CONTEXT = "/api/bisys/hent-utvidet-barnetrygd"
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -56,9 +56,9 @@ internal class PensjonConsumerTest {
{ assertThat(hentBarnetilleggPensjonResponse).isNotNull },
{ assertThat(hentBarnetilleggPensjonResponse.size).isEqualTo(2) },
{ assertThat(hentBarnetilleggPensjonResponse[0].barn).isEqualTo("barnIdent") },
{ assertThat(hentBarnetilleggPensjonResponse[0].beloep).isEqualTo(BigDecimal.valueOf(1000.11)) },
{ assertThat(hentBarnetilleggPensjonResponse[0].beloep).isEqualTo(BigDecimal.valueOf(1001.45)) },
{ assertThat(hentBarnetilleggPensjonResponse[1].barn).isEqualTo("barnIdent") },
{ assertThat(hentBarnetilleggPensjonResponse[1].beloep).isEqualTo(BigDecimal.valueOf(2000.22)) },
{ assertThat(hentBarnetilleggPensjonResponse[1].beloep).isEqualTo(BigDecimal.valueOf(2002.50)) },
)
}

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -44,8 +44,8 @@ class HentBarnetilleggServiceMockTest {
{ assertThat(barnetilleggPensjonListe).isNotNull() },
{ assertThat(barnetilleggPensjonListe.grunnlagListe).isNotEmpty() },
{ assertThat(barnetilleggPensjonListe.grunnlagListe).hasSize(2) },
{ assertThat(barnetilleggPensjonListe.grunnlagListe[0].beløpBrutto).isEqualTo(BigDecimal.valueOf(1000.11)) },
{ assertThat(barnetilleggPensjonListe.grunnlagListe[1].beløpBrutto).isEqualTo(BigDecimal.valueOf(2000.22)) },
{ assertThat(barnetilleggPensjonListe.grunnlagListe[0].beløpBrutto).isEqualTo(BigDecimal.valueOf(1001)) },
{ assertThat(barnetilleggPensjonListe.grunnlagListe[1].beløpBrutto).isEqualTo(BigDecimal.valueOf(2003)) },
{ assertThat(barnetilleggPensjonListe.feilrapporteringListe).isEmpty() },
)
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -322,7 +322,7 @@ class OppdaterGrunnlagspakkeServiceTest {
},
{ Assertions.assertThat(barnetilleggListe[0].periodeFra).isEqualTo(LocalDate.parse("2021-01-01")) },
{ Assertions.assertThat(barnetilleggListe[0].periodeTil).isEqualTo(LocalDate.parse("2022-01-01")) },
{ Assertions.assertThat(barnetilleggListe[0].belopBrutto).isEqualTo(BigDecimal.valueOf(1000.11)) },
{ Assertions.assertThat(barnetilleggListe[0].belopBrutto).isEqualTo(BigDecimal.valueOf(1001.45)) },
{ Assertions.assertThat(barnetilleggListe[0].barnType).isEqualTo(BarnType.FELLES.toString()) },

{ Assertions.assertThat(barnetilleggListe[1].partPersonId).isEqualTo("12345678910") },
Expand All @@ -333,7 +333,7 @@ class OppdaterGrunnlagspakkeServiceTest {
},
{ Assertions.assertThat(barnetilleggListe[1].periodeFra).isEqualTo(LocalDate.parse("2022-01-01")) },
{ Assertions.assertThat(barnetilleggListe[1].periodeTil).isEqualTo(LocalDate.parse("2023-01-01")) },
{ Assertions.assertThat(barnetilleggListe[1].belopBrutto).isEqualTo(BigDecimal.valueOf(2000.22)) },
{ Assertions.assertThat(barnetilleggListe[1].belopBrutto).isEqualTo(BigDecimal.valueOf(2002.50)) },
{ Assertions.assertThat(barnetilleggListe[1].barnType).isEqualTo(BarnType.FELLES.toString()) },

// sjekk oppdatertGrunnlagspakke
Expand Down

0 comments on commit 35744b8

Please sign in to comment.