TFP-5383: Datointerval for avtaler og permisjoner må gjøres nullsafe … (#2544)

TFP-5383: Datointerval for avtaler og permisjoner må gjøres nullsafe før LocaldateSegmenter

Author: espenwaaga

domenetjenester/iay/src/main/java/no/nav/foreldrepenger/abakus/iay/tjeneste/dto/arbeidsforhold/ArbeidsforholdDtoTjeneste.java

@@ -34,6 +34,8 @@
 import no.nav.fpsak.tidsserie.LocalDateTimeline;
 import no.nav.vedtak.konfig.Tid;
 
+import static no.nav.abakus.iaygrunnlag.kodeverk.PermisjonsbeskrivelseType.finnForKodeverkEiersKode;
+
 @ApplicationScoped
 public class ArbeidsforholdDtoTjeneste {
 
@@ -74,38 +76,50 @@ private ArbeidsforholdDto mapTilArbeidsforholdMedPermisjoner(Map.Entry<Arbeidsfo
         ArbeidsforholdDto dto = new ArbeidsforholdDto(arbeidsgiver, arbeidType);
         dto.setArbeidsforholdId(mapArbeidsforholdId(key.getArbeidsforholdId()));
         dto.setAnsettelsesperiode(mapAnsettelsesPerioder(arbeidsforholdEntry.getValue()));
-        dto.setArbeidsavtaler(arbeidsforholdEntry.getValue().stream().map(this::mapArbeidsavtaler).flatMap(Collection::stream).toList());
-        dto.setPermisjoner(arbeidsforholdEntry.getValue().stream().map(this::mapPermisjoner).flatMap(Collection::stream).toList());
+        dto.setArbeidsavtaler(tilArbeidsavtaler(arbeidsforholdEntry.getValue()));
+        dto.setPermisjoner(tilPermisjoner(arbeidsforholdEntry.getValue()));
         return dto;
     }
 
+    private List<PermisjonDto> tilPermisjoner(List<Arbeidsforhold> arbeidsforhold) {
+        return arbeidsforhold.stream().map(this::mapPermisjoner).flatMap(Collection::stream).toList();
+    }
+
+    private List<ArbeidsavtaleDto> tilArbeidsavtaler(List<Arbeidsforhold> arbeidsforhold) {
+        return arbeidsforhold.stream().map(this::mapArbeidsavtaler).flatMap(Collection::stream).toList();
+    }
+
     private List<ArbeidsavtaleDto> mapArbeidsavtaler(Arbeidsforhold arbeidsforhold) {
-        LocalDateInterval ansettelse = new LocalDateInterval(arbeidsforhold.getArbeidFom(), arbeidsforhold.getArbeidTom());
-        var tidslinje = arbeidsforhold.getArbeidsavtaler().stream()
+        var ansettelse = new LocalDateInterval(arbeidsforhold.getArbeidFom(), arbeidsforhold.getArbeidTom());
+        var arbeidsavtalerTidlinje = arbeidsforhold.getArbeidsavtaler().stream()
             .filter(arbeidsavtale -> !arbeidsavtale.getErAnsettelsesPerioden())
             .filter(arbeidsavtale -> arbeidsavtale.getStillingsprosent() != null)
-            .map(a -> new LocalDateSegment<>(a.getArbeidsavtaleFom(), a.getArbeidsavtaleTom(), a.getStillingsprosent()))
+            .map(a -> new LocalDateSegment<>(safeFom(a.getArbeidsavtaleFom()), safeTom(a.getArbeidsavtaleTom()), a.getStillingsprosent()))
             .collect(Collectors.collectingAndThen(Collectors.toList(), LocalDateTimeline::new));
-        return tidslinje.intersection(ansettelse).stream().map(s -> new ArbeidsavtaleDto(new Periode(safeFom(s.getFom()), safeTom(s.getTom())), s.getValue())).toList();
+        return arbeidsavtalerTidlinje.intersection(ansettelse).stream()
+            .map(s -> new ArbeidsavtaleDto(new Periode(s.getFom(), s.getTom()), s.getValue()))
+            .toList();
     }
 
     private List<PermisjonDto> mapPermisjoner(Arbeidsforhold arbeidsforhold) {
-        LocalDateInterval ansettelse = new LocalDateInterval(arbeidsforhold.getArbeidFom(), arbeidsforhold.getArbeidTom());
-        var tidslinje = arbeidsforhold.getPermisjoner()
-            .stream()
+        var ansettelse = new LocalDateInterval(arbeidsforhold.getArbeidFom(), arbeidsforhold.getArbeidTom());
+        var permisjonTidslinje = arbeidsforhold.getPermisjoner().stream()
             .filter(permisjon -> permisjon.getPermisjonsprosent() != null)
-            .map(p -> new LocalDateSegment<>(p.getPermisjonFom(), p.getPermisjonTom(),
+            .map(p -> new LocalDateSegment<>(safeFom(p.getPermisjonFom()), safeTom(p.getPermisjonTom()),
                 new PermisjonTidslinjeObjekt(p.getPermisjonsprosent(), p.getPermisjonsÅrsak())))
             .collect(Collectors.collectingAndThen(Collectors.toList(), LocalDateTimeline::new));
 
-        return tidslinje.intersection(ansettelse)
+        return permisjonTidslinje.intersection(ansettelse)
             .stream()
-            .map(s -> new PermisjonDto(new Periode(safeFom(s.getFom()), safeTom(s.getTom())),
-                PermisjonsbeskrivelseType.finnForKodeverkEiersKode(s.getValue().permisjonsÅrsak))
-                .medProsentsats(s.getValue().permisjonsprosent()))
+            .map(ArbeidsforholdDtoTjeneste::tilPermisjonDto)
             .toList();
     }
 
+    private static PermisjonDto tilPermisjonDto(LocalDateSegment<PermisjonTidslinjeObjekt> s) {
+        return new PermisjonDto(new Periode(s.getFom(), s.getTom()), finnForKodeverkEiersKode(s.getValue().permisjonsÅrsak))
+            .medProsentsats(s.getValue().permisjonsprosent());
+    }
+
     private static LocalDate safeFom(LocalDate fom) {
         return fom != null ? fom : Tid.TIDENES_BEGYNNELSE;
     }