Endringsperiode serialisert i JSON (#187)

* Legger til feilende tester som viser at det ikke serialiseres i JSON.

* Fikser så endringsperioder havner i JSON & at getSøknadsperiode() bruker rett endringsperioder.

Author: fraadsbrandth

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

@@ -42,11 +42,6 @@ public class PleiepengerSyktBarn implements Ytelse {
     @JsonProperty(value = "søknadsperiode", required = true)
     private List<@Valid Periode> søknadsperiode = new ArrayList<>();
 
-    @Valid
-    @JsonFormat(with = JsonFormat.Feature.ACCEPT_SINGLE_VALUE_AS_ARRAY)
-    @JsonProperty(value = "endringsperiode", required = true)
-    private List<@Valid Periode> endringsperiode = new ArrayList<>();
-
     @Valid
     @JsonProperty(value = "trekkKravPerioder", required = true)
     private List<@Valid Periode> trekkKravPerioder = new ArrayList<>();
@@ -126,7 +121,7 @@ public Person getAnnenPart() {
     @Override
     public Periode getSøknadsperiode() {
         final List<Periode> perioder = new ArrayList<>(søknadsperiode);
-        perioder.addAll(endringsperiode);
+        perioder.addAll(getEndringsperiode());
 
         final var fom = perioder
                 .stream()
@@ -155,6 +150,7 @@ public Person getAnnenPart() {
         return this;
     }
 
+    @JsonProperty(value = "endringsperiode")
     public List<Periode> getEndringsperiode() {
         return EndringsperiodeKalkulator.getEndringsperiode(this);
     }
@@ -283,7 +279,4 @@ public Type getType() {
     public YtelseValidator getValidator() {
         return new PleiepengerSyktBarnYtelseValidator();
     }
-
-
-
 }

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

@@ -7,8 +7,12 @@
 import static org.assertj.core.api.Assertions.assertThat;
 
 import java.time.LocalDate;
+import java.util.ArrayList;
 import java.util.List;
 
+import com.fasterxml.jackson.databind.node.ArrayNode;
+import no.nav.k9.søknad.JsonUtils;
+import no.nav.k9.søknad.ytelse.psb.v1.PleiepengerSyktBarn;
 import org.junit.jupiter.api.Test;
 
 import no.nav.k9.søknad.felles.type.Periode;
@@ -23,6 +27,7 @@ class EndringTest {
 
         var psb = YtelseEksempel.komplettEndringssøknad(endringsperiode);
         verifyIngenFeil(psb, List.of(gyldigEndringsInterval));
+        assertEndringsperioderIJson(psb);
     }
 
     @Test
@@ -32,6 +37,7 @@ class EndringTest {
 
         var psb = YtelseEksempel.standardYtelseMedEndring(søknadsperiode, endringsperiode);
         verifyIngenFeil(psb, List.of(endringsperiode));
+        assertEndringsperioderIJson(psb);
     }
 
     @Test
@@ -41,7 +47,7 @@ class EndringTest {
 
         var psb = YtelseEksempel.standardYtelseMedEndring(søknadsperiode, endringsperiode);
         verifyIngenFeil(psb, List.of(endringsperiode));
-
+        assertEndringsperioderIJson(psb);
     }
 
     @Test
@@ -56,6 +62,7 @@ class EndringTest {
         feilInneholderFeltOgFeilkode(feil, "uttak.perioder", "ugyldigPeriode");
         assertThat(feil).size().isEqualTo(1);
         assertThat(endringsperiodeList).isEqualTo(psb.getEndringsperiode());
+        assertEndringsperioderIJson(psb);
     }
 
     @Test
@@ -83,7 +90,8 @@ class EndringTest {
         var psb = YtelseEksempel.komplettYtelse(søknadsperiode);
 
         var feil = verifyHarFeil(psb, List.of(new Periode(LocalDate.now().plusDays(2), LocalDate.now())));
-        feilInneholderFeilkode(feil, "ugyldigPeriode");    }
+        feilInneholderFeilkode(feil, "ugyldigPeriode");
+    }
 
     @Test
     public void kalkulertEndringsperiodeFinnerFlereSøknadsperioder() {
@@ -106,8 +114,20 @@ class EndringTest {
         assertThat(endringsperiode).contains(søknadsperiodeTo);
         assertThat(endringsperiode).contains(søknadsperiodeTre);
         assertThat(endringsperiode).contains(søknadsperiodeFire);
-
-
+        assertEndringsperioderIJson(ytelse);
     }
 
+    private void assertEndringsperioderIJson(PleiepengerSyktBarn ytelse) {
+        var endringsperioder = new ArrayList<Periode>();
+        try {
+            var jsonArray = (ArrayNode) JsonUtils.getObjectMapper()
+                    .readTree(JsonUtils.toString(ytelse))
+                    .get("endringsperiode");
+
+            jsonArray.forEach(jsonNode -> endringsperioder.add(new Periode(jsonNode.asText())));
+        } catch (Exception ex) {
+            throw new IllegalStateException("Feil ved sjekk på endringsperioder i JSON", ex);
+        }
+        assertThat(endringsperioder).containsExactlyElementsOf(ytelse.getEndringsperiode());
+    }
 }