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

Flere begrunnelser v1 - Oppdatert IM modell og PDF støtte #837

Draft
wants to merge 6 commits into
base: main
Choose a base branch
from
Draft
Show file tree
Hide file tree
Changes from 3 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
3 changes: 3 additions & 0 deletions apps/api/build.gradle.kts
Original file line number Diff line number Diff line change
Expand Up @@ -22,4 +22,7 @@ dependencies {
testImplementation("io.ktor:ktor-client-core:$ktorVersion")
testImplementation("io.ktor:ktor-server-test-host:$ktorVersion")
testImplementation("no.nav.security:mock-oauth2-server:$mockOauth2ServerVersion")

// midlertidlig for flere endringsaarsaker
testImplementation("io.kotest:kotest-assertions-json:5.8.0")
}
Original file line number Diff line number Diff line change
Expand Up @@ -15,6 +15,7 @@ import no.nav.helsearbeidsgiver.felles.metrics.Metrics
import no.nav.helsearbeidsgiver.felles.rapidsrivers.redis.RedisConnection
import no.nav.helsearbeidsgiver.felles.rapidsrivers.redis.RedisPrefix
import no.nav.helsearbeidsgiver.felles.rapidsrivers.redis.RedisStore
import no.nav.helsearbeidsgiver.felles.utils.fyllUtMangledeEndringsAarsaker
import no.nav.helsearbeidsgiver.inntektsmelding.api.RedisPoller
import no.nav.helsearbeidsgiver.inntektsmelding.api.Routes
import no.nav.helsearbeidsgiver.inntektsmelding.api.auth.Tilgangskontroll
Expand Down Expand Up @@ -94,6 +95,19 @@ fun Route.innsending(
}
}

// // midlertidlig duplisering for å støtte flere endringsÅrsaker
Copy link
Contributor

Choose a reason for hiding this comment

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

Kan gjerne fjerne utkommentert kode, heller ta det inn når det faktisk skal tas i bruk

// fun SkjemaInntektsmelding.fyllUtMangledeEndringsAarsaker(): SkjemaInntektsmelding {
// val medEndringsAarsakerfyllt = this.copy(inntekt = this.inntekt?.copy(endringAarsaker = listOfNotNull(this.inntekt?.endringAarsak)))
// val verdiMangler = this.inntekt?.endringAarsaker.isNullOrEmpty()
// return if (verdiMangler) medEndringsAarsakerfyllt else this
// }
//
// fun Inntektsmelding.fyllUtMangledeEndringsAarsaker(): Inntektsmelding {
// val medEndringsAarsakerfyllt = this.copy(inntekt = this.inntekt?.copy(endringAarsaker = listOfNotNull(this.inntekt?.endringAarsak)))
// val verdiMangler = this.inntekt?.endringAarsaker.isNullOrEmpty()
// return if (verdiMangler) medEndringsAarsakerfyllt else this
// }

