endret navn på objektet for info fra punsj til journalposter, lagt ti… (#146)

christofferlohne · web-flow · commit 822d4a9e87c0 · 2021-05-06T11:30:14.000+02:00
* endret navn på objektet for info fra punsj til journalposter, lagt til journalpostId

* fiks etter kommentarer fra Stian

* Lagt tilbake InfoFraPunsj, flyttet Journalposter til søknad
diff --git a/soknad/src/main/java/no/nav/k9/søknad/Søknad.java b/soknad/src/main/java/no/nav/k9/søknad/Søknad.java
@@ -2,11 +2,15 @@
 
 import java.io.IOException;
 import java.time.ZonedDateTime;
+import java.util.ArrayList;
 import java.util.Collections;
 import java.util.List;
+import java.util.Objects;
 
 import javax.validation.Valid;
+import javax.validation.constraints.Max;
 import javax.validation.constraints.NotNull;
+import javax.validation.constraints.Size;
 
 import com.fasterxml.jackson.annotation.JsonAutoDetect;
 import com.fasterxml.jackson.annotation.JsonCreator;
@@ -18,10 +22,12 @@
 
 import no.nav.k9.søknad.felles.Versjon;
 import no.nav.k9.søknad.felles.personopplysninger.Søker;
+import no.nav.k9.søknad.felles.type.Journalpost;
 import no.nav.k9.søknad.felles.type.Person;
 import no.nav.k9.søknad.felles.type.Språk;
 import no.nav.k9.søknad.felles.type.SøknadId;
 import no.nav.k9.søknad.ytelse.Ytelse;
+import no.nav.k9.søknad.ytelse.psb.v1.PleiepengerSyktBarn;
 
 @JsonIgnoreProperties(ignoreUnknown = true)
 @JsonAutoDetect(fieldVisibility = JsonAutoDetect.Visibility.NONE, getterVisibility = JsonAutoDetect.Visibility.NONE, setterVisibility = JsonAutoDetect.Visibility.NONE, isGetterVisibility = JsonAutoDetect.Visibility.NONE, creatorVisibility = JsonAutoDetect.Visibility.NONE)
@@ -52,6 +58,11 @@ public class Søknad implements Innsending {
     @JsonProperty(value = "språk", required = false)
     private Språk språk = Språk.NORSK_BOKMÅL;
 
+    @Valid
+    @Size(max=1000)
+    @JsonProperty(value = "journalposter")
+    private List<Journalpost> journalposter = new ArrayList<>();
+
     @JsonManagedReference
     @Valid
     @NotNull
@@ -169,6 +180,20 @@ public void setMottattDato(ZonedDateTime mottattDato) {
         this.søker = søker;
     }
 
+    public List<Journalpost> getJournalposter() {
+        return journalposter;
+    }
+
+    public Søknad medJournalpost(Journalpost journalpost) {
+        this.journalposter.add(Objects.requireNonNull(journalpost, "journalpost"));
+        return this;
+    }
+
+    public Søknad medJournalposter(List<Journalpost> journalposter) {
+        this.journalposter.addAll(Objects.requireNonNull(journalposter, "journalposter"));
+        return this;
+    }
+
     @SuppressWarnings("unchecked")
     public <Y extends Ytelse> Y getYtelse() {
         return (Y) ytelse;
diff --git a/soknad/src/main/java/no/nav/k9/søknad/felles/type/Journalpost.java b/soknad/src/main/java/no/nav/k9/søknad/felles/type/Journalpost.java
@@ -0,0 +1,50 @@
+package no.nav.k9.søknad.felles.type;
+
+import java.util.Objects;
+
+import javax.validation.Valid;
+import javax.validation.constraints.NotNull;
+import javax.validation.constraints.Pattern;
+import javax.validation.constraints.Size;
+
+import com.fasterxml.jackson.annotation.JsonAutoDetect;
+import com.fasterxml.jackson.annotation.JsonIgnoreProperties;
+import com.fasterxml.jackson.annotation.JsonProperty;
+
+@JsonIgnoreProperties(ignoreUnknown = true)
+@JsonAutoDetect(fieldVisibility = JsonAutoDetect.Visibility.NONE, getterVisibility = JsonAutoDetect.Visibility.NONE, setterVisibility = JsonAutoDetect.Visibility.NONE, isGetterVisibility = JsonAutoDetect.Visibility.NONE, creatorVisibility = JsonAutoDetect.Visibility.NONE)
+public class Journalpost {
+
+    private static final String GYLDIG = "^[\\p{Alnum}]+$";
+
+    @JsonProperty(value = "inneholderInfomasjonSomIkkeKanPunsjes")
+    @Valid
+    private Boolean inneholderInfomasjonSomIkkeKanPunsjes;
+
+    @JsonProperty(value = "journalpostId", required = true)
+    @NotNull
+    @Size(max = 50, min = 3)
+    @Pattern(regexp = GYLDIG, message = "Saksnummer [${validatedValue}] matcher ikke tillatt pattern [{regexp}]")
+    private String journalpostId;
+
+    public Journalpost() {
+    }
+
+    public Boolean getInneholderInfomasjonSomIkkeKanPunsjes() {
+        return inneholderInfomasjonSomIkkeKanPunsjes;
+    }
+
+    public Journalpost medInfomasjonSomIkkeKanPunsjes(Boolean søknadenInneholderInfomasjonSomIkkeKanPunsjes) {
+        this.inneholderInfomasjonSomIkkeKanPunsjes = søknadenInneholderInfomasjonSomIkkeKanPunsjes;
+        return this;
+    }
+
+    public String getJournalpostId() {
+        return journalpostId;
+    }
+
+    public Journalpost medJournalpostId(String journalpostId) {
+        this.journalpostId = Objects.requireNonNull(journalpostId);
+        return this;
+    }
+}
diff --git a/soknad/src/main/java/no/nav/k9/søknad/ytelse/psb/v1/InfoFraPunsj.java b/soknad/src/main/java/no/nav/k9/søknad/ytelse/psb/v1/InfoFraPunsj.java
@@ -27,4 +27,4 @@ public InfoFraPunsj() {
         this.søknadenInneholderInfomasjonSomIkkeKanPunsjes = søknadenInneholderInfomasjonSomIkkeKanPunsjes;
         return this;
     }
-}
+}
diff --git a/soknad/src/main/java/no/nav/k9/søknad/ytelse/psb/v1/PleiepengerSyktBarn.java b/soknad/src/main/java/no/nav/k9/søknad/ytelse/psb/v1/PleiepengerSyktBarn.java
@@ -5,7 +5,6 @@
 import java.util.Collections;
 import java.util.List;
 import java.util.Objects;
-import java.util.TreeMap;
 
 import javax.validation.Valid;
 import javax.validation.constraints.NotNull;
@@ -23,6 +22,7 @@
 import no.nav.k9.søknad.felles.personopplysninger.Barn;
 import no.nav.k9.søknad.felles.personopplysninger.Bosteder;
 import no.nav.k9.søknad.felles.personopplysninger.Utenlandsopphold;
+import no.nav.k9.søknad.felles.type.Journalpost;
 import no.nav.k9.søknad.felles.type.Periode;
 import no.nav.k9.søknad.felles.type.Person;
 import no.nav.k9.søknad.ytelse.Ytelse;
@@ -152,16 +152,12 @@ public Person getAnnenPart() {
     }
 
     public PleiepengerSyktBarn medSøknadsperiode(List<Periode> søknadsperiodeList) {
-        if (this.søknadsperiode == null)
-            this.søknadsperiode = new ArrayList<>();
-        this.søknadsperiode.addAll(søknadsperiodeList);
+        this.søknadsperiode.addAll(Objects.requireNonNull(søknadsperiodeList, "søknadsperiodeList"));
         return this;
     }
 
     public PleiepengerSyktBarn medSøknadsperiode(Periode søknadsperiode) {
-        if (this.søknadsperiode == null)
-            this.søknadsperiode = new ArrayList<>();
-        this.søknadsperiode.add(søknadsperiode);
+        this.søknadsperiode.add(Objects.requireNonNull(søknadsperiode, "søknadsperiode"));
         return this;
     }
 
diff --git a/soknad/src/test/java/no/nav/k9/søknad/ytelse/psb/PleiepengerBarnSøknadValidatorTest.java b/soknad/src/test/java/no/nav/k9/søknad/ytelse/psb/PleiepengerBarnSøknadValidatorTest.java
@@ -67,6 +67,13 @@ public class PleiepengerBarnSøknadValidatorTest {
         verifyIngenFeil(søknad);
     }
 
+    @Test
+    public void komplettSøknadSkalIkkeHaFeil() {
+        var søknadsperiode = new Periode(LocalDate.now(), LocalDate.now().plusMonths(3));
+        var psb = TestUtils.komplettYtelsePsb(søknadsperiode);
+        verifyIngenFeil(psb);
+    }
+
     @Test
     public void minimumSøknadMedOmsorgNullPåFelterSkalIkkeHaValideringsfeil() {
         var psb = TestUtils.minimumSøknadPleiepengerSyktBarnMedDelperioder();
@@ -89,16 +96,6 @@ public class PleiepengerBarnSøknadValidatorTest {
         verifyHarFeil(ytelse);
     }
 
-    @Disabled //Disabled siden det er usikkert om dette er et krav
-    @Test
-    public void måVæreSøknadHvisDetErInfoOmOmsorg() {
-        var ytelse = TestUtils.minimumEndringssøknad(new Periode(LocalDate.now().minusMonths(1), LocalDate.now().plusMonths(1)))
-                .medOmsorg(new Omsorg().medRelasjonTilBarnet(Omsorg.BarnRelasjon.MOR).medBeskrivelseAvOmsorgsrollen(TestUtils.testTekst()));
-        var feil = valider(ytelse);
-        assertThat(feil.size()).isEqualTo(1);
-        assertThat(feil.get(0).getFeilkode()).isEqualTo("IllegalArgumentException");
-    }
-
     @Test
     public void endringssøknadUtenFeil() {
         var endringsperiode = new Periode(LocalDate.now().minusMonths(1), LocalDate.now().plusMonths(1));
@@ -117,6 +114,7 @@ public class PleiepengerBarnSøknadValidatorTest {
         verifyIngenFeil(endringssøknad);
 
     }
+
     @Test
     public void søknadOgEndringMedFeil() {
         var søknadsperiode = new Periode(LocalDate.now(), LocalDate.now().plusMonths(2));
diff --git a/soknad/src/test/java/no/nav/k9/søknad/ytelse/psb/TestUtils.java b/soknad/src/test/java/no/nav/k9/søknad/ytelse/psb/TestUtils.java
@@ -8,27 +8,26 @@
 import java.util.List;
 import java.util.Map;
 
-import no.nav.k9.søknad.JsonUtils;
 import no.nav.k9.søknad.Søknad;
-import no.nav.k9.søknad.ytelse.psb.v1.InfoFraPunsj;
-import no.nav.k9.søknad.ytelse.psb.v1.arbeidstid.Arbeidstaker;
 import no.nav.k9.søknad.felles.personopplysninger.Barn;
 import no.nav.k9.søknad.felles.personopplysninger.Bosteder;
 import no.nav.k9.søknad.felles.personopplysninger.Utenlandsopphold;
 import no.nav.k9.søknad.felles.type.Landkode;
 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.ytelse.Ytelse;
 import no.nav.k9.søknad.ytelse.psb.v1.Beredskap;
 import no.nav.k9.søknad.ytelse.psb.v1.DataBruktTilUtledning;
+import no.nav.k9.søknad.felles.type.Journalpost;
+import no.nav.k9.søknad.ytelse.psb.v1.InfoFraPunsj;
 import no.nav.k9.søknad.ytelse.psb.v1.LovbestemtFerie;
 import no.nav.k9.søknad.ytelse.psb.v1.LovbestemtFerie.LovbestemtFeriePeriodeInfo;
 import no.nav.k9.søknad.ytelse.psb.v1.Nattevåk;
 import no.nav.k9.søknad.ytelse.psb.v1.Omsorg;
 import no.nav.k9.søknad.ytelse.psb.v1.PleiepengerSyktBarn;
 import no.nav.k9.søknad.ytelse.psb.v1.Uttak;
 import no.nav.k9.søknad.ytelse.psb.v1.UttakPeriodeInfo;
+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;
@@ -144,7 +143,8 @@ static PleiepengerSyktBarn komplettYtelsePsb(Periode søknadsperiode) {
         var søknadInfo = new DataBruktTilUtledning( true, true,
                 false, false, true );
 
-        var infoFraPunsj = new InfoFraPunsj().medSøknadenInneholderInfomasjonSomIkkeKanPunsjes(false);
+        var infoFraPunsj = new InfoFraPunsj()
+                .medSøknadenInneholderInfomasjonSomIkkeKanPunsjes(false);
 
         return new PleiepengerSyktBarn()
                 .medSøknadsperiode(søknadsperiode)
@@ -235,4 +235,8 @@ static String testTekst() {
         return "Lorem ipsum dolor sit amet, consectetur adipisicing elit, sed do eiusmod tempor incididunt ut labore et dolore magna aliqua.";
     }
 
+    static String journalpostId() {
+        return "sajhdasd83724234";
+    }
+
 }
diff --git a/soknad/src/test/resources/ytelse/psb/komplett-søknad.json b/soknad/src/test/resources/ytelse/psb/komplett-søknad.json
@@ -5,6 +5,10 @@
   "søker": {
     "norskIdentitetsnummer": "11111111111"
   },
+  "journalposter" : [ {
+    "inneholderInfomasjonSomIkkeKanPunsjes" : false,
+    "journalpostId" : "sajhdasd83724234"
+  } ],
   "ytelse": {
     "type" : "PLEIEPENGER_SYKT_BARN",
     "søknadsperiode" : [ "2018-12-30/2019-10-20" ],

Original file line number	Diff line number	Diff line change
`@@ -27,4 +27,4 @@ public InfoFraPunsj() {`
`27`	`27`	`this.søknadenInneholderInfomasjonSomIkkeKanPunsjes = søknadenInneholderInfomasjonSomIkkeKanPunsjes;`
`28`	`28`	`return this;`
`29`	`29`	`}`
`30`		`-}`
	`30`	`+}`