Skip to content

Commit

Permalink
OK-754: Siivotaan tarpeetonta koodia ja testi pois
Browse files Browse the repository at this point in the history
  • Loading branch information
@heidilm
heidilm committed Jan 8, 2025
1 parent dd24a10 commit 887201b
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 151 deletions.
13 changes: 0 additions & 13 deletions ovara-backend/src/main/scala/fi/oph/ovara/backend/utils/ExcelWriter.scala
View file
Original file line number Diff line number Diff line change
Expand Up @@ -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,
Expand Down Expand Up @@ -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) {
Expand Down
138 changes: 0 additions & 138 deletions ovara-backend/src/test/scala/fi/oph/ovara/backend/utils/ExcelWriterSpec.scala
View file
Original file line number Diff line number Diff line change
Expand Up @@ -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"),
Expand Down

0 comments on commit 887201b

Please sign in to comment.