litt endring på feilhåndtering/logging

navikt · Nov 21, 2024 · c75f7a7 · c75f7a7
1 parent 9b04870
commit c75f7a7
Show file tree

Hide file tree

Showing 2 changed files with 16 additions and 8 deletions.
diff --git a/.../opptjening/omsorgsopptjening/start/innlesning/barnetrygd/domain/KompletteringsService.kt b/.../opptjening/omsorgsopptjening/start/innlesning/barnetrygd/domain/KompletteringsService.kt
@@ -55,14 +55,15 @@ class KompletteringsService(
                     hentBarnetrygd(komplettering.barnetrygdmottaker, gyldigÅrsIntervall)
                 )
             } catch (e: BarnetrygdException.OverlappendePerioder) {
-                secureLog.warn("Feil ved henting av barnetrygd", e)
+                secureLog.warn("Feil ved henting av barnetrygd. Overlappende perioder: ${e.perioder}", e)
                 komplettering
                     .withLøseRådata(e.rådata)
                     .withFeilinformasjon(
-                        Feilinformasjon.FeilIDataGrunnlag(
+                        Feilinformasjon.OverlappendeBarnetrygdperioder(
                             message = "Feil i datagrunnlag: ${e.message}",
                             exceptionType = e::class.java.canonicalName,
                             exceptionMessage = e.message ?: "",
+                            perioder = e.perioder,
                         )
                     )
             } catch (e: BarnetrygdException.FeilIGrunnlagsdata) {
@@ -132,7 +133,7 @@ class KompletteringsService(
                     hentHjelpestønadGrunnlag(komplettering.persongrunnlag!!, gyldigÅrsIntervall)
                 )
             } catch (e: UgyldigPersongrunnlag.OverlappendeOmsorgsperiode) {
-                secureLog.warn("Overlappende perioder ved henting av hjelpestønadsgrunnlag", e)
+                secureLog.warn("Overlappende perioder ved henting av hjelpestønadsgrunnlag: ${e.perioder}", e)
                 komplettering.withFeilinformasjon(
                     Feilinformasjon.OverlappendeHjelpestønadperioder(
                         message = "Overlappende perioder ved henting av hjelpestønadsgrunnlag",
@@ -170,7 +171,10 @@ class KompletteringsService(
                     )
                 )
             } catch (e: BarnetrygdException.OverlappendePerioder) {
-                secureLog.warn("Feil ved oppdatering av fødselsnummer etter henting av hjelpestønadgrunnlag", e)
+                secureLog.warn(
+                    "Feil ved oppdatering av fødselsnummer etter henting av hjelpestønadgrunnlag. Overlappende perioder: ${e.perioder}",
+                    e
+                )
                 komplettering.withFeilinformasjon(
                     Feilinformasjon.OverlappendeHjelpestønadperioder(
                         message = "Overlappende hjelpestønadperioder",
@@ -186,7 +190,10 @@ class KompletteringsService(
                     komplettering.persongrunnlag!!.komprimer()
                 )
             } catch (e: BarnetrygdException.OverlappendePerioder) {
-                secureLog.warn("Feil ved komprimering av persongrunnlag etter henting av hjelpestønadgrunnlag", e)
+                secureLog.warn(
+                    "Overlappende perioder ved komprimering av persongrunnlag etter henting av hjelpestønadgrunnlag: ${e.perioder}",
+                    e
+                )
                 komplettering.withFeilinformasjon(
                     Feilinformasjon.OverlappendeHjelpestønadperioder(
                         message = e.message ?: "",

diff --git a/...tjening/omsorgsopptjening/start/innlesning/barnetrygd/domain/KompletteringsServiceTest.kt b/...tjening/omsorgsopptjening/start/innlesning/barnetrygd/domain/KompletteringsServiceTest.kt
@@ -436,11 +436,12 @@ class KompletteringsServiceTest : SpringContextTest.NoKafka() {
             }
 
         assertThat(komplettert.feilinformasjon)
-            .hasSize(1)
-            .first()
-            .isInstanceOf(Feilinformasjon.FeilIDataGrunnlag::class.java)
+            .hasSize(1)            .first()
+            .isInstanceOf(Feilinformasjon.OverlappendeBarnetrygdperioder::class.java)
         println(komplettert)
         assertThat(komplettert.rådata).hasSize(1)
+        assertThat((komplettert.feilinformasjon.first() as Feilinformasjon.OverlappendeBarnetrygdperioder).perioder)
+            .hasSize(2)
     }