Utvider fraværperiode med fraværårsak og legger på sluttdato på frilanser. (#120)

ramrock93 · web-flow · commit 4f6fa1f2a621 · 2021-03-24T13:19:24.000+01:00
* Utvider fraværperiode med fraværårsak.

* Legger på sluttdato på frilanser og validerer på felter.

* Fikser tester

* Markerer frilanser builder som deprecated og bruker constructor.
Legger også til setter metoder med chaining for bedre lesbarhet.
diff --git a/soknad-omsorgspenger-utbetaling/src/test/java/no/nav/k9/søknad/omsorgspenger/utbetaling/snf/TestUtils.java b/soknad-omsorgspenger-utbetaling/src/test/java/no/nav/k9/søknad/omsorgspenger/utbetaling/snf/TestUtils.java
@@ -80,6 +80,7 @@ class TestUtils {
                 ))
                 .frilanser(Frilanser.builder()
                         .startdato(LocalDate.parse("2019-01-01"))
+                        .sluttdato(LocalDate.parse("2020-01-01"))
                         .jobberFortsattSomFrilans(false)
                         .build())
                 .søker(Søker.builder()
diff --git a/soknad-omsorgspenger-utbetaling/src/test/resources/snf/komplett-søknad.json b/soknad-omsorgspenger-utbetaling/src/test/resources/snf/komplett-søknad.json
@@ -33,6 +33,7 @@
   }],
   "frilanser": {
     "startdato": "2019-01-01",
+    "sluttdato": "2020-01-01",
     "jobberFortsattSomFrilans": false
   }
 }
diff --git a/soknad/src/main/java/no/nav/k9/søknad/felles/aktivitet/Frilanser.java b/soknad/src/main/java/no/nav/k9/søknad/felles/aktivitet/Frilanser.java
@@ -4,6 +4,7 @@
 import com.fasterxml.jackson.annotation.JsonCreator;
 import com.fasterxml.jackson.annotation.JsonIgnoreProperties;
 import com.fasterxml.jackson.annotation.JsonProperty;
+import com.fasterxml.jackson.databind.util.ClassUtil;
 
 import javax.validation.constraints.NotNull;
 import java.time.LocalDate;
@@ -14,35 +15,63 @@ public class Frilanser {
 
     @JsonProperty(value = "startdato")
     @NotNull
-    private final LocalDate startdato;
+    private LocalDate startdato;
+
+    @JsonProperty(value = "sluttdato")
+    private LocalDate sluttdato;
 
     @JsonProperty(value = "jobberFortsattSomFrilans")
-    private final Boolean jobberFortsattSomFrilans;
+    private Boolean jobberFortsattSomFrilans;
 
+    /**@deprecated brukt ctor.*/
+    @Deprecated(forRemoval = true)
     public static Frilanser.Builder builder() {
         return new Frilanser.Builder();
     }
 
+    public Frilanser() {}
+
     @JsonCreator
     public Frilanser(
                      @JsonProperty("startdato") LocalDate startdato,
+                     @JsonProperty("sluttdato") LocalDate sluttdato,
                      @JsonProperty("jobberFortsattSomFrilans") Boolean jobberFortsattSomFrilans) {
         this.startdato = startdato;
+        this.sluttdato = sluttdato;
         this.jobberFortsattSomFrilans = jobberFortsattSomFrilans;
     }
+
+    public Frilanser medStartDato(LocalDate startdato) {
+        this.startdato = startdato;
+        return this;
+    }
+
+    public Frilanser medSluttDato(LocalDate sluttdato) {
+        this.sluttdato = sluttdato;
+        return this;
+    }
+
+    public Frilanser medJobberFortsattSomFrilans(Boolean jobberFortsattSomFrilans) {
+        this.jobberFortsattSomFrilans = jobberFortsattSomFrilans;
+        return this;
+    }
     
     public LocalDate getStartdato() {
         return startdato;
     }
-    
+    public LocalDate getSluttdato() { return sluttdato; }
     public Boolean getJobberFortsattSomFrilans() {
         return jobberFortsattSomFrilans;
     }
 
+    /**@deprecated brukt ctor.*/
+    @Deprecated(forRemoval = true)
     public static final class Builder {
         private LocalDate startdato;
+        private LocalDate sluttdato;
         private Boolean jobberFortsattSomFrilans;
 
+
         private Builder() {
         }
 
@@ -51,13 +80,18 @@ public Builder startdato(LocalDate startdato) {
             return this;
         }
 
+        public Builder sluttdato(LocalDate sluttdato) {
+            this.sluttdato = sluttdato;
+            return this;
+        }
+
         public Frilanser.Builder jobberFortsattSomFrilans(Boolean jobberFortsattSomFrilans) {
             this.jobberFortsattSomFrilans = jobberFortsattSomFrilans;
             return this;
         }
 
         public Frilanser build() {
-            return new Frilanser(startdato, jobberFortsattSomFrilans);
+            return new Frilanser(startdato, sluttdato, jobberFortsattSomFrilans);
         }
     }
 }
