Rename klasse til Organisasjon og flytt til mapping

Co-authored-by: Vegard Skjefstad <[email protected]>

Author: JMLindseth

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

@@ -65,8 +65,3 @@ 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

@@ -13,7 +13,7 @@ import java.time.YearMonth
 
 fun InntektkomponentResponse.mapToFrontend(
     person: Inntektsmottaker,
-    organisasjonsInfoListe: List<OrganisasjonNavnOgIdMapping>,
+    organisasjoner: List<Organisasjon>,
 ): InntekterDto {
     val inntekt = arbeidsInntektMaaned
     val virksomheter: MutableList<Virksomhet> = mutableListOf()
@@ -22,9 +22,7 @@ fun InntektkomponentResponse.mapToFrontend(
         val inntektsInformasjon = arbeidsInntektMaaned.arbeidsInntektInformasjon
         inntektsInformasjon?.inntektListe?.forEach { inntekt ->
             val virksomhet = inntekt.virksomhet
-            val virksomhetNavn =
-                organisasjonsInfoListe.find { it.organisasjonsnummer == virksomhet?.identifikator }?.organisasjonNavn
-                    ?: ""
+            val virksomhetNavn = organisasjoner.find { it.organisasjonsnummer == virksomhet?.identifikator }?.navn ?: ""
             val inntekter = mutableListOf<InntektMaaned>()
             inntekter.add(
                 InntektMaaned(
@@ -143,3 +141,8 @@ data class InntektMaaned(
     val begrunnelse: String,
     val aarMaaned: YearMonth,
 )
+
+data class Organisasjon(
+    val organisasjonsnummer: String,
+    val navn: String,
+)

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

@@ -30,7 +30,7 @@ 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.Organisasjon
 import no.nav.dagpenger.inntekt.mapping.dataGrunnlagKlassifiseringToVerdikode
 import no.nav.dagpenger.inntekt.mapping.mapToDetachedInntekt
 import no.nav.dagpenger.inntekt.mapping.mapToFrontend
@@ -150,8 +150,8 @@ fun Route.uklassifisertInntekt(
                         .let {
                             val person = personOppslag.hentPerson(it.fødselsnummer)
                             val inntektsmottaker = Inntektsmottaker(it.fødselsnummer, person.sammensattNavn())
-                            val hentOrganisasjonsInfoListe =
-                                hentOrganisasjonNavn(
+                            val organisasjoner =
+                                hentOrganisasjoner(
                                     enhetsregisterClient,
                                     it.inntekt.arbeidsInntektMaaned
                                         ?.flatMap { it.arbeidsInntektInformasjon?.inntektListe.orEmpty() }
@@ -162,7 +162,7 @@ fun Route.uklassifisertInntekt(
                                         ?.toTypedArray()
                                         ?.toList() ?: emptyList(),
                                 )
-                            it.inntekt.mapToFrontend(inntektsmottaker, hentOrganisasjonsInfoListe)
+                            it.inntekt.mapToFrontend(inntektsmottaker, organisasjoner)
                         }.let {
                             call.respond(HttpStatusCode.OK, it)
                         }
@@ -280,27 +280,27 @@ fun Route.uklassifisertInntekt(
     }
 }
 
-private suspend fun hentOrganisasjonNavn(
+private suspend fun hentOrganisasjoner(
     enhetsregisterClient: EnhetsregisterClient,
     organisasjonsNummerListe: List<String>?,
-): MutableList<OrganisasjonNavnOgIdMapping> {
-    val organisasjonNavnOgIdMappingListe = mutableListOf<OrganisasjonNavnOgIdMapping>()
+): List<Organisasjon> {
+    val organisasjoner = mutableListOf<Organisasjon>()
     organisasjonsNummerListe?.forEach { orgNr ->
         runCatching {
             enhetsregisterClient.hentEnhet(orgNr)
         }.onFailure {
             logger.error(it) { "Feil ved henting av organisasjonsnavn for $it" }
         }.onSuccess {
             val organisasjonsNavnOgIdMapping =
-                OrganisasjonNavnOgIdMapping(
+                Organisasjon(
                     organisasjonsnummer = orgNr,
-                    organisasjonNavn = it,
+                    navn = it,
                 )
-            organisasjonNavnOgIdMappingListe.add(organisasjonsNavnOgIdMapping)
+            organisasjoner.add(organisasjonsNavnOgIdMapping)
         }
     }
 
-    return organisasjonNavnOgIdMappingListe
+    return organisasjoner
 }
 
 private fun ApplicationCall.getSubject(): String {

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

@@ -170,15 +170,15 @@ val mottaker =
         navn = "Ola Nordmann",
     )
 
-val organisasjonNavnOgIdMappingForTesting =
+val organisasjoner =
     mutableListOf(
-        OrganisasjonNavnOgIdMapping(
+        Organisasjon(
             organisasjonsnummer = "896929119",
-            organisasjonNavn = "Test Org 119",
+            navn = "Test Org 119",
         ),
-        OrganisasjonNavnOgIdMapping(
+        Organisasjon(
             organisasjonsnummer = "896929120",
-            organisasjonNavn = "Test Org 120",
+            navn = "Test Org 120",
         ),
     )
 
@@ -188,7 +188,7 @@ class MapToInntektFrontendTest {
         val mappedToInntektFrontend =
             inntektkomponentResponse.mapToFrontend(
                 mottaker,
-                organisasjonNavnOgIdMappingForTesting,
+                organisasjoner,
             )
 
         assertEquals(2, mappedToInntektFrontend.virksomheter.size)
@@ -331,7 +331,7 @@ class MapToInntektFrontendTest {
             )
 
         val mapTilFrontendMedNullVirksomhet =
-            inntektkomponentResponseMedTomVirksomhet.mapToFrontend(mottaker, organisasjonNavnOgIdMappingForTesting)
+            inntektkomponentResponseMedTomVirksomhet.mapToFrontend(mottaker, organisasjoner)
         assertEquals(3, mapTilFrontendMedNullVirksomhet.virksomheter.size)
         assertEquals(2, mapTilFrontendMedNullVirksomhet.virksomheter.filter { it.virksomhetsnummer == "" }.size)
         assertEquals(1, mapTilFrontendMedNullVirksomhet.virksomheter.filter { it.virksomhetsnummer == "896929120" }.size)