Legger til info erFiskerPåBladB. (#311)

* Legger til info erFiskerPåBladB.

Gir valideringsfeil dersom virksomhetstype inneholder FISKE, men erFiskerPåBladB er null.

* Fikser feilende test

Author: ramrock93

soknad-omsorgspenger-utbetaling/src/test/java/no/nav/k9/søknad/omsorgspenger/utbetaling/snf/TestUtils.java

@@ -71,6 +71,7 @@ class TestUtils {
                                                 .endringDato(LocalDate.parse("2018-12-12"))
                                                 .erNyoppstartet(false)
                                                 .erVarigEndring(true)
+                                                .erFiskerPåBladB(true)
                                                 .regnskapsførerNavn("Regnskapsfører Svensen")
                                                 .regnskapsførerTelefon("12345678")
                                                 .registrertIUtlandet(true)
@@ -106,6 +107,7 @@ class TestUtils {
                                                 .endringBegrunnelse("fordi")
                                                 .endringDato(LocalDate.parse("2018-12-12"))
                                                 .erNyoppstartet(false)
+                                                .erFiskerPåBladB(true)
                                                 .erVarigEndring(true)
                                                 .regnskapsførerNavn("Regnskapsfører Svensen")
                                                 .regnskapsførerTelefon("12345678")

soknad-omsorgspenger-utbetaling/src/test/resources/snf/komplett-søknad-med-barn.json

@@ -19,6 +19,7 @@
       "2018-11-11/2018-11-30": {
         "virksomhetstyper":  [
           "DAGMAMMA", "ENK", "FISKE", "FRILANSER", "JORDBRUK_SKOGBRUK", "ANNEN", "-"],
+        "erFiskerPåBladB": true,
         "regnskapsførerNavn": "Regnskapsfører Svensen",
         "regnskapsførerTlf":  "12345678",
         "erVarigEndring":  true,

soknad-omsorgspenger-utbetaling/src/test/resources/snf/komplett-søknad-uten-næringsinntekt.json

@@ -19,6 +19,7 @@
       "2018-11-11/2018-11-30": {
         "virksomhetstyper":  [
           "DAGMAMMA", "FISKE", "JORDBRUK_SKOGBRUK", "ANNEN", "-"],
+        "erFiskerPåBladB": true,
         "regnskapsførerNavn": "Regnskapsfører Svensen",
         "regnskapsførerTlf":  "12345678",
         "erVarigEndring":  true,

soknad-omsorgspenger-utbetaling/src/test/resources/snf/komplett-søknad.json

@@ -19,6 +19,7 @@
       "2018-11-11/2018-11-30": {
         "virksomhetstyper":  [
           "DAGMAMMA", "FISKE", "JORDBRUK_SKOGBRUK", "ANNEN", "-"],
+        "erFiskerPåBladB": true,
         "regnskapsførerNavn": "Regnskapsfører Svensen",
         "regnskapsførerTlf":  "12345678",
         "erVarigEndring":  true,

soknad/src/main/java/no/nav/k9/søknad/felles/opptjening/SelvstendigNæringsdrivende.java

@@ -43,12 +43,16 @@ public class SelvstendigNæringsdrivende {
     @NotEmpty
     private Map<@NotNull Periode, @NotNull SelvstendigNæringsdrivendePeriodeInfo> perioder;
 
-    /** Orgnummer - påkrevd for norske selskaper, ikke for utenlandske enn så lenge. */
+    /**
+     * Orgnummer - påkrevd for norske selskaper, ikke for utenlandske enn så lenge.
+     */
     @JsonProperty(value = "organisasjonsnummer", required = false)
     @Valid
     private Organisasjonsnummer organisasjonsnummer;
 
-    /** Virsomhetsnavn - påkrevd for norske og utenlandske selskaper. */
+    /**
+     * Virsomhetsnavn - påkrevd for norske og utenlandske selskaper.
+     */
     @JsonProperty(value = "virksomhetNavn", required = false)
     @Pattern(regexp = "^[\\p{Graph}\\p{Space}\\p{Sc}\\p{L}\\p{M}\\p{N}]+$", message = "[ugyldigSyntaks] '${validatedValue}' matcher ikke tillatt pattern '{regexp}'")
     private String virksomhetNavn;
@@ -112,7 +116,9 @@ boolean isOkOrganisasjonsnummer() {
         return true;
     }
 
-    /**@Deprecated bruk tom ctor*/
+    /**
+     * @Deprecated bruk tom ctor
+     */
     public static final class Builder {
         private Map<Periode, SelvstendigNæringsdrivendePeriodeInfo> perioder;
         private Organisasjonsnummer organisasjonsnummer;
@@ -195,7 +201,10 @@ public static class SelvstendigNæringsdrivendePeriodeInfo {
         //TODO validering etter "ISO 3166 alpha-3"
         @JsonProperty("landkode")
         @Valid
-        private  Landkode landkode;
+        private Landkode landkode;
+
+        @JsonProperty("erFiskerPåBladB")
+        private Boolean erFiskerPåBladB;
 
         public SelvstendigNæringsdrivendePeriodeInfo() {
 
@@ -213,7 +222,9 @@ public static class SelvstendigNæringsdrivendePeriodeInfo {
                 @JsonProperty("bruttoInntekt") BigDecimal bruttoInntekt,
                 @JsonProperty("erNyoppstartet") Boolean erNyoppstartet,
                 @JsonProperty("registrertIUtlandet") Boolean registrertIUtlandet,
-                @JsonProperty("landkode") Landkode landkode) {
+                @JsonProperty("landkode") Landkode landkode,
+                @JsonProperty("erFiskerPåBladB") Boolean erFiskerPåBladB
+        ) {
             this.virksomhetstyper = Objects.requireNonNull(virksomhetstyper, "virksomhetstyper");
             this.regnskapsførerNavn = regnskapsførerNavn;
             this.regnskapsførerTlf = regnskapsførerTlf;
@@ -225,6 +236,7 @@ public static class SelvstendigNæringsdrivendePeriodeInfo {
             this.erNyoppstartet = erNyoppstartet;
             this.registrertIUtlandet = registrertIUtlandet;
             this.landkode = landkode;
+            this.erFiskerPåBladB = erFiskerPåBladB;
         }
 
         public SelvstendigNæringsdrivendePeriodeInfo medVirksomhetstyper(List<VirksomhetType> virksomhetstyper) {
@@ -277,6 +289,11 @@ public static class SelvstendigNæringsdrivendePeriodeInfo {
             return this;
         }
 
+        public SelvstendigNæringsdrivendePeriodeInfo medErFiskerPåBladB(Boolean erFiskerPåBladB) {
+            this.erFiskerPåBladB = Objects.requireNonNull(erFiskerPåBladB, "erFiskerPåBladB");
+            return this;
+        }
+
         public SelvstendigNæringsdrivendePeriodeInfo medLandkode(Landkode landkode) {
             this.landkode = Objects.requireNonNull(landkode, "landkode");
             return this;
@@ -326,6 +343,10 @@ public List<VirksomhetType> getVirksomhetstyper() {
             return virksomhetstyper;
         }
 
+        public Boolean getErFiskerPåBladB() {
+            return erFiskerPåBladB;
+        }
+
         public static Builder builder() {
             return new Builder();
         }
@@ -363,6 +384,7 @@ private boolean isVarigEndringBegrunnelseSatt() {
             }
             return !regnskapsførerNavn.isEmpty();
         }
+
         @AssertTrue(message = "Kan ikke være blankt")
         private boolean isRegnskapsførerTlfEmpty() {
             if (regnskapsførerTlf == null) {
@@ -371,6 +393,19 @@ private boolean isVarigEndringBegrunnelseSatt() {
             return !regnskapsførerTlf.isEmpty();
         }
 
+        @AssertTrue(message = "erFiskerPåBladB kan ikke være null dersom virksomhetstyper er FISKE.")
+        private boolean isFiskerPåBladBValid() {
+            if (virksomhetstyper.contains(VirksomhetType.FISKE)) {
+                try {
+                    Objects.requireNonNull(erFiskerPåBladB, "erFiskerPåBladB");
+                    return true;
+                } catch (Exception e) {
+                    return false;
+                }
+            }
+            return true;
+        }
+
         /* Deaktivert pga søknader med feil
         @AssertTrue(message = "[ugyldigVerdi] Norge kan ikke være en landkode")
         private boolean isLandkodeNotNor() {
@@ -381,7 +416,7 @@ private boolean isLandkodeNotNor() {
         }
          */
 
-        @Deprecated (forRemoval = true)
+        @Deprecated(forRemoval = true)
         public static final class Builder {
             private List<VirksomhetType> virksomhetstyper;
             private String regnskapsførerNavn;
@@ -395,6 +430,8 @@ public static final class Builder {
             private Boolean registrertIUtlandet;
             private Landkode landkode;
 
+            private Boolean erFiskerPåBladB;
+
             private Builder() {
             }
 
@@ -453,19 +490,27 @@ public Builder landkode(Landkode landkode) {
                 return this;
             }
 
+            public Builder erFiskerPåBladB(Boolean erFiskerPåBladB) {
+                this.erFiskerPåBladB = erFiskerPåBladB;
+                return this;
+            }
+
             @Deprecated
             public SelvstendigNæringsdrivendePeriodeInfo build() {
                 return new SelvstendigNæringsdrivendePeriodeInfo(
-                    virksomhetstyper,
-                    regnskapsførerNavn,
-                    regnskapsførerTelefon,
-                    erVarigEndring,
-                    erNyIArbeidslivet,
-                    endringDato,
-                    endringBegrunnelse,
-                    bruttoInntekt,
-                    erNyoppstartet,
-                    registrertIUtlandet, landkode);
+                        virksomhetstyper,
+                        regnskapsførerNavn,
+                        regnskapsførerTelefon,
+                        erVarigEndring,
+                        erNyIArbeidslivet,
+                        endringDato,
+                        endringBegrunnelse,
+                        bruttoInntekt,
+                        erNyoppstartet,
+                        registrertIUtlandet,
+                        landkode,
+                        erFiskerPåBladB
+                );
             }
         }
     }

soknad/src/test/java/no/nav/k9/søknad/felles/opptjening/SelvstendigNæringsdrvendeTest.java

@@ -108,5 +108,15 @@ class SelvstendigNæringsdrvendeTest {
 
         validator.verifyIngenFeil(opptjeningAktivitet);
     }
+
+    @Test
+    public void erFiskerPåBladBKanIkkeVæreNullDersomVirksomhetstypeErFiske() {
+        var periode = new Periode(LocalDate.now(), LocalDate.now().plusWeeks(2));
+        var snInfo = new SelvstendigNæringsdrivende.SelvstendigNæringsdrivendePeriodeInfo()
+                .medVirksomhetstyper(List.of(VirksomhetType.FISKE));
+        var sn = new SelvstendigNæringsdrivende().medPerioder(Map.of(periode, snInfo));
+        var feil = validator.verifyHarFeil(sn);
+        feilInneholder(feil, "påkrevd");
+    }
 }
 

soknad/src/test/resources/ytelse/omp/utbetaling/v1.0.0/komplett-søknad-omp-utbetaling-snf.json

@@ -25,6 +25,7 @@
                 "DAGMAMMA",
                 "ANNEN"
               ],
+              "erFiskerPåBladB": true,
               "regnskapsførerNavn": "Kjell",
               "regnskapsførerTlf": "84554",
               "erVarigEndring": true,

soknad/src/test/resources/ytelse/omp/utbetaling/v1.1.0/komplett-søknad-omp-utbetaling-snf.json

@@ -25,6 +25,7 @@
                 "DAGMAMMA",
                 "ANNEN"
               ],
+              "erFiskerPåBladB": true,
               "regnskapsførerNavn": "Kjell",
               "regnskapsførerTlf": "84554",
               "erVarigEndring": true,

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

@@ -22,6 +22,7 @@
         "perioder": {
           "2018-12-30/..": {
             "virksomhetstyper": ["FISKE", "JORDBRUK_SKOGBRUK", "DAGMAMMA", "ANNEN"],
+            "erFiskerPåBladB": true,
             "regnskapsførerNavn": "Regskapsfører",
             "regnskapsførerTlf": "88888889",
             "erVarigEndring": true,
@@ -143,4 +144,4 @@
       "beskrivelseAvOmsorgsrollen": "Noe tilleggsinformasjon. Lorem ipsum æÆøØåÅ."
     }
   }
-}
+}