Skip to content

Commit

Permalink
Refaktorering underhold respons
Browse files Browse the repository at this point in the history
  • Loading branch information
ugur93 committed Dec 23, 2024
1 parent 460ba5d commit 9ce63f7
Show file tree
Hide file tree
Showing 12 changed files with 17 additions and 64 deletions.
3 changes: 2 additions & 1 deletion .nais/feature.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,7 @@ env:
BIDRAG_SJABLON_URL: https://bidrag-sjablon.dev-fss-pub.nais.io/bidrag-sjablon
ARBEID_OG_INNTEKT_URL: https://arbeid-og-inntekt-q1.dev-fss-pub.nais.io/api/v2
GRUNNLAG_MIN_ANT_MINUTTER_SIDEN_FORRIGE_INNHENTING: 1
KODEVERK_URL: https://kodeverk-api.nav.no
KODEVERK_URL: http://bidrag-kodeverk
JAVA_TOOL_OPTIONS: -agentlib:jdwp=transport=dt_socket,server=y,suspend=n,address=*:5005
kafka:
pool: nav-dev
Expand Down Expand Up @@ -58,6 +58,7 @@ outbound-access-apps:
- bidrag-tilgangskontroll-feature
- bidrag-vedtak-q1
- bidrag-dokument-produksjon
- bidrag-kodeverk
outbound-access-hosts:
- bidrag-dokument-feature.dev-fss-pub.nais.io
- bidrag-person-feature.dev-fss-pub.nais.io
Expand Down
3 changes: 2 additions & 1 deletion .nais/main.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -29,7 +29,7 @@ env:
BIDRAG_SJABLON_URL: https://bidrag-sjablon.dev-fss-pub.nais.io/bidrag-sjablon
TOPIC_VEDTAK: bidrag.vedtak-q2
ARBEID_OG_INNTEKT_URL: https://arbeid-og-inntekt.dev-fss-pub.nais.io/api/v2
KODEVERK_URL: https://kodeverk-api.nav.no
KODEVERK_URL: http://bidrag-kodeverk
GRUNNLAG_MIN_ANT_MINUTTER_SIDEN_FORRIGE_INNHENTING: 15
JAVA_TOOL_OPTIONS: -agentlib:jdwp=transport=dt_socket,server=y,suspend=n,address=*:5005
kafka:
Expand Down Expand Up @@ -58,6 +58,7 @@ outbound-access-apps:
- bidrag-tilgangskontroll
- bidrag-vedtak-q2
- bidrag-dokument-produksjon
- bidrag-kodeverk
outbound-access-hosts:
- bidrag-dokument.dev-fss-pub.nais.io
- bidrag-person.dev-fss-pub.nais.io
Expand Down
3 changes: 2 additions & 1 deletion .nais/prod.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -29,7 +29,7 @@ env:
BIDRAG_SJABLON_URL: https://bidrag-sjablon.prod-fss-pub.nais.io/bidrag-sjablon
ARBEID_OG_INNTEKT_URL: https://arbeid-og-inntekt.prod-fss-pub.nais.io/api/v2
GRUNNLAG_MIN_ANT_MINUTTER_SIDEN_FORRIGE_INNHENTING: 600
KODEVERK_URL: https://kodeverk-api.nav.no
KODEVERK_URL: http://bidrag-kodeverk
TOPIC_VEDTAK: bidrag.vedtak
kafka:
pool: nav-prod
Expand Down Expand Up @@ -57,6 +57,7 @@ outbound-access-apps:
- bidrag-tilgangskontroll
- bidrag-vedtak
- bidrag-dokument-produksjon
- bidrag-kodeverk
outbound-access-hosts:
- bidrag-dokument.prod-fss-pub.nais.io
- bidrag-person.prod-fss-pub.nais.io
Expand Down
3 changes: 2 additions & 1 deletion .nais/q1.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,7 @@ env:
BIDRAG_SJABLON_URL: https://bidrag-sjablon.dev-fss-pub.nais.io/bidrag-sjablon
ARBEID_OG_INNTEKT_URL: https://arbeid-og-inntekt-q1.dev-fss-pub.nais.io/api/v2
GRUNNLAG_MIN_ANT_MINUTTER_SIDEN_FORRIGE_INNHENTING: 1
KODEVERK_URL: https://kodeverk-api.nav.no
KODEVERK_URL: http://bidrag-kodeverk
JAVA_TOOL_OPTIONS: -agentlib:jdwp=transport=dt_socket,server=y,suspend=n,address=*:5005
kafka:
pool: nav-dev
Expand Down Expand Up @@ -59,6 +59,7 @@ outbound-access-apps:
- bidrag-dokument-forsendelse-feature
- bidrag-tilgangskontroll-feature
- bidrag-dokument-produksjon-feature
- bidrag-kodeverk
outbound-access-hosts:
- bidrag-dokument-feature.dev-fss-pub.nais.io
- bidrag-person-feature.dev-fss-pub.nais.io
Expand Down
3 changes: 2 additions & 1 deletion .nais/q2.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,7 @@ env:
BIDRAG_SJABLON_URL: https://bidrag-sjablon.dev-fss-pub.nais.io/bidrag-sjablon
ARBEID_OG_INNTEKT_URL: https://arbeid-og-inntekt.dev-fss-pub.nais.io/api/v2
GRUNNLAG_MIN_ANT_MINUTTER_SIDEN_FORRIGE_INNHENTING: 1
KODEVERK_URL: https://kodeverk-api.nav.no
KODEVERK_URL: http://bidrag-kodeverk
JAVA_TOOL_OPTIONS: -agentlib:jdwp=transport=dt_socket,server=y,suspend=n,address=*:5005
kafka:
pool: nav-dev
Expand Down Expand Up @@ -58,6 +58,7 @@ outbound-access-apps:
- bidrag-tilgangskontroll
- bidrag-vedtak-q2
- bidrag-dokument-produksjon
- bidrag-kodeverk
outbound-access-hosts:
- bidrag-dokument.dev-fss-pub.nais.io
- bidrag-person-q2.dev-fss-pub.nais.io
Expand Down
3 changes: 2 additions & 1 deletion .nais/q4.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,7 @@ env:
BIDRAG_SJABLON_URL: https://bidrag-sjablon.dev-fss-pub.nais.io/bidrag-sjablon
ARBEID_OG_INNTEKT_URL: https://arbeid-og-inntekt-q1.dev-fss-pub.nais.io/api/v2
GRUNNLAG_MIN_ANT_MINUTTER_SIDEN_FORRIGE_INNHENTING: 1
KODEVERK_URL: https://kodeverk-api.nav.no
KODEVERK_URL: http://bidrag-kodeverk
JAVA_TOOL_OPTIONS: -agentlib:jdwp=transport=dt_socket,server=y,suspend=n,address=*:5005
kafka:
pool: nav-dev
Expand Down Expand Up @@ -58,6 +58,7 @@ outbound-access-apps:
- bidrag-tilgangskontroll
- bidrag-vedtak-q4
- bidrag-dokument-produksjon
- bidrag-kodeverk
outbound-access-hosts:
- bidrag-dokument.dev-fss-pub.nais.io
- bidrag-person-q4.dev-fss-pub.nais.io
Expand Down
2 changes: 1 addition & 1 deletion pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,7 @@
<properties>
<java.version>21</java.version>
<kotlin.version>2.1.0</kotlin.version>
<bidrag-felles.version>2024.12.19.080813</bidrag-felles.version>
<bidrag-felles.version>2024.12.23.100918</bidrag-felles.version>
<bidrag-beregn-felles.version>2024.12.16.134803</bidrag-beregn-felles.version>
<h2.version>2.3.232</h2.version>
<logback-encoder.version>8.0</logback-encoder.version>
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,6 @@ import no.nav.bidrag.behandling.dto.v2.underhold.SletteUnderholdselement
import no.nav.bidrag.behandling.dto.v2.underhold.StønadTilBarnetilsynDto
import no.nav.bidrag.behandling.dto.v2.underhold.Underholdselement
import no.nav.bidrag.behandling.fantIkkeFødselsdatoTilPerson
import no.nav.bidrag.behandling.transformers.Dtomapper
import no.nav.bidrag.behandling.transformers.behandling.hentAlleBearbeidaBarnetilsyn
import no.nav.bidrag.behandling.transformers.underhold.aktivereBarnetilsynHvisIngenEndringerMåAksepteres
import no.nav.bidrag.behandling.transformers.underhold.erstatteOffentligePerioderIBarnetilsynstabellMedOppdatertGrunnlag
Expand Down Expand Up @@ -52,7 +51,6 @@ class UnderholdService(
private val underholdskostnadRepository: UnderholdskostnadRepository,
private val personRepository: PersonRepository,
private val notatService: NotatService,
private val dtomapper: Dtomapper,
private val personService: PersonService,
) {
fun oppdatereBegrunnelse(
Expand Down Expand Up @@ -181,11 +179,6 @@ class UnderholdService(
)
underholdskostnad.barnetilsyn.add(barnetilsyn)
underholdskostnad.harTilsynsordning = true
underholdskostnadRepository
.save(underholdskostnad)
.barnetilsyn
.sortedBy { it.id }
.last()
}
}

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -116,14 +116,12 @@ fun SletteUnderholdselement.validere(behandling: Behandling) {
}
}

