Skip to content

Commit

Permalink
Merge branch 'main' into dev/hent_tilganger_via_fager_api
Browse files Browse the repository at this point in the history
  • Loading branch information
magnusae committed Jan 20, 2025
2 parents 93ad0c8 + 2f2fde8 commit 33d8b4f
Show file tree
Hide file tree
Showing 13 changed files with 79 additions and 193 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -314,7 +314,7 @@ private fun ForespurtData.hardcodedJson(): String =
},
"inntekt": {
"paakrevd": ${inntekt.paakrevd},
"forslag": ${inntekt.forslag.hardcodedJson()}
"forslag": ${inntekt.forslag?.hardcodedJson()}
},
"refusjon": {
"paakrevd": ${refusjon.paakrevd},
Expand All @@ -324,23 +324,11 @@ private fun ForespurtData.hardcodedJson(): String =
"""

private fun ForslagInntekt.hardcodedJson(): String =
when (this) {
is ForslagInntekt.Grunnlag ->
"""
{
"type": "ForslagInntektGrunnlag",
"forrigeInntekt": ${forrigeInntekt?.hardcodedJson()}
}
"""

is ForslagInntekt.Fastsatt ->
"""
{
"type": "ForslagInntektFastsatt",
"fastsattInntekt": $fastsattInntekt
}
"""
"""
{
"forrigeInntekt": ${forrigeInntekt?.hardcodedJson()}
}
"""

private fun ForrigeInntekt.hardcodedJson(): String =
"""
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -2,12 +2,10 @@ package no.nav.helsearbeidsgiver.inntektsmelding.berikinntektsmeldingservice

import no.nav.helsearbeidsgiver.domene.inntektsmelding.v1.AarsakInnsending
import no.nav.helsearbeidsgiver.domene.inntektsmelding.v1.Avsender
import no.nav.helsearbeidsgiver.domene.inntektsmelding.v1.Inntekt
import no.nav.helsearbeidsgiver.domene.inntektsmelding.v1.Inntektsmelding
import no.nav.helsearbeidsgiver.domene.inntektsmelding.v1.Sykmeldt
import no.nav.helsearbeidsgiver.domene.inntektsmelding.v1.skjema.SkjemaInntektsmelding
import no.nav.helsearbeidsgiver.felles.domene.Forespoersel
import no.nav.helsearbeidsgiver.felles.domene.ForslagInntekt
import no.nav.helsearbeidsgiver.felles.utils.zoneIdOslo
import java.time.LocalDateTime
import java.util.UUID
Expand Down Expand Up @@ -38,12 +36,7 @@ fun mapInntektsmelding(
)
}
} else {
Inntekt(
beloep = (forespoersel.forespurtData.inntekt.forslag as ForslagInntekt.Fastsatt).fastsattInntekt,
inntektsdato = forespoersel.forslagInntektsdato(),
naturalytelser = emptyList(),
endringAarsak = null,
)
null
}

val refusjon =
Expand Down
Original file line number Diff line number Diff line change
@@ -1,7 +1,6 @@
package no.nav.helsearbeidsgiver.inntektsmelding.berikinntektsmeldingservice

import io.kotest.core.spec.style.FunSpec
import io.kotest.matchers.collections.shouldBeEmpty
import io.kotest.matchers.collections.shouldNotBeEmpty
import io.kotest.matchers.nulls.shouldBeNull
import io.kotest.matchers.nulls.shouldNotBeNull
Expand Down Expand Up @@ -131,7 +130,7 @@ class MapInntektsmeldingKtTest :
inntektsmelding.agp.shouldBeNull()
}

test("bruker fastsatt inntekt (fra Spleis) dersom inntekt _ikke_ er påkrevd") {
test("fjerner inntekt dersom inntekt _ikke_ er påkrevd") {
val forespoersel = mockForespoersel().utenPaakrevdInntekt()
val skjema = mockSkjemaInntektsmelding()

Expand All @@ -146,16 +145,8 @@ class MapInntektsmeldingKtTest :
mottatt = 6.desember.atStartOfDay(),
)

val fastsattInntekt = 8795.0

inntektsmelding.inntekt shouldNotBe skjema.inntekt
inntektsmelding.inntekt.also {
it.shouldNotBeNull()
it.beloep shouldBe fastsattInntekt
it.inntektsdato shouldBe forespoersel.forslagInntektsdato()
it.naturalytelser.shouldBeEmpty()
it.endringAarsak.shouldBeNull()
}
skjema.inntekt.shouldNotBeNull()
inntektsmelding.inntekt.shouldBeNull()
}

