From 003e107fc7b464e8809ffe834a046ded8a6118b9 Mon Sep 17 00:00:00 2001 From: Olli-Pekka Lehtokallio Date: Tue, 31 Dec 2024 12:51:13 +0200 Subject: [PATCH] oppiaine-kooditettu-nimi-fix --- .../eperusteet/dto/yl/OppiaineBaseDto.java | 19 +++++----- .../sade/eperusteet/dto/yl/OppiaineDto.java | 9 ----- .../eperusteet/dto/yl/OppiaineLaajaDto.java | 3 +- .../NavigationBuilderPerusopetus.java | 8 ++--- .../NavigationBuilderPublicPerusopetus.java | 12 +++---- .../impl/validators/ValidatorPeruste.java | 6 ++-- .../service/mapping/DtoMapperConfig.java | 36 +++++++++---------- .../service/mapping/OppiaineCustomMapper.java | 32 +++++++++++++++++ .../LukiokoulutuksenSisaltoServiceIT.java | 2 +- .../eperusteet/service/OppiaineServiceIT.java | 6 ++-- .../PerusopetusPerusteNavigationIT.java | 2 +- .../eperusteet/service/util/OppiaineUtil.java | 2 +- 12 files changed, 80 insertions(+), 57 deletions(-) create mode 100644 eperusteet/eperusteet-service/src/main/java/fi/vm/sade/eperusteet/service/mapping/OppiaineCustomMapper.java diff --git a/eperusteet/eperusteet-service/src/main/java/fi/vm/sade/eperusteet/dto/yl/OppiaineBaseDto.java b/eperusteet/eperusteet-service/src/main/java/fi/vm/sade/eperusteet/dto/yl/OppiaineBaseDto.java index 080f13bf3c..6291820692 100644 --- a/eperusteet/eperusteet-service/src/main/java/fi/vm/sade/eperusteet/dto/yl/OppiaineBaseDto.java +++ b/eperusteet/eperusteet-service/src/main/java/fi/vm/sade/eperusteet/dto/yl/OppiaineBaseDto.java @@ -4,6 +4,7 @@ import com.fasterxml.jackson.annotation.JsonInclude; import fi.vm.sade.eperusteet.dto.Reference; import fi.vm.sade.eperusteet.dto.ReferenceableDto; +import fi.vm.sade.eperusteet.dto.tutkinnonrakenne.KoodiDto; import fi.vm.sade.eperusteet.dto.util.LokalisoituTekstiDto; import lombok.AllArgsConstructor; import lombok.Data; @@ -25,19 +26,11 @@ public abstract class OppiaineBaseDto implements ReferenceableDto { private Optional oppiaine; private Optional koosteinen; private Optional abstrakti; - protected Optional nimi; + protected LokalisoituTekstiDto nimi; + private KoodiDto koodi; private Optional jnro; private Date muokattu; - @JsonIgnore - public LokalisoituTekstiDto getNimiOrDefault(LokalisoituTekstiDto defaultNimi) { - if (nimi != null) { - return nimi.orElse(defaultNimi); - } else { - return defaultNimi; - } - } - @JsonIgnore public Long getJnroOrDefault(Long defaultJnro) { if (jnro != null) { @@ -46,4 +39,10 @@ public Long getJnroOrDefault(Long defaultJnro) { return defaultJnro; } } + + public LokalisoituTekstiDto getNimi() { + return Optional.ofNullable(koodi) + .map(KoodiDto::getNimi) + .orElse(Optional.ofNullable(nimi).orElse(LokalisoituTekstiDto.of(""))); + } } diff --git a/eperusteet/eperusteet-service/src/main/java/fi/vm/sade/eperusteet/dto/yl/OppiaineDto.java b/eperusteet/eperusteet-service/src/main/java/fi/vm/sade/eperusteet/dto/yl/OppiaineDto.java index c6de739b01..900e5023bc 100644 --- a/eperusteet/eperusteet-service/src/main/java/fi/vm/sade/eperusteet/dto/yl/OppiaineDto.java +++ b/eperusteet/eperusteet-service/src/main/java/fi/vm/sade/eperusteet/dto/yl/OppiaineDto.java @@ -22,14 +22,5 @@ public class OppiaineDto extends OppiaineBaseUpdateDto { private Set oppimaarat; private Set kohdealueet; private Set vuosiluokkakokonaisuudet; - private KoodiDto koodi; private List vapaatTekstit; - - public Optional getNimi() { - if (super.getNimi() != null && super.getNimi().isPresent()) { - return super.getNimi(); - } - - return Optional.ofNullable(koodi).map(KoodiDto::getNimi); - } } diff --git a/eperusteet/eperusteet-service/src/main/java/fi/vm/sade/eperusteet/dto/yl/OppiaineLaajaDto.java b/eperusteet/eperusteet-service/src/main/java/fi/vm/sade/eperusteet/dto/yl/OppiaineLaajaDto.java index 399818ed32..bb3078f2f3 100644 --- a/eperusteet/eperusteet-service/src/main/java/fi/vm/sade/eperusteet/dto/yl/OppiaineLaajaDto.java +++ b/eperusteet/eperusteet-service/src/main/java/fi/vm/sade/eperusteet/dto/yl/OppiaineLaajaDto.java @@ -4,6 +4,7 @@ import com.fasterxml.jackson.annotation.JsonInclude; import fi.vm.sade.eperusteet.dto.KevytTekstiKappaleDto; import fi.vm.sade.eperusteet.dto.peruste.PerusteRakenneOsa; +import fi.vm.sade.eperusteet.dto.tutkinnonrakenne.KoodiDto; import lombok.AllArgsConstructor; import lombok.Data; import lombok.EqualsAndHashCode; @@ -29,6 +30,6 @@ public class OppiaineLaajaDto extends OppiaineBaseDto { private String koodiArvo; public PerusteRakenneOsa getPerusteenOsa() { - return new PerusteRakenneOsa("perusopetus_oppiaine", nimi.get()); + return new PerusteRakenneOsa("perusopetus_oppiaine", getNimi()); } } diff --git a/eperusteet/eperusteet-service/src/main/java/fi/vm/sade/eperusteet/service/impl/navigation/NavigationBuilderPerusopetus.java b/eperusteet/eperusteet-service/src/main/java/fi/vm/sade/eperusteet/service/impl/navigation/NavigationBuilderPerusopetus.java index e6670753a5..a8e3b8898f 100644 --- a/eperusteet/eperusteet-service/src/main/java/fi/vm/sade/eperusteet/service/impl/navigation/NavigationBuilderPerusopetus.java +++ b/eperusteet/eperusteet-service/src/main/java/fi/vm/sade/eperusteet/service/impl/navigation/NavigationBuilderPerusopetus.java @@ -91,10 +91,10 @@ private NavigationNodeDto vuosiluokkakokonaisuudet(Long perusteId, String kieli) private NavigationNodeDto oppiaineet(Long perusteId, String kieli) { return NavigationNodeDto.of(NavigationType.perusopetusoppiaineet) .addAll(sisallot.getOppiaineet(perusteId, OppiaineSuppeaDto.class).stream() - .sorted(Comparator.comparing(oppiaine -> LokalisoituTekstiDto.getOrDefault(oppiaine.getNimiOrDefault(LokalisoituTekstiDto.of("")), Kieli.of(kieli), ""))) + .sorted(Comparator.comparing(oppiaine -> oppiaine.getNimi().get(Kieli.of(kieli)))) .sorted(Comparator.comparing(oppiaine -> oppiaine.getJnroOrDefault(99L))) .map(oppiaine -> - NavigationNodeDto.of(NavigationType.perusopetusoppiaine, oppiaine.getNimiOrDefault(null), oppiaine.getId()) + NavigationNodeDto.of(NavigationType.perusopetusoppiaine, oppiaine.getNimi(), oppiaine.getId()) .addAll(!ObjectUtils.isEmpty(oppiaine.getOppimaarat()) ? oppimaarat(oppiaine.getOppimaarat(), kieli) : null) ).collect(Collectors.toList())); } @@ -103,9 +103,9 @@ private Stream oppimaarat(Set oppimaarat, return Stream.concat( Stream.of(NavigationNodeDto.of(NavigationType.oppimaarat).meta("navigation-subtype", true)), oppimaarat.stream() - .sorted(Comparator.comparing(oppiaine -> LokalisoituTekstiDto.getOrDefault(oppiaine.getNimiOrDefault(LokalisoituTekstiDto.of("")), Kieli.of(kieli), ""))) + .sorted(Comparator.comparing(oppiaine -> oppiaine.getNimi().get(Kieli.of(kieli)))) .sorted(Comparator.comparing(oppiaine -> oppiaine.getJnroOrDefault(99l))) - .map(oppimaara -> NavigationNodeDto.of(NavigationType.perusopetusoppiaine, oppimaara.getNimiOrDefault(null), oppimaara.getId()).meta("oppimaara", true))); + .map(oppimaara -> NavigationNodeDto.of(NavigationType.perusopetusoppiaine, oppimaara.getNimi(), oppimaara.getId()).meta("oppimaara", true))); } } diff --git a/eperusteet/eperusteet-service/src/main/java/fi/vm/sade/eperusteet/service/impl/navigationpublic/NavigationBuilderPublicPerusopetus.java b/eperusteet/eperusteet-service/src/main/java/fi/vm/sade/eperusteet/service/impl/navigationpublic/NavigationBuilderPublicPerusopetus.java index 9f25899c61..3f9b6a861e 100644 --- a/eperusteet/eperusteet-service/src/main/java/fi/vm/sade/eperusteet/service/impl/navigationpublic/NavigationBuilderPublicPerusopetus.java +++ b/eperusteet/eperusteet-service/src/main/java/fi/vm/sade/eperusteet/service/impl/navigationpublic/NavigationBuilderPublicPerusopetus.java @@ -61,10 +61,10 @@ private List vuosiluokat(PerusteKaikkiDto peruste, String kie private List vuosiluokanOppiaineet(PerusteKaikkiDto peruste, Long vlkId, String kieli) { return peruste.getPerusopetuksenPerusteenSisalto().getOppiaineet().stream() .filter(oppiaine -> oppiaineJaOppimaaraVuosiluokkakokonaisuusIdt(oppiaine).contains(vlkId)) - .sorted(Comparator.comparing(oppiaine -> LokalisoituTekstiDto.getOrDefault(oppiaine.getNimiOrDefault(LokalisoituTekstiDto.of("")), Kieli.of(kieli), ""))) + .sorted(Comparator.comparing(oppiaine -> oppiaine.getNimi().get(Kieli.of(kieli)))) .sorted(Comparator.comparing(oppiaine -> oppiaine.getJnroOrDefault(99l))) .map(oppiaine -> - NavigationNodeDto.of(NavigationType.perusopetusoppiaine, oppiaine.getNimiOrDefault(null), oppiaine.getId()).meta("vlkId", vlkId) + NavigationNodeDto.of(NavigationType.perusopetusoppiaine, oppiaine.getNimi(), oppiaine.getId()).meta("vlkId", vlkId) .add(!ObjectUtils.isEmpty(oppiaine.getOppimaarat()) ? oppimaarat(oppiaine.getOppimaarat(), vlkId, kieli) : null) ).collect(Collectors.toList()); } @@ -82,10 +82,10 @@ private List oppiaineJaOppimaaraVuosiluokkakokonaisuusIdt(OppiaineLaajaDto private NavigationNodeDto oppiaineet(PerusteKaikkiDto peruste, String kieli) { return NavigationNodeDto.of(NavigationType.perusopetusoppiaineet) .addAll(peruste.getPerusopetuksenPerusteenSisalto().getOppiaineet().stream() - .sorted(Comparator.comparing(oppiaine -> LokalisoituTekstiDto.getOrDefault(oppiaine.getNimiOrDefault(LokalisoituTekstiDto.of("")), Kieli.of(kieli), ""))) + .sorted(Comparator.comparing(oppiaine -> oppiaine.getNimi().get(Kieli.of(kieli)))) .sorted(Comparator.comparing(oppiaine -> oppiaine.getJnroOrDefault(99l))) .map(oppiaine -> - NavigationNodeDto.of(NavigationType.perusopetusoppiaine, oppiaine.getNimiOrDefault(null), oppiaine.getId()) + NavigationNodeDto.of(NavigationType.perusopetusoppiaine, oppiaine.getNimi(), oppiaine.getId()) .add(!ObjectUtils.isEmpty(oppiaine.getOppimaarat()) ? oppimaarat(oppiaine.getOppimaarat(), null, kieli) : null) ).collect(Collectors.toList())); } @@ -94,11 +94,11 @@ private NavigationNodeDto oppimaarat(Set oppimaarat, Long vlkId, St return NavigationNodeDto.of(NavigationType.oppimaarat).meta("navigation-subtype", true) .addAll( oppimaarat.stream() - .sorted(Comparator.comparing(oppiaine -> LokalisoituTekstiDto.getOrDefault(oppiaine.getNimiOrDefault(LokalisoituTekstiDto.of("")), Kieli.of(kieli), ""))) + .sorted(Comparator.comparing(oppiaine -> oppiaine.getNimi().get(Kieli.of(kieli)))) .sorted(Comparator.comparing(oppiaine -> oppiaine.getJnroOrDefault(99l))) .map(oppimaara -> { NavigationNodeDto node = NavigationNodeDto - .of(NavigationType.perusopetusoppiaine, oppimaara.getNimiOrDefault(null), oppimaara.getId()); + .of(NavigationType.perusopetusoppiaine, oppimaara.getNimi(), oppimaara.getId()); if (vlkId != null) { node = node.meta("vlkId", vlkId); } diff --git a/eperusteet/eperusteet-service/src/main/java/fi/vm/sade/eperusteet/service/impl/validators/ValidatorPeruste.java b/eperusteet/eperusteet-service/src/main/java/fi/vm/sade/eperusteet/service/impl/validators/ValidatorPeruste.java index 7db2849668..ad9c5dafa3 100644 --- a/eperusteet/eperusteet-service/src/main/java/fi/vm/sade/eperusteet/service/impl/validators/ValidatorPeruste.java +++ b/eperusteet/eperusteet-service/src/main/java/fi/vm/sade/eperusteet/service/impl/validators/ValidatorPeruste.java @@ -209,7 +209,7 @@ private void tarkistaPerusopetuksenOppiaine( OppiaineSuppeaDto oaDto = mapper.map(oa, OppiaineSuppeaDto.class); if (oa.getKoodiArvo() == null && oa.getKoodiUri() == null) { - validointi.virhe("peruste-validointi-oppiaine-koodi", NavigationNodeDto.of(NavigationType.perusopetusoppiaine, oaDto.getNimiOrDefault(null), oa.getId())); + validointi.virhe("peruste-validointi-oppiaine-koodi", NavigationNodeDto.of(NavigationType.perusopetusoppiaine, oaDto.getNimi(), oa.getId())); } if (oa.getTehtava() != null) { @@ -279,11 +279,11 @@ private void tarkistaPerusopetuksenOppiaine( .flatMap(Collection::stream) .collect(Collectors.toSet()); if (!sisaltoalueetIdt.containsAll(tavoitteidenSisaltoalueetIdt)) { - validointi.virhe("peruste-validointi-oppiaine-vlk-tavoite-sisaltoalueet", NavigationNodeDto.of(NavigationType.perusopetusoppiaine, oaDto.getNimiOrDefault(null), oa.getId())); + validointi.virhe("peruste-validointi-oppiaine-vlk-tavoite-sisaltoalueet", NavigationNodeDto.of(NavigationType.perusopetusoppiaine, oaDto.getNimi(), oa.getId())); } } - virheellisetKielet.entrySet().forEach(entry -> validointi.virhe(entry.getKey(), NavigationNodeDto.of(NavigationType.perusopetusoppiaine, oaDto.getNimiOrDefault(null), oa.getId()))); + virheellisetKielet.entrySet().forEach(entry -> validointi.virhe(entry.getKey(), NavigationNodeDto.of(NavigationType.perusopetusoppiaine, oaDto.getNimi(), oa.getId()))); if (oa.getOppimaarat() != null) { for (Oppiaine oppimaara : oa.getOppimaarat()) { diff --git a/eperusteet/eperusteet-service/src/main/java/fi/vm/sade/eperusteet/service/mapping/DtoMapperConfig.java b/eperusteet/eperusteet-service/src/main/java/fi/vm/sade/eperusteet/service/mapping/DtoMapperConfig.java index f7c2036e5a..2d4812cd1b 100644 --- a/eperusteet/eperusteet-service/src/main/java/fi/vm/sade/eperusteet/service/mapping/DtoMapperConfig.java +++ b/eperusteet/eperusteet-service/src/main/java/fi/vm/sade/eperusteet/service/mapping/DtoMapperConfig.java @@ -108,6 +108,8 @@ import fi.vm.sade.eperusteet.dto.vst.OpintokokonaisuusDto; import fi.vm.sade.eperusteet.dto.vst.TavoitesisaltoalueDto; import fi.vm.sade.eperusteet.dto.yl.LukioOppiaineUpdateDto; +import fi.vm.sade.eperusteet.dto.yl.OppiaineBaseDto; +import fi.vm.sade.eperusteet.dto.yl.OppiaineBaseUpdateDto; import fi.vm.sade.eperusteet.dto.yl.OppiaineDto; import fi.vm.sade.eperusteet.dto.yl.OppiaineKevytDto; import fi.vm.sade.eperusteet.dto.yl.OppiaineLaajaDto; @@ -673,32 +675,30 @@ public void mapAtoB(Koodi a, OsaamisalaDto b, MappingContext context) { .byDefault() .register(); + factory.classMap(OppiaineBaseDto.class, Oppiaine.class) + .mapNulls(true) + .customize(OppiaineCustomMapper.create(koodistoClient)) + .byDefault() + .register(); + factory.classMap(OppiaineDto.class, Oppiaine.class) .mapNulls(true) .fieldBToA("vuosiluokkakokonaisuudet", "vuosiluokkakokonaisuudet") - .customize(new CustomMapper() { - @Override - public void mapBtoA(Oppiaine oppiaine, OppiaineDto oppiaineDto, MappingContext context) { - super.mapBtoA(oppiaine, oppiaineDto, context); - if (oppiaine.getKoodiUri() != null) { - oppiaineDto.setKoodi(koodistoClient.getKoodi(oppiaine.getKoodiUri().split("_")[0], oppiaine.getKoodiUri())); - } - } + .customize(OppiaineCustomMapper.create(koodistoClient)) + .byDefault() + .register(); - @Override - public void mapAtoB(OppiaineDto oppiaineDto, Oppiaine oppiaine, MappingContext context) { - super.mapAtoB(oppiaineDto, oppiaine, context); - if (oppiaineDto.getKoodi() != null) { - oppiaine.setKoodiUri(oppiaineDto.getKoodi().getUri()); - oppiaine.setKoodiArvo(oppiaineDto.getKoodi().getArvo()); - } - } - }) + factory.classMap(OppiaineLaajaDto.class, Oppiaine.class) + .mapNulls(true) + .fieldBToA("vuosiluokkakokonaisuudet", "vuosiluokkakokonaisuudet") + .customize(OppiaineCustomMapper.create(koodistoClient)) .byDefault() .register(); factory.classMap(OppiaineSuppeaDto.class, Oppiaine.class) - .fieldBToA("muokattu", "muokattu") + .mapNulls(true) + .fieldBToA("vuosiluokkakokonaisuudet", "vuosiluokkakokonaisuudet") + .customize(OppiaineCustomMapper.create(koodistoClient)) .byDefault() .register(); diff --git a/eperusteet/eperusteet-service/src/main/java/fi/vm/sade/eperusteet/service/mapping/OppiaineCustomMapper.java b/eperusteet/eperusteet-service/src/main/java/fi/vm/sade/eperusteet/service/mapping/OppiaineCustomMapper.java new file mode 100644 index 0000000000..ea3a6c96e0 --- /dev/null +++ b/eperusteet/eperusteet-service/src/main/java/fi/vm/sade/eperusteet/service/mapping/OppiaineCustomMapper.java @@ -0,0 +1,32 @@ +package fi.vm.sade.eperusteet.service.mapping; + +import fi.vm.sade.eperusteet.domain.yl.Oppiaine; +import fi.vm.sade.eperusteet.dto.yl.OppiaineBaseDto; +import fi.vm.sade.eperusteet.service.KoodistoClient; +import ma.glasnost.orika.CustomMapper; +import ma.glasnost.orika.MappingContext; + +public class OppiaineCustomMapper { + + public static CustomMapper create(KoodistoClient koodistoClient) { + return new CustomMapper() { + + @Override + public void mapBtoA(Oppiaine oppiaine, T oppiaineDto, MappingContext context) { + super.mapBtoA(oppiaine, oppiaineDto, context); + if (oppiaine.getKoodiUri() != null) { + oppiaineDto.setKoodi(koodistoClient.getKoodi(oppiaine.getKoodiUri().split("_")[0], oppiaine.getKoodiUri())); + } + } + + @Override + public void mapAtoB(T oppiaineDto, Oppiaine oppiaine, MappingContext context) { + super.mapAtoB(oppiaineDto, oppiaine, context); + if (oppiaineDto.getKoodi() != null) { + oppiaine.setKoodiUri(oppiaineDto.getKoodi().getUri()); + oppiaine.setKoodiArvo(oppiaineDto.getKoodi().getArvo()); + } + } + }; + } +} \ No newline at end of file diff --git a/eperusteet/eperusteet-service/src/test/java/fi/vm/sade/eperusteet/service/LukiokoulutuksenSisaltoServiceIT.java b/eperusteet/eperusteet-service/src/test/java/fi/vm/sade/eperusteet/service/LukiokoulutuksenSisaltoServiceIT.java index df0f059c7a..9a8dbd8a7d 100644 --- a/eperusteet/eperusteet-service/src/test/java/fi/vm/sade/eperusteet/service/LukiokoulutuksenSisaltoServiceIT.java +++ b/eperusteet/eperusteet-service/src/test/java/fi/vm/sade/eperusteet/service/LukiokoulutuksenSisaltoServiceIT.java @@ -96,7 +96,7 @@ public void testAddAndListOppiaineet() { List aineet = lukiokoulutuksenPerusteenSisaltoService.getOppiaineet(perusteId, OppiaineDto.class); assertEquals(1, aineet.size()); - assertEquals("Finska", aineet.get(0).getNimi().get().get(SV)); + assertEquals("Finska", aineet.get(0).getNimi().get(SV)); assertEquals(2, aineet.get(0).getOppimaarat().size()); } } diff --git a/eperusteet/eperusteet-service/src/test/java/fi/vm/sade/eperusteet/service/OppiaineServiceIT.java b/eperusteet/eperusteet-service/src/test/java/fi/vm/sade/eperusteet/service/OppiaineServiceIT.java index 9863fbc544..a2130bb70d 100644 --- a/eperusteet/eperusteet-service/src/test/java/fi/vm/sade/eperusteet/service/OppiaineServiceIT.java +++ b/eperusteet/eperusteet-service/src/test/java/fi/vm/sade/eperusteet/service/OppiaineServiceIT.java @@ -65,7 +65,7 @@ public void testCRUD() throws IOException { vk = vkrepo.save(vk); OppiaineDto oppiaineDto = new OppiaineDto(); - oppiaineDto.setNimi(Optional.of(lt("Oppiaine"))); + oppiaineDto.setNimi(lt("Oppiaine")); oppiaineDto.setTehtava(to("TehtävänOtsikko", "Tehtava")); oppiaineDto.setKoosteinen(Optional.of(false)); oppiaineDto.setVapaatTekstit(List.of(KevytTekstiKappaleDto.of("nimi", "teksti"), KevytTekstiKappaleDto.of("nimi2", "teksti2"))); @@ -186,7 +186,7 @@ public void testAddAndUpdateOppimaara() throws IOException { vk = vkrepo.save(vk); OppiaineDto oppiaineDto = new OppiaineDto(); - oppiaineDto.setNimi(Optional.of(lt("Oppiaine"))); + oppiaineDto.setNimi(lt("Oppiaine")); oppiaineDto.setTehtava(to("TehtävänOtsikko", "Tehtava")); oppiaineDto.setKoosteinen(Optional.of(true)); @@ -205,7 +205,7 @@ public void testAddAndUpdateOppimaara() throws IOException { assertEquals(0, oa1.getOppimaarat().size()); OppiaineDto oppimaara = new OppiaineDto(); - oppimaara.setNimi(olt("OppimaaranNimi")); + oppimaara.setNimi(lt("OppimaaranNimi")); oppimaara.setOppiaine(Optional.of(new Reference(oa1.getId()))); vkDto = new OppiaineenVuosiluokkaKokonaisuusDto(); vkDto.setTehtava(Optional.of(to("Tehtävä", ""))); diff --git a/eperusteet/eperusteet-service/src/test/java/fi/vm/sade/eperusteet/service/navigation/PerusopetusPerusteNavigationIT.java b/eperusteet/eperusteet-service/src/test/java/fi/vm/sade/eperusteet/service/navigation/PerusopetusPerusteNavigationIT.java index 8c2d08e666..7da7695e4b 100644 --- a/eperusteet/eperusteet-service/src/test/java/fi/vm/sade/eperusteet/service/navigation/PerusopetusPerusteNavigationIT.java +++ b/eperusteet/eperusteet-service/src/test/java/fi/vm/sade/eperusteet/service/navigation/PerusopetusPerusteNavigationIT.java @@ -153,7 +153,7 @@ private OppiaineSuppeaDto oppiainesuppera(String nimi, Long jnro, Kieli kieli, O private T oppiaineBase(T baseOppiaine, String nimi, Long jnro, Kieli kieli) { if (nimi != null) { - baseOppiaine.setNimi(Optional.of(lt(nimi, kieli))); + baseOppiaine.setNimi(lt(nimi, kieli)); } if (jnro != null) { baseOppiaine.setJnro(Optional.of(jnro)); diff --git a/eperusteet/eperusteet-service/src/test/java/fi/vm/sade/eperusteet/service/util/OppiaineUtil.java b/eperusteet/eperusteet-service/src/test/java/fi/vm/sade/eperusteet/service/util/OppiaineUtil.java index cb9c904e22..206594b113 100644 --- a/eperusteet/eperusteet-service/src/test/java/fi/vm/sade/eperusteet/service/util/OppiaineUtil.java +++ b/eperusteet/eperusteet-service/src/test/java/fi/vm/sade/eperusteet/service/util/OppiaineUtil.java @@ -69,7 +69,7 @@ public static OppiaineLuontiPuu oppiaine(LokalisoituTekstiDto nimi) { public static OppiaineDto oppaine(LokalisoituTekstiDto nimi, Long parentId) { OppiaineDto dto = new OppiaineDto(); - dto.setNimi(Optional.of(nimi)); + dto.setNimi(nimi); dto.setOppiaine(parentId == null ? Optional.empty() : Optional.of(new fi.vm.sade.eperusteet.dto.Reference(parentId.toString()))); return dto;