Bruk wrappers i forespoersel

navikt · Dec 16, 2024 · 11a912a · 11a912a
1 parent 96b128c
commit 11a912a
Show file tree

Hide file tree

Showing 25 changed files with 127 additions and 132 deletions.
diff --git a/.../kotlin/no/nav/helsearbeidsgiver/inntektsmelding/aktiveorgnrservice/AktiveOrgnrService.kt b/.../kotlin/no/nav/helsearbeidsgiver/inntektsmelding/aktiveorgnrservice/AktiveOrgnrService.kt
@@ -1,7 +1,6 @@
 package no.nav.helsearbeidsgiver.inntektsmelding.aktiveorgnrservice
 
 import com.github.navikt.tbd_libs.rapids_and_rivers_api.RapidsConnection
-import kotlinx.serialization.builtins.MapSerializer
 import kotlinx.serialization.builtins.serializer
 import kotlinx.serialization.json.JsonElement
 import no.nav.helsearbeidsgiver.felles.BehovType
@@ -12,6 +11,7 @@ import no.nav.helsearbeidsgiver.felles.domene.Arbeidsforhold
 import no.nav.helsearbeidsgiver.felles.domene.Person
 import no.nav.helsearbeidsgiver.felles.domene.ResultJson
 import no.nav.helsearbeidsgiver.felles.json.les
+import no.nav.helsearbeidsgiver.felles.json.orgMapSerializer
 import no.nav.helsearbeidsgiver.felles.json.personMapSerializer
 import no.nav.helsearbeidsgiver.felles.json.toJson
 import no.nav.helsearbeidsgiver.felles.rapidsrivers.KafkaKey
@@ -28,6 +28,7 @@ import no.nav.helsearbeidsgiver.utils.json.toJson
 import no.nav.helsearbeidsgiver.utils.log.logger
 import no.nav.helsearbeidsgiver.utils.log.sikkerLogger
 import no.nav.helsearbeidsgiver.utils.wrapper.Fnr
