Filtrerer bort husstandsmedlemsperioder utenfor aktuell periode

navikt · Mar 14, 2024 · b14b40a · b14b40a
1 parent 028ef6c
commit b14b40a
Show file tree

Hide file tree

Showing 2 changed files with 53 additions and 24 deletions.
diff --git a/src/main/kotlin/no/nav/bidrag/grunnlag/service/HentRelatertePersonerService.kt b/src/main/kotlin/no/nav/bidrag/grunnlag/service/HentRelatertePersonerService.kt
@@ -12,6 +12,7 @@ import no.nav.bidrag.grunnlag.util.GrunnlagUtil.Companion.evaluerFeiltype
 import no.nav.bidrag.transport.behandling.grunnlag.response.BorISammeHusstandDto
 import no.nav.bidrag.transport.behandling.grunnlag.response.FeilrapporteringDto
 import no.nav.bidrag.transport.behandling.grunnlag.response.RelatertPersonGrunnlagDto
+import no.nav.bidrag.transport.person.Husstandsmedlem
 import no.nav.bidrag.transport.person.NavnFødselDødDto
 import java.time.LocalDate
 
@@ -26,7 +27,7 @@ class HentRelatertePersonerService(
 
         relatertPersonRequestListe.forEach { personIdOgPeriode ->
             // Henter alle husstandsmedlemmer til BM/BP
-            val husstandsmedlemmerListe = hentHusstandsmedlemmer(ident = personIdOgPeriode.personId, feilrapporteringListe = feilrapporteringListe)
+            val husstandsmedlemmerListe = hentHusstandsmedlemmer(personIdOgPeriode = personIdOgPeriode, feilrapporteringListe = feilrapporteringListe)
 
             // Henter alle barn av BM/BP
             val barnListe = hentBarn(personident = Personident(personIdOgPeriode.personId), feilrapporteringListe = feilrapporteringListe)
@@ -90,37 +91,46 @@ class HentRelatertePersonerService(
     }
 
     // Henter alle husstandsmedlemmer til BM/BP
-    private fun hentHusstandsmedlemmer(ident: String, feilrapporteringListe: MutableList<FeilrapporteringDto>): List<PersonBo> {
+    private fun hentHusstandsmedlemmer(
+        personIdOgPeriode: PersonIdOgPeriodeRequest,
+        feilrapporteringListe: MutableList<FeilrapporteringDto>,
+    ): List<PersonBo> {
         val husstandsmedlemListe = mutableListOf<PersonBo>()
 
         when (
-            val restResponseHusstandsmedlemmer = bidragPersonConsumer.hentHusstandsmedlemmer(Personident(ident))
+            val restResponseHusstandsmedlemmer = bidragPersonConsumer.hentHusstandsmedlemmer(Personident(personIdOgPeriode.personId))
         ) {
             is RestResponse.Success -> {
                 val husstandsmedlemmerResponseDto = restResponseHusstandsmedlemmer.body
-                SECURE_LOGGER.info("Bidrag-person ga følgende respons på husstandsmedlemmer for $ident: $husstandsmedlemmerResponseDto")
+                SECURE_LOGGER.info(
+                    "Bidrag-person ga følgende respons på husstandsmedlemmer for ${personIdOgPeriode.personId}: $husstandsmedlemmerResponseDto",
+                )
 
                 husstandsmedlemmerResponseDto.husstandListe.forEach { husstand ->
-                    husstand.husstandsmedlemListe.forEach {
-                        husstandsmedlemListe.add(
-                            PersonBo(
-                                personId = it.personId.verdi,
-                                navn = it.navn,
-                                fodselsdato = it.fødselsdato,
-                                husstandsmedlemPeriodeFra = it.gyldigFraOgMed,
-                                husstandsmedlemPeriodeTil = it.gyldigTilOgMed,
-                            ),
-                        )
+                    husstand.husstandsmedlemListe.forEach { husstandsmedlem ->
+                        if (husstandsmedlem.personId.toString() != personIdOgPeriode.personId &&
+                            husstandsmedlemInnenforPeriode(personIdOgPeriode, husstandsmedlem)
+                        ) {
+                            husstandsmedlemListe.add(
+                                PersonBo(
+                                    personId = husstandsmedlem.personId.verdi,
+                                    navn = husstandsmedlem.navn,
+                                    fodselsdato = husstandsmedlem.fødselsdato,
+                                    husstandsmedlemPeriodeFra = husstandsmedlem.gyldigFraOgMed,
+                                    husstandsmedlemPeriodeTil = husstandsmedlem.gyldigTilOgMed,
+                                ),
+                            )
+                        }
                     }
                 }
             }
 
             is RestResponse.Failure -> {
-                SECURE_LOGGER.warn("Feil ved henting av husstandsmedlemmer for $ident")
+                SECURE_LOGGER.warn("Feil ved henting av husstandsmedlemmer for ${personIdOgPeriode.personId}")
                 feilrapporteringListe.add(
                     FeilrapporteringDto(
                         grunnlagstype = GrunnlagRequestType.HUSSTANDSMEDLEMMER_OG_EGNE_BARN,
-                        personId = ident,
+                        personId = personIdOgPeriode.personId,
                         periodeFra = null,
                         periodeTil = null,
                         feiltype = evaluerFeiltype(
@@ -273,6 +283,27 @@ class HentRelatertePersonerService(
         return resultatListe
     }
 
+    private fun husstandsmedlemInnenforPeriode(personIdOgPeriode: PersonIdOgPeriodeRequest, husstandsmedlem: Husstandsmedlem): Boolean {
+        if (husstandsmedlem.gyldigFraOgMed == null) {
+            return husstandsmedlem.gyldigTilOgMed == null || husstandsmedlem.gyldigTilOgMed!!.isAfter(
+                personIdOgPeriode.periodeFra,
+            )
+        }
+
+        if (husstandsmedlem.gyldigTilOgMed == null) {
+            return husstandsmedlem.gyldigFraOgMed!!.isBefore(personIdOgPeriode.periodeTil)
+        }
+
+        if (husstandsmedlem.gyldigFraOgMed!!.isAfter(personIdOgPeriode.periodeTil.minusDays(1))) {
+            return false
+        }
+
+        if (husstandsmedlem.gyldigTilOgMed!!.isAfter(personIdOgPeriode.periodeFra)) {
+            return true
+        }
+        return false
+    }
+
     // Intern dataklasse brukt for å simulere funksjonalitet fra oppdater- og hent-grunnlagspakke-tjenestene
     data class RelatertPersonIntern(
         val partPersonId: String?,

diff --git a/src/test/kotlin/no/nav/bidrag/grunnlag/service/HentRelatertePersonerServiceMockTest.kt b/src/test/kotlin/no/nav/bidrag/grunnlag/service/HentRelatertePersonerServiceMockTest.kt
@@ -46,17 +46,15 @@ class HentRelatertePersonerServiceMockTest {
         assertAll(
             { assertThat(relatertPersonListe).isNotNull() },
             { assertThat(relatertPersonListe.grunnlagListe).isNotEmpty() },
-            { assertThat(relatertPersonListe.grunnlagListe).hasSize(5) },
+            { assertThat(relatertPersonListe.grunnlagListe).hasSize(4) },
             { assertThat(relatertPersonListe.grunnlagListe[0].relatertPersonPersonId).isEqualTo("111") },
-            { assertThat(relatertPersonListe.grunnlagListe[1].relatertPersonPersonId).isEqualTo("333") },
-            { assertThat(relatertPersonListe.grunnlagListe[2].relatertPersonPersonId).isEqualTo("444") },
-            { assertThat(relatertPersonListe.grunnlagListe[3].relatertPersonPersonId).isEqualTo("555") },
-            { assertThat(relatertPersonListe.grunnlagListe[4].relatertPersonPersonId).isEqualTo("222") },
+            { assertThat(relatertPersonListe.grunnlagListe[1].relatertPersonPersonId).isEqualTo("555") },
+            { assertThat(relatertPersonListe.grunnlagListe[2].relatertPersonPersonId).isEqualTo("222") },
+            { assertThat(relatertPersonListe.grunnlagListe[3].relatertPersonPersonId).isEqualTo("333") },
             { assertThat(relatertPersonListe.grunnlagListe[0].borISammeHusstandDtoListe).isNotEmpty },
             { assertThat(relatertPersonListe.grunnlagListe[1].borISammeHusstandDtoListe).isNotEmpty },
-            { assertThat(relatertPersonListe.grunnlagListe[2].borISammeHusstandDtoListe).isNotEmpty },
-            { assertThat(relatertPersonListe.grunnlagListe[3].borISammeHusstandDtoListe).isNotEmpty },
-            { assertThat(relatertPersonListe.grunnlagListe[4].borISammeHusstandDtoListe).isEmpty() },
+            { assertThat(relatertPersonListe.grunnlagListe[2].borISammeHusstandDtoListe).isEmpty() },
+            { assertThat(relatertPersonListe.grunnlagListe[3].borISammeHusstandDtoListe).isEmpty() },
             { assertThat(relatertPersonListe.feilrapporteringListe).isEmpty() },
         )
     }