Forbedring på periodevalidering OMP_UT + validator på søknadsnivå (#219)

Author: espennav

soknad/src/main/java/no/nav/k9/søknad/ytelse/omsorgspenger/v1/OmsorgspengerUtbetalingSøknadValidator.java

@@ -0,0 +1,42 @@
+package no.nav.k9.søknad.ytelse.omsorgspenger.v1;
+
+import java.util.List;
+import java.util.stream.Collectors;
+
+import javax.validation.Validation;
+import javax.validation.ValidatorFactory;
+
+import no.nav.k9.søknad.Søknad;
+import no.nav.k9.søknad.SøknadValidator;
+import no.nav.k9.søknad.felles.Feil;
+import no.nav.k9.søknad.felles.Versjon;
+
+public class OmsorgspengerUtbetalingSøknadValidator extends SøknadValidator<Søknad> {
+
+
+    private static final ValidatorFactory VALIDATOR_FACTORY = Validation.buildDefaultValidatorFactory();
+
+    public OmsorgspengerUtbetalingSøknadValidator() {
+    }
+
+    private static void validerVersjon(Versjon versjon, List<Feil> feil) {
+        if (versjon != null && !versjon.erGyldig()) {
+            feil.add(new Feil("versjon", "ugyldigVersjon", "Versjonen er på ugyldig format."));
+        }
+    }
+
+
+    @Override
+    public List<Feil> valider(Søknad søknad) {
+        var validate = VALIDATOR_FACTORY.getValidator().validate(søknad);
+
+        List<Feil> feil = validate.stream()
+                .map(Feil::toFeil)
+                .collect(Collectors.toList());
+
+        validerVersjon(søknad.getVersjon(), feil);
+        feil.addAll(new OmsorgspengerUtbetalingValidator().valider(søknad.getYtelse()));
+
+        return feil;
+    }
+}

soknad/src/main/java/no/nav/k9/søknad/ytelse/omsorgspenger/v1/OmsorgspengerUtbetalingValidator.java

@@ -2,10 +2,11 @@
 
 import java.time.Duration;
 import java.util.ArrayList;
+import java.util.LinkedHashMap;
 import java.util.List;
+import java.util.Map;
 import java.util.Objects;
 import java.util.Optional;
-import java.util.stream.Collectors;
 
 import no.nav.k9.søknad.PeriodeValidator;
 import no.nav.k9.søknad.felles.Feil;
@@ -15,6 +16,7 @@
 import no.nav.k9.søknad.felles.opptjening.SelvstendigNæringsdrivende;
 import no.nav.k9.søknad.felles.personopplysninger.Barn;
 import no.nav.k9.søknad.felles.type.Organisasjonsnummer;
+import no.nav.k9.søknad.felles.type.Periode;
 import no.nav.k9.søknad.ytelse.Ytelse;
 import no.nav.k9.søknad.ytelse.YtelseValidator;
 
@@ -27,6 +29,7 @@ public OmsorgspengerUtbetalingValidator() {
 
     @Override
     public List<Feil> valider(Ytelse ytelse) {
+
         var omsorgspengerUtbetaling = (OmsorgspengerUtbetaling) ytelse;
 
         List<Feil> feil = new ArrayList<>();
@@ -203,9 +206,21 @@ private List<Feil> validerBosteder(OmsorgspengerUtbetaling ytelse) {
             index++;
         }
 
-        // Valider perioder på tvers
-        var perioder = fraværsperioderKorrigeringIm.stream().collect(Collectors.toMap(e -> e.getPeriode(), e -> e.getArbeidsgiverOrgNr()));
-        feil.addAll(periodeValidator.validerIkkeTillattOverlapp(perioder, "fraværsperioderKorrigeringIm.perioder"));
+
+
+        Map<Periode, Organisasjonsnummer> perioder = new LinkedHashMap<>();
+        for (FraværPeriode fraværPeriode : fraværsperioderKorrigeringIm) {
+            if (perioder.containsKey(fraværPeriode.getPeriode())) {
+                feil.add(new Feil("fraværsperioderKorrigeringIm.perioder[" + fraværPeriode.getPeriode().getIso8601() + "]", "likePerioder", "To identiske perioder oppgitt"));
+            } else {
+                perioder.put(fraværPeriode.getPeriode(), fraværPeriode.getArbeidsgiverOrgNr());
+            }
+            index++;
+        }
+        if (feil.stream().noneMatch(it -> it.getFeilkode().equals("likePerioder"))) {
+            // Valider perioder på tvers
+            feil.addAll(periodeValidator.validerIkkeTillattOverlapp(perioder, "fraværsperioderKorrigeringIm.perioder"));
+        }
 
         return feil;
     }

soknad/src/test/java/no/nav/k9/søknad/ytelse/omsorgspenger/v1/OmsorgspengerUtbetalingValidatorTest.java

@@ -32,7 +32,7 @@ class OmsorgspengerUtbetalingValidatorTest {
     void skal_returnere_ingen_feil_for_minimum_json_søknad() {
         var søknad = TestUtils.minimumJsonSøknad();
 
-        var feil = validatorSøknad.valider(søknad.getYtelse());
+        var feil = new OmsorgspengerUtbetalingSøknadValidator().valider(søknad);
 
         assertThat(feil).isEmpty();
     }
@@ -106,6 +106,22 @@ public void skal_returnere_feil_for_overlappende_perioder() {
         feilInneholder(feil, "fraværsperioderKorrigeringIm.perioder[2021-09-01/2021-09-02]", "overlappendePerioder");
     }
 
+    @Test
+    public void skal_returnere_feil_for_identiske_perioder() {
+        var fraværPeriode1 = new Periode(LocalDate.parse("2021-09-01"), LocalDate.parse("2021-09-02"));
+        var fraværPeriode2 = new Periode(LocalDate.parse("2021-09-01"), LocalDate.parse("2021-09-02"));
+
+        OmsorgspengerUtbetaling ytelse = byggOmsorgspengerUtbetalingMedFraværskorrigeringIm(
+                lagFraværskorrigeringIm(orgnr1, arbforholdId1, fraværPeriode1, null),
+                lagFraværskorrigeringIm(orgnr1, arbforholdId1, fraværPeriode2, null));
+
+        var feil = validatorSøknad.valider(ytelse);
+
+
+        assertThat(feil).hasSize(1);
+        feilInneholder(feil, "fraværsperioderKorrigeringIm.perioder[2021-09-01/2021-09-02]", "likePerioder");
+    }
+
     @Test
     public void skal_returnere_feil_for_flere_orgnr() {
         var fraværPeriode1 = new Periode(LocalDate.parse("2021-09-01"), LocalDate.parse("2021-09-02"));

soknad/src/test/resources/ytelse/omp/utbetaling/minimum-søknad-omp-utbetaling-snf.json

@@ -1,5 +1,5 @@
 {
-  "søknadId": "$søknadId",
+  "søknadId": "123",
   "mottattDato": "2018-01-02T03:04:05.000Z",
   "versjon": "1.0.0",
   "språk": "nb",
@@ -10,8 +10,10 @@
     "type": "OMP_UT",
     "fosterbarn": null,
     "aktivitet": {
-      "selvstendigNæringsdrivende": null,
-      "frilanser": null
+      "frilanser": {
+        "startdato": "2020-01-01",
+        "jobberFortsattSomFrilans": true
+      }
     },
     "fraværsperioder": [
       {