Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

update dto's to/from afp-beholdninger-api #91

Merged
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Original file line number Diff line number Diff line change
Expand Up @@ -2,4 +2,4 @@ package no.nav.tjenestepensjon.simulering.model.domain.pen

import java.time.LocalDate

data class AFPOffentligLivsvarigYtelse(val ar: Int, val belop: Double, val fom: LocalDate, val tom: LocalDate?)
data class AFPOffentligLivsvarigYtelse(val ar: Int, val belop: Double, val fom: LocalDate)
Original file line number Diff line number Diff line change
Expand Up @@ -5,4 +5,4 @@ import com.fasterxml.jackson.annotation.JsonProperty
import java.time.LocalDate

@JsonIgnoreProperties(ignoreUnknown = true)
data class AFPGrunnlagBeholdningPeriode(val beholdning: Int, @JsonProperty("fraOgMedDato") val fom: LocalDate, @JsonProperty("tilOgMedDato") val tom: LocalDate?)
data class AFPGrunnlagBeholdningPeriode(val pensjonsBeholdning: Int, @JsonProperty("fraOgMedDato") val fom: LocalDate)
Original file line number Diff line number Diff line change
@@ -1,3 +1,6 @@
package no.nav.tjenestepensjon.simulering.model.domain.popp

data class InntektPeriode(val inntektAr: Int, val belop: Int)
import com.fasterxml.jackson.annotation.JsonProperty
import java.time.LocalDate

data class InntektPeriode(@JsonProperty("fraOgMedDato") val fom: LocalDate, @JsonProperty("arligInntekt") val arligInntekt: Int)
Original file line number Diff line number Diff line change
Expand Up @@ -3,4 +3,4 @@ package no.nav.tjenestepensjon.simulering.model.domain.popp
import com.fasterxml.jackson.annotation.JsonProperty
import java.time.LocalDate

data class SimulerAFPBeholdningGrunnlagRequest(val personId: String, @JsonProperty("fraOgMedDato") val fom: LocalDate, val inntekter: List<InntektPeriode>)
data class SimulerAFPBeholdningGrunnlagRequest(val personId: String, @JsonProperty("fraOgMedDato") val fom: LocalDate, @JsonProperty("fremtidigInntektListe") val inntekter: List<InntektPeriode>)
Original file line number Diff line number Diff line change
Expand Up @@ -3,4 +3,4 @@ package no.nav.tjenestepensjon.simulering.model.domain.popp
import com.fasterxml.jackson.annotation.JsonIgnoreProperties

@JsonIgnoreProperties(ignoreUnknown = true)
data class SimulerAFPBeholdningGrunnlagResponse(val personId: String, val afpGrunnlagBeholdninger: List<AFPGrunnlagBeholdningPeriode>)
data class SimulerAFPBeholdningGrunnlagResponse(val pensjonsBeholdningsPeriodeListe: List<AFPGrunnlagBeholdningPeriode>)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Denne burde kanskje ha en

@JsonProperty("pensjonsBeholdningsPeriodeListe") val afpGrunnlagBeholdninger: ...

så man hadde unngått endringer i interne Service-klasser når en ekstern avhengighet endres.
Men ikke viktig, så la det være som det er.

Original file line number Diff line number Diff line change
Expand Up @@ -21,12 +21,12 @@ class AFPOffentligLivsvarigSimuleringService(val afpBeholdningClient: AFPBeholdn
log.info("Henter delingstall for fødselsår: ${request.fodselsdato.year} og alder $alder")
val dt = penClient.hentDelingstall(request.fodselsdato.year, alder)

val requestToAFPBeholdninger = SimulerAFPBeholdningGrunnlagRequest(request.fnr, request.fom, request.fremtidigeInntekter.map { InntektPeriode(it.fom.year, it.belop) })
val requestToAFPBeholdninger = SimulerAFPBeholdningGrunnlagRequest(request.fnr, request.fom, request.fremtidigeInntekter.map { InntektPeriode(it.fom, it.belop) })
log.info("Henter AFP beholdninger for request: $requestToAFPBeholdninger") //TODO fjern fnr før produksjon
val afpBeholdningsgrunnlag = afpBeholdningClient.simulerAFPBeholdningGrunnlag(requestToAFPBeholdninger)

log.info("Beregner AFP Offentlig Livsvarig for request: $request") //TODO fjern fnr før produksjon
val response = beregnAfpOffentligLivsvarigYtelser(dt.delingstall, afpBeholdningsgrunnlag.afpGrunnlagBeholdninger)
val response = beregnAfpOffentligLivsvarigYtelser(dt.delingstall, afpBeholdningsgrunnlag.pensjonsBeholdningsPeriodeListe)

log.info("Simulering av AFP Offentlig Livsvarig for request: $request ga response: $response") //TODO fjern fnr før produksjon
return response
Expand All @@ -37,7 +37,7 @@ class AFPOffentligLivsvarigSimuleringService(val afpBeholdningClient: AFPBeholdn
afpGrunnlagBeholdninger: List<AFPGrunnlagBeholdningPeriode>
): List<AFPOffentligLivsvarigYtelse> {
return afpGrunnlagBeholdninger.map {
AFPOffentligLivsvarigYtelse(it.fom.year, OffentligAFPYtelseBeregner.beregn(it.beholdning, delingstall), it.fom, it.tom)
AFPOffentligLivsvarigYtelse(it.fom.year, OffentligAFPYtelseBeregner.beregn(it.pensjonsBeholdning, delingstall), it.fom)
}
}

Expand Down
Loading