private suspend fun PipelineContext<Unit, ApplicationCall>.lesRequestOrNull(): SkjemaInntektsmelding? =
call
.receiveText()
Expand All @@ -105,6 +119,7 @@ private suspend fun PipelineContext<Unit, ApplicationCall>.lesRequestOrNull(): S
sikkerLogger.info("$it\n${json.toPretty()}")
}
}.fromJson(SkjemaInntektsmelding.serializer())
.fyllUtMangledeEndringsAarsaker()
}.getOrElse { error ->
"Klarte ikke parse json for inntektsmeldingsskjema.".also {
logger.error(it)
Expand Down
Original file line number Diff line number Diff line change
@@ -1,5 +1,7 @@
package no.nav.helsearbeidsgiver.inntektsmelding.api.hentselvbestemtim

import io.kotest.assertions.assertSoftly
import io.kotest.assertions.json.shouldEqualJson
import io.kotest.matchers.shouldBe
import io.kotest.matchers.string.shouldBeEmpty
import io.ktor.client.statement.bodyAsText
Expand Down Expand Up @@ -77,6 +79,14 @@ class HentSelvbestemtImRouteKtTest : ApiTest() {

val actualJson = response.bodyAsText()

println("expected vs actual json:")
println(actualJson)
println(Mock.successResponseJson(expectedInntektsmelding))
Copy link
Contributor

Choose a reason for hiding this comment

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

Kan vel droppe println og assertSoftly egentlig?


assertSoftly {
actualJson shouldEqualJson Mock.successResponseJson(expectedInntektsmelding)
}

response.status shouldBe HttpStatusCode.OK
actualJson shouldBe Mock.successResponseJson(expectedInntektsmelding)
}
Expand Down Expand Up @@ -318,7 +328,8 @@ private fun Inntekt.hardcodedJson(): String =
"beloep": $beloep,
"inntektsdato": "$inntektsdato",
"naturalytelser": [${naturalytelser.joinToString(transform = Naturalytelse::hardcodedJson)}],
"endringAarsak": ${endringAarsak?.hardcodedJson()}
"endringAarsak": ${endringAarsak?.hardcodedJson()},
"endringAarsaker": [${endringAarsak?.hardcodedJson()}]
}
"""

Expand Down
Original file line number Diff line number Diff line change
@@ -0,0 +1,25 @@
package no.nav.helsearbeidsgiver.felles.utils

import no.nav.helsearbeidsgiver.domene.inntektsmelding.v1.Inntektsmelding
import no.nav.helsearbeidsgiver.domene.inntektsmelding.v1.skjema.SkjemaInntektsmelding

// midlertidlig duplisering for å støtte flere endringsÅrsaker
fun SkjemaInntektsmelding.fyllUtMangledeEndringsAarsaker(): SkjemaInntektsmelding {
Copy link
Contributor

Choose a reason for hiding this comment

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

Kanskje ha et annet navn en fyllUtManglede - noe mer som ala "konverterEndringAarsakTilListe"
Og så tror jeg det kan være greit å sjekke at endringAarsaker er tom, før man gjør dette. Da slipper vi å holde styr på når frontend evt endres.

val medEndringsAarsakerfyllt = this.copy(inntekt = this.inntekt?.copy(endringAarsaker = listOfNotNull(this.inntekt?.endringAarsak)))
val verdiMangler = this.inntekt?.endringAarsaker.isNullOrEmpty()
return if (verdiMangler) medEndringsAarsakerfyllt else this
}

fun Inntektsmelding.fyllUtMangledeEndringsAarsaker(): Inntektsmelding {
val medEndringsAarsakerfyllt = this.copy(inntekt = this.inntekt?.copy(endringAarsaker = listOfNotNull(this.inntekt?.endringAarsak)))
val verdiMangler = this.inntekt?.endringAarsaker.isNullOrEmpty()
return if (verdiMangler) medEndringsAarsakerfyllt else this
}

// fun LagretInntektsmelding.fyllUtMangledeEndringsAarsaker(): LagretInntektsmelding {
//
//
// val medEndringsAarsakerfyllt = this.copy(inntekt = this.inntekt?.copy(endringAarsaker = listOfNotNull(this.inntekt?.endringAarsak)))
// val verdiMangler = this.inntekt?.endringAarsaker.isNullOrEmpty()
// return if (verdiMangler) medEndringsAarsakerfyllt else this
// }
Original file line number Diff line number Diff line change
Expand Up @@ -130,6 +130,12 @@ fun mockInntekt(): Inntekt =
NyStillingsprosent(
gjelderFra = 16.oktober,
),
endringAarsaker =
listOf(
NyStillingsprosent(
gjelderFra = 16.oktober,
),
),
)

fun mockRefusjon(): Refusjon =
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -18,14 +18,16 @@ import no.nav.helsearbeidsgiver.domene.inntektsmelding.v1.Sykefravaer
import no.nav.helsearbeidsgiver.domene.inntektsmelding.v1.Tariffendring
import no.nav.helsearbeidsgiver.domene.inntektsmelding.v1.VarigLoennsendring
import no.nav.helsearbeidsgiver.felles.utils.tilNorskFormat
import no.nav.helsearbeidsgiver.utils.pipe.orDefault

private const val FORKLARING_ENDRING = "Forklaring for endring"
private const val FORKLARING_ENDRING = "Endringsårsak"

class PdfDokument(
val inntektsmelding: Inntektsmelding,
) {
private val pdf = PdfBuilder(bodySize = 20, topText = "Innsendt: ${inntektsmelding.mottatt.tilNorskFormat()}") // Setter skriftstørrelsen på labels og text
private var y = 0
private var forklaringEndring = FORKLARING_ENDRING
private val kolonneEn = 0
private val kolonneTo = 420
private val naturalytelse1 = kolonneEn
Expand Down Expand Up @@ -197,29 +199,33 @@ class PdfDokument(
"Registrert inntekt (per ${inntektsmelding.inntekt?.inntektsdato?.tilNorskFormat()})",
"${inntektsmelding.inntekt?.beloep?.tilNorskFormat()} kr/måned",
)
val endringAarsak = inntektsmelding.inntekt?.endringAarsak
when (endringAarsak) {
null -> return // trenger ikke sende inn årsak...
is Bonus -> addBonus()
is Feilregistrert -> addFeilregistrert()
is Ferie -> addFerie(endringAarsak)
is Ferietrekk -> addFerietrekk()
is NyStilling -> addNyStilling(endringAarsak)
is NyStillingsprosent -> addNyStillingsprosent(endringAarsak)
is Nyansatt -> addNyAnsatt()
is Permisjon -> addPermisjon(endringAarsak)
is Permittering -> addPermittering(endringAarsak)
is Sykefravaer -> addSykefravaer(endringAarsak)
is Tariffendring -> addTariffendring(endringAarsak)
is VarigLoennsendring -> addVarigLonnsendring(endringAarsak)
val endringAarsaker = inntektsmelding.inntekt?.endringAarsaker.orDefault(emptyList())
val antall = endringAarsaker.size
endringAarsaker.forEachIndexed { indeks, endringAarsak ->
forklaringEndring = FORKLARING_ENDRING + if (antall > 1) " (${indeks + 1} av $antall)" else ""

when (endringAarsak) {
is Bonus -> addBonus()
is Feilregistrert -> addFeilregistrert()
is Ferie -> addFerie(endringAarsak)
is Ferietrekk -> addFerietrekk()
is NyStilling -> addNyStilling(endringAarsak)
is NyStillingsprosent -> addNyStillingsprosent(endringAarsak)
is Nyansatt -> addNyAnsatt()
is Permisjon -> addPermisjon(endringAarsak)
is Permittering -> addPermittering(endringAarsak)
is Sykefravaer -> addSykefravaer(endringAarsak)
is Tariffendring -> addTariffendring(endringAarsak)
is VarigLoennsendring -> addVarigLonnsendring(endringAarsak)
}
}
}

private fun addInntektEndringPerioder(
endringAarsak: String,
perioder: List<Periode>,
) {
addLabel(FORKLARING_ENDRING, endringAarsak, linefeed = false)
addLabel(forklaringEndring, endringAarsak, linefeed = false)
addPerioder(kolonneTo, perioder)
}

Expand All @@ -240,42 +246,42 @@ class PdfDokument(
}

private fun addNyAnsatt() {
addLabel(FORKLARING_ENDRING, "Nyansatt")
addLabel(forklaringEndring, "Nyansatt")
}

private fun addFeilregistrert() {
addLabel(FORKLARING_ENDRING, "Mangelfull eller uriktig rapportering til A-ordningen")
addLabel(forklaringEndring, "Mangelfull eller uriktig rapportering til A-ordningen")
}

private fun addTariffendring(tariffendring: Tariffendring) {
addLabel(FORKLARING_ENDRING, "Tariffendring")
addLabel(forklaringEndring, "Tariffendring")
addLabel("Gjelder fra", tariffendring.gjelderFra.tilNorskFormat(), linefeed = false)
addLabel("Ble kjent", tariffendring.bleKjent.tilNorskFormat(), kolonneTo)
}

private fun addVarigLonnsendring(varigLoennsendring: VarigLoennsendring) {
addLabel(FORKLARING_ENDRING, "Varig lønnsendring")
addLabel(forklaringEndring, "Varig lønnsendring")
addLabel("Gjelder fra", varigLoennsendring.gjelderFra.tilNorskFormat())
}

private fun addNyStilling(nyStilling: NyStilling) {
addLabel(FORKLARING_ENDRING, "Ny stilling", linefeed = false)
addLabel(forklaringEndring, "Ny stilling", linefeed = false)
addLabel("Gjelder fra", nyStilling.gjelderFra.tilNorskFormat(), kolonneTo)
}

private fun addNyStillingsprosent(nyStillingsprosent: NyStillingsprosent) {
addLabel(FORKLARING_ENDRING, "Ny stillingsprosent", linefeed = false)
addLabel(forklaringEndring, "Ny stillingsprosent", linefeed = false)
addLabel("Gjelder fra", nyStillingsprosent.gjelderFra.tilNorskFormat(), kolonneTo)
}

private fun addBonus() {
addLabel(FORKLARING_ENDRING, "Bonus")
addLabel(forklaringEndring, "Bonus")
// addLabel("Estimert årlig bonus", årligBonus.tilNorskFormat())
// addLabel("Dato siste bonus", datoBonus.tilNorskFormat())
}

private fun addFerietrekk() {
addLabel(FORKLARING_ENDRING, "Ferietrekk/Utbetaling av feriepenger")
addLabel(forklaringEndring, "Ferietrekk/Utbetaling av feriepenger")
}

private fun addRefusjon() {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -247,6 +247,7 @@ class PdfDokumentTest {
im.inntekt.shouldNotBeNull().copy(
beloep = 123.0,
endringAarsak = it.value,
endringAarsaker = listOf(it.value),
),
),
)
Expand All @@ -258,6 +259,7 @@ class PdfDokumentTest {
im.inntekt.shouldNotBeNull().copy(
beloep = 123.0,
endringAarsak = null,
endringAarsaker = emptyList(),
),
),
)
Expand All @@ -267,7 +269,7 @@ class PdfDokumentTest {
title: String,
im: Inntektsmelding,
) {
// val file = File(System.getProperty("user.home"), "/Desktop/$title.pdf")
// val file = File(System.getProperty("user.home"), "/Desktop/pdf/$title.pdf")
val file = File.createTempFile(title, ".pdf")
val writer = FileOutputStream(file)
writer.write(PdfDokument(im).export())
Expand Down
2 changes: 1 addition & 1 deletion gradle.properties
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@ kotlinVersion=2.0.21
kotlinterVersion=4.4.0

# Dependency versions
hagDomeneInntektsmeldingVersion=0.1.7
hagDomeneInntektsmeldingVersion=0.1.8-SNAPSHOT
Copy link
Contributor

Choose a reason for hiding this comment

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

Bør bygge en final release (uten snapshot) som brukes her, før vi merger

junitJupiterVersion=5.11.3
kotestVersion=5.9.1
kotlinCoroutinesVersion=1.9.0
Expand Down