Legg til henting av orgnavn fra enhetsregister og map i response til frontend

Author: ArselanSultani

dp-inntekt-api/src/main/kotlin/no/nav/dagpenger/inntekt/InntektApi.kt

@@ -107,7 +107,8 @@ internal fun Application.inntektApi(
         }
         exception<InntektskomponentenHttpClientException> { call, cause ->
             val statusCode =
-                if (HttpStatusCode.fromValue(cause.status)
+                if (HttpStatusCode
+                        .fromValue(cause.status)
                         .isSuccess()
                 ) {
                     HttpStatusCode.InternalServerError
@@ -218,7 +219,7 @@ internal fun Application.inntektApi(
     routing {
         route("/v1") {
             route("/inntekt") {
-                uklassifisertInntekt(inntektskomponentHttpClient, inntektStore, personOppslag)
+                uklassifisertInntekt(inntektskomponentHttpClient, inntektStore, personOppslag, enhetsregisterClient)
             }
             opptjeningsperiodeApi(inntektStore)
             enhetsregisteret(enhetsregisterClient)

dp-inntekt-api/src/main/kotlin/no/nav/dagpenger/inntekt/mapping/GUIInntektsKomponentResponse.kt

@@ -20,7 +20,10 @@ data class GUIInntekt(
     val inntektsmottaker: Inntektsmottaker? = null,
 )
 
-data class Inntektsmottaker(val pnr: String?, val navn: String?)
+data class Inntektsmottaker(
+    val pnr: String?,
+    val navn: String?,
+)
 
 data class GUIInntektsKomponentResponse(
     val fraDato: YearMonth?,
@@ -62,3 +65,8 @@ data class InntektMedVerdikode(
     val tilleggsinformasjon: TilleggInformasjon? = null,
     val verdikode: String,
 )
+
+data class OrganisasjonNavnOgIdMapping(
+    val organisasjonsnummer: String,
+    val organisasjonNavn: String,
+)

dp-inntekt-api/src/main/kotlin/no/nav/dagpenger/inntekt/mapping/MapToInntektFrontend.kt

@@ -9,19 +9,20 @@ import no.nav.dagpenger.inntekt.inntektskomponenten.v1.TilleggInformasjon
 import java.math.BigDecimal
 import java.time.YearMonth
 
-fun mapToInntektFrontend(
-    inntektResponse: InntektkomponentResponse,
+fun InntektkomponentResponse.mapToFrontend(
     person: Inntektsmottaker,
+    organisasjonsInfoListe: MutableList<OrganisasjonNavnOgIdMapping>,
 ): InntektForVirksomhetMedPersonInformasjon {
-    val inntekt = inntektResponse.arbeidsInntektMaaned
-    var virksomhetListe: MutableList<InntektForVirksomhet> = mutableListOf()
+    val inntekt = arbeidsInntektMaaned
+    val virksomhetListe: MutableList<InntektForVirksomhet> = mutableListOf()
 
     inntekt?.forEach { arbeidsInntektMaaned ->
         val inntektsInformasjon = arbeidsInntektMaaned.arbeidsInntektInformasjon
         inntektsInformasjon?.inntektListe?.forEach { inntekt ->
             val virksomhet = inntekt.virksomhet
-            // TODO: Replace with actual logic to get virksomhetNavn
-            val virksomhetNavn = "testNavn"
+            val virksomhetNavn =
+                organisasjonsInfoListe.find { it.organisasjonsnummer == virksomhet?.identifikator }?.organisasjonNavn
+                    ?: ""
             val inntekter = mutableListOf<InntektVirksomhetMaaned>()
             inntekter.add(
                 InntektVirksomhetMaaned(
@@ -62,7 +63,7 @@ fun mapToInntektFrontend(
             } else {
                 virksomhetListe.add(
                     InntektForVirksomhet(
-                        virksomhet = virksomhet!!.identifikator,
+                        virksomhet = virksomhet?.identifikator ?: "",
                         virksomhetNavn = virksomhetNavn,
                         periode =
                             InntektForVirksomhetPeriode(
@@ -71,7 +72,7 @@ fun mapToInntektFrontend(
                             ),
                         inntekter = inntekter,
                         avvikListe =
-                            arbeidsInntektMaaned.avvikListe?.filter { it.virksomhet?.identifikator == virksomhet.identifikator }
+                            arbeidsInntektMaaned.avvikListe?.filter { it.virksomhet?.identifikator == virksomhet?.identifikator }
                                 ?: emptyList(),
                     ),
                 )

dp-inntekt-api/src/main/kotlin/no/nav/dagpenger/inntekt/v1/UklassifisertInntektRoute.kt

@@ -28,13 +28,15 @@ import no.nav.dagpenger.inntekt.inntektskomponenten.v1.InntektkomponentRequest
 import no.nav.dagpenger.inntekt.inntektskomponenten.v1.InntektskomponentClient
 import no.nav.dagpenger.inntekt.mapping.GUIInntekt
 import no.nav.dagpenger.inntekt.mapping.Inntektsmottaker
+import no.nav.dagpenger.inntekt.mapping.OrganisasjonNavnOgIdMapping
 import no.nav.dagpenger.inntekt.mapping.dataGrunnlagKlassifiseringToVerdikode
 import no.nav.dagpenger.inntekt.mapping.mapToDetachedInntekt
+import no.nav.dagpenger.inntekt.mapping.mapToFrontend
 import no.nav.dagpenger.inntekt.mapping.mapToGUIInntekt
-import no.nav.dagpenger.inntekt.mapping.mapToInntektFrontend
 import no.nav.dagpenger.inntekt.mapping.mapToStoredInntekt
 import no.nav.dagpenger.inntekt.oppslag.Person
 import no.nav.dagpenger.inntekt.oppslag.PersonOppslag
+import no.nav.dagpenger.inntekt.oppslag.enhetsregister.EnhetsregisterClient
 import no.nav.dagpenger.inntekt.opptjeningsperiode.Opptjeningsperiode
 import java.time.LocalDate
 
@@ -65,6 +67,7 @@ fun Route.uklassifisertInntekt(
     inntektskomponentClient: InntektskomponentClient,
     inntektStore: InntektStore,
     personOppslag: PersonOppslag,
+    enhetsregisterClient: EnhetsregisterClient,
 ) {
     authenticate("azure") {
         route("/uklassifisert/{aktørId}/{kontekstType}/{kontekstId}/{beregningsDato}") {
@@ -145,8 +148,17 @@ fun Route.uklassifisertInntekt(
                         .let {
                             val person = personOppslag.hentPerson(it.fødselsnummer)
                             val inntektsmottaker = Inntektsmottaker(it.fødselsnummer, person.sammensattNavn())
-                            mapToInntektFrontend(it.inntekt, inntektsmottaker)
-                        }?.let {
+                            val hentOrganisasjonsInfoListe =
+                                hentOrganisasjonNavn(
+                                    enhetsregisterClient,
+                                    it.inntekt.arbeidsInntektMaaned
+                                        ?.flatMap { it.arbeidsInntektInformasjon?.inntektListe.orEmpty() }
+                                        ?.mapNotNull { it.virksomhet?.identifikator }
+                                        ?.toTypedArray()
+                                        ?.toList() ?: emptyList(),
+                                )
+                            it.inntekt.mapToFrontend(inntektsmottaker, hentOrganisasjonsInfoListe)
+                        }.let {
                             call.respond(HttpStatusCode.OK, it)
                         }
                 }
@@ -228,6 +240,29 @@ fun Route.uklassifisertInntekt(
     }
 }
 
+private suspend fun hentOrganisasjonNavn(
+    enhetsregisterClient: EnhetsregisterClient,
+    organisasjonsNummerListe: List<String>?,
+): MutableList<OrganisasjonNavnOgIdMapping> {
+    val organisasjonNavnOgIdMappingListe = mutableListOf<OrganisasjonNavnOgIdMapping>()
+    organisasjonsNummerListe?.forEach { orgNr ->
+        runCatching {
+            enhetsregisterClient.hentEnhet(orgNr)
+        }.onFailure {
+            logger.error(it) { "Feil ved henting av organisasjonsnavn for $it" }
+        }.onSuccess {
+            val organisasjonsNavnOgIdMapping =
+                OrganisasjonNavnOgIdMapping(
+                    organisasjonsnummer = it,
+                    organisasjonNavn = it,
+                )
+            organisasjonNavnOgIdMappingListe.add(organisasjonsNavnOgIdMapping)
+        }
+    }
+
+    return organisasjonNavnOgIdMappingListe
+}
+
 private fun ApplicationCall.getSubject(): String {
     return runCatching {
         this.authentication

dp-inntekt-api/src/test/kotlin/no/nav/dagpenger/inntekt/mapping/MapToInntektFrontendTest.kt

@@ -169,13 +169,25 @@ val mottaker =
         navn = "Ola Nordmann",
     )
 
+val organisasjonNavnOgIdMappingForTesting =
+    mutableListOf(
+        OrganisasjonNavnOgIdMapping(
+            organisasjonsnummer = "896929119",
+            organisasjonNavn = "Test Org 119",
+        ),
+        OrganisasjonNavnOgIdMapping(
+            organisasjonsnummer = "896929120",
+            organisasjonNavn = "Test Org 120",
+        ),
+    )
+
 class MapToInntektFrontendTest {
     @Test
     fun `Map inntekt til InntektForVirksomhetMedPersonInformasjon`() {
         val mappedToInntektFrontend =
-            mapToInntektFrontend(
-                inntektkomponentResponse,
+            inntektkomponentResponse.mapToFrontend(
                 mottaker,
+                organisasjonNavnOgIdMappingForTesting,
             )
 
         assertEquals(2, mappedToInntektFrontend.inntektVirksomhetMaaned.size)