test("bruker innsendt inntektsdato dersom AGP er påkrevd") {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,6 @@

package no.nav.helsearbeidsgiver.felles.domene

import kotlinx.serialization.SerialName
import kotlinx.serialization.Serializable
import kotlinx.serialization.UseSerializers
import no.nav.helsearbeidsgiver.utils.json.serializer.LocalDateSerializer
Expand All @@ -23,7 +22,7 @@ data class ForespurtData(
@Serializable
data class Inntekt(
val paakrevd: Boolean,
val forslag: ForslagInntekt,
val forslag: ForslagInntekt?,
)

@Serializable
Expand All @@ -34,19 +33,10 @@ data class ForespurtData(
}

@Serializable
sealed class ForslagInntekt {
@Serializable
@SerialName("ForslagInntektGrunnlag")
data class Grunnlag(
val forrigeInntekt: ForrigeInntekt?,
) : ForslagInntekt()

@Serializable
@SerialName("ForslagInntektFastsatt")
data class Fastsatt(
val fastsattInntekt: Double,
) : ForslagInntekt()
}
data class ForslagInntekt(
// TODO fjern default etter overgangsfase
val forrigeInntekt: ForrigeInntekt? = null,
)

@Serializable
data class ForslagRefusjon(
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -34,8 +34,6 @@ object Metrics {

val pdlRequest = requestMetric("PDL")

val forespoerslerBesvartFraSimba = counterMetric("forespoersler besvart fra Simba")

val forespoerslerBesvartFraSpleis = counterMetric("forespoersler besvart fra Spleis")

object Expose {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -6,8 +6,8 @@ import io.kotest.data.row
import io.kotest.matchers.shouldBe
import no.nav.helsearbeidsgiver.felles.domene.ForespurtData
import no.nav.helsearbeidsgiver.felles.test.mock.mockForespurtData
import no.nav.helsearbeidsgiver.felles.test.mock.mockForespurtDataMedFastsattInntekt
import no.nav.helsearbeidsgiver.felles.test.mock.mockForespurtDataMedForrigeInntekt
import no.nav.helsearbeidsgiver.felles.test.mock.mockForespurtDataMedTomtInntektForslag
import no.nav.helsearbeidsgiver.utils.json.fromJson
import no.nav.helsearbeidsgiver.utils.json.toJsonStr
import no.nav.helsearbeidsgiver.utils.test.json.removeJsonWhitespace
Expand All @@ -17,7 +17,7 @@ class ForespurtDataTest :
FunSpec({
listOf(
row("forespurtData", ::mockForespurtData),
row("forespurtDataMedFastsattInntekt", ::mockForespurtDataMedFastsattInntekt),
row("forespurtDataMedTomtInntektForslag", ::mockForespurtDataMedTomtInntektForslag),
row("forespurtDataMedForrigeInntekt", ::mockForespurtDataMedForrigeInntekt),
).forEach { (fileName, mockDataFn) ->
val expectedJson = "json/$fileName.json".readResource().removeJsonWhitespace()
Expand Down
5 changes: 1 addition & 4 deletions apps/felles/src/test/resources/json/forespurtData.json
Original file line number Diff line number Diff line change
Expand Up @@ -4,10 +4,7 @@
},
"inntekt": {
"paakrevd": true,
"forslag": {
"type": "ForslagInntektGrunnlag",
"forrigeInntekt": null
}
"forslag": null
},
"refusjon": {
"paakrevd": true,
Expand Down

This file was deleted.

Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,6 @@
"inntekt": {
"paakrevd": true,
"forslag": {
"type": "ForslagInntektGrunnlag",
"forrigeInntekt": {
"skjæringstidspunkt": "2017-01-01",
"kilde": "INNTEKTSMELDING",
Expand Down
Original file line number Diff line number Diff line change
@@ -0,0 +1,25 @@
{
"arbeidsgiverperiode": {
"paakrevd": true
},
"inntekt": {
"paakrevd": true,
"forslag": {}
},
"refusjon": {
"paakrevd": true,
"forslag": {
"perioder": [
{
"fom": "2017-01-10",
"beloep": 10.48
},
{
"fom": "2017-02-02",
"beloep": 98.26
}
],
"opphoersdato": "2017-02-26"
}
}
}
Original file line number Diff line number Diff line change
Expand Up @@ -41,101 +41,56 @@ fun mockForespurtData(): ForespurtData =
inntekt =
ForespurtData.Inntekt(
paakrevd = true,
forslag = ForslagInntekt.Grunnlag(forrigeInntekt = null),
forslag = null,
),
refusjon =
ForespurtData.Refusjon(
paakrevd = true,
forslag =
ForslagRefusjon(
perioder =
listOf(
ForslagRefusjon.Periode(
fom = 10.januar(2017),
beloep = 10.48,
),
ForslagRefusjon.Periode(
fom = 2.februar(2017),
beloep = 98.26,
),
),
opphoersdato = 26.februar(2017),
),
forslag = mockForespurtDataForslagRefusjon(),
),
)

fun mockForespurtDataMedForrigeInntekt(): ForespurtData =
fun mockForespurtDataMedTomtInntektForslag(): ForespurtData =
ForespurtData(
arbeidsgiverperiode =
ForespurtData.Arbeidsgiverperiode(
paakrevd = false,
paakrevd = true,
),
inntekt =
ForespurtData.Inntekt(
paakrevd = true,
forslag =
ForslagInntekt.Grunnlag(
forrigeInntekt =
ForrigeInntekt(
skjæringstidspunkt = 1.januar.minusYears(1),
kilde = "INNTEKTSMELDING",
beløp = 10000.0,
),
),
forslag = ForslagInntekt(forrigeInntekt = null),
),
refusjon =
ForespurtData.Refusjon(
paakrevd = true,
forslag =
ForslagRefusjon(
perioder =
listOf(
ForslagRefusjon.Periode(
fom = 10.januar(2017),
beloep = 10.48,
),
ForslagRefusjon.Periode(
fom = 2.februar(2017),
beloep = 98.26,
),
),
opphoersdato = 26.februar(2017),
),
forslag = mockForespurtDataForslagRefusjon(),
),
)

fun mockForespurtDataMedFastsattInntekt(): ForespurtData =
fun mockForespurtDataMedForrigeInntekt(): ForespurtData =
ForespurtData(
arbeidsgiverperiode =
ForespurtData.Arbeidsgiverperiode(
paakrevd = true,
paakrevd = false,
),
inntekt =
ForespurtData.Inntekt(
paakrevd = false,
paakrevd = true,
forslag =
ForslagInntekt.Fastsatt(
fastsattInntekt = 31415.92,
ForslagInntekt(
forrigeInntekt =
ForrigeInntekt(
skjæringstidspunkt = 1.januar.minusYears(1),
kilde = "INNTEKTSMELDING",
beløp = 10000.0,
),
),
),
refusjon =
ForespurtData.Refusjon(
paakrevd = true,
forslag =
ForslagRefusjon(
perioder =
listOf(
ForslagRefusjon.Periode(
fom = 1.januar,
beloep = 31415.92,
),
ForslagRefusjon.Periode(
fom = 15.januar,
beloep = 3.14,
),
),
opphoersdato = null,
),
forslag = mockForespurtDataForslagRefusjon(),
),
)

Expand All @@ -157,10 +112,7 @@ fun Forespoersel.utenPaakrevdInntekt(): Forespoersel =
inntekt =
ForespurtData.Inntekt(
paakrevd = false,
forslag =
ForslagInntekt.Fastsatt(
fastsattInntekt = 8795.0,
),
forslag = null,
),
),
)
Expand All @@ -180,3 +132,19 @@ fun Forespoersel.utenPaakrevdRefusjon(): Forespoersel =
),
),
)

private fun mockForespurtDataForslagRefusjon(): ForslagRefusjon =
ForslagRefusjon(
perioder =
listOf(
ForslagRefusjon.Periode(
fom = 10.januar(2017),
beloep = 10.48,
),
ForslagRefusjon.Periode(
fom = 2.februar(2017),
beloep = 98.26,
),
),
opphoersdato = 26.februar(2017),
)
Loading

0 comments on commit 33d8b4f

Please sign in to comment.