+import no.nav.helsearbeidsgiver.utils.wrapper.Orgnr
 import java.util.UUID
 
 data class Steg0(
@@ -47,7 +48,7 @@ sealed class Steg1 {
 }
 
 data class Steg2(
-    val virksomheter: Map<String, String>,
+    val virksomheter: Map<Orgnr, String>,
 )
 
 class AktiveOrgnrService(
@@ -89,7 +90,7 @@ class AktiveOrgnrService(
 
     override fun lesSteg2(melding: Map<Key, JsonElement>): Steg2 =
         Steg2(
-            virksomheter = Key.VIRKSOMHETER.les(MapSerializer(String.serializer(), String.serializer()), melding),
+            virksomheter = Key.VIRKSOMHETER.les(orgMapSerializer, melding),
         )
 
     override fun utfoerSteg0(
@@ -179,7 +180,7 @@ class AktiveOrgnrService(
             val gyldigeUnderenheter =
                 steg2.virksomheter.map {
                     AktiveArbeidsgivere.Arbeidsgiver(
-                        orgnrUnderenhet = it.key,
+                        orgnrUnderenhet = it.key.verdi,
                         virksomhetsnavn = it.value,
                     )
                 }

diff --git a/...in/no/nav/helsearbeidsgiver/inntektsmelding/api/hentforespoersel/HentForespoerselRoute.kt b/...in/no/nav/helsearbeidsgiver/inntektsmelding/api/hentforespoersel/HentForespoerselRoute.kt
@@ -100,8 +100,8 @@ private fun HentForespoerselResultat.toResponse(): HentForespoerselResponse {
             navn = sykmeldtNavn,
             innsenderNavn = avsenderNavn,
             orgNavn = orgNavn,
-            identitetsnummer = forespoersel.fnr,
-            orgnrUnderenhet = forespoersel.orgnr,
+            identitetsnummer = forespoersel.fnr.verdi,
+            orgnrUnderenhet = forespoersel.orgnr.verdi,
             fravaersperioder = forespoersel.sykmeldingsperioder,
             egenmeldingsperioder = forespoersel.egenmeldingsperioder,
             bestemmendeFravaersdag = forespoersel.forslagBestemmendeFravaersdag(),

diff --git a/...earbeidsgiver/inntektsmelding/api/hentforespoerselIdListe/HentForespoerselIdListeRoute.kt b/...earbeidsgiver/inntektsmelding/api/hentforespoerselIdListe/HentForespoerselIdListeRoute.kt
@@ -113,7 +113,7 @@ suspend fun PipelineContext<Unit, ApplicationCall>.hentForespoersler(
                 }
 
                 else -> {
-                    orgnrSet.firstOrNull()?.also { orgnr -> tilgangskontroll.validerTilgangTilOrg(call.request, orgnr) }
+                    orgnrSet.firstOrNull()?.also { orgnr -> tilgangskontroll.validerTilgangTilOrg(call.request, orgnr.verdi) }
 
                     val respons =
                         resultat.map { (id, forespoersel) ->

diff --git a/...nav/helsearbeidsgiver/inntektsmelding/api/hentforespoersel/HentForespoerselRouteKtTest.kt b/...nav/helsearbeidsgiver/inntektsmelding/api/hentforespoersel/HentForespoerselRouteKtTest.kt
@@ -36,6 +36,9 @@ import no.nav.helsearbeidsgiver.utils.test.date.februar
 import no.nav.helsearbeidsgiver.utils.test.date.januar
 import no.nav.helsearbeidsgiver.utils.test.date.mars
 import no.nav.helsearbeidsgiver.utils.test.json.removeJsonWhitespace
+import no.nav.helsearbeidsgiver.utils.test.wrapper.genererGyldig
+import no.nav.helsearbeidsgiver.utils.wrapper.Fnr
+import no.nav.helsearbeidsgiver.utils.wrapper.Orgnr
 import org.junit.jupiter.api.Assertions.assertEquals
 import org.junit.jupiter.api.Assertions.assertNotNull
 import org.junit.jupiter.api.Assertions.assertNull
@@ -148,12 +151,14 @@ class HentForespoerselRouteKtTest : ApiTest() {
 }
 
 private object Mock {
+    private val orgnr = Orgnr.genererGyldig()
+
     val request = HentForespoerselRequest(UUID.randomUUID())
 
     private val forespoersel =
         Forespoersel(
-            orgnr = "123",
-            fnr = "abc",
+            orgnr = orgnr,
+            fnr = Fnr.genererGyldig(),
             sykmeldingsperioder =
                 listOf(
                     1.april til 20.april,
@@ -164,7 +169,7 @@ private object Mock {
                     29.mars til 29.mars,
                     31.mars til 31.mars,
                 ),
-            bestemmendeFravaersdager = mapOf("123" to 25.april),
+            bestemmendeFravaersdager = mapOf(orgnr to 25.april),
             forespurtData = mockForespurtData(),
             erBesvart = false,
             vedtaksperiodeId = UUID.randomUUID(),

diff --git a/...dsgiver/inntektsmelding/api/hentforespoerselIdListe/HentForespoerselIdListeRouteKtTest.kt b/...dsgiver/inntektsmelding/api/hentforespoerselIdListe/HentForespoerselIdListeRouteKtTest.kt
@@ -294,16 +294,16 @@ class HentForespoerselIdListeRouteKtTest : ApiTest() {
 
 private object Mock {
     val orgnr = Orgnr.genererGyldig()
-    val vedtaksPeriodeId1 = UUID.randomUUID()
-    val forespoerselId1 = UUID.randomUUID()
+    val vedtaksPeriodeId1: UUID = UUID.randomUUID()
+    val forespoerselId1: UUID = UUID.randomUUID()
 
-    val vedtaksPeriodeId2 = UUID.randomUUID()
-    val forespoerselId2 = UUID.randomUUID()
+    val vedtaksPeriodeId2: UUID = UUID.randomUUID()
+    val forespoerselId2: UUID = UUID.randomUUID()
 
     fun mockResultat(): Map<UUID, Forespoersel> =
         mapOf(
-            forespoerselId1 to mockForespoersel().copy(vedtaksperiodeId = vedtaksPeriodeId1, orgnr = orgnr.toString()),
-            forespoerselId2 to mockForespoersel().copy(vedtaksperiodeId = vedtaksPeriodeId2, orgnr = orgnr.toString()),
+            forespoerselId1 to mockForespoersel().copy(vedtaksperiodeId = vedtaksPeriodeId1, orgnr = orgnr),
+            forespoerselId2 to mockForespoersel().copy(vedtaksperiodeId = vedtaksPeriodeId2, orgnr = orgnr),
         )
 
     fun mockResultatMedUlikeOrgnr(): Map<UUID, Forespoersel> =

diff --git a/...searbeidsgiver/inntektsmelding/berikinntektsmeldingservice/BerikInntektsmeldingService.kt b/...searbeidsgiver/inntektsmelding/berikinntektsmeldingservice/BerikInntektsmeldingService.kt
@@ -127,7 +127,7 @@ class BerikInntektsmeldingService(
                 Key.KONTEKST_ID to steg0.transaksjonId.toJson(),
                 Key.DATA to
                     data
-                        .plus(Key.ORGNR_UNDERENHETER to setOf(steg1.forespoersel.orgnr).toJson(String.serializer()))
+                        .plus(Key.ORGNR_UNDERENHETER to setOf(steg1.forespoersel.orgnr).toJson(Orgnr.serializer()))
                         .toJson(),
             ).also { loggBehovPublisert(BehovType.HENT_VIRKSOMHET_NAVN, it) }
     }
@@ -149,7 +149,7 @@ class BerikInntektsmeldingService(
                         .plus(
                             Key.FNR_LISTE to
                                 listOf(
-                                    steg1.forespoersel.fnr.let(::Fnr),
+                                    steg1.forespoersel.fnr,
                                     steg0.avsenderFnr,
                                 ).toJson(Fnr.serializer()),
                         ).toJson(),
@@ -163,8 +163,8 @@ class BerikInntektsmeldingService(
         steg2: Steg2,
         steg3: Steg3,
     ) {
-        val orgNavn = steg2.orgnrMedNavn[steg1.forespoersel.orgnr.let(::Orgnr)] ?: UKJENT_VIRKSOMHET
-        val sykmeldtNavn = steg3.personer[steg1.forespoersel.fnr.let(::Fnr)]?.navn ?: UKJENT_NAVN
+        val orgNavn = steg2.orgnrMedNavn[steg1.forespoersel.orgnr] ?: UKJENT_VIRKSOMHET
+        val sykmeldtNavn = steg3.personer[steg1.forespoersel.fnr]?.navn ?: UKJENT_NAVN
         val avsenderNavn = steg3.personer[steg0.avsenderFnr]?.navn ?: UKJENT_NAVN
         val aarsakInnsending = if (steg1.forespoersel.erBesvart) AarsakInnsending.Endring else AarsakInnsending.Ny
 

diff --git a/...o/nav/helsearbeidsgiver/inntektsmelding/berikinntektsmeldingservice/MapInntektsmelding.kt b/...o/nav/helsearbeidsgiver/inntektsmelding/berikinntektsmeldingservice/MapInntektsmelding.kt
@@ -8,8 +8,6 @@ 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.utils.wrapper.Fnr
-import no.nav.helsearbeidsgiver.utils.wrapper.Orgnr
 import java.time.ZonedDateTime
 import java.util.UUID
 
@@ -61,12 +59,12 @@ fun mapInntektsmelding(
             ),
         sykmeldt =
             Sykmeldt(
-                fnr = forespoersel.fnr.let(::Fnr),
+                fnr = forespoersel.fnr,
                 navn = sykmeldtNavn,
             ),
         avsender =
             Avsender(
-                orgnr = forespoersel.orgnr.let(::Orgnr),
+                orgnr = forespoersel.orgnr,
                 orgNavn = virksomhetNavn,
                 navn = avsenderNavn,
                 tlf = skjema.avsenderTlf,

diff --git a/...beidsgiver/inntektsmelding/berikinntektsmeldingservice/BerikInntektsmeldingServiceTest.kt b/...beidsgiver/inntektsmelding/berikinntektsmeldingservice/BerikInntektsmeldingServiceTest.kt
@@ -157,11 +157,11 @@ private object Mock {
 
     private val sykmeldt =
         Person(
-            fnr = Fnr(forespoersel.fnr),
+            fnr = forespoersel.fnr,
             navn = "Dolly Duck",
         )
 
-    private val orgnrMedNavn = mapOf(Orgnr(forespoersel.orgnr) to "Lasses kasserollesjappe")
+    private val orgnrMedNavn = mapOf(forespoersel.orgnr to "Lasses kasserollesjappe")
 
     private val personer =
         mapOf(

diff --git a/...helsearbeidsgiver/inntektsmelding/berikinntektsmeldingservice/MapInntektsmeldingKtTest.kt b/...helsearbeidsgiver/inntektsmelding/berikinntektsmeldingservice/MapInntektsmeldingKtTest.kt
@@ -21,8 +21,6 @@ import no.nav.helsearbeidsgiver.felles.test.mock.utenPaakrevdRefusjon
 import no.nav.helsearbeidsgiver.utils.test.date.august
 import no.nav.helsearbeidsgiver.utils.test.date.desember
 import no.nav.helsearbeidsgiver.utils.test.date.juli
-import no.nav.helsearbeidsgiver.utils.wrapper.Fnr
-import no.nav.helsearbeidsgiver.utils.wrapper.Orgnr
 import java.time.ZonedDateTime
 import kotlin.time.Duration.Companion.seconds
 import kotlin.time.toJavaDuration
@@ -57,13 +55,13 @@ class MapInntektsmeldingKtTest :
 
                     sykmeldt shouldBe
                         Sykmeldt(
-                            fnr = forespoersel.fnr.let(::Fnr),
+                            fnr = forespoersel.fnr,
                             navn = sykmeldtNavn,
                         )
 
                     avsender shouldBe
                         Avsender(
-                            orgnr = forespoersel.orgnr.let(::Orgnr),
+                            orgnr = forespoersel.orgnr,
                             orgNavn = virksomhetNavn,
                             navn = avsenderNavn,
                             tlf = skjema.avsenderTlf,

diff --git a/felles/src/main/kotlin/no/nav/helsearbeidsgiver/felles/domene/Forespoersel.kt b/felles/src/main/kotlin/no/nav/helsearbeidsgiver/felles/domene/Forespoersel.kt
@@ -8,17 +8,19 @@ import no.nav.helsearbeidsgiver.domene.inntektsmelding.v1.Periode
 import no.nav.helsearbeidsgiver.domene.inntektsmelding.v1.bestemmendeFravaersdag
 import no.nav.helsearbeidsgiver.utils.json.serializer.LocalDateSerializer
 import no.nav.helsearbeidsgiver.utils.json.serializer.UuidSerializer
+import no.nav.helsearbeidsgiver.utils.wrapper.Fnr
+import no.nav.helsearbeidsgiver.utils.wrapper.Orgnr
 import java.time.LocalDate
 import java.util.UUID
 
 @Serializable
 data class Forespoersel(
-    val orgnr: String,
-    val fnr: String,
+    val orgnr: Orgnr,
+    val fnr: Fnr,
     val vedtaksperiodeId: UUID,
     val sykmeldingsperioder: List<Periode>,
     val egenmeldingsperioder: List<Periode>,
-    val bestemmendeFravaersdager: Map<String, LocalDate>,
+    val bestemmendeFravaersdager: Map<Orgnr, LocalDate>,
     val forespurtData: ForespurtData,
     val erBesvart: Boolean,
     val opprettetUpresisIkkeBruk: LocalDate,

diff --git a/felles/src/main/kotlin/no/nav/helsearbeidsgiver/felles/domene/ForespoerselFraBro.kt b/felles/src/main/kotlin/no/nav/helsearbeidsgiver/felles/domene/ForespoerselFraBro.kt
@@ -28,12 +28,12 @@ data class ForespoerselFraBro(
 ) {
     fun toForespoersel(): Forespoersel =
         Forespoersel(
-            orgnr = orgnr.verdi,
-            fnr = fnr.verdi,
+            orgnr = orgnr,
+            fnr = fnr,
             vedtaksperiodeId = vedtaksperiodeId,
             sykmeldingsperioder = sykmeldingsperioder,
             egenmeldingsperioder = egenmeldingsperioder,
-            bestemmendeFravaersdager = bestemmendeFravaersdager.mapKeys { it.key.verdi },
+            bestemmendeFravaersdager = bestemmendeFravaersdager,
             forespurtData = forespurtData,
             erBesvart = erBesvart,
             opprettetUpresisIkkeBruk = opprettetUpresisIkkeBruk,