Legger til organisasjonsnavn til arbeidsgiver (#652)

* Legger til organisasjonsnavn til arbeidsgiver * fjerner unødvendig println --------- Co-authored-by: Stian S. Douzette <[email protected]>
navikt · Feb 26, 2024 · 20a4a82 · 20a4a82
1 parent 80d4601
commit 20a4a82
Show file tree

Hide file tree

Showing 12 changed files with 45 additions and 14 deletions.
diff --git a/src/main/kotlin/no/nav/medlemskap/domene/arbeidsforhold/Arbeidsgiver.kt b/src/main/kotlin/no/nav/medlemskap/domene/arbeidsforhold/Arbeidsgiver.kt
@@ -1,6 +1,7 @@
 package no.nav.medlemskap.domene.arbeidsforhold
 
 data class Arbeidsgiver(
+    val navn: String?,
     val organisasjonsnummer: String?,
     val ansatte: List<Ansatte>?,
     val konkursStatus: List<String?>?,

diff --git a/src/main/kotlin/no/nav/medlemskap/routes/EvalueringRoute.kt b/src/main/kotlin/no/nav/medlemskap/routes/EvalueringRoute.kt
@@ -313,10 +313,18 @@ private fun loggResponse(fnr: String, response: Response, endpoint: String = "/"
             kv("har_utflytting", response.datagrunnlag.pdlpersonhistorikk.harUtflyttingFraNorge()),
             kv("yrkeskoder", response.datagrunnlag.arbeidsforhold.alleAktiveYrkeskoderDerTomErNull()),
             kv("har_medlperiode_uten_arbeidsforhold", response.datagrunnlag.harPeriodeUtenMedlemskapOgIkkeArbeidsforhold()),
-            kv("Antall_ansatte_hos_arbeidsgiver", response.datagrunnlag.arbeidsforhold.antallAnsatteHosArbeidsgivere(
-                Kontrollperiode.kontrollPeriodeForArbeidsforhold(response.datagrunnlag.startDatoForYtelse))),
-            kv("antall_ansatte_for_juridiske_enheter", response.datagrunnlag.arbeidsforhold.antallAnsatteHosArbeidsgiversJuridiskeEnheter(
-                Kontrollperiode.kontrollPeriodeForArbeidsforhold(response.datagrunnlag.startDatoForYtelse))),
+            kv(
+                "Antall_ansatte_hos_arbeidsgiver",
+                response.datagrunnlag.arbeidsforhold.antallAnsatteHosArbeidsgivere(
+                    Kontrollperiode.kontrollPeriodeForArbeidsforhold(response.datagrunnlag.startDatoForYtelse)
+                )
+            ),
+            kv(
+                "antall_ansatte_for_juridiske_enheter",
+                response.datagrunnlag.arbeidsforhold.antallAnsatteHosArbeidsgiversJuridiskeEnheter(
+                    Kontrollperiode.kontrollPeriodeForArbeidsforhold(response.datagrunnlag.startDatoForYtelse)
+                )
+            ),
             kv("endpoint", endpoint)
             // kv("regleroverstyrt", response.resultat.erReglerOverstyrt())
         )

diff --git a/src/main/kotlin/no/nav/medlemskap/services/aareg/V2tilV1Converter.kt b/src/main/kotlin/no/nav/medlemskap/services/aareg/V2tilV1Converter.kt
@@ -14,7 +14,7 @@ fun List<Arbeidsforhold>.mapTilV1(): List<no.nav.medlemskap.domene.arbeidsforhol
             periode = mapAnsettelsesperiodeTilV1(it),
             utenlandsopphold = mapUtenlandsoppholdTilV1(it),
             arbeidsgivertype = mapOpplysningspliktigTypeTilV1(it.opplysningspliktig),
-            arbeidsgiver = Arbeidsgiver(null, null, null, null),
+            arbeidsgiver = Arbeidsgiver(null, null, null, null, null),
             arbeidsforholdstype = mapArbeidsforholdstypeTilV1(it.ansettelsesdetaljer.sortedBy { it.rapporteringsmaaneder.fra }.last()),
             arbeidsavtaler = mapAnsettelsesdetaljerTilArbeidsavtaler(it.ansettelsesdetaljer),
             permisjonPermittering = mapPermisjonPermitteringTilV1(it)

diff --git a/src/main/kotlin/no/nav/medlemskap/services/ereg/EregMapper.kt b/src/main/kotlin/no/nav/medlemskap/services/ereg/EregMapper.kt
@@ -1,5 +1,6 @@
 package no.nav.medlemskap.services.ereg
 
+import no.nav.medlemskap.clients.ereg.Navn
 import no.nav.medlemskap.clients.ereg.Organisasjon
 import no.nav.medlemskap.domene.Periode
 import no.nav.medlemskap.domene.arbeidsforhold.Ansatte
@@ -11,6 +12,7 @@ fun mapOrganisasjonTilArbeidsgiver(organisasjon: Organisasjon, juridiskeEnheter:
     val ansatte = organisasjon.organisasjonDetaljer?.ansatte
     val konkursStatus = organisasjon.organisasjonDetaljer?.statuser?.map { it -> it?.kode }
     return Arbeidsgiver(
+        navn = finnOrgNavn(organisasjon.navn),
         organisasjonsnummer = organisasjon.organisasjonsnummer,
         ansatte = mapAnsatte(ansatte),
         konkursStatus = konkursStatus,
@@ -41,3 +43,13 @@ fun mapGyldighetsperiode(gyldighetsperiode: no.nav.medlemskap.clients.ereg.Gyldi
 
 fun mapBruksperiode(bruksperiode: no.nav.medlemskap.clients.ereg.Bruksperiode?): Periode? =
     bruksperiode?.let { Periode(fom = it.fom, tom = it.tom) }
+
+fun finnOrgNavn(orgNavn: Navn?): String {
+    return orgNavn?.navnelinje1
+        ?: orgNavn?.navnelinje2
+        ?: orgNavn?.navnelinje3
+        ?: orgNavn?.navnelinje4
+        ?: orgNavn?.navnelinje5
+        ?: orgNavn?.redigertnavn
+        ?: "null"
+}
diff --git a/src/main/resources/lovme.yaml b/src/main/resources/lovme.yaml
@@ -162,6 +162,9 @@ components:
             $ref: "#/components/schemas/PermisjonPermittering"
     Arbeidsgiver:
       properties:
+        navn:
+          type: string
+          nullable: true
         organisasjonsnummer:
           type: string
           nullable: true

diff --git a/src/test/kotlin/no/nav/medlemskap/clients/aareg/AaregMapperTest.kt b/src/test/kotlin/no/nav/medlemskap/clients/aareg/AaregMapperTest.kt
@@ -30,6 +30,7 @@ class AaregMapperTest {
         juridiskEnhetOrgnummerEnhetstype["123456789"] = "AS"
 
         val arbeidsgiver = Arbeidsgiver(
+            navn = "null",
             organisasjonsnummer = "985672744",
             ansatte = listOf(
                 Ansatte(

diff --git a/src/test/kotlin/no/nav/medlemskap/cucumber/DomenespråkParser.kt b/src/test/kotlin/no/nav/medlemskap/cucumber/DomenespråkParser.kt
@@ -20,7 +20,7 @@ import java.time.YearMonth
 object DomenespråkParser : BasisDomeneParser() {
     val ANSATTE_9 = listOf(Ansatte(9, null))
     val VANLIG_NORSK_ARBEIDSGIVER =
-        Arbeidsgiver(organisasjonsnummer = "1", ansatte = ANSATTE_9, konkursStatus = null, juridiskeEnheter = null)
+        Arbeidsgiver(navn = "null", organisasjonsnummer = "1", ansatte = ANSATTE_9, konkursStatus = null, juridiskeEnheter = null)
 
     fun parseValgfriYtelse(domenebegrep: Domenebegrep, rad: Map<String, String>): Ytelse? {
         val valgfriVerdi = valgfriVerdi(domenebegrep.nøkkel, rad)

diff --git a/.../kotlin/no/nav/medlemskap/cucumber/SpraakParserDomene/ArbeidsforholdDomeneSpraakParser.kt b/.../kotlin/no/nav/medlemskap/cucumber/SpraakParserDomene/ArbeidsforholdDomeneSpraakParser.kt
@@ -8,7 +8,7 @@ import java.time.YearMonth
 
 object ArbeidsforholdDomeneSpraakParser : BasisDomeneParser() {
     val ANSATTE_9 = listOf(Ansatte(9, null))
-    val VANLIG_NORSK_ARBEIDSGIVER = Arbeidsgiver(organisasjonsnummer = "1", ansatte = ANSATTE_9, konkursStatus = null, juridiskeEnheter = null)
+    val VANLIG_NORSK_ARBEIDSGIVER = Arbeidsgiver(navn = "null", organisasjonsnummer = "1", ansatte = ANSATTE_9, konkursStatus = null, juridiskeEnheter = null)
 
     fun mapArbeidsforhold(
         dataTable: DataTable,
@@ -301,6 +301,7 @@ object ArbeidsforholdDomeneSpraakParser : BasisDomeneParser() {
             }
 
             return Arbeidsgiver(
+                navn = parseValgfriString(ArbeidDomenebegrep.Navn, rad),
                 organisasjonsnummer = parseValgfriString(ArbeidDomenebegrep.IDENTIFIKATOR, rad),
                 ansatte = listOf(Ansatte(parseValgfriInt(ArbeidDomenebegrep.ANTALL_ANSATTE, rad), null)),
                 konkursStatus = konkursStatuser,
@@ -339,6 +340,7 @@ object ArbeidsforholdDomeneSpraakParser : BasisDomeneParser() {
 }
 
 enum class ArbeidDomenebegrep(val nøkkel: String) : Domenenøkkel {
+    Navn("Navn"),
     ANTALL_ANSATTE("Antall ansatte"),
     ARBEIDSFORHOLDSTYPE("Arbeidsforholdstype"),
     ARBEIDSGIVER_ID("Arbeidsgiver Id"),

diff --git a/src/test/kotlin/no/nav/medlemskap/cucumber/steps/RegelSteps.kt b/src/test/kotlin/no/nav/medlemskap/cucumber/steps/RegelSteps.kt
@@ -40,6 +40,7 @@ import java.io.File
 class RegelSteps : No {
     private val ANSATTE_9 = listOf(Ansatte(9, null))
     private val VANLIG_NORSK_ARBEIDSGIVER = Arbeidsgiver(
+        navn = "null",
         organisasjonsnummer = "1",
         ansatte = ANSATTE_9,
         konkursStatus = null,

diff --git a/src/test/kotlin/no/nav/medlemskap/domene/arbeidsforhold/ArbeidsforholdTest.kt b/src/test/kotlin/no/nav/medlemskap/domene/arbeidsforhold/ArbeidsforholdTest.kt
@@ -426,7 +426,7 @@ class ArbeidsforholdTest {
             arbeidsforholdPeriode,
             null,
             OpplysningspliktigArbeidsgiverType.Organisasjon,
-            Arbeidsgiver(null, listOf(Ansatte(antall = 10, gyldighetsperiode = null)), null, null),
+            Arbeidsgiver(null, null, listOf(Ansatte(antall = 10, gyldighetsperiode = null)), null, null),
             Arbeidsforholdstype.NORMALT,
             listOf(arbeidsavtale),
             emptyList()
@@ -462,6 +462,7 @@ class ArbeidsforholdTest {
             utenlandsopphold = null,
             arbeidsgivertype = OpplysningspliktigArbeidsgiverType.Organisasjon,
             arbeidsgiver = Arbeidsgiver(
+                navn = null,
                 organisasjonsnummer = null,
                 ansatte = listOf(Ansatte(antall = antall, gyldighetsperiode = null)),
                 konkursStatus = null,

diff --git a/src/test/kotlin/no/nav/medlemskap/services/FamilieServiceTest.kt b/src/test/kotlin/no/nav/medlemskap/services/FamilieServiceTest.kt
@@ -142,7 +142,7 @@ internal class FamilieServiceTest {
             periode = Periode(null, null),
             utenlandsopphold = null,
             arbeidsgivertype = OpplysningspliktigArbeidsgiverType.Organisasjon,
-            arbeidsgiver = Arbeidsgiver("123", listOf(Ansatte(5, null)), null, null),
+            arbeidsgiver = Arbeidsgiver("Navn", "123", listOf(Ansatte(5, null)), null, null),
             arbeidsforholdstype = Arbeidsforholdstype.NORMALT,
             arbeidsavtaler = emptyList(),
             permisjonPermittering = null

diff --git a/src/test/resources/testpersoner/bakoverkompatibeltest/forventetRespons.json b/src/test/resources/testpersoner/bakoverkompatibeltest/forventetRespons.json
@@ -1,6 +1,6 @@
 {
-  "tidspunkt" : "2023-09-27T11:04:57.238412",
-  "vurderingsID": "7b006d2c-ff75-4404-9663-7babf23f45de",
+  "vurderingsID" : "7b006d2c-ff75-4404-9663-7babf23f45de",
+  "tidspunkt" : "2024-02-26T09:46:08.89591",
   "versjonTjeneste" : "",
   "versjonRegler" : "v1",
   "kanal" : "/",
@@ -13,10 +13,10 @@
     "førsteDagForYtelse" : "2019-01-01",
     "brukerinput" : {
       "arbeidUtenforNorge" : false,
-      "oppholdUtenforEos" : null,
-      "oppholdUtenforNorge" : null,
       "oppholdstilatelse" : null,
-      "utfortAarbeidUtenforNorge" : null
+      "utfortAarbeidUtenforNorge" : null,
+      "oppholdUtenforEos" : null,
+      "oppholdUtenforNorge" : null
     },
     "pdlpersonhistorikk" : {
       "statsborgerskap" : [ {
@@ -83,6 +83,7 @@
       } ],
       "arbeidsgivertype" : "Organisasjon",
       "arbeidsgiver" : {
+        "navn" : null,
         "organisasjonsnummer" : "organisasjonsnummer",
         "ansatte" : [ {
           "antall" : 10,
@@ -188,6 +189,7 @@
         "utenlandsopphold" : [ ],
         "arbeidsgivertype" : "Organisasjon",
         "arbeidsgiver" : {
+          "navn" : "null",
           "organisasjonsnummer" : "1",
           "ansatte" : [ {
             "antall" : 9,