diff --git a/ovara-backend/src/main/scala/fi/oph/ovara/backend/utils/ExcelWriter.scala b/ovara-backend/src/main/scala/fi/oph/ovara/backend/utils/ExcelWriter.scala index 6a98283..aaa8616 100644 --- a/ovara-backend/src/main/scala/fi/oph/ovara/backend/utils/ExcelWriter.scala +++ b/ovara-backend/src/main/scala/fi/oph/ovara/backend/utils/ExcelWriter.scala @@ -29,17 +29,6 @@ object ExcelWriter { } } - def flattenHierarkiat( - organisaatioHierarkiaWithHakukohteet: OrganisaatioHierarkiaWithHakukohteet - ): List[OrganisaatioHierarkiaWithHakukohteet] = { - if (organisaatioHierarkiaWithHakukohteet.children.isEmpty) { - List(organisaatioHierarkiaWithHakukohteet) - } else { - val childHierarkiat = organisaatioHierarkiaWithHakukohteet.children.flatMap(child => flattenHierarkiat(child)) - organisaatioHierarkiaWithHakukohteet :: childHierarkiat - } - } - def createOrganisaatioHeadingRow( sheet: XSSFSheet, initialRowIndex: Int, @@ -122,8 +111,6 @@ object ExcelWriter { asiointikieli: String, raporttiColumnTitlesWithIndex: List[(String, Int)] ): Unit = { - val allHierarkiat = hierarkiatWithHakukohteet.flatMap(child => flattenHierarkiat(child)) - val hakukohteet = allHierarkiat.flatMap(_.hakukohteet) var currentRowIndex = initialRowIndex if (hierarkiatWithHakukohteet.nonEmpty) { diff --git a/ovara-backend/src/test/scala/fi/oph/ovara/backend/utils/ExcelWriterSpec.scala b/ovara-backend/src/test/scala/fi/oph/ovara/backend/utils/ExcelWriterSpec.scala index df88c7c..5d37552 100644 --- a/ovara-backend/src/test/scala/fi/oph/ovara/backend/utils/ExcelWriterSpec.scala +++ b/ovara-backend/src/test/scala/fi/oph/ovara/backend/utils/ExcelWriterSpec.scala @@ -298,144 +298,6 @@ class ExcelWriterSpec extends AnyFlatSpec { ) } - "flattenHierarkiat" should "return list with one toimipiste hakukohde and two oppilaitos hakukohde in hierarkia" in { - val oppilaitoksenKth1 = OrganisaationKoulutusToteutusHakukohde( - organisaatio_oid = Some("1.2.246.562.10.278170642010"), - koulutusToteutusHakukohde = KoulutusToteutusHakukohdeResult( - hakukohdeNimi = Map(En -> "Hakukohteen nimi en", Fi -> "Hakukohteen nimi fi", Sv -> "Hakukohteen nimi sv"), - hakukohdeOid = "1.2.246.562.20.00000000000000021565", - koulutuksenTila = Some("julkaistu"), - toteutuksenTila = Some("julkaistu"), - hakukohteenTila = Some("julkaistu"), - aloituspaikat = Some(8), - onValintakoe = Some(false) - ) - ) - - val oppilaitoksenKth2 = oppilaitoksenKth1.copy(koulutusToteutusHakukohde = - oppilaitoksenKth1._2.copy( - hakukohdeNimi = - Map(En -> "Hakukohteen 2 nimi en", Fi -> "Hakukohteen 2 nimi fi", Sv -> "Hakukohteen 2 nimi sv"), - hakukohdeOid = "1.2.246.562.20.0000000000000002156667" - ) - ) - - val toimipisteenKth = oppilaitoksenKth1.copy( - organisaatio_oid = Some("1.2.246.562.10.2781706420101111"), - koulutusToteutusHakukohde = oppilaitoksenKth1._2.copy( - hakukohdeNimi = Map( - En -> "Toimipiste hakukohteen nimi en", - Fi -> "Toimipiste hakukohteen nimi fi", - Sv -> "Toimipiste hakukohteen nimi sv" - ), - hakukohdeOid = "1.2.246.562.20.000000000000000215521" - ) - ) - - val hierarkiaWithHakukohteet = - OrganisaatioHierarkiaWithHakukohteet( - "1.2.246.562.10.278170642010", - Map(En -> "Oppilaitoksen nimi en", Fi -> "Oppilaitoksen nimi fi", Sv -> "Oppilaitoksen nimi sv"), - List("02"), - List("1.2.246.562.10.00000000001", "1.2.246.562.10.2781706420000", "1.2.246.562.10.278170642010"), - List( - OrganisaatioHierarkiaWithHakukohteet( - "1.2.246.562.10.2781706420101111", - Map(En -> "Toimipisteen nimi en", Fi -> "Toimipisteen nimi fi", Sv -> "Toimipisteen nimi sv"), - List("03"), - List( - "1.2.246.562.10.00000000001", - "1.2.246.562.10.2781706420000", - "1.2.246.562.10.278170642010", - "1.2.246.562.10.2781706420101111" - ), - List(), - List(toimipisteenKth) - ), - OrganisaatioHierarkiaWithHakukohteet( - "1.2.246.562.10.2781706420101112", - Map(En -> "Toimipisteen 2 nimi en", Fi -> "Toimipisteen 2 nimi fi", Sv -> "Toimipisteen 2 nimi sv"), - List("03"), - List( - "1.2.246.562.10.00000000001", - "1.2.246.562.10.2781706420000", - "1.2.246.562.10.278170642010", - "1.2.246.562.10.2781706420101112" - ), - List(), - List() - ) - ), - List(oppilaitoksenKth1, oppilaitoksenKth2) - ) - - assert( - ExcelWriter.flattenHierarkiat(hierarkiaWithHakukohteet) == List( - OrganisaatioHierarkiaWithHakukohteet( - "1.2.246.562.10.278170642010", - Map(En -> "Oppilaitoksen nimi en", Fi -> "Oppilaitoksen nimi fi", Sv -> "Oppilaitoksen nimi sv"), - List("02"), - List("1.2.246.562.10.00000000001", "1.2.246.562.10.2781706420000", "1.2.246.562.10.278170642010"), - List( - OrganisaatioHierarkiaWithHakukohteet( - "1.2.246.562.10.2781706420101111", - Map(En -> "Toimipisteen nimi en", Fi -> "Toimipisteen nimi fi", Sv -> "Toimipisteen nimi sv"), - List("03"), - List( - "1.2.246.562.10.00000000001", - "1.2.246.562.10.2781706420000", - "1.2.246.562.10.278170642010", - "1.2.246.562.10.2781706420101111" - ), - List(), - List(toimipisteenKth) - ), - OrganisaatioHierarkiaWithHakukohteet( - "1.2.246.562.10.2781706420101112", - Map(En -> "Toimipisteen 2 nimi en", Fi -> "Toimipisteen 2 nimi fi", Sv -> "Toimipisteen 2 nimi sv"), - List("03"), - List( - "1.2.246.562.10.00000000001", - "1.2.246.562.10.2781706420000", - "1.2.246.562.10.278170642010", - "1.2.246.562.10.2781706420101112" - ), - List(), - List() - ) - ), - List(oppilaitoksenKth1, oppilaitoksenKth2) - ), - OrganisaatioHierarkiaWithHakukohteet( - "1.2.246.562.10.2781706420101111", - Map(En -> "Toimipisteen nimi en", Fi -> "Toimipisteen nimi fi", Sv -> "Toimipisteen nimi sv"), - List("03"), - List( - "1.2.246.562.10.00000000001", - "1.2.246.562.10.2781706420000", - "1.2.246.562.10.278170642010", - "1.2.246.562.10.2781706420101111" - ), - List(), - List(toimipisteenKth) - ), - OrganisaatioHierarkiaWithHakukohteet( - "1.2.246.562.10.2781706420101112", - Map(En -> "Toimipisteen 2 nimi en", Fi -> "Toimipisteen 2 nimi fi", Sv -> "Toimipisteen 2 nimi sv"), - List("03"), - List( - "1.2.246.562.10.00000000001", - "1.2.246.562.10.2781706420000", - "1.2.246.562.10.278170642010", - "1.2.246.562.10.2781706420101112" - ), - List(), - List() - ) - ) - ) - } - "createOrganisaatioHeadingRow" should "create row with org name and total count of aloituspaikat" in { val oppilaitoksenKth1 = OrganisaationKoulutusToteutusHakukohde( organisaatio_oid = Some("1.2.246.562.10.2781706420000"),