Merge pull request #286 from navikt/fix

Micrometer-revert + litt cleanup

Author: bjornhun

build.gradle.kts

@@ -36,7 +36,7 @@ dependencies {
     val ktorVersion = "2.3.11"
     val logbackVersion = "1.5.6"
     val logstashVersion = "7.4"
-    val micrometerVersion = "1.13.0"
+    val micrometerVersion = "1.12.5"
     val mockkVersion = "1.13.11"
     val navSecurityVersion = "4.1.7"
     val kotlinVersion = "1.9.23"

src/main/kotlin/no/nav/personopplysninger/common/consumer/kodeverk/dto/Kodeverk.kt

@@ -8,8 +8,6 @@ class Kodeverk(
     val navn: String,
     val koder: List<Kode>
 ) {
-    private val log = LoggerFactory.getLogger(Kodeverk::class.java)
-
     private val koderByNavn: Map<String, Kode> by lazy {
         koder.associateBy { it.navn }
     }
@@ -51,6 +49,8 @@ class Kodeverk(
     }
 
     companion object {
+        private val log = LoggerFactory.getLogger(Kodeverk::class.java)
+
         fun fromKoderBetydningerResponse(navn: String, response: GetKodeverkKoderBetydningerResponse): Kodeverk {
             return response.betydninger
                 .entries

src/main/kotlin/no/nav/personopplysninger/endreopplysninger/consumer/PdlMottakConsumer.kt

@@ -108,8 +108,7 @@ class PdlMottakConsumer(
                     header(HEADER_NAV_CALL_ID, UUID.randomUUID())
                     header(HEADER_NAV_CONSUMER_ID, CONSUMER_ID)
                 }
-            val endringList = response.body<List<Endring>>()
-            endring = endringList[0]
+            endring = response.body<List<Endring>>().first()
         } while (++i < MAX_POLLS && endring.isPending())
         log.info("Antall polls for status: $i")
 

src/main/kotlin/no/nav/personopplysninger/personalia/PersonaliaService.kt

@@ -50,10 +50,8 @@ class PersonaliaService(
             foedelandterm = kodeverkService.hentLandKoder().term(pdlPerson.foedsel.firstOrNull()?.foedeland)
             gtLandterm = kodeverkService.hentLandKoder().term(pdlGeografiskTilknytning?.gtLand)
             statsborgerskaptermer = hentGyldigeStatsborgerskap(pdlPerson.statsborgerskap)
-            utenlandskbanklandterm =
-                if (inboundKonto?.utenlandskKontoInfo != null) hentLandKodeterm(inboundKonto.utenlandskKontoInfo.bankLandkode) else null
-            utenlandskbankvalutaterm =
-                if (inboundKonto?.utenlandskKontoInfo != null) hentValutaKodeterm(inboundKonto.utenlandskKontoInfo.valutakode) else null
+            utenlandskbanklandterm = inboundKonto?.utenlandskKontoInfo?.let { hentLandKodeterm(it.bankLandkode) }
+            utenlandskbankvalutaterm = inboundKonto?.utenlandskKontoInfo?.let { hentValutaKodeterm(it.valutakode) }
             kontaktadresseKodeverk =
                 kontaktadresse.map { hentAdresseKodeverk(it.postnummer, it.landkode, it.kommunenummer) }
             bostedsadresseKodeverk =
@@ -67,7 +65,7 @@ class PersonaliaService(
 
     private suspend fun hentGyldigeStatsborgerskap(statsborgerskap: List<PdlStatsborgerskap>): List<String> {
         return statsborgerskap
-            .filter { it.land != "XUK" && it.gyldigTilOgMed?.isBefore(LocalDate.now()) != true } // Filtrer ut ukjent og ugyldige
+            .filter { it.land != UKJENT_LAND && it.gyldigTilOgMed?.isBefore(LocalDate.now()) != true } // Filtrer ut ukjent og ugyldige
             .map { kodeverkService.hentStatsborgerskap().term(it.land) }
             .filter { it.isNotEmpty() }
     }
@@ -99,5 +97,9 @@ class PersonaliaService(
     private suspend fun hentLandKodeterm(kode: String?): String? {
         return kontoregisterConsumer.hentLandkoder().find { land -> kode == land.kode }?.tekst
     }
+
+    companion object {
+        private const val UKJENT_LAND = "XUK"
+    }
 }