sender mer info i Feilinformasjon

navikt · Nov 20, 2024 · 34bddad · 34bddad
1 parent 4fc1152
commit 34bddad
Show file tree

Hide file tree

Showing 5 changed files with 18 additions and 13 deletions.
diff --git a/build.gradle.kts b/build.gradle.kts
@@ -3,7 +3,7 @@ import org.gradle.api.tasks.testing.logging.TestLogEvent
 import org.jetbrains.kotlin.gradle.dsl.JvmTarget
 import org.jetbrains.kotlin.gradle.tasks.KotlinCompile
 
-val domeneVersion = "2.0.9"
+val domeneVersion = "2.0.10"
 val navTokenSupportVersion = "5.0.8"
 val hibernateValidatorVersion = "8.0.1.Final"
 val logbackEncoderVersion = "8.0"

diff --git a/.../opptjening/omsorgsopptjening/start/innlesning/barnetrygd/domain/KompletteringsService.kt b/.../opptjening/omsorgsopptjening/start/innlesning/barnetrygd/domain/KompletteringsService.kt
@@ -60,6 +60,8 @@ class KompletteringsService(
                     .withFeilinformasjon(
                         Feilinformasjon.FeilIDataGrunnlag(
                             message = "Feil i datagrunnlag: ${e.message}",
+                            exceptionType = e::class.java.canonicalName,
+                            exceptionMessage = e.message ?: "",
                         )
                     )
             }
@@ -84,6 +86,8 @@ class KompletteringsService(
                 komplettering.withFeilinformasjon(
                     Feilinformasjon.OverlappendeBarnetrygdperioder(
                         message = "Overlappende barnetrygdperioder",
+                        exceptionType = e::class.java.canonicalName,
+                        exceptionMessage = e.message ?: "",
                     )
                 )
             }
@@ -96,7 +100,9 @@ class KompletteringsService(
                 secureLog.warn("Feil ved komprimering av persongrunnlag etter henting av barnetrygdgrunnlag", e)
                 komplettering.withFeilinformasjon(
                     Feilinformasjon.OverlappendeBarnetrygdperioder(
-                        message = "Overlappende barnetrygdperioder"
+                        message = "Overlappende barnetrygdperioder",
+                        exceptionType = e::class.java.canonicalName,
+                        exceptionMessage = e.message ?: "",
                     )
                 )
             }
@@ -109,7 +115,9 @@ class KompletteringsService(
                 secureLog.warn("Feil ved henting av hjelpestønadgrunnlag", e)
                 komplettering.withFeilinformasjon(
                     Feilinformasjon.FeilIDataGrunnlag(
-                        message = "Feil i datagrunnlag ved henting av hjelpestønadgrunnlag"
+                        message = "Feil i datagrunnlag ved henting av hjelpestønadgrunnlag",
+                        exceptionType = e::class.java.canonicalName,
+                        exceptionMessage = e.message ?: "",
                     )
                 )
             }
@@ -136,6 +144,8 @@ class KompletteringsService(
                 komplettering.withFeilinformasjon(
                     Feilinformasjon.OverlappendeHjelpestønadperioder(
                         message = "Overlappende hjelpestønadperioder",
+                        exceptionType = e::class.java.canonicalName,
+                        exceptionMessage = e.message ?: "",
                     )
                 )
             }
@@ -149,6 +159,8 @@ class KompletteringsService(
                 komplettering.withFeilinformasjon(
                     Feilinformasjon.OverlappendeHjelpestønadperioder(
                         message = "Overlappende hjelpestønadperioder",
+                        exceptionType = e::class.java.canonicalName,
+                        exceptionMessage = e.message ?: "",
                     )
                 )
             }

diff --git a/...omsorgsopptjening/start/innlesning/barnetrygd/external/hjelpestønad/HjelpestønadClient.kt b/...omsorgsopptjening/start/innlesning/barnetrygd/external/hjelpestønad/HjelpestønadClient.kt
@@ -30,9 +30,7 @@ class HjelpestønadClient(
     internal val metrikker: Metrikker,
     webClientBuilder: WebClient.Builder,
 ) {
-    private val webClient: WebClient = webClientBuilder.baseUrl(baseUrl).filter(
-        logRequest()
-    ).build()
+    private val webClient: WebClient = webClientBuilder.baseUrl(baseUrl).build()
 
     internal fun hentHjelpestønad(
         fnr: Ident,

diff --git a/...sjon/opptjening/omsorgsopptjening/start/innlesning/barnetrygd/external/pdl/PdlResponse.kt b/...sjon/opptjening/omsorgsopptjening/start/innlesning/barnetrygd/external/pdl/PdlResponse.kt
@@ -41,8 +41,6 @@ data class HentPersonQueryResponse(
     }
 
     fun toDomain(): PersonId {
-        println("identhistorikk:gjeldende: ${identhistorikk().gjeldende().ident}")
-        println("identhistorikk:historikk: ${identhistorikk().historikk()}")
         return PersonId(
             fnr = Ident(identhistorikk().gjeldende().ident),
             historiske = identhistorikk().historikk().map { it.ident }.toSet()

diff --git a/src/test/kotlin/no/nav/pensjon/opptjening/omsorgsopptjening/start/innlesning/EndToEndTest.kt b/src/test/kotlin/no/nav/pensjon/opptjening/omsorgsopptjening/start/innlesning/EndToEndTest.kt
@@ -2,6 +2,7 @@ package no.nav.pensjon.opptjening.omsorgsopptjening.start.innlesning
 
 import com.github.tomakehurst.wiremock.core.WireMockConfiguration
 import com.github.tomakehurst.wiremock.junit5.WireMockExtension
+import net.javacrumbs.jsonunit.assertj.JsonAssertions.assertThatJson
 import no.nav.pensjon.opptjening.omsorgsopptjening.felles.deserialize
 import no.nav.pensjon.opptjening.omsorgsopptjening.felles.domene.kafka.messages.domene.Kilde
 import no.nav.pensjon.opptjening.omsorgsopptjening.felles.domene.kafka.messages.domene.Landstilknytning
@@ -88,14 +89,10 @@ class EndToEndTest : SpringContextTest.WithKafka() {
         sendTilBestemService.sendTilBestem()
 
         listener.removeFirstRecord(3).let { consumerRecord ->
-            val expectedKey =
+            assertThatJson(consumerRecord.key()).isEqualTo(
                 """
                     {"ident":"12345678910"}
                 """.trimIndent()
-            JSONAssert.assertEquals(
-                consumerRecord.key(),
-                expectedKey,
-                true
             )
 
             deserialize<PersongrunnlagMelding>(consumerRecord.value()).also {