Merge pull request #3040 from Opetushallitus/eperuste-validation-logging

TOR-2205: Lisää flägi ja lokitus ammatillisten E-peruste laajuusvalidaatiolle

Author: a544jh

src/main/resources/reference.conf

@@ -236,6 +236,7 @@ validaatiot = {
     perusopetukseenValmistavaOpetusKokonaislaajuusYksikköTunneissaAlkaa = "2024-01-01"
     jotpaAsianumeroVaatimusAlkaa = "2024-01-01"
     ohitaValidaatiovirheetKäyttäjällä = []
+    ammatillinenEPerusteValidaatio = true
 }
 
 elaketurvakeskus = {

src/main/scala/fi/oph/koski/config/KoskiApplication.scala

@@ -102,7 +102,7 @@ class KoskiApplication(
   lazy val ytrHistoryRepository = YtrOpiskeluoikeusHistoryRepository(masterDatabase.db)
   lazy val virta = TimedProxy[AuxiliaryOpiskeluoikeusRepository](VirtaOpiskeluoikeusRepository(virtaClient, oppilaitosRepository, koodistoViitePalvelu, organisaatioRepository, virtaAccessChecker, Some(validator)))
   lazy val henkilöCache = new KoskiHenkilöCache(masterDatabase.db)
-  lazy val ePerusteetValidator = new EPerusteisiinPerustuvaValidator(ePerusteet, tutkintoRepository, koodistoViitePalvelu)
+  lazy val ePerusteetValidator = new EPerusteisiinPerustuvaValidator(ePerusteet, tutkintoRepository, koodistoViitePalvelu, config)
   lazy val ePerusteetChangeValidator = new EPerusteetOpiskeluoikeusChangeValidator(ePerusteet, tutkintoRepository, koodistoViitePalvelu)
   lazy val ePerusteetFiller = new EPerusteetFiller(ePerusteet, tutkintoRepository, koodistoViitePalvelu)
   lazy val ePerusteetLops2019Validator = new EPerusteetLops2019Validator(config, ePerusteet)

src/main/scala/fi/oph/koski/eperusteetvalidation/EPerusteisiinPerustuvaValidator.scala

@@ -1,5 +1,6 @@
 package fi.oph.koski.eperusteetvalidation
 
+import com.typesafe.config.Config
 import fi.oph.koski.eperusteet.{EPerusteRakenne, EPerusteetRepository}
 import fi.oph.koski.http.{HttpStatus, KoskiErrorCategory}
 import fi.oph.koski.koodisto.KoodistoViitePalvelu
@@ -13,9 +14,10 @@ import java.time.LocalDate
 class EPerusteisiinPerustuvaValidator(
   ePerusteet: EPerusteetRepository,
   tutkintoRepository: TutkintoRepository,
-  koodistoViitePalvelu: KoodistoViitePalvelu
+  koodistoViitePalvelu: KoodistoViitePalvelu,
+  config: Config
 ) extends EPerusteetValidationUtils(tutkintoRepository, koodistoViitePalvelu) with Logging {
-  private val tutkintorakenneValidator: TutkintoRakenneValidator = TutkintoRakenneValidator(tutkintoRepository, koodistoViitePalvelu)
+  private val tutkintorakenneValidator: TutkintoRakenneValidator = TutkintoRakenneValidator(tutkintoRepository, koodistoViitePalvelu, config)
 
   def validateTutkintorakenne(opiskeluoikeus: KoskeenTallennettavaOpiskeluoikeus): HttpStatus = HttpStatus.fold(
     opiskeluoikeus.suoritukset.map(

src/main/scala/fi/oph/koski/eperusteetvalidation/TutkintoRakenneValidator.scala

@@ -1,5 +1,6 @@
 package fi.oph.koski.eperusteetvalidation
 
+import com.typesafe.config.Config
 import fi.oph.koski.http.{HttpStatus, KoskiErrorCategory}
 import fi.oph.koski.koodisto.KoodistoViitePalvelu
 import fi.oph.koski.log.Logging
@@ -10,7 +11,7 @@ import fi.oph.koski.tutkinto.{Koulutustyyppi, _}
 
 import java.time.LocalDate
 
-case class TutkintoRakenneValidator(tutkintoRepository: TutkintoRepository, koodistoViitePalvelu: KoodistoViitePalvelu)
+case class TutkintoRakenneValidator(tutkintoRepository: TutkintoRepository, koodistoViitePalvelu: KoodistoViitePalvelu, config: Config)
   extends EPerusteetValidationUtils(tutkintoRepository, koodistoViitePalvelu) with Logging {
 
   def validate(suoritus: PäätasonSuoritus, alkamispäiväLäsnä: Option[LocalDate], vaadittuPerusteenVoimassaolopäivä: LocalDate): HttpStatus = {
@@ -359,7 +360,17 @@ case class TutkintoRakenneValidator(tutkintoRepository: TutkintoRepository, kood
           case _ => HttpStatus.ok // mm. paikalliset tutkinnon osat
         }
       ))
-      HttpStatus.fold(List(osaStatus) ++ osaAlueStatuses)
+      val result = HttpStatus.fold(List(osaStatus) ++ osaAlueStatuses)
+
+      val validaatioPäällä = config.getBoolean("validaatiot.ammatillinenEPerusteValidaatio")
+      if (validaatioPäällä) {
+        result
+      } else {
+        if (result.isError) {
+          logger.warn(s"Ammatillisen suoritusken E-peruste validaatio poissa päältä mutta epäonnistuisi virheillä: ${result.errorString.getOrElse("")}")
+        }
+        HttpStatus.ok
+      }
   }
 
   private def validateLukio2019Diaarinumero(s: LukionPäätasonSuoritus2019) = {

src/test/scala/fi/oph/koski/api/misc/OpiskeluoikeusValidationSpec.scala

@@ -93,7 +93,8 @@ class OpiskeluoikeusValidationSpec extends AnyFreeSpec with Matchers with Opiske
       new EPerusteisiinPerustuvaValidator(
         KoskiApplicationForTests.ePerusteet,
         KoskiApplicationForTests.tutkintoRepository,
-        KoskiApplicationForTests.koodistoViitePalvelu
+        KoskiApplicationForTests.koodistoViitePalvelu,
+        config
       ),
       new EPerusteetLops2019Validator(KoskiApplicationForTests.config, KoskiApplicationForTests.ePerusteet),
       new EPerusteetFiller(

src/test/scala/fi/oph/koski/api/oppijavalidation/OppijaValidationAmmatillinenSpec.scala

@@ -1478,7 +1478,8 @@ class OppijaValidationAmmatillinenSpec extends TutkinnonPerusteetTest[Ammatillin
       new EPerusteisiinPerustuvaValidator(
         KoskiApplicationForTests.ePerusteet,
         KoskiApplicationForTests.tutkintoRepository,
-        KoskiApplicationForTests.koodistoViitePalvelu
+        KoskiApplicationForTests.koodistoViitePalvelu,
+        config
       ),
       new EPerusteetLops2019Validator(KoskiApplicationForTests.config, KoskiApplicationForTests.ePerusteet),
       new EPerusteetFiller(

src/test/scala/fi/oph/koski/api/oppijavalidation/OppijaValidationTutkintokoulutukseenValmentavaKoulutusSpec.scala

@@ -888,7 +888,8 @@ class OppijaValidationTutkintokoulutukseenValmentavaKoulutusSpec extends Tutkinn
       new EPerusteisiinPerustuvaValidator(
         KoskiApplicationForTests.ePerusteet,
         KoskiApplicationForTests.tutkintoRepository,
-        KoskiApplicationForTests.koodistoViitePalvelu
+        KoskiApplicationForTests.koodistoViitePalvelu,
+        config
       ),
       new EPerusteetLops2019Validator(KoskiApplicationForTests.config, KoskiApplicationForTests.ePerusteet),
       new EPerusteetFiller(

src/test/scala/fi/oph/koski/api/oppijavalidation/OppijaValidationVapaaSivistystyoOsaamismerkkiSpec.scala

@@ -240,7 +240,8 @@ class OppijaValidationVapaaSivistystyoOsaamismerkkiSpec extends AnyFreeSpec with
       new EPerusteisiinPerustuvaValidator(
         KoskiApplicationForTests.ePerusteet,
         KoskiApplicationForTests.tutkintoRepository,
-        KoskiApplicationForTests.koodistoViitePalvelu
+        KoskiApplicationForTests.koodistoViitePalvelu,
+        config
       ),
       new EPerusteetLops2019Validator(KoskiApplicationForTests.config, KoskiApplicationForTests.ePerusteet),
       new EPerusteetFiller(