fun Set<Tilleggsstønad>.finneTilleggsstønadsperioderSomIkkeOverlapperMedFaktiskTilsynsutgiftsperioder(
perioderTilsynsutgift: Set<FaktiskTilsynsutgift>,
): Set<DatoperiodeDto> {
val datoperioderTilsynsutgift = perioderTilsynsutgift.tilsynsutgiftTilDatoperioder()
fun Underholdskostnad.finnTilleggsstønadsperioderSomIkkeOverlapperMedFaktiskTilsynsutgiftsperioder(): Set<DatoperiodeDto> {
val datoperioderTilsynsutgift = faktiskeTilsynsutgifter.tilsynsutgiftTilDatoperioder()
val datoperioderTillegsstønadSomIkkeOverlapperMedTilsynsutgift = mutableListOf<DatoperiodeDto>()

val tilleggsstønadsperioderSomIkkeErDekketAvTilsynsutgift =
this.tilleggsstønadTilDatoperioder().redusereMed(datoperioderTilsynsutgift)
this.tilleggsstønad.tilleggsstønadTilDatoperioder().redusereMed(datoperioderTilsynsutgift)

tilleggsstønadsperioderSomIkkeErDekketAvTilsynsutgift.forEach { periode ->
datoperioderTillegsstønadSomIkkeOverlapperMedTilsynsutgift.add(periode)
Expand Down Expand Up @@ -180,10 +178,7 @@ fun Underholdskostnad.valider(): UnderholdskostnadValideringsfeil =
stønadTilBarnetilsyn = barnetilsyn.validerePerioderBarnetilsyn().takeIf { it.harFeil },
tilleggsstønad = tilleggsstønad.validerePerioderTilleggsstønad().takeIf { it.harFeil },
faktiskTilsynsutgift = faktiskeTilsynsutgifter.validerePerioderFaktiskTilsynsutgift().takeIf { it.harFeil },
tilleggsstønadsperioderUtenFaktiskTilsynsutgift =
tilleggsstønad.finneTilleggsstønadsperioderSomIkkeOverlapperMedFaktiskTilsynsutgiftsperioder(
faktiskeTilsynsutgifter,
),
tilleggsstønadsperioderUtenFaktiskTilsynsutgift = finnTilleggsstønadsperioderSomIkkeOverlapperMedFaktiskTilsynsutgiftsperioder(),
manglerPerioderForTilsynsordning = manglerPerioderForTilsynsordning(),
manglerBegrunnelse = manglerBegrunnelse(),
)
Expand Down Expand Up @@ -218,43 +213,6 @@ fun Set<Tilleggsstønad>.validerePerioderTilleggsstønad() =
overlappendePerioder = tilleggsstønadTilUnderholdsperioder().finneOverlappendePerioder(),
)

/*
fun Underholdskostnad.validerePerioder(perioderUnderholdskostnadDto: Set<UnderholdskostnadDto>) =
ValideringsfeilUnderhold(
gjelderUnderholdskostnad = this,
overlappendePerioder =
finneOverlappendePerioder(
this.barnetilsyn.barnetilsynTilUnderholdsperioder(),
) +
finneOverlappendePerioder(
this.tilleggsstønad.tilleggsstønadTilUnderholdsperioder(),
),
tilleggsstønadsperioderUtenFaktiskTilsynsutgift =
this.tilleggsstønad.finneTilleggsstønadsperioderSomIkkeOverlapperMedFaktiskTilsynsutgiftsperioder(
this.faktiskeTilsynsutgifter,
),
fremtidigePerioder =
this.barnetilsyn
.barnetilsynTilDatoperioder()
.finneFremtidigePerioder(Underholdselement.STØNAD_TIL_BARNETILSYN) +
this.faktiskeTilsynsutgifter
.tilsynsutgiftTilDatoperioder()
.finneFremtidigePerioder(Underholdselement.FAKTISK_TILSYNSUTGIFT) +
this.tilleggsstønad
.tilleggsstønadTilDatoperioder()
.finneFremtidigePerioder(Underholdselement.TILLEGGSSTØNAD),
harIngenPerioder = this.barnetsRolleIBehandlingen?.let { perioderUnderholdskostnadDto.isEmpty() } ?: false,
manglerPerioderForTilsynsutgifter =
this.harTilsynsordning?.let {
this.barnetilsyn.isEmpty() &&
this.faktiskeTilsynsutgifter.isEmpty() &&
this.tilleggsstønad.isEmpty()
}
?: false,
)
*/

fun StønadTilBarnetilsynDto.validerePerioderStønadTilBarnetilsyn(underholdskostnad: Underholdskostnad) {
this.id?.let { id ->
if (id > 0 && underholdskostnad.barnetilsyn.find { id == it.id } == null) {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -98,7 +98,6 @@ abstract class CommonVedtakTilBehandlingTest {
underholdskostnadRepository,
personRepository,
notatService,
dtomapper,
personService,
)
val vedtakTilBehandlingMapping = VedtakTilBehandlingMapping(validerBeregning, underholdService = underholdService)
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,6 @@ import io.kotest.matchers.collections.shouldHaveSize
import io.kotest.matchers.collections.shouldNotBeEmpty
import io.kotest.matchers.comparables.shouldBeGreaterThan
import io.kotest.matchers.date.shouldBeBefore
import io.kotest.matchers.maps.shouldHaveSize
import io.kotest.matchers.nulls.shouldBeNull
import io.kotest.matchers.nulls.shouldNotBeNull
import io.kotest.matchers.shouldBe
Expand Down Expand Up @@ -142,7 +141,6 @@ class UnderholdServiceTest {
underholdskostnadRepository,
personRepository,
notatService,
dtomapper,
personService,
)

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -169,7 +169,6 @@ class VedtakserviceTest : TestContainerRunner() {
underholdskostnadRepository,
personRepository,
notatService,
dtomapper,
personService,
)
val vedtakTilBehandlingMapping = VedtakTilBehandlingMapping(validerBeregning, underholdService)
Expand Down

0 comments on commit 9ce63f7

Please sign in to comment.