lagt på validering av arbeidstid med faktisk og normal arbeidstid (#205)

Author: christofferlohne

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

@@ -3,8 +3,10 @@
 import java.time.Duration;
 
 import javax.validation.Valid;
+import javax.validation.constraints.AssertTrue;
 import javax.validation.constraints.NotNull;
 
+import org.hibernate.validator.constraints.time.DurationMax;
 import org.hibernate.validator.constraints.time.DurationMin;
 
 import com.fasterxml.jackson.annotation.JsonAutoDetect;
@@ -19,12 +21,14 @@ public class ArbeidstidPeriodeInfo {
     @Valid
     @NotNull
     @DurationMin
+    @DurationMax(hours = 24)
     @JsonProperty(value = "jobberNormaltTimerPerDag", required = true)
     private Duration jobberNormaltTimerPerDag;
 
     @Valid
     @NotNull
     @DurationMin
+    @DurationMax(hours = 24)
     @JsonProperty(value = "faktiskArbeidTimerPerDag", required = true)
     private Duration faktiskArbeidTimerPerDag;
 
@@ -56,4 +60,12 @@ public ArbeidstidPeriodeInfo medJobberNormaltTimerPerDag(Duration jobberNormaltT
         this.jobberNormaltTimerPerDag = jobberNormaltTimerPerDag;
         return this;
     }
+
+    @AssertTrue(message="[ugyldigArbeidstid] Faktisk arbeid er større enn jobber normalt")
+    private boolean isValid() {
+        if (jobberNormaltTimerPerDag == null || faktiskArbeidTimerPerDag == null) {
+            return true;
+        }
+        return jobberNormaltTimerPerDag.compareTo(faktiskArbeidTimerPerDag) >= 0;
+    }
 }

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

@@ -225,10 +225,13 @@ public static Beredskap lagBeredskap(Periode... perioder) {
 
     public static Arbeidstaker lagArbeidstaker(Periode... perioder) {
         ArbeidstidPeriodeInfo arbeidstidPeriodeInfo = new ArbeidstidPeriodeInfo(Duration.ofHours(7).plusMinutes(30), Duration.ofHours(7).plusMinutes(30));
-        var arbeidstaker = new Arbeidstaker(null, Organisasjonsnummer.of("999999999"),
+        return lagArbeidstaker(arbeidstidPeriodeInfo, perioder);
+    }
+
+    public static Arbeidstaker lagArbeidstaker(ArbeidstidPeriodeInfo arbeidstidPeriodeInfo, Periode... perioder) {
+        return new Arbeidstaker(null, Organisasjonsnummer.of("999999999"),
                 new ArbeidstidInfo(
                         lagPerioder(perioder, arbeidstidPeriodeInfo)));
-        return arbeidstaker;
     }
 
     public static Uttak lagUttak(Periode... perioder) {

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

@@ -23,6 +23,7 @@
 import no.nav.k9.søknad.ytelse.psb.SøknadEksempel;
 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;
 import no.nav.k9.søknad.ytelse.psb.v1.arbeidstid.ArbeidstidInfo;
 import no.nav.k9.søknad.ytelse.psb.v1.arbeidstid.ArbeidstidPeriodeInfo;
 import no.nav.k9.søknad.ytelse.psb.v1.tilsyn.TilsynPeriodeInfo;
@@ -215,7 +216,8 @@ public void overlappendePerioderForUttaksperiodeMap() {
 
         var arbeidstaker = new Arbeidstaker(null, Organisasjonsnummer.of("88888888"), arbeidstidInfo);
         ((PleiepengerSyktBarn) søknad.getYtelse()).getArbeidstid().leggeTilArbeidstaker(arbeidstaker);
-        verifyHarFeil(søknad);
+        var feil = verifyHarFeil(søknad);
+        feilInneholder(feil, "påkrevd");
     }
 
     @Test
@@ -277,4 +279,17 @@ public void overlappendePerioderForUttaksperiodeMap() {
         feilInneholder(feil, "ikkeEntydigId");
     }
 
+    @Test
+    public void søknadMedFaktisArbeidStørreEnnFaktisArbeid() {
+        var søknadsperiode = new Periode(LocalDate.now(), LocalDate.now().plusMonths(2));
+        var søknad = SøknadEksempel.søknad(YtelseEksempel.komplettYtelse(søknadsperiode));
+        var jobberNormalt = Duration.ofHours(3);
+        var jobberFaktisk = Duration.ofHours(7);
+        var arbeidstaker = YtelseEksempel.lagArbeidstaker(new ArbeidstidPeriodeInfo(jobberNormalt, jobberFaktisk), søknadsperiode);
+
+        ((PleiepengerSyktBarn) søknad.getYtelse()).medArbeidstid(new Arbeidstid().medArbeidstaker(List.of(arbeidstaker)));
+        var feil = verifyHarFeil(søknad);
+        feilInneholder(feil, "ugyldigArbeidstid");
+    }
+
 }

soknad/src/test/resources/ytelse/psb/søknad-med-endring.json

@@ -108,25 +108,25 @@
         "arbeidstidInfo": {
           "perioder": {
             "2018-12-30/2019-10-20": {
-              "jobberNormaltTimerPerDag": "PT7H28M",
-              "faktiskArbeidTimerPerDag": "PT37H30M"
+              "jobberNormaltTimerPerDag": "PT7H30M",
+              "faktiskArbeidTimerPerDag": "PT3H"
             }
           }
         }
       }],
       "frilanserArbeidstidInfo": {
         "perioder": {
           "2018-12-30/2019-10-20": {
-            "jobberNormaltTimerPerDag": "PT5H",
-            "faktiskArbeidTimerPerDag": "PT6H"
+            "jobberNormaltTimerPerDag": "PT6H",
+            "faktiskArbeidTimerPerDag": "PT5H"
           }
         }
       },
       "selvstendigNæringsdrivendeArbeidstidInfo": {
         "perioder": {
           "2018-12-30/2019-10-20": {
-            "jobberNormaltTimerPerDag": "PT4H",
-            "faktiskArbeidTimerPerDag": "PT7H"
+            "jobberNormaltTimerPerDag": "PT7H",
+            "faktiskArbeidTimerPerDag": "PT4H"
           }
         }
       }