Fix: OMP Validering av periode som ikke er satt ordentlig (#312)

* Fix: Unngår NPE i validering når periode ikke er satt ordentlig
* Fix: validerer at periode er satt ordentlig i OMP fraværsperioe
* bruker også søknadvalidator isdf ytelsevalidator i test for OMP
* Fix: Validering tom før fom

Author: tendestad

soknad/src/main/java/no/nav/k9/søknad/felles/fravær/FraværPeriode.java

@@ -5,6 +5,7 @@
 import java.util.Objects;
 
 import javax.validation.Valid;
+import javax.validation.constraints.AssertFalse;
 import javax.validation.constraints.NotNull;
 import javax.validation.constraints.Size;
 
@@ -76,6 +77,21 @@ public class FraværPeriode implements Comparable<FraværPeriode> {
         this.arbeidsforholdId = arbeidsforholdId;
     }
 
+    @AssertFalse(message = "Mangler fra-og-med-dato for perioden")
+    boolean isPeriodeManglerFom(){
+        return periode != null && periode.getFraOgMed() == null;
+    }
+
+    @AssertFalse(message = "Mangler til-og-med-dato for perioden")
+    boolean isPeriodeManglerTom(){
+        return periode != null && periode.getTilOgMed() == null;
+    }
+
+    @AssertFalse(message = "Til-og-med-dato er før fra-og-med-dato for perioden")
+    boolean isPeriodeTomFørFom(){
+        return periode != null && periode.getTilOgMed() != null && periode.getFraOgMed() != null && periode.getTilOgMed().isBefore(periode.getFraOgMed());
+    }
+
     public FraværPeriode medPeriode(Periode periode) {
         this.periode = periode;
         return this;

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

@@ -57,15 +57,20 @@ public List<Feil> valider(Ytelse ytelse) {
     }
 
     private List<Feil> validerPeriodeInnenforEttÅr(OmsorgspengerUtbetaling ytelse) {
-        List<Feil> feil = new ArrayList<>();
+        if (ytelse.getFraværsperioder().stream().anyMatch(OmsorgspengerUtbetalingValidator::erPeriodeUgyldig)) {
+            //avbryter her for å unngå NPE i koden under når periode ikke er satt, eller ikke er satt ordentlig
+            //nødvendig siden validatoren brukes mens søknaden bygges i k9-punsj, og søknad vil således være ukomplett underveis
+            //periode valideres andre steder uansett
+            return List.of();
+        }
         var søknadsperiode = ytelse.getSøknadsperiode();
         var yearMin = søknadsperiode.getFraOgMed().getYear();
         var yearMax = søknadsperiode.getTilOgMed().getYear();
 
         if (yearMin != yearMax) {
-            feil.add(new Feil("fraværsperioder", "perioderOverFlereÅr", "Perioder kan ikke overstige ett år"));
+            return List.of(new Feil("fraværsperioder", "perioderOverFlereÅr", "Perioder kan ikke overstige ett år"));
         }
-        return feil;
+        return List.of();
     }
 
     private List<Feil> validerAktivitet(OmsorgspengerUtbetaling ytelse) {
@@ -235,6 +240,12 @@ private List<Feil> validerOverlappendePerioderPrAktivitet(List<FraværPeriode> f
         Map<Aktivitet, Set<Periode>> unikePerioderPrAktivitet = new HashMap<>();
         int index = 0;
         for (FraværPeriode fraværPeriode : fraværPerioder) {
+            if (erPeriodeUgyldig(fraværPeriode)){
+                //avbryter her for å unngå NPE i koden under når periode ikke er satt, eller ikke er satt ordentlig
+                //nødvendig siden validatoren brukes mens søknaden bygges i k9-punsj, og søknad vil således være ukomplett underveis
+                //periode valideres andre steder uansett
+                continue;
+            }
             for (AktivitetFravær aktivitetType : fraværPeriode.getAktivitetFravær()) {
                 Aktivitet aktivitet = new Aktivitet(aktivitetType, fraværPeriode.getArbeidsgiverOrgNr());
                 Set<Periode> unikePerioder = unikePerioderPrAktivitet.computeIfAbsent(aktivitet, k -> new HashSet<>());
@@ -263,6 +274,12 @@ private List<Feil> validerOverlappendePerioderPrAktivitet(List<FraværPeriode> f
         return feil;
     }
 
+    private static boolean erPeriodeUgyldig(FraværPeriode fraværPeriode){
+        Periode periode = fraværPeriode.getPeriode();
+        return periode == null || periode.getFraOgMed() == null || periode.getTilOgMed() == null || periode.getTilOgMed().isBefore(periode.getFraOgMed());
+    }
+
+
     private List<Feil> validerOverlappendePerioderKorrigerIm(List<FraværPeriode> fraværPerioder) {
         List<Feil> feil = new ArrayList<>();
         List<LocalDateSegment<Integer>> perioder = new ArrayList<>();

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

@@ -8,25 +8,31 @@
 import java.nio.file.Path;
 import java.time.Duration;
 import java.time.LocalDate;
+import java.time.ZonedDateTime;
 import java.util.Arrays;
 import java.util.List;
 
 import org.junit.jupiter.api.Test;
 
 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;
 import no.nav.k9.søknad.felles.fravær.AktivitetFravær;
 import no.nav.k9.søknad.felles.fravær.DelvisFravær;
 import no.nav.k9.søknad.felles.fravær.FraværPeriode;
 import no.nav.k9.søknad.felles.fravær.FraværÅrsak;
 import no.nav.k9.søknad.felles.fravær.SøknadÅrsak;
 import no.nav.k9.søknad.felles.opptjening.OpptjeningAktivitet;
+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.Organisasjonsnummer;
 import no.nav.k9.søknad.felles.type.Periode;
+import no.nav.k9.søknad.felles.type.Språk;
+import no.nav.k9.søknad.felles.type.SøknadId;
 
 class OmsorgspengerUtbetalingValidatorTest {
-
-    private OmsorgspengerUtbetalingValidator validatorSøknad = new OmsorgspengerUtbetalingValidator(Versjon.of("1.1.0"));
+    private SøknadValidator<Søknad> søknadValidator = new OmsorgspengerUtbetalingSøknadValidator();
 
     private Organisasjonsnummer orgnr1 = Organisasjonsnummer.of("999999999");
     private Organisasjonsnummer orgnr2 = Organisasjonsnummer.of("816338352");
@@ -35,14 +41,14 @@ class OmsorgspengerUtbetalingValidatorTest {
     @Test
     void skal_returnere_ingen_feil_for_minimum_json_søknad() {
         var søknad = TestUtils.minimumJsonSøknad();
-        var feil = new OmsorgspengerUtbetalingSøknadValidator().valider(søknad);
+        var feil = søknadValidator.valider(søknad);
         assertThat(feil).isEmpty();
     }
 
     @Test
     void skal_returnere_ingen_feil_for_komplett_json_søknad() {
         var søknad = TestUtils.komplettJsonSøknad();
-        var feil = new OmsorgspengerUtbetalingSøknadValidator().valider(søknad);
+        var feil = søknadValidator.valider(søknad);
         assertThat(feil).isEmpty();
     }
 
@@ -55,7 +61,7 @@ class OmsorgspengerUtbetalingValidatorTest {
                 lagFraværskorrigeringIm(orgnr1, arbforholdId1, fulltFraværPeriode, null),
                 lagFraværskorrigeringImGammelVariant(orgnr1, arbforholdId1, delvisFraværPeriode, Duration.ofHours(4)));
 
-        var feil = validatorSøknad.valider(ytelse);
+        List<Feil> feil = lagSøknadOgValider(ytelse);
 
         assertThat(feil).isEmpty();
     }
@@ -67,7 +73,7 @@ class OmsorgspengerUtbetalingValidatorTest {
         OmsorgspengerUtbetaling ytelse = byggOmsorgspengerUtbetalingMedFraværskorrigeringIm(
                 lagFraværskorrigeringIm(orgnr1, arbforholdId1, fraværPeriode, null));
 
-        var feil = validatorSøknad.valider(ytelse);
+        List<Feil> feil = lagSøknadOgValider(ytelse);
 
         assertThat(feil).hasSize(1);
         feilInneholder(feil, "fraværsperioder", "perioderOverFlereÅr");
@@ -80,7 +86,7 @@ void skal_returnere_feil_for_nulling_som_overstiger_enkeltdag() {
         OmsorgspengerUtbetaling ytelse = byggOmsorgspengerUtbetalingMedFraværskorrigeringIm(
                 lagFraværskorrigeringImGammelVariant(orgnr1, arbforholdId1, fraværPeriode, Duration.ofHours(0)));
 
-        var feil = validatorSøknad.valider(ytelse);
+        List<Feil> feil = lagSøknadOgValider(ytelse);
 
         assertThat(feil).hasSize(1);
         feilInneholder(feil, "fraværsperioderKorrigeringIm[0]", "nullingPeriodeOversteget");
@@ -93,7 +99,7 @@ void skal_returnere_feil_for_nulling_som_overstiger_enkeltdag() {
         OmsorgspengerUtbetaling ytelse = byggOmsorgspengerUtbetalingMedFraværskorrigeringIm(
                 lagFraværskorrigeringImGammelVariant(orgnr1, arbforholdId1, fraværPeriode, Duration.parse("PT7H31M")));
 
-        var feil = validatorSøknad.valider(ytelse);
+        List<Feil> feil = lagSøknadOgValider(ytelse);
 
         assertThat(feil).hasSize(1);
         feilInneholder(feil, "fraværsperioderKorrigeringIm[0]", "varighetOversteget");
@@ -108,13 +114,39 @@ void skal_returnere_feil_for_overlappende_perioder_ved_korrigering_IM() {
                 lagFraværskorrigeringIm(orgnr1, arbforholdId1, fraværPeriode1, null),
                 lagFraværskorrigeringIm(orgnr1, arbforholdId1, fraværPeriode2, null));
 
-        var feil = validatorSøknad.valider(ytelse);
+        List<Feil> feil = lagSøknadOgValider(ytelse);
 
 
         assertThat(feil).hasSize(1);
         feilInneholder(feil, "fraværsperioderKorrigeringIm.perioder[0, 1]", "overlappendePerioder");
     }
 
+    @Test
+    void skal_gi_fornuftig_feilmelding_når_periode_ikke_er_satt() {
+        Periode periode = null;
+        Periode periode1 = new Periode(null, null);
+        Periode periode2 = new Periode(null, LocalDate.now());
+        Periode periode3 = new Periode(LocalDate.now(), null);
+        Periode periode4 = new Periode(LocalDate.now(), LocalDate.now().minusDays(1));
+        OmsorgspengerUtbetaling ytelse = byggOmsorgspengerUtbetalingSøknadBruker(
+                lagSøknadsperiode(orgnr1, periode, null),
+                lagSøknadsperiode(orgnr1, periode1, null),
+                lagSøknadsperiode(orgnr1, periode2, null),
+                lagSøknadsperiode(orgnr1, periode3, null),
+                lagSøknadsperiode(orgnr1, periode4, null)
+        );
+
+        List<Feil> feil = lagSøknadOgValider(ytelse);
+
+        assertThat(feil).hasSize(6);
+        feilInneholder(feil, "ytelse.fraværsperioder['0'].periode", "nullFeil");
+        feilInneholder(feil, "ytelse.fraværsperioder['1'].periodeManglerFom", "påkrevd");
+        feilInneholder(feil, "ytelse.fraværsperioder['1'].periodeManglerTom", "påkrevd");
+        feilInneholder(feil, "ytelse.fraværsperioder['2'].periodeManglerFom", "påkrevd");
+        feilInneholder(feil, "ytelse.fraværsperioder['3'].periodeManglerTom", "påkrevd");
+        feilInneholder(feil, "ytelse.fraværsperioder['4'].periodeTomFørFom", "påkrevd");
+    }
+
     @Test
     void skal_være_lov_å_ha_fravær_fra_flere_arbeidsgivere_samtidig() {
         var fraværPeriode1 = new Periode(LocalDate.parse("2021-09-01"), LocalDate.parse("2021-09-02"));
@@ -124,7 +156,7 @@ void skal_returnere_feil_for_overlappende_perioder_ved_korrigering_IM() {
                 lagSøknadsperiode(orgnr1, fraværPeriode1, null),
                 lagSøknadsperiode(orgnr2, fraværPeriode2, null));
 
-        var feil = validatorSøknad.valider(ytelse);
+        List<Feil> feil = lagSøknadOgValider(ytelse);
 
         assertThat(feil).isEmpty();
     }
@@ -138,7 +170,7 @@ void skal_returnere_feil_for_identiske_perioder() {
                 lagFraværskorrigeringIm(orgnr1, arbforholdId1, fraværPeriode1, null),
                 lagFraværskorrigeringIm(orgnr1, arbforholdId1, fraværPeriode2, null));
 
-        var feil = validatorSøknad.valider(ytelse);
+        List<Feil> feil = lagSøknadOgValider(ytelse);
 
 
         assertThat(feil).hasSize(1);
@@ -152,10 +184,10 @@ void skal_returnere_feil_for_identiske_perioder() {
         var fraværPeriode2 = new Periode(LocalDate.parse("2021-09-01"), LocalDate.parse("2021-09-02"));
 
         OmsorgspengerUtbetaling ytelse = byggOmsorgspengerUtbetalingSøknadBruker(
-                    lagSøknadsperiode(orgnr1, fraværPeriode1, null),
-                    lagSøknadsperiode(orgnr1, fraværPeriode2, null));
+                lagSøknadsperiode(orgnr1, fraværPeriode1, null),
+                lagSøknadsperiode(orgnr1, fraværPeriode2, null));
 
-        var feil = validatorSøknad.valider(ytelse);
+        List<Feil> feil = lagSøknadOgValider(ytelse);
 
         assertThat(feil).hasSize(1);
         feilInneholder(feil, "fraværsperioder[2021-09-01/2021-09-02]", "likePerioder");
@@ -169,7 +201,7 @@ void skal_returnere_feil_for_flere_orgnr() {
                 lagFraværskorrigeringIm(orgnr1, arbforholdId1, fraværPeriode1, null),
                 lagFraværskorrigeringIm(orgnr2, arbforholdId1, fraværPeriode2, null));
 
-        var feil = validatorSøknad.valider(ytelse);
+        List<Feil> feil = lagSøknadOgValider(ytelse);
 
         assertThat(feil).hasSize(1);
         feilInneholder(feil, "fraværsperioderKorrigeringIm[1]", "ikkeUniktOrgNr");
@@ -183,7 +215,7 @@ void skal_returnere_feil_for_flere_arbeidsforhold_id() {
                 lagFraværskorrigeringIm(orgnr1, arbforholdId1, fraværPeriode1, null),
                 lagFraværskorrigeringIm(orgnr1, null, fraværPeriode2, null));
 
-        var feil = validatorSøknad.valider(ytelse);
+        List<Feil> feil = lagSøknadOgValider(ytelse);
 
         assertThat(feil).hasSize(1);
         feilInneholder(feil, "fraværsperioderKorrigeringIm[1]", "ikkeUnikArbeidsforholdId");
@@ -195,10 +227,12 @@ void skal_returnere_feil_for_flere_arbeidsforhold_id() {
         OmsorgspengerUtbetaling ytelse = byggOmsorgspengerUtbetalingSøknadBruker(
                 lagSøknadsperiode(orgnr1, periode, new DelvisFravær(null, Duration.ofHours(2))));
 
-        var feil = validatorSøknad.valider(ytelse);
+        List<Feil> feil = lagSøknadOgValider(ytelse);
 
-        assertThat(feil).hasSize(1);
+        assertThat(feil).hasSize(2);
+        //TODO bør holde med en feil - begge feilmeldinger rapporterer samme feil
         feilInneholder(feil, "fraværsperioder[0].delvisFravær.normalarbeidstid", "manglerNormalarbeidstidForDelvisFravær");
+        feilInneholder(feil, "ytelse.fraværsperioder['0'].delvisFravær.normalarbeidstid", "nullFeil");
     }
 
     @Test
@@ -207,10 +241,12 @@ void skal_returnere_feil_for_flere_arbeidsforhold_id() {
         OmsorgspengerUtbetaling ytelse = byggOmsorgspengerUtbetalingSøknadBruker(
                 lagSøknadsperiode(orgnr1, periode, new DelvisFravær(Duration.ofHours(2), null)));
 
-        var feil = validatorSøknad.valider(ytelse);
+        List<Feil> feil = lagSøknadOgValider(ytelse);
 
-        assertThat(feil).hasSize(1);
+        assertThat(feil).hasSize(2);
+        //TODO bør holde med en feil - begge feilmeldinger rapporterer samme feil
         feilInneholder(feil, "fraværsperioder[0].delvisFravær.fravær", "manglerFraværForDelvisFravær");
+        feilInneholder(feil, "ytelse.fraværsperioder['0'].delvisFravær.fravær", "nullFeil");
     }
 
     @Test
@@ -222,7 +258,7 @@ void skal_ikke_tillate_overlappende_periode() {
                 lagSøknadsperiode(orgnr1, periode2, null)
         );
 
-        var feil = validatorSøknad.valider(ytelse);
+        List<Feil> feil = lagSøknadOgValider(ytelse);
 
         assertThat(feil).hasSize(1);
         feilInneholder(feil, "fraværsperioder[0, 1]", "overlappendePerioder");
@@ -231,29 +267,29 @@ void skal_ikke_tillate_overlappende_periode() {
     @Test
     void skal_ikke_tillate_at_duration_ikke_tilsvarer_som_delvisFravær() {
         var periode = new Periode(LocalDate.parse("2021-09-01"), LocalDate.parse("2021-09-02"));
-            FraværPeriode fraværPeriode = new FraværPeriode(
-                    periode,
-                    Duration.ofHours(2),
-                    new DelvisFravær(Duration.ofHours(4), Duration.ofHours(2)), //50% fravær fra 4 times-jobb skal oppgis som 4 timer i duration ( 50% av 7.5 timer, og rundet oppover til nærmeste halvtime)
-                    FraværÅrsak.ORDINÆRT_FRAVÆR,
-                    SøknadÅrsak.NYOPPSTARTET_HOS_ARBEIDSGIVER,
-                    List.of(AktivitetFravær.ARBEIDSTAKER),
-                    Organisasjonsnummer.of("999999999"),
-                    null
-            );
-
-            OmsorgspengerUtbetaling ytelse = byggOmsorgspengerUtbetalingSøknadBruker(
-                fraværPeriode
+        FraværPeriode fraværPeriode = new FraværPeriode(
+                periode,
+                Duration.ofHours(2),
+                new DelvisFravær(Duration.ofHours(4), Duration.ofHours(2)), //50% fravær fra 4 times-jobb skal oppgis som 4 timer i duration ( 50% av 7.5 timer, og rundet oppover til nærmeste halvtime)
+                FraværÅrsak.ORDINÆRT_FRAVÆR,
+                SøknadÅrsak.NYOPPSTARTET_HOS_ARBEIDSGIVER,
+                List.of(AktivitetFravær.ARBEIDSTAKER),
+                Organisasjonsnummer.of("999999999"),
+                null
         );
 
+        OmsorgspengerUtbetaling ytelse = byggOmsorgspengerUtbetalingSøknadBruker(
+                fraværPeriode
+        );
 
 
-        var feil = validatorSøknad.valider(ytelse);
+        List<Feil> feil = lagSøknadOgValider(ytelse);
 
         assertThat(feil).hasSize(1);
         feilInneholder(feil, "fraværsperioder[0]", "avvikDurationOgDelvisFravær");
     }
 
+
     // Søknad med fraværskorrigering av IM
     private OmsorgspengerUtbetaling byggOmsorgspengerUtbetalingMedFraværskorrigeringIm(FraværPeriode... fraværPerioder) {
         return new OmsorgspengerUtbetaling(
@@ -326,5 +362,22 @@ private static String jsonFromFile(String filename) {
         static Søknad komplettJsonSøknad() {
             return Søknad.SerDes.deserialize(jsonFromFile("komplett-søknad-omp-utbetaling-snf.json"));
         }
+
+
+    }
+
+    private List<Feil> lagSøknadOgValider(OmsorgspengerUtbetaling ytelse) {
+        Søknad søknad = lagSøknad(ytelse);
+        return søknadValidator.valider(søknad);
+    }
+
+    private Søknad lagSøknad(OmsorgspengerUtbetaling ytelse) {
+        return new Søknad(
+                new SøknadId("foo"),
+                new Versjon("1.1.0"),
+                ZonedDateTime.now(),
+                new Søker(NorskIdentitetsnummer.of("11111111111")),
+                Språk.NORSK_BOKMÅL,
+                ytelse);
     }
 }