From 34bddadad5e7ec3c3e66ce1f8b6650d80ce1d0b4 Mon Sep 17 00:00:00 2001 From: Jan Kroken Date: Wed, 20 Nov 2024 12:41:57 +0100 Subject: [PATCH] sender mer info i Feilinformasjon --- build.gradle.kts | 2 +- .../barnetrygd/domain/KompletteringsService.kt | 16 ++++++++++++++-- .../Hjelpest\303\270nadClient.kt" | 4 +--- .../barnetrygd/external/pdl/PdlResponse.kt | 2 -- .../start/innlesning/EndToEndTest.kt | 7 ++----- 5 files changed, 18 insertions(+), 13 deletions(-) diff --git a/build.gradle.kts b/build.gradle.kts index 021882a..83232ea 100644 --- 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/src/main/kotlin/no/nav/pensjon/opptjening/omsorgsopptjening/start/innlesning/barnetrygd/domain/KompletteringsService.kt b/src/main/kotlin/no/nav/pensjon/opptjening/omsorgsopptjening/start/innlesning/barnetrygd/domain/KompletteringsService.kt index 300463d..f2fa366 100644 --- a/src/main/kotlin/no/nav/pensjon/opptjening/omsorgsopptjening/start/innlesning/barnetrygd/domain/KompletteringsService.kt +++ b/src/main/kotlin/no/nav/pensjon/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/src/main/kotlin/no/nav/pensjon/opptjening/omsorgsopptjening/start/innlesning/barnetrygd/external/hjelpest\303\270nad/Hjelpest\303\270nadClient.kt" "b/src/main/kotlin/no/nav/pensjon/opptjening/omsorgsopptjening/start/innlesning/barnetrygd/external/hjelpest\303\270nad/Hjelpest\303\270nadClient.kt" index 1478165..70417f7 100644 --- "a/src/main/kotlin/no/nav/pensjon/opptjening/omsorgsopptjening/start/innlesning/barnetrygd/external/hjelpest\303\270nad/Hjelpest\303\270nadClient.kt" +++ "b/src/main/kotlin/no/nav/pensjon/opptjening/omsorgsopptjening/start/innlesning/barnetrygd/external/hjelpest\303\270nad/Hjelpest\303\270nadClient.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/src/main/kotlin/no/nav/pensjon/opptjening/omsorgsopptjening/start/innlesning/barnetrygd/external/pdl/PdlResponse.kt b/src/main/kotlin/no/nav/pensjon/opptjening/omsorgsopptjening/start/innlesning/barnetrygd/external/pdl/PdlResponse.kt index 94324f5..4d33c44 100644 --- a/src/main/kotlin/no/nav/pensjon/opptjening/omsorgsopptjening/start/innlesning/barnetrygd/external/pdl/PdlResponse.kt +++ b/src/main/kotlin/no/nav/pensjon/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 index ca3fc8a..d911fed 100644 --- 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(consumerRecord.value()).also {