Bosteder kan være utenfor søknadsperiode og overlappe med trekkKravPe… (#192)

christofferlohne · web-flow · commit 2d904ff70be1 · 2021-09-17T14:43:48.000+02:00
* Bosteder kan være utenfor søknadsperiode og overlappe med trekkKravPeriode

* lagt tilbake validering på perioder som kan være utenfor søknadsperiode i valider ytelse
diff --git a/soknad/src/main/java/no/nav/k9/søknad/ytelse/psb/v1/PerioderMedEndringUtil.java b/soknad/src/main/java/no/nav/k9/søknad/ytelse/psb/v1/PerioderMedEndringUtil.java
@@ -17,16 +17,15 @@ class PerioderMedEndringUtil {
 
     public static List<Periode> getEndringsperiode(PleiepengerSyktBarn psb) {
         var allePerioderMedEndringTidsserie =
-                tilTidsserie(getAllePerioderMedEndring(psb));
+                tilTidsserie(getAllePerioderSomMåVæreInnenforSøknadsperiode(psb));
         var søknadsperiode = toLocalDateTimeline(psb.getSøknadsperiodeList());
         var endringsperiodeTidsserie = allePerioderMedEndringTidsserie.disjoint(søknadsperiode);
         return TidsserieUtils.toPeriodeList(endringsperiodeTidsserie);
     }
 
-    public static List<PerioderMedEndring> getAllePerioderMedEndring(PleiepengerSyktBarn psb) {
+    public static List<PerioderMedEndring> getAllePerioderSomMåVæreInnenforSøknadsperiode(PleiepengerSyktBarn psb) {
         var listen = new ArrayList<PerioderMedEndring>();
         listen.add(new PerioderMedEndring().medPerioder("utenlandsopphold", psb.getUtenlandsopphold().getPerioder()));
-        listen.add(new PerioderMedEndring().medPerioder("bosteder", psb.getBosteder().getPerioder()));
         listen.add(new PerioderMedEndring().medPerioder("beredskap", psb.getBeredskap().getPerioder()));
         listen.add(new PerioderMedEndring().medPerioder("nattevåk", psb.getNattevåk().getPerioder()));
         listen.add(new PerioderMedEndring().medPerioder("tilsynsordning", psb.getTilsynsordning().getPerioder()));
@@ -43,8 +42,8 @@ public static List<PerioderMedEndring> getArbeidstidPerioder(Arbeidstid arbeidst
         }
         if (arbeidstid.getArbeidstakerList() != null && !arbeidstid.getArbeidstakerList().isEmpty()) {
             int i = 0;
-            for (var at: arbeidstid.getArbeidstakerList()) {
-                listen.add(new PerioderMedEndring().medPerioder("arbeidstid.arbeidstaker[" + i + "]" , at.getArbeidstidInfo().getPerioder()));
+            for (var at : arbeidstid.getArbeidstakerList()) {
+                listen.add(new PerioderMedEndring().medPerioder("arbeidstid.arbeidstaker[" + i + "]", at.getArbeidstidInfo().getPerioder()));
                 i++;
             }
         }
@@ -61,7 +60,7 @@ public static List<PerioderMedEndring> getArbeidstidPerioder(Arbeidstid arbeidst
 
     public static LocalDateTimeline<Boolean> tilTidsserie(List<PerioderMedEndring> listen) {
         var temp = new LocalDateTimeline<Boolean>(Collections.emptyList());
-        for (PerioderMedEndring yp: listen) {
+        for (PerioderMedEndring yp : listen) {
             temp = temp.union(
                     toLocalDateTimeline(yp.getPeriodeList()),
                     StandardCombinators::coalesceLeftHandSide);
@@ -90,5 +89,11 @@ PerioderMedEndring medPerioder(String felt, Map<Periode, ?> periodeMap) {
             this.periodeList = new ArrayList<>(periodeMap.keySet());
             return this;
         }
+
+        PerioderMedEndring medPerioder(String felt, List<Periode> periodeList) {
+            this.felt = felt;
+            this.periodeList = periodeList;
+            return this;
+        }
     }
 }
diff --git a/soknad/src/main/java/no/nav/k9/søknad/ytelse/psb/v1/PleiepengerSyktBarnYtelseValidator.java b/soknad/src/main/java/no/nav/k9/søknad/ytelse/psb/v1/PleiepengerSyktBarnYtelseValidator.java
@@ -50,11 +50,11 @@ public List<Feil> valider(Ytelse ytelse, List<Periode> gyldigeEndringsperioder)
         feil.addAll(inneholderSøknadsperiodeEllerGyldigeEndringsperioder(psb, gyldigeEndringsperioder));
         feil.addAll(validerKomplettSøknad(psb));
 
-        var perioderMedEndring = PerioderMedEndringUtil.getAllePerioderMedEndring(psb);
-        feil.addAll(validerPerioderErLukketOgIkkeFeilRekkefølge(psb.getSøknadsperiodeList(), "søknadsperiode"));
         feil.addAll(validerPerioderErLukketOgIkkeFeilRekkefølge(gyldigeEndringsperioder, "gyldigeEndringsperioder"));
+        feil.addAll(validerPerioderErLukketOgIkkeFeilRekkefølge(psb.getSøknadsperiodeList(), "søknadsperioder"));
         feil.addAll(validerPerioderErLukketOgIkkeFeilRekkefølge(psb.getTrekkKravPerioder(), "trekkKravPerioder"));
-        feil.addAll(validerPerioderErLukketOgIkkeFeilRekkefølge(perioderMedEndring));
+        feil.addAll(validerPerioderErLukketOgIkkeFeilRekkefølge(psb.getBosteder().getPerioder(), "bosteder"));
+        feil.addAll(validerPerioderErLukketOgIkkeFeilRekkefølge(PerioderMedEndringUtil.getAllePerioderSomMåVæreInnenforSøknadsperiode(psb)));
 
         var søknadsperiode = toLocalDateTimeline(psb.getSøknadsperiodeList(), "søknadsperiode", feil);
         var gyldigeIntervalForEndring = søknadsperiode.union(
@@ -64,7 +64,9 @@ public List<Feil> valider(Ytelse ytelse, List<Periode> gyldigeEndringsperioder)
 
         feil.addAll(finnPerioderInnenforTrekkKrav(trekkKravPerioder, søknadsperiode, "søknadperiode"));
 
-        feil.addAll(innenforGyldigIntervalForEndringOgIkkeInnenforTrekkAvKrav(gyldigeIntervalForEndring, trekkKravPerioder, perioderMedEndring));
+        feil.addAll(innenforGyldigIntervalForEndringOgIkkeInnenforTrekkAvKrav(gyldigeIntervalForEndring,
+                trekkKravPerioder,
+                PerioderMedEndringUtil.getAllePerioderSomMåVæreInnenforSøknadsperiode(psb)));
         feil.addAll(periodeneErKomplett(søknadsperiode, psb.getUttak().getPerioder(), "uttak"));
 
         return feil;
@@ -168,6 +170,10 @@ private Feil toFeil(ConstraintViolation<PleiepengerSyktBarn> constraintViolation
         return feil;
     }
 
+    private static List<Feil> validerPerioderErLukketOgIkkeFeilRekkefølge(Map<Periode, ?> perioder, String felt) {
+        return validerPerioderErLukketOgIkkeFeilRekkefølge(new ArrayList<>(perioder.keySet()), felt);
+    }
+
     private static List<Feil> validerPerioderErLukketOgIkkeFeilRekkefølge(List<Periode> perioder, String felt) {
         var feil = new ArrayList<Feil>();
         for (int i = 0; i < perioder.size(); i++ ) {
diff --git a/soknad/src/test/java/no/nav/k9/søknad/ytelse/psb/SøknadTest.java b/soknad/src/test/java/no/nav/k9/søknad/ytelse/psb/SøknadTest.java
@@ -5,6 +5,7 @@
 import static no.nav.k9.søknad.ytelse.psb.ValiderUtil.verifyIngenFeil;
 
 import java.time.LocalDate;
+import java.util.List;
 
 import org.junit.jupiter.api.Test;
 
@@ -13,6 +14,7 @@
 import no.nav.k9.søknad.felles.personopplysninger.Søker;
 import no.nav.k9.søknad.felles.type.NorskIdentitetsnummer;
 import no.nav.k9.søknad.felles.type.Periode;
+import no.nav.k9.søknad.ytelse.psb.v1.PleiepengerSyktBarn;
 
 class SøknadTest {
     private static final Periode TEST_PERIODE = new Periode(LocalDate.now(), LocalDate.now().plusMonths(2));
@@ -25,7 +27,6 @@ class SøknadTest {
         verifyIngenFeil(KOMPLETT_SØKNAD);
     }
 
-
     @Test
     public void minimumSøknadHarIngenValideringsFeil() {
         verifyIngenFeil(MINIMUM_SØKNAD);
@@ -50,6 +51,41 @@ class SøknadTest {
         feilInneholder(feil, new Feil("søker", PÅKREVD, "must not be null"));
     }
 
+    @Test
+    public void bostederKanVæreUtenforSøknadsperiode() {
+        var søknadsperiode = new Periode(LocalDate.now(), LocalDate.now().plusMonths(2));
+        var bostedperiode = new Periode(LocalDate.now().minusMonths(2), søknadsperiode.getTilOgMed());
+
+        var søknad = SøknadEksempel.komplettSøknad(søknadsperiode);
+        ((PleiepengerSyktBarn)søknad.getYtelse()).medBosteder(YtelseEksempel.lagBosteder(List.of(bostedperiode)));
+
+        verifyIngenFeil(søknad);
+    }
+
+    @Test
+    public void bostederKanIkkeHaFeilRekkefølgeIPerioden() {
+        var søknadsperiode = new Periode(LocalDate.now(), LocalDate.now().plusMonths(2));
+        var bostedperiode = new Periode(søknadsperiode.getTilOgMed(), LocalDate.now().minusMonths(2));
+
+        var søknad = SøknadEksempel.komplettSøknad(søknadsperiode);
+        ((PleiepengerSyktBarn)søknad.getYtelse()).medBosteder(YtelseEksempel.lagBosteder(List.of(bostedperiode)));
+
+        var feil = verifyHarFeil(søknad);
+        feilInneholder(feil, "bosteder[0]", "ugyldigPeriode", "Fra og med (FOM) må være før eller lik til og med (TOM).");
+    }
+
+    @Test
+    public void utenlandsoppholdKanIkkeVæreUtenforSøknadsperiode() {
+        var søknadsperiode = new Periode(LocalDate.now(), LocalDate.now().plusMonths(2));
+        var bostedperiode = new Periode(LocalDate.now().minusMonths(2), søknadsperiode.getTilOgMed());
+
+        var søknad = SøknadEksempel.komplettSøknad(søknadsperiode);
+        ((PleiepengerSyktBarn)søknad.getYtelse()).medUtenlandsopphold(YtelseEksempel.lagUtenlandsopphold(List.of(bostedperiode)));
+
+        var feil = verifyHarFeil(søknad);
+        feilInneholder(feil, "utenlandsopphold.perioder", "ugyldigPeriode");
+    }
+
     //TODO legge på getSøknadsperioder test
 
 }
diff --git a/soknad/src/test/java/no/nav/k9/søknad/ytelse/psb/TestUtils.java b/soknad/src/test/java/no/nav/k9/søknad/ytelse/psb/TestUtils.java
@@ -43,6 +43,15 @@ public static void feilInneholder(List<Feil> feil, String felt, String feilkode)
                 .isNotEmpty();
     }
 
+    public static void feilInneholder(List<Feil> feil, String felt, String feilkode, String feilmelding) {
+        assertThat(feil
+                .stream()
+                .filter(f -> f.getFeilkode().equals(feilkode) && f.getFelt().equals(felt) && f.getFeilmelding().equals(feilmelding))
+                .collect(Collectors.toList()))
+                .withFailMessage("Finner ikke fetl og feilkode: " + felt + ", " + feilkode + ", " + feilmelding)
+                .isNotEmpty();
+    }
+
     public static void feilInneholder(List<Feil> feilList, Feil feil) {
         assertThat(feilList
                 .stream()
diff --git a/soknad/src/test/java/no/nav/k9/søknad/ytelse/psb/TrekkKravTest.java b/soknad/src/test/java/no/nav/k9/søknad/ytelse/psb/TrekkKravTest.java
@@ -75,8 +75,8 @@ class TrekkKravTest
         ((PleiepengerSyktBarn)søknad.getYtelse()).addTrekkKravPeriode(trekkKravPerioderSomOverlapper);
 
         var feil = verifyHarFeil(søknad);
-        feilInneholder(feil,"søknadperiode.perioder", "ugyldigTrekkKrav");
-        feilInneholder(feil,"uttak.perioder", "ugyldigTrekkKrav");
+        feilInneholder(feil, "søknadperiode.perioder", "ugyldigTrekkKrav");
+        feilInneholder(feil, "uttak.perioder", "ugyldigTrekkKrav");
     }
 
 }

Original file line number	Diff line number	Diff line change
`@@ -75,8 +75,8 @@ class TrekkKravTest`
`75`	`75`	`((PleiepengerSyktBarn)søknad.getYtelse()).addTrekkKravPeriode(trekkKravPerioderSomOverlapper);`
`76`	`76`
`77`	`77`	`var feil = verifyHarFeil(søknad);`
`78`		`- feilInneholder(feil,"søknadperiode.perioder", "ugyldigTrekkKrav");`
`79`		`- feilInneholder(feil,"uttak.perioder", "ugyldigTrekkKrav");`
	`78`	`+ feilInneholder(feil, "søknadperiode.perioder", "ugyldigTrekkKrav");`
	`79`	`+ feilInneholder(feil, "uttak.perioder", "ugyldigTrekkKrav");`
`80`	`80`	`}`
`81`	`81`
`82`	`82`	`}`