diff --git a/soknad/src/main/java/no/nav/k9/søknad/felles/fravær/FraværPeriode.java b/soknad/src/main/java/no/nav/k9/søknad/felles/fravær/FraværPeriode.java
@@ -4,6 +4,7 @@
 import java.util.Objects;
 
 import javax.validation.Valid;
+import javax.validation.constraints.NotNull;
 
 import com.fasterxml.jackson.annotation.JsonAutoDetect;
 import com.fasterxml.jackson.annotation.JsonCreator;
@@ -22,10 +23,19 @@ public class FraværPeriode implements Comparable<FraværPeriode> {
     @JsonProperty("duration")
     private final Duration duration;
 
+    @NotNull
+    @JsonProperty(value = "årsak", required = true)
+    private final FraværÅrsak årsak;
+
     @JsonCreator
-    public FraværPeriode(@JsonProperty("periode") @Valid Periode periode, @JsonProperty("duration") Duration duration) {
+    public FraværPeriode(
+            @JsonProperty("periode") @Valid Periode periode,
+            @JsonProperty("duration") Duration duration,
+            @JsonProperty("årsak") FraværÅrsak årsak
+    ) {
         this.periode = periode;
         this.duration = duration;
+        this.årsak = årsak;
     }
 
     public Periode getPeriode() {
@@ -36,30 +46,34 @@ public Duration getDuration() {
         return duration;
     }
 
+    public FraværÅrsak getÅrsak() { return årsak; }
+
     @Override
     public boolean equals(Object o) {
         if (this == o) return true;
         if (o == null || getClass() != o.getClass()) return false;
         FraværPeriode that = (FraværPeriode) o;
         return periode.equals(that.periode) &&
-                Objects.equals(duration, that.duration);
+                Objects.equals(duration, that.duration) &&
+                Objects.equals(årsak, that.årsak);
     }
 
     @Override
     public int hashCode() {
-        return Objects.hash(periode, duration);
+        return Objects.hash(periode, duration, årsak);
     }
 
     @Override
     public int compareTo(FraværPeriode b) {
         return this.getPeriode().compareTo(b.getPeriode());
     }
-    
+
     @Override
     public String toString() {
         return "FraværPeriode{" +
                 "periode=" + periode +
                 ", duration=" + duration +
+                ", årsak=" + årsak +
                 '}';
     }
 }
diff --git a/soknad/src/main/java/no/nav/k9/søknad/felles/fravær/FraværÅrsak.java b/soknad/src/main/java/no/nav/k9/søknad/felles/fravær/FraværÅrsak.java
@@ -0,0 +1,50 @@
+package no.nav.k9.søknad.felles.fravær;
+
+import com.fasterxml.jackson.annotation.JsonCreator;
+import com.fasterxml.jackson.annotation.JsonValue;
+
+public enum FraværÅrsak {
+    STENGT_SKOLE_ELLER_BARNEHAGE(
+            "STENGT_SKOLE_ELLER_BARNEHAGE",
+            "Stengt skole eller barnehage"),
+
+    SMITTEVERNHENSYN(
+            "SMITTEVERNHENSYN",
+            "Smitteverhensyn"
+    ),
+
+    ORDINÆRT_FRAVÆR(
+            "ORDINÆRT_FRAVÆR",
+            "Ordinært fravær"
+    );
+
+
+    private String kode;
+    private String årsak;
+
+
+    FraværÅrsak(String kode, String årsak) {
+        this.kode = kode;
+        this.årsak = årsak;
+    }
+
+    public String getÅrsak() {
+        return årsak;
+    }
+
+    @JsonValue
+    public String getKode() { return kode; }
+
+    @JsonCreator
+    public static FraværÅrsak of(String value) {
+        if (value == null || value.isBlank()) {
+            return null;
+        }
+        for (FraværÅrsak årsak : values()) {
+            if (årsak.kode.equals(value)) {
+                return årsak;
+            }
+        }
+        throw new IllegalArgumentException("Ikke støttet årsak '" + value + "'");
+    }
+}
diff --git a/soknad/src/main/java/no/nav/k9/søknad/ytelse/YtelseValidator.java b/soknad/src/main/java/no/nav/k9/søknad/ytelse/YtelseValidator.java
@@ -7,6 +7,7 @@
 
 public abstract class YtelseValidator {
     protected static final String PÅKREVD = "påkrevd";
+    protected static final String UGYLDIG_ARGUMENT = "ugyldig argument";
 
     public abstract List<Feil> valider(Ytelse søknad);
 
diff --git a/soknad/src/main/java/no/nav/k9/søknad/ytelse/omsorgspenger/v1/OmsorgspengerUtbetalingValidator.java b/soknad/src/main/java/no/nav/k9/søknad/ytelse/omsorgspenger/v1/OmsorgspengerUtbetalingValidator.java
@@ -33,10 +33,11 @@ private void getAktivitet(OmsorgspengerUtbetaling søknad, List<Feil> feil) {
 
         if (aktivitet != null) {
 
-            if (aktivitet.getFrilanser() != null
-                || (aktivitet.getSelvstendigNæringsdrivende() != null && !aktivitet.getSelvstendigNæringsdrivende().isEmpty())) {
+            if ((aktivitet.getFrilanser() != null)
+                    || ((aktivitet.getSelvstendigNæringsdrivende() != null) && !aktivitet.getSelvstendigNæringsdrivende().isEmpty())) {
                 validerFrilanserOgSelvstendingNæringsdrivende(aktivitet.getSelvstendigNæringsdrivende(), aktivitet.getFrilanser(), feil);
                 validerSelvstendingNæringsdrivende(aktivitet.getSelvstendigNæringsdrivende(), feil);
+                validerFrilanser(aktivitet.getFrilanser(), feil);
             }
         }
     }
@@ -48,6 +49,18 @@ private void getAktivitet(OmsorgspengerUtbetaling søknad, List<Feil> feil) {
         }
     }
 
+    private void validerFrilanser(Frilanser frilanser, List<Feil> feil) {
+        if (frilanser == null) return;
+
+        if (!frilanser.getJobberFortsattSomFrilans() && frilanser.getSluttdato() == null) {
+            feil.add(new Feil("frilanser.sluttdato", PÅKREVD, "'sluttdato' kan ikke være null, dersom 'jobberFortsattSomFrilans' er false."));
+        }
+
+        if (frilanser.getSluttdato() != null && frilanser.getStartdato().isAfter(frilanser.getSluttdato())) {
+            feil.add(new Feil("frilanser.startdato", UGYLDIG_ARGUMENT, "'startdato' kan ikke være etter 'sluttdato'"));
+        }
+    }
+
     private void validerSelvstendingNæringsdrivende(List<SelvstendigNæringsdrivende> selvstendigeVirksomheter, List<Feil> feil) {
         if (selvstendigeVirksomheter == null || selvstendigeVirksomheter.isEmpty())
             return;
@@ -72,11 +85,11 @@ private void getAktivitet(OmsorgspengerUtbetaling søknad, List<Feil> feil) {
                 if (snInfo.getRegistrertIUtlandet() != null) {
                     if (snInfo.getRegistrertIUtlandet() && snInfo.getLandkode() == null) {
                         feil.add(new Feil(snInfoFelt + ".landkode", PÅKREVD,
-                            "landkode må være satt, og kan ikke være null, dersom virksomhet er registrert i utlandet."));
+                                "landkode må være satt, og kan ikke være null, dersom virksomhet er registrert i utlandet."));
                     }
                 } else if (sn.getOrganisasjonsnummer() == null) {
                     feil.add(new Feil(snInfoFelt + ".registrertIUtlandet og " + snFelt + ".organisasjonsnummer", PÅKREVD,
-                        "Dersom virksomheten er registrert i norge, må organisasjonsnummeret være satt."));
+                            "Dersom virksomheten er registrert i norge, må organisasjonsnummeret være satt."));
                 }
             });
             index++;
diff --git a/soknad/src/test/java/no/nav/k9/søknad/ytelse/omsorgspenger/v1/OmsorgspengerUtbetalingValidatorTest.java b/soknad/src/test/java/no/nav/k9/søknad/ytelse/omsorgspenger/v1/OmsorgspengerUtbetalingValidatorTest.java
diff --git a/soknad/src/test/resources/ytelse/omp/utbetaling/komplett-søknad-omp-utbetaling-snf.json b/soknad/src/test/resources/ytelse/omp/utbetaling/komplett-søknad-omp-utbetaling-snf.json
diff --git a/soknad/src/test/resources/ytelse/omp/utbetaling/minimum-søknad-omp-utbetaling-snf.json b/soknad/src/test/resources/ytelse/omp/utbetaling/minimum-søknad-omp-utbetaling-snf.json

Original file line number	Diff line number	Diff line change
`@@ -33,6 +33,7 @@`
`33`	`33`	`}],`
`34`	`34`	`"frilanser": {`
`35`	`35`	`"startdato": "2019-01-01",`
	`36`	`+ "sluttdato": "2020-01-01",`
`36`	`37`	`"jobberFortsattSomFrilans": false`
`37`	`38`	`}`
`38`	`39`	`}`