fikse fnutt og ryddet i map til list til map (#213)

Author: christofferlohne

soknad/src/main/java/no/nav/k9/søknad/ytelse/psb/v1/PerioderMedEndringUtil.java

@@ -7,7 +7,6 @@
 import java.util.List;
 import java.util.Map;
 
-import no.nav.fpsak.tidsserie.LocalDateSegment;
 import no.nav.fpsak.tidsserie.LocalDateTimeline;
 import no.nav.fpsak.tidsserie.StandardCombinators;
 import no.nav.k9.søknad.TidsserieUtils;
@@ -61,7 +60,7 @@ public static List<PerioderMedEndring> getArbeidstidPerioder(Arbeidstid arbeidst
     private static LocalDateTimeline<Boolean> tilTidsserie(List<PerioderMedEndring> listen) {
         var temp = new LocalDateTimeline<Boolean>(Collections.emptyList());
         for (PerioderMedEndring yp : listen) {
-            for (Periode p : yp.getPeriodeList()) {
+            for (Periode p : yp.getPeriodeMap().keySet()) {
                 temp = temp.union(
                         new LocalDateTimeline<>(p.getFraOgMed(), p.getTilOgMed(), Boolean.TRUE),
                         StandardCombinators::coalesceLeftHandSide);
@@ -72,7 +71,7 @@ private static LocalDateTimeline<Boolean> tilTidsserie(List<PerioderMedEndring>
 
     public static class PerioderMedEndring {
         private String felt;
-        private List<Periode> periodeList;
+        private Map<Periode, ?> periodeMap;
 
         public PerioderMedEndring() {
 
@@ -82,20 +81,19 @@ public String getFelt() {
             return felt;
         }
 
+        public Map<Periode, ?> getPeriodeMap() {
+            return periodeMap;
+        }
+
         public List<Periode> getPeriodeList() {
-            return periodeList;
+            return new ArrayList<>(periodeMap.keySet());
         }
 
         PerioderMedEndring medPerioder(String felt, Map<Periode, ?> periodeMap) {
             this.felt = felt;
-            this.periodeList = new ArrayList<>(periodeMap.keySet());
+            this.periodeMap = periodeMap;
             return this;
         }
 
-        PerioderMedEndring medPerioder(String felt, List<Periode> periodeList) {
-            this.felt = felt;
-            this.periodeList = periodeList;
-            return this;
-        }
     }
 }

soknad/src/main/java/no/nav/k9/søknad/ytelse/psb/v1/PleiepengerSyktBarnYtelseValidator.java

@@ -89,7 +89,7 @@ private List<Feil> validerOgLeggTilFeilene(PleiepengerSyktBarn psb,
         }
 
         for (var ytelsePeriode : PerioderMedEndringUtil.getAllePerioderSomMåVæreInnenforSøknadsperiode(psb)) {
-            var ytelsePeriodeTidsserie = lagTidslinjeOgValiderForYtelseperioder(ytelsePeriode.getPeriodeList(), ytelsePeriode.getFelt() + ".perioder");
+            var ytelsePeriodeTidsserie = lagTidslinjeOgValider(ytelsePeriode.getPeriodeMap(), ytelsePeriode.getFelt() + ".perioder");
             feilene.addAll(validerAtYtelsePerioderErInnenforIntervalForEndring(intervalForEndringTidslinje, ytelsePeriodeTidsserie, ytelsePeriode.getFelt() + ".perioder"));
             feilene.addAll(validerAtIngenPerioderOverlapperMedTrekkKravPerioder(trekkKravPerioderTidslinje, ytelsePeriodeTidsserie, ytelsePeriode.getFelt() + ".perioder"));
         }
@@ -175,32 +175,23 @@ private LocalDateTimeline<Boolean> lagTidslinjeOgValider(List<Periode> periodeLi
         }
     }
 
-    //TODO skrive om sånn at det ikke trengs en metode for list og en annen for map
-    private LocalDateTimeline<Boolean> lagTidslinjeOgValiderForYtelseperioder(List<Periode> periodeList, String felt) throws ValideringsAvbrytendeFeilException {
-        var feil = validerPerioderErLukketOgGyldigForYtelseperioder(periodeList, felt);
+    private LocalDateTimeline<Boolean> lagTidslinjeOgValider(Map<Periode, ? > periodeMap, String felt) throws ValideringsAvbrytendeFeilException {
+        var feil = validerPerioderErLukketOgGyldig(periodeMap, felt);
         if (!feil.isEmpty()) {
             throw new ValideringsAvbrytendeFeilException(feil);
         }
         try {
-            return toLocalDateTimeline(periodeList);
+            return toLocalDateTimeline(new ArrayList<>(periodeMap.keySet()));
         } catch (IllegalArgumentException e) {
             throw new ValideringsAvbrytendeFeilException(List.of(lagFeil(felt, "IllegalArgumentException", e.getMessage())));
         }
     }
 
-    private List<Feil> validerPerioderErLukketOgGyldigForYtelseperioder(List<Periode> perioder, String felt) {
-        var feil = new ArrayList<Feil>();
-        perioder.forEach(p -> {
-            validerPerioderErLukket(p, felt + "[" + p + "]", feil);
-            validerPerioderIkkeErInvertert(p, felt + "[" + p + "]", feil);});
-        return feil;
-    }
-
     private List<Feil> validerPerioderErLukketOgGyldig(Map<Periode, ?> perioder, String felt) {
         var feil = new ArrayList<Feil>();
         perioder.keySet().forEach(p -> {
-            validerPerioderErLukket(p, felt + "[" + p + "]", feil);
-            validerPerioderIkkeErInvertert(p, felt + "[" + p + "]", feil);});
+            validerPerioderErLukket(p, felt + "['" + p + "']", feil);
+            validerPerioderIkkeErInvertert(p, felt + "['" + p + "']", feil);});
         return feil;
     }
 

soknad/src/test/java/no/nav/k9/søknad/ytelse/psb/TestUtils.java

@@ -70,8 +70,9 @@ public static void feilInneholder(List<Feil> feilList, Feil feil) {
     public static String periodeString(int i) {
         return "[" + i + "]";
     }
+
     public static String periodeString(Periode periode) {
-        return "[" + periode + "]";
+        return "['" + periode + "']";
     }
 
 }

soknad/src/test/java/no/nav/k9/søknad/ytelse/psb/v1/YtelseTest.java

@@ -21,6 +21,7 @@
 import no.nav.k9.søknad.felles.type.Periode;
 import no.nav.k9.søknad.felles.type.VirksomhetType;
 import no.nav.k9.søknad.ytelse.psb.SøknadEksempel;
+import no.nav.k9.søknad.ytelse.psb.TestUtils;
 import no.nav.k9.søknad.ytelse.psb.YtelseEksempel;
 import no.nav.k9.søknad.ytelse.psb.v1.arbeidstid.Arbeidstaker;
 import no.nav.k9.søknad.ytelse.psb.v1.arbeidstid.Arbeidstid;
@@ -227,7 +228,7 @@ public void invertertPeriodeForArbeidstakerPeriode() {
         ((PleiepengerSyktBarn) søknad.getYtelse()).medArbeidstid(new Arbeidstid().medArbeidstaker(List.of(arbeidstaker)));
 
         var feil = verifyHarFeil(søknad);
-        feilInneholder(feil, "ytelse.arbeidstid.arbeidstakerList[0].perioder[" + arbeidstidperiode + "]", "ugyldigPeriode");
+        feilInneholder(feil, "ytelse.arbeidstid.arbeidstakerList" + TestUtils.periodeString(0) + ".perioder" + TestUtils.periodeString(arbeidstidperiode) , "ugyldigPeriode");
     }
 
     @Test