deprecate holder (navn på institusjon) (#339)

Author: oyvemb

soknad/src/main/java/no/nav/k9/søknad/ytelse/olp/v1/OpplæringspengerYtelseValidator.java

@@ -9,7 +9,6 @@
 import java.util.List;
 import java.util.Map;
 import java.util.Objects;
-import java.util.UUID;
 import java.util.stream.Collectors;
 
 import no.nav.fpsak.tidsserie.LocalDateTimeline;
@@ -20,7 +19,6 @@
 import no.nav.k9.søknad.ytelse.Ytelse;
 import no.nav.k9.søknad.ytelse.YtelseValidator;
 import no.nav.k9.søknad.ytelse.olp.v1.kurs.KursPeriodeMedReisetid;
-import no.nav.k9.søknad.ytelse.olp.v1.kurs.Kursholder;
 
 class OpplæringspengerYtelseValidator extends YtelseValidator {
 
@@ -98,8 +96,6 @@ List<Feil> validerOgLeggTilFeilene(Opplæringspenger olp,
 
         validerAtYtelsePeriodenErKomplettMedSøknad(søknadsperiodeTidslinje, olp.getUttak().getPerioder(), "uttak", feilene);
 
-        validerKursholderOgInstitusjonUuid(olp.getKurs().getKursholder(), "kurs.kursholder", feilene);
-
         validerReisetidMotKursperioden(olp.getKurs().getKursperioder(), "kurs.kursperioder", feilene);
 
         return feilene;
@@ -133,17 +129,6 @@ private List<Feil> validerAtYtelsePerioderErInnenforIntervalForEndring(LocalDate
                 .collect(Collectors.toCollection(ArrayList::new)));
     }
 
-    private void validerKursholderOgInstitusjonUuid(Kursholder kursholder, String felt, List<Feil> feil) {
-        String holder = kursholder.getHolder();
-        UUID institusjonUuid = kursholder.getInstitusjonUuid();
-
-        if (holder == null && institusjonUuid == null) {
-            feil.add(lagFeil(felt, "ugyldigHolderEllerInstitusjonUuid", "Enten holder eller institusjonUuid må være satt."));
-        } else if (holder != null && institusjonUuid != null) {
-            feil.add(lagFeil(felt, "ugyldigHolderEllerInstitusjonUuid", "Kan ikke ha både holder og institusjonUuid satt samtidig."));
-        }
-    }
-
     private void validerReisetidMotKursperioden(List<KursPeriodeMedReisetid> kursperioder, String felt, List<Feil> feil) {
         for (KursPeriodeMedReisetid kursPeriode : kursperioder) {
             if (kursPeriode != null) {

soknad/src/main/java/no/nav/k9/søknad/ytelse/olp/v1/kurs/Kursholder.java

@@ -14,7 +14,8 @@
 @JsonAutoDetect(fieldVisibility = JsonAutoDetect.Visibility.NONE, getterVisibility = JsonAutoDetect.Visibility.NONE, setterVisibility = JsonAutoDetect.Visibility.NONE, isGetterVisibility = JsonAutoDetect.Visibility.NONE, creatorVisibility = JsonAutoDetect.Visibility.NONE)
 public class Kursholder {
 
-    @JsonProperty(value = "holder", required = true)
+    @Deprecated(forRemoval = true)
+    @JsonProperty(value = "holder")
     @Valid
     @Size(max = 100)
     @Pattern(regexp = "^[\\p{Pd}\\p{Graph}\\p{Space}\\p{Sc}\\p{L}\\p{M}\\p{N}§]*$", message = "[${validatedValue}] matcher ikke tillatt pattern [{regexp}]")
@@ -27,11 +28,17 @@ public class Kursholder {
     public Kursholder() {
     }
 
+    public Kursholder(UUID institusjonUuid) {
+        this.institusjonUuid = institusjonUuid;
+    }
+
+    @Deprecated(forRemoval = true)
     public Kursholder(String holder, UUID institusjonUuid) {
         this.holder = holder;
         this.institusjonUuid = institusjonUuid;
     }
 
+    @Deprecated(forRemoval = true)
     public String getHolder() {
         return holder;
     }

soknad/src/test/java/no/nav/k9/søknad/ytelse/olp/v1/OpplæringspengerYtelseValidatorTest.java

@@ -8,7 +8,6 @@
 
 import org.junit.jupiter.api.Test;
 
-import no.nav.k9.søknad.TestUtils;
 import no.nav.k9.søknad.felles.Feil;
 import no.nav.k9.søknad.felles.type.Periode;
 import no.nav.k9.søknad.ytelse.olp.v1.kurs.Kurs;
@@ -20,34 +19,18 @@ class OpplæringspengerYtelseValidatorTest {
 
     private final OpplæringspengerYtelseValidator ytelseValidator = new OpplæringspengerYtelseValidator();
 
-    private Opplæringspenger lagYtelse(String holder, UUID institusjonUuid) {
+    private Opplæringspenger lagYtelse() {
         Periode søknadsperiode = new Periode(LocalDate.now(), LocalDate.now().plusWeeks(1));
         KursPeriodeMedReisetid kursPeriode = new KursPeriodeMedReisetid(søknadsperiode, søknadsperiode.getFraOgMed(), søknadsperiode.getTilOgMed(), null, null);
-        Kurs kurs = new Kurs(new Kursholder(holder, institusjonUuid), List.of(kursPeriode));
+        Kurs kurs = new Kurs(new Kursholder(UUID.randomUUID()), List.of(kursPeriode));
         return new Opplæringspenger().medBarn(YtelseEksempel.lagBarn()).medSøknadsperiode(List.of(søknadsperiode)).medUttak(YtelseEksempel.lagUttak(søknadsperiode)).medKurs(kurs);
     }
 
     @Test
     void skalValidereOk() {
-        Opplæringspenger olpYtelse = lagYtelse(null, UUID.randomUUID());
+        Opplæringspenger olpYtelse = lagYtelse();
 
         List<Feil> feil = ytelseValidator.valider(olpYtelse);
         assertThat(feil).isEmpty();
     }
-
-    @Test
-    void skalGiFeilHvisBådeHolderOgUuidErNull() {
-        Opplæringspenger olpYtelse = lagYtelse(null, null);
-
-        List<Feil> feil = ytelseValidator.valider(olpYtelse);
-        TestUtils.feilInneholder(feil, "ytelse.kurs.kursholder", "ugyldigHolderEllerInstitusjonUuid", "Enten holder eller institusjonUuid må være satt.");
-    }
-
-    @Test
-    void skalGiFeilHvisBådeHolderOgUuidErSatt() {
-        Opplæringspenger olpYtelse = lagYtelse("Franz Holder", UUID.randomUUID());
-
-        List<Feil> feil = ytelseValidator.valider(olpYtelse);
-        TestUtils.feilInneholder(feil, "ytelse.kurs.kursholder", "ugyldigHolderEllerInstitusjonUuid", "Kan ikke ha både holder og institusjonUuid satt samtidig.");
-    }
 }