Validerer at mottattdato ikke kan være i fremtiden. Flytter validering av versjon av mottattdato inn i en felles valideringsmetode for søknad. (#282)

Author: stiangre

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

@@ -1,8 +1,10 @@
 package no.nav.k9.søknad;
 
+import java.time.ZonedDateTime;
 import java.util.List;
 
 import no.nav.k9.søknad.felles.Feil;
+import no.nav.k9.søknad.felles.Versjon;
 
 public abstract class SøknadValidator<S> {
     protected static final String PÅKREVD = "påkrevd";
@@ -16,4 +18,30 @@ public void forsikreValidert(S søknad) {
             throw new ValideringsFeil(feil);
         }
     }
+    
+    
+    /**
+     * Felles validering av felter på søknad. Må kalles eksplisitt på av
+     * subklassene til denne klassen hvis den skal brukes,
+     * 
+     * @param søknad Søknaden som valideres.
+     * @param feil En liste der eventuelt feil skal legges til.
+     */
+    public final void validerFelterPåSøknad(Søknad søknad, List<Feil> feil) {
+        validerMottattDato(søknad, feil);
+        validerVersjon(søknad.getVersjon(), feil);
+    }
+    
+    private void validerMottattDato(Søknad søknad, List<Feil> feil) {
+        final int maximumClockSkewSeconds = 10;
+        if (søknad.getMottattDato().minusSeconds(maximumClockSkewSeconds).isAfter(ZonedDateTime.now())) {
+            feil.add(new Feil("mottattDato", "fremtidigDatoIkkeTillat", "Mottattdato kan ikke være satt til fremtiden."));
+        }
+    }
+    
+    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."));
+        }
+    }
 }

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

@@ -19,13 +19,6 @@ public class OmsorgspengerUtbetalingSøknadValidator extends SøknadValidator<S
     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);
@@ -34,7 +27,7 @@ public List<Feil> valider(Søknad søknad) {
                 .map(Feil::toFeil)
                 .collect(Collectors.toList());
 
-        validerVersjon(søknad.getVersjon(), feil);
+        validerFelterPåSøknad(søknad, feil);
         feil.addAll(new OmsorgspengerUtbetalingValidator().valider(søknad.getYtelse()));
 
         return feil;

soknad/src/main/java/no/nav/k9/søknad/ytelse/pls/v1/PleiepengerLivetsSluttfaseSøknadValidator.java

@@ -39,7 +39,7 @@ public List<Feil> valider(Søknad søknad, List<Periode> gyldigeEndringsperioder
                 .collect(Collectors.toList());
 
         validerInneholderBegrunnelseDersomTrekkAvPerioder(søknad, feil);
-        validerVersjon(søknad.getVersjon(), feil);
+        validerFelterPåSøknad(søknad, feil);
         validerPleietrengendeIkkeErSøker(søknad.getSøker(), søknad.getBerørtePersoner(), feil);
         validerPleietrengendeIkkeErSøker(søknad.getSøker(), List.of(((PleipengerLivetsSluttfase) søknad.getYtelse()).getPleietrengende()), feil);
         feil.addAll(new PleiepengerLivetsSluttfaseYtelseValidator().valider(søknad.getYtelse(), gyldigeEndringsperioder));
@@ -56,12 +56,6 @@ private static void validerInneholderBegrunnelseDersomTrekkAvPerioder(Søknad s
         }
     }
 
-    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."));
-        }
-    }
-
     private static void validerPleietrengendeIkkeErSøker(Søker søker, List<Person> pleietrengende, List<Feil> feil) {
         if (søker == null || pleietrengende == null || pleietrengende.isEmpty()) {
             return;

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

@@ -1,5 +1,7 @@
 package no.nav.k9.søknad.ytelse.psb.v1;
 
+import java.time.ZonedDateTime;
+import java.time.chrono.ChronoZonedDateTime;
 import java.util.List;
 import java.util.stream.Collectors;
 
@@ -23,12 +25,6 @@ public class PleiepengerSyktBarnSøknadValidator extends SøknadValidator<Søkna
     public PleiepengerSyktBarnSø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."));
-        }
-    }
-
     private static void validerBarnIkkeErSøker(Søker søker, List<Person> barnList, List<Feil> feil) {
         if (søker == null || barnList == null || barnList.isEmpty()) {
             return;
@@ -49,11 +45,11 @@ public List<Feil> valider(Søknad søknad, List<Periode> gyldigeEndringsperioder
         List<Feil> feil = validate.stream()
                 .map(Feil::toFeil)
                 .collect(Collectors.toList());
+        
+        validerFelterPåSøknad(søknad, feil);
 
         PleiepengerSyktBarn ytelse = (PleiepengerSyktBarn) søknad.getYtelse();
         validerInneholderBegrunnelseForInnsending(søknad, ytelse, feil);
-
-        validerVersjon(søknad.getVersjon(), feil);
         validerBarnIkkeErSøker(søknad.getSøker(), søknad.getBerørtePersoner(), feil);
         feil.addAll(new PleiepengerSyktBarnYtelseValidator().validerMedGyldigEndringsperodeHvisDenFinnes(søknad.getYtelse(), gyldigeEndringsperioder));
 

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

@@ -12,6 +12,7 @@
 
 import java.time.Duration;
 import java.time.LocalDate;
+import java.time.ZonedDateTime;
 import java.util.List;
 
 import org.junit.jupiter.api.Test;
@@ -45,6 +46,15 @@ class SøknadTest {
     public void minimumSøknadHarIngenValideringsFeil() {
         verifyIngenFeil(MINIMUM_SØKNAD);
     }
+    
+    @Test
+    public void mottattDatoKanIkkeVæreIFremtiden() {
+        final Periode søknadsperiode = new Periode(LocalDate.now(), LocalDate.now().plusMonths(2));
+        final Søknad søknad = SøknadEksempel.minimumSøknad(søknadsperiode);
+        verifyIngenFeil(søknad);
+        søknad.setMottattDato(ZonedDateTime.now().plusSeconds(60));
+        verifyHarFeil(søknad);
+    }
 
     @Test
     public void barnKanIkkeVæreSøker() {