startet å fikse valideringsmeldinger, ta bort duplikater (#199)

* startet å fikse valideringsmeldinger, ta bort duplikater

* endret på validering av perioder og tidslinjer

* bruker tidslinje for å validere overlapp, lagt på key i felt for maps

* Flyttet tester til V1, gjort ytelse validering protected. Bruk søknad validering

* fikset på felt validering duplikater og feilkode

* lagt på regx fra Stian

* fikset feil

* fikest trekkKravTest

Author: christofferlohne

soknad/src/main/java/no/nav/k9/søknad/TidsserieUtils.java

@@ -1,35 +1,18 @@
 package no.nav.k9.søknad;
 
-import java.util.ArrayList;
-import java.util.Collections;
 import java.util.List;
-import java.util.Map;
 import java.util.stream.Collectors;
 
 import no.nav.fpsak.tidsserie.LocalDateSegment;
 import no.nav.fpsak.tidsserie.LocalDateTimeline;
-import no.nav.k9.søknad.felles.Feil;
 import no.nav.k9.søknad.felles.type.Periode;
 
 public class TidsserieUtils {
-    public static List<Periode> toPeriodeList(LocalDateTimeline<?> t) {
+    public static List<Periode> tilPeriodeList(LocalDateTimeline<?> t) {
         return t.stream().map(l -> new Periode(l.getFom(), l.getTom())).collect(Collectors.toList());
     }
 
-    public static LocalDateTimeline<Boolean> toLocalDateTimeline(Map<Periode, ?> periodeMap, String felt, List<Feil> feil) {
-        return toLocalDateTimeline(new ArrayList<>(periodeMap.keySet()), felt, feil);
-    }
-
-    public static LocalDateTimeline<Boolean> toLocalDateTimeline(List<Periode> perioder, String felt, List<Feil> feil) {
-        try {
-            return toLocalDateTimeline(perioder);
-        } catch (IllegalArgumentException e) {
-            feil.add(new Feil(felt, "IllegalArgumentException", e.getMessage()));
-        }
-        return new LocalDateTimeline<>(Collections.emptyList());
-    }
-
-    public static LocalDateTimeline<Boolean> toLocalDateTimeline(List<Periode> perioder) throws IllegalArgumentException{
+    public static LocalDateTimeline<Boolean> toLocalDateTimeline(List<Periode> perioder) {
         return new LocalDateTimeline<Boolean>(perioder
                 .stream()
                 .map(no.nav.k9.søknad.TidsserieUtils::mapLocalDateSegment)

soknad/src/main/java/no/nav/k9/søknad/felles/type/Organisasjonsnummer.java

@@ -35,7 +35,7 @@ public static Organisasjonsnummer of(String verdi) {
     }
 
 
-    @AssertTrue(message="Organisasjonsnummer må være gyldig")
+    @AssertTrue(message="[ugyldigOrgNummer] Organisasjonsnummer må være gyldig.")
     private boolean isValid() {
         return OrganisasjonsNummerValidator.erGyldig(verdi);
     }

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

@@ -20,12 +20,11 @@ public static List<Periode> getEndringsperiode(PleiepengerSyktBarn psb) {
                 tilTidsserie(getAllePerioderSomMåVæreInnenforSøknadsperiode(psb));
         var søknadsperiode = toLocalDateTimeline(psb.getSøknadsperiodeList());
         var endringsperiodeTidsserie = allePerioderMedEndringTidsserie.disjoint(søknadsperiode);
-        return TidsserieUtils.toPeriodeList(endringsperiodeTidsserie);
+        return TidsserieUtils.tilPeriodeList(endringsperiodeTidsserie);
     }
 
     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("beredskap", psb.getBeredskap().getPerioder()));
         listen.add(new PerioderMedEndring().medPerioder("nattevåk", psb.getNattevåk().getPerioder()));
         listen.add(new PerioderMedEndring().medPerioder("tilsynsordning", psb.getTilsynsordning().getPerioder()));

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

@@ -1,6 +1,8 @@
 package no.nav.k9.søknad.ytelse.psb.v1;
 
 import java.util.List;
+import java.util.regex.Matcher;
+import java.util.regex.Pattern;
 import java.util.stream.Collectors;
 
 import javax.validation.ConstraintViolation;
@@ -18,6 +20,7 @@
 
 public class PleiepengerSyktBarnSøknadValidator extends SøknadValidator<Søknad> {
 
+
     private static final ValidatorFactory VALIDATOR_FACTORY = Validation.buildDefaultValidatorFactory();
 
     public PleiepengerSyktBarnSøknadValidator() {
@@ -39,10 +42,26 @@ private static void validerVersjon(Versjon versjon, List<Feil> feil) {
     }
 
     private Feil toFeil(ConstraintViolation<Søknad> constraintViolation) {
+        var constraintMessage = constraintViolation.getMessage();
+
+        final Pattern feilkodePattern = Pattern.compile("^[^\\[]*\\[([^]]*)\\](.*)$");
+        final Matcher feilkodeMatcher = feilkodePattern.matcher(constraintMessage);
+
+        final String feilkode;
+        final String feilmelding;
+
+        if (feilkodeMatcher.matches() && feilkodeMatcher.groupCount() >= 2) {
+            feilkode = feilkodeMatcher.group(1).trim();
+            feilmelding = feilkodeMatcher.group(2).trim();
+        } else {
+            feilkode = "påkrevd";
+            feilmelding = constraintMessage;
+        }
+
         return new Feil(
                 constraintViolation.getPropertyPath().toString(),
-                PÅKREVD,
-                constraintViolation.getMessage());
+                feilkode,
+                feilmelding);
     }
 
     @Override