Tarkenna kirjainkoodeihin

Opetushallitus · Jan 2, 2025 · a7b4f3c · a7b4f3c
1 parent 882cbdf
commit a7b4f3c
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 2 deletions.
diff --git a/src/main/scala/fi/oph/koski/virta/VirtaXMLConverter.scala b/src/main/scala/fi/oph/koski/virta/VirtaXMLConverter.scala
@@ -374,7 +374,8 @@ case class VirtaXMLConverter(oppilaitosRepository: OppilaitosRepository, koodist
       .map(l => koodistoViitePalvelu.validateRequired(koodistoUri, l))
 
   private def parsePatevyys(parentNode: Node, koodistoUri: String): List[Koodistokoodiviite] = (parentNode \ "Patevyys")
-    .map(_.text).filter(s => s.nonEmpty && s.length == 2).toList
+    .map(_.text)
+    .filter(s => s.nonEmpty && s.length == 2 && s.forall(_.isLetter)).toList
     .map(l => koodistoViitePalvelu.validateRequired(koodistoUri, l))
 
   private def laajuudetYhteensä(osasuoritukset: List[KorkeakoulunOpintojaksonSuoritus]) = {

diff --git a/src/test/scala/fi/oph/koski/virta/VirtaXMLConverterSpec.scala b/src/test/scala/fi/oph/koski/virta/VirtaXMLConverterSpec.scala
@@ -54,6 +54,8 @@ class VirtaXMLConverterSpec extends AnyFreeSpec with TestEnvironment with Matche
     <virta:Patevyys>kl</virta:Patevyys>
     <virta:Patevyys>aj</virta:Patevyys>
     <virta:Patevyys>ob</virta:Patevyys>
+    <virta:Patevyys>ll4</virta:Patevyys>
+    <virta:Patevyys>12</virta:Patevyys>
   </virta:Opintosuoritus>
 
   val virtaOpiskeluoikeudet: Elem = opiskeluoikeusWithOrganisaatio(None)
@@ -118,6 +120,8 @@ class VirtaXMLConverterSpec extends AnyFreeSpec with TestEnvironment with Matche
         <virta:Patevyys>ew</virta:Patevyys>
         <virta:Patevyys>oa</virta:Patevyys>
         <virta:Patevyys>ob</virta:Patevyys>
+        <virta:Patevyys>far</virta:Patevyys>
+        <virta:Patevyys>16</virta:Patevyys>
       </virta:Jakso>
       <virta:Laajuus>
         <virta:Opintopiste>240</virta:Opintopiste>
@@ -182,7 +186,7 @@ class VirtaXMLConverterSpec extends AnyFreeSpec with TestEnvironment with Matche
     }
 
     "Patevyys" - {
-      "parsitaan opintosuorituksilta sekä jaksoilta ilman duplikaatteja" in {
+      "parsitaan opintosuorituksilta sekä jaksoilta ilman duplikaatteja ja vain kaksimerkkiset kirjainkoodit huomioidaan" in {
         val oo = converter.convertToOpiskeluoikeudet(opiskeluoikeusSuorituksella())
         oo should have size (1)
         val patevyydet = oo.head.lisätiedot.flatMap(_.opettajapatevyys.map(x => x.map(_.koodiarvo))).getOrElse(List())