fikser feilmelding teskst, fikk nullpointer ved utenthing av info. Fi… (#145)

* fikser feilmelding teskst, fikk nullpointer ved utenthing av info. Fikset på tester i TestUtil

Author: christofferlohne

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

@@ -48,7 +48,7 @@ public List<Feil> valider(Ytelse ytelse) {
             //TODO valider Utlandsopphold
 
         } catch (IllegalArgumentException e) {
-            feil.add(new Feil(e.getClass().getName(), e.getCause() + "IllegalArgumentException", e.getMessage()));
+            feil.add(new Feil(e.getClass().getName(), "IllegalArgumentException", e.getMessage()));
         }
 
         validerKomplettSøknad(psb, feil);

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

@@ -5,9 +5,6 @@
 import no.nav.k9.søknad.ValideringsFeil;
 import no.nav.k9.søknad.felles.Feil;
 import no.nav.k9.søknad.felles.opptjening.*;
-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.ytelse.psb.v1.Beredskap;
 import no.nav.k9.søknad.ytelse.psb.v1.Nattevåk;
 import no.nav.k9.søknad.ytelse.psb.v1.Omsorg;
@@ -31,7 +28,6 @@
 
 import java.time.Duration;
 import java.time.LocalDate;
-import java.util.Collections;
 import java.util.List;
 import java.util.Map;
 
@@ -42,7 +38,7 @@ public class PleiepengerBarnSøknadValidatorTest {
 
     @Test
     public void minimumSøknadNullTest() {
-        var psb = TestUtils.minimumSøknadPleiepengerSyktBarn();
+        var psb = TestUtils.minimumSøknadPleiepengerSyktBarnMedDelperioder();
         JsonUtils.toString(psb);
         verifyIngenFeil(psb);
     }
@@ -73,11 +69,19 @@ public class PleiepengerBarnSøknadValidatorTest {
 
     @Test
     public void minimumSøknadMedOmsorgNullPåFelterSkalIkkeHaValideringsfeil() {
-        var psb = TestUtils.minimumSøknadPleiepengerSyktBarn();
+        var psb = TestUtils.minimumSøknadPleiepengerSyktBarnMedDelperioder();
         psb.medOmsorg(new Omsorg().medBeskrivelseAvOmsorgsrollen(null).medRelasjonTilBarnet(null));
         verifyIngenFeil(psb);
     }
 
+    @Test void søknadPerioderFeilFomFørTom() {
+        var søknadperiode = new Periode(LocalDate.now().plusDays(10), LocalDate.now().minusDays(10));
+        var psb = TestUtils.minimumSøknadPleiepengerSyktBarn(søknadperiode);
+
+        final List<Feil> feil = valider(psb);
+        assertThat(feil).isNotEmpty();
+    }
+
     @Test
     public void uttakKanIkkeVæreTom() {
         var ytelse = TestUtils.komplettYtelsePsbMedDelperioder();

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

@@ -48,6 +48,10 @@ private static String jsonFromFile(String filename) {
         }
     }
 
+    /*
+    JSON
+     */
+
     static Søknad komplettSøknadJson() {
         return Søknad.SerDes.deserialize(jsonFromFile("komplett-søknad.json"));
     }
@@ -64,9 +68,9 @@ private static String jsonFromFile(String filename) {
         return Søknad.SerDes.deserialize(jsonFromFile("5.1.33/minimum-søknad.json"));
     }
 
-    static Ytelse komplettYtelsePsbJson(String ytelse) {
-        return JsonUtils.fromString(ytelse, Ytelse.class);
-    }
+    /*
+    Komplett
+     */
 
     static PleiepengerSyktBarn komplettYtelsePsbMedDelperioder() {
 
@@ -157,7 +161,11 @@ static PleiepengerSyktBarn komplettYtelsePsb(Periode søknadsperiode) {
                 .medBosteder(bosteder);
     }
 
-    static PleiepengerSyktBarn minimumSøknadPleiepengerSyktBarn() {
+    /*@
+    Minimum
+     */
+
+    static PleiepengerSyktBarn minimumSøknadPleiepengerSyktBarnMedDelperioder() {
         var søknadsperiode = new Periode(LocalDate.parse("2018-12-30"), LocalDate.parse("2019-10-20"));
         var uttakperiode = new Periode(LocalDate.parse("2018-12-30"), LocalDate.parse("2019-02-20"));
         var uttakperiode2 = new Periode(LocalDate.parse("2019-02-21"), LocalDate.parse("2019-10-20"));
@@ -168,17 +176,28 @@ uttakperiode, new UttakPeriodeInfo(Duration.ofHours(7).plusMinutes(30)),
 
         var barn = new Barn(null, LocalDate.now());
 
-        var omsorg = new Omsorg()
-                .medRelasjonTilBarnet(Omsorg.BarnRelasjon.MOR)
-                .medBeskrivelseAvOmsorgsrollen(TestUtils.testTekst());
+        return new PleiepengerSyktBarn()
+                .medSøknadsperiode(søknadsperiode)
+                .medBarn(barn)
+                .medUttak(uttak);
+    }
+
+    static PleiepengerSyktBarn minimumSøknadPleiepengerSyktBarn(Periode søknadsperiode) {
+        var uttak = new Uttak().medPerioder(Map.of(
+                søknadsperiode, new UttakPeriodeInfo(Duration.ofHours(7).plusMinutes(30))));
+
+        var barn = new Barn(null, LocalDate.now());
 
         return new PleiepengerSyktBarn()
                 .medSøknadsperiode(søknadsperiode)
                 .medBarn(barn)
-                .medUttak(uttak)
-                .medOmsorg(omsorg);
+                .medUttak(uttak);
     }
 
+    /*
+    Endring
+     */
+
     static PleiepengerSyktBarn minimumEndringssøknad(Periode endringsperiode) {
         return new PleiepengerSyktBarn()
                 .medEndringsperiode(endringsperiode)
@@ -208,6 +227,10 @@ uttakperiode, new UttakPeriodeInfo(Duration.ofHours(7).plusMinutes(30)),
                 .medUtenlandsopphold(new Utenlandsopphold());
     }
 
+    /*
+    Util
+     */
+
     static String testTekst() {
         return "Lorem ipsum dolor sit amet, consectetur adipisicing elit, sed do eiusmod tempor incididunt ut labore et dolore magna aliqua.";
     }