OY-4799 filtteröidään pelkästään hylätyt osasuoritukset

Opetushallitus · Apr 18, 2024 · 71d27f5 · 71d27f5
1 parent 36649ee
commit 71d27f5
Show file tree

Hide file tree

Showing 4 changed files with 3,018 additions and 4 deletions.
diff --git a/src/main/scala/fi/vm/sade/hakurekisteri/integration/koski/KoskiDataHandler.scala b/src/main/scala/fi/vm/sade/hakurekisteri/integration/koski/KoskiDataHandler.scala
@@ -628,11 +628,9 @@ class KoskiDataHandler(
           Oids.perusopetuksenOppiaineenOppimaaraOid.contains(s.suoritus.komo)
         )
       }
-      // Ei tallenneta peruskoulun oppiaineen oppimäärän suorituksia joilla ei ole arvosanaa tai joiden arvosana on 4
+      // Ei tallenneta peruskoulun oppiaineen oppimäärän suorituksia joilla ei ole arvosanaa
       tallennettavatSuoritukset = tallennettavatSuoritukset.filterNot(s =>
-        Oids.perusopetuksenOppiaineenOppimaaraOid.contains(
-          s.suoritus.komo
-        ) && (s.arvosanat.isEmpty || s.arvosanat.exists(a => a.arvio.equals(Arvio410("4"))))
+        Oids.perusopetuksenOppiaineenOppimaaraOid.contains(s.suoritus.komo) && s.arvosanat.isEmpty
       )
 
       // Tallennetaan mahdollinen 7-8-valmistava vain silloin, kun valmista ja vahvistettua

diff --git a/src/main/scala/fi/vm/sade/hakurekisteri/integration/koski/KoskiSuoritusArvosanaParser.scala b/src/main/scala/fi/vm/sade/hakurekisteri/integration/koski/KoskiSuoritusArvosanaParser.scala
@@ -460,6 +460,9 @@ class KoskiSuoritusArvosanaParser {
                 .koodistoUri
                 .contentEquals("koskioppiaineetyleissivistava")
             )
+            // Filtteröidään hylätyt
+            s = s.filter(osaSuoritus =>
+              osaSuoritus.arviointi.exists(a => a.hyväksytty.isDefined && a.hyväksytty.get))
             osasuoritusToArvosana(
               personOid,
               komoOid,

diff --git a/src/test/scala/fi/vm/sade/hakurekisteri/integration/koski/KoskiDataHandlerTest.scala b/src/test/scala/fi/vm/sade/hakurekisteri/integration/koski/KoskiDataHandlerTest.scala
@@ -4466,6 +4466,36 @@ class KoskiDataHandlerTest
 
   }
 
+  it should "import hyväksytty suoritus even if there is arvosana 4 for nuorten perusopetuksen oppiaineen oppimäärä" in {
+    val json: String =
+      scala.io.Source
+        .fromFile(jsonDir + "koski_perusopituksen_oppiaine_4_ja_hyvaksytty.json")
+        .mkString
+    val henkilo: KoskiHenkiloContainer = parse(json).extract[KoskiHenkiloContainer]
+    henkilo should not be null
+    henkilo.opiskeluoikeudet.head.tyyppi should not be empty
+
+    KoskiUtil.deadlineDate = LocalDate.now().plusDays(30)
+
+    Await.result(
+      koskiDatahandler.processHenkilonTiedotKoskesta(
+        henkilo,
+        PersonOidsWithAliases(henkilo.henkilö.oid.toSet),
+        new KoskiSuoritusHakuParams(saveLukio = true, saveAmmatillinen = true)
+      ),
+      5.seconds
+    )
+
+    val matikkaArvosanat = run(
+      database.run(
+        sql"select count(*) from arvosana where aine = 'MA'"
+          .as[String]
+      )
+    )
+    matikkaArvosanat.head should equal("2")
+
+  }
+
   it should "not import arvosanat for aikuisten perusopetuksen oppiaineen oppimäärä without arvosana" in {
     val json: String =
       scala.io.Source