Skip to content

Commit

Permalink
search-api: Fix grep tests after rewriting to dump fetching
Browse files Browse the repository at this point in the history 
I broke some tests in the rewrite to fetching grep information via the
zipped dump. This patch fixes those.
  • Loading branch information
@jnatten
jnatten committed Jan 8, 2025
1 parent 2013855 commit 1a230d9
Show file tree
Hide file tree
Showing 5 changed files with 119 additions and 98 deletions.
6 changes: 0 additions & 6 deletions search-api/src/main/scala/no/ndla/searchapi/model/api/grep/GrepResultDTO.scala
View file
Original file line number Diff line number Diff line change
Expand Up @@ -9,14 +9,9 @@
package no.ndla.searchapi.model.api.grep

import cats.implicits.*
import com.scalatsi.TSType.fromCaseClass
import com.scalatsi.TypescriptType.{TSLiteralString, TSString, TSUnion}
import com.scalatsi.{TSIType, TSNamedType, TSType}
import com.scalatsi.dsl.*
import io.circe.generic.auto.*
import sttp.tapir.generic.auto.*

import scala.reflect.runtime.universe.*
import io.circe.syntax.*
import io.circe.{Decoder, Encoder, Json}
import no.ndla.language.Language
Expand All @@ -35,7 +30,6 @@ import no.ndla.searchapi.model.search.SearchableGrepElement
import sttp.tapir.Schema
import sttp.tapir.Schema.annotations.description

import scala.reflect.ClassTag
import scala.util.{Success, Try}

@description("Information about a single grep search result entry")
Expand Down
49 changes: 28 additions & 21 deletions search-api/src/test/scala/no/ndla/searchapi/TestData.scala
View file
Original file line number Diff line number Diff line change
Expand Up @@ -1587,27 +1587,34 @@ object TestData {
)

val grepBundle: GrepBundle = emptyGrepBundle.copy(
// kjerneelementer = List(
// GrepKjerneelement(
// "KE12",
// GrepTextObj(List(GrepTitle("default", "Utforsking og problemløysing"))),
// BelongsToObj("LP1")
// ),
// GrepKjerneelement(
// "KE34",
// GrepTextObj(List(GrepTitle("default", "Abstraksjon og generalisering"))),
// BelongsToObj("LP1")
// )
// ),
// kompetansemaal = List(
// GrepKompetansemaal(
// "KM123",
// GrepTextObj(List(GrepTitle("default", "bruke ulike kilder på en kritisk, hensiktsmessig og etterrettelig måte"))),
// BelongsToObj("LP1")
// )
// ),
// tverrfagligeTemaer = List(GrepTverrfagligTema("TT2", Seq(GrepTitle("default", "Demokrati og medborgerskap")))),
// laereplaner = List(GrepLaererplan("LP1", GrepTextObj(List(GrepTitle("default", "Læreplan i norsk (NOR01-04)")))))
kjerneelementer = List(
GrepKjerneelement(
kode = "KE12",
tittel = GrepTextObj(List(GrepTitle("default", "Utforsking og problemløysing"))),
beskrivelse = GrepTextObj(List(GrepTitle("default", ""))),
`tilhoerer-laereplan` = BelongsToObj("LP1", "Dette er LP1")
),
GrepKjerneelement(
kode = "KE34",
tittel = GrepTextObj(List(GrepTitle("default", "Abstraksjon og generalisering"))),
beskrivelse = GrepTextObj(List(GrepTitle("default", ""))),
`tilhoerer-laereplan` = BelongsToObj("LP1", "Dette er LP2")
)
),
kompetansemaal = List(
GrepKompetansemaal(
kode = "KM123",
tittel = GrepTextObj(
List(GrepTitle("default", "bruke ulike kilder på en kritisk, hensiktsmessig og etterrettelig måte"))
),
`tilhoerer-laereplan` = BelongsToObj("LP1", "Dette er LP1"),
`tilhoerer-kompetansemaalsett` = BelongsToObj("KMS1", "Dette er KMS1"),
`tilknyttede-tverrfaglige-temaer` = List(),
`tilknyttede-kjerneelementer` = List()
)
),
tverrfagligeTemaer = List(GrepTverrfagligTema("TT2", Seq(GrepTitle("default", "Demokrati og medborgerskap")))),
laereplaner = List(GrepLaererplan("LP1", GrepTextObj(List(GrepTitle("default", "Læreplan i norsk (NOR01-04)")))))
)

val searchSettings: SearchSettings = SearchSettings(
Expand Down
21 changes: 0 additions & 21 deletions search-api/src/test/scala/no/ndla/searchapi/integration/GrepApiClientTest.scala
View file

This file was deleted.

77 changes: 41 additions & 36 deletions search-api/src/test/scala/no/ndla/searchapi/service/search/GrepSearchServiceTest.scala
View file
Original file line number Diff line number Diff line change
Expand Up @@ -48,49 +48,54 @@ class GrepSearchServiceTest extends IntegrationSuite(EnableElasticsearchContaine

val grepTestBundle: GrepBundle = GrepBundle(
kjerneelementer = List(
// GrepKjerneelement(
// "KE12",
// GrepTextObj(
// List(GrepTitle("default", "Utforsking og problemløysing"), GrepTitle("nob", "Utforsking og problemløsning"))
// ),
// BelongsToObj("LP1")
// ),
// GrepKjerneelement(
// "KE34",
// GrepTextObj(
// List(GrepTitle("default", "Abstraksjon og generalisering"), GrepTitle("nob", "Abstraksjon og generalisering"))
// ),
// BelongsToObj("LP2")
// )
GrepKjerneelement(
"KE12",
GrepTextObj(
List(GrepTitle("default", "Utforsking og problemløysing"), GrepTitle("nob", "Utforsking og problemløsning"))
),
GrepTextObj(List(GrepTitle("default", ""))),
BelongsToObj("LP1", "Dette er LP1")
),
GrepKjerneelement(
"KE34",
GrepTextObj(
List(GrepTitle("default", "Abstraksjon og generalisering"), GrepTitle("nob", "Abstraksjon og generalisering"))
),
GrepTextObj(List(GrepTitle("default", ""))),
BelongsToObj("LP2", "Dette er LP2")
)
),
kompetansemaal = List(
// GrepKompetansemaal(
// "KM123",
// GrepTextObj(
// List(
// GrepTitle("default", "bruke ulike kilder på en kritisk, hensiktsmessig og etterrettelig måte"),
// GrepTitle("nob", "bruke ulike kilder på en kritisk, hensiktsmessig og etterrettelig måte")
// )
// ),
// BelongsToObj("LP2")
// )
GrepKompetansemaal(
kode = "KM123",
tittel = GrepTextObj(
List(
GrepTitle("default", "bruke ulike kilder på en kritisk, hensiktsmessig og etterrettelig måte"),
GrepTitle("nob", "bruke ulike kilder på en kritisk, hensiktsmessig og etterrettelig måte")
)
),
`tilhoerer-laereplan` = BelongsToObj("LP2", "Dette er LP2"),
`tilhoerer-kompetansemaalsett` = BelongsToObj("KE200", "Kompetansemaalsett"),
`tilknyttede-tverrfaglige-temaer` = List(),
`tilknyttede-kjerneelementer` = List()
)
),
kompetansemaalsett = List.empty,
tverrfagligeTemaer = List(
// GrepTverrfagligTema(
// "TT2",
// Seq(GrepTitle("default", "Demokrati og medborgerskap"), GrepTitle("nob", "Demokrati og medborgerskap"))
// )
GrepTverrfagligTema(
"TT2",
Seq(GrepTitle("default", "Demokrati og medborgerskap"), GrepTitle("nob", "Demokrati og medborgerskap"))
)
),
laereplaner = List(
// GrepLaererplan(
// "LP1",
// GrepTextObj(List(GrepTitle("default", "Læreplan i norsk"), GrepTitle("nob", "Læreplan i norsk")))
// ),
// GrepLaererplan(
// "LP2",
// GrepTextObj(List(GrepTitle("default", "Læreplan i engelsk"), GrepTitle("nob", "Læreplan i engelsk")))
// )
GrepLaererplan(
"LP1",
GrepTextObj(List(GrepTitle("default", "Læreplan i norsk"), GrepTitle("nob", "Læreplan i norsk")))
),
GrepLaererplan(
"LP2",
GrepTextObj(List(GrepTitle("default", "Læreplan i engelsk"), GrepTitle("nob", "Læreplan i engelsk")))
)
)
)

Expand Down
64 changes: 50 additions & 14 deletions search-api/src/test/scala/no/ndla/searchapi/service/search/SearchConverterServiceTest.scala
View file
Original file line number Diff line number Diff line change
Expand Up @@ -496,13 +496,31 @@ class SearchConverterServiceTest extends UnitSuite with TestEnvironment {
test("That asSearchableDraft converts grepContexts correctly based on grepBundle if draft has grepCodes") {
val draft = TestData.emptyDomainDraft.copy(id = Some(99), grepCodes = Seq("KE12", "KM123", "TT2"))
val grepBundle = TestData.emptyGrepBundle.copy(
// kjerneelementer = List(
// GrepKjerneelement("KE12", GrepTextObj(List(GrepTitle("default", "tittel12"))), BelongsToObj("LP123")),
// GrepKjerneelement("KE34", GrepTextObj(List(GrepTitle("default", "tittel34"))), BelongsToObj("LP123"))
// ),
// kompetansemaal =
// List(GrepKompetansemaal("KM123", GrepTextObj(List(GrepTitle("default", "tittel123"))), BelongsToObj("LP123"))),
// tverrfagligeTemaer = List(GrepTverrfagligTema("TT2", Seq(GrepTitle("default", "tittel2"))))
kjerneelementer = List(
GrepKjerneelement(
"KE12",
GrepTextObj(List(GrepTitle("default", "tittel12"))),
GrepTextObj(List(GrepTitle("default", ""))),
BelongsToObj("LP123", "Dette er LP123")
),
GrepKjerneelement(
"KE34",
GrepTextObj(List(GrepTitle("default", "tittel34"))),
GrepTextObj(List(GrepTitle("default", ""))),
BelongsToObj("LP123", "Dette er LP123")
)
),
kompetansemaal = List(
GrepKompetansemaal(
"KM123",
GrepTextObj(List(GrepTitle("default", "tittel123"))),
BelongsToObj("LP123", "Dette er LP123"),
BelongsToObj("KMS123", "Dette er KMS123"),
List(),
List()
)
),
tverrfagligeTemaer = List(GrepTverrfagligTema("TT2", Seq(GrepTitle("default", "tittel2"))))
)
val grepContexts = List(
SearchableGrepContext("KE12", Some("tittel12")),
Expand All @@ -517,13 +535,31 @@ class SearchConverterServiceTest extends UnitSuite with TestEnvironment {
test("That asSearchableDraft converts grepContexts correctly based on grepBundle if draft has no grepCodes") {
val draft = TestData.emptyDomainDraft.copy(id = Some(99), grepCodes = Seq.empty)
val grepBundle = TestData.emptyGrepBundle.copy(
// kjerneelementer = List(
// GrepKjerneelement("KE12", GrepTextObj(List(GrepTitle("default", "tittel12"))), BelongsToObj("LP123")),
// GrepKjerneelement("KE34", GrepTextObj(List(GrepTitle("default", "tittel34"))), BelongsToObj("LP123"))
// ),
// kompetansemaal =
// List(GrepKompetansemaal("KM123", GrepTextObj(List(GrepTitle("default", "tittel123"))), BelongsToObj("LP123"))),
// tverrfagligeTemaer = List(GrepTverrfagligTema("TT2", List(GrepTitle("default", "tittel2"))))
kjerneelementer = List(
GrepKjerneelement(
"KE12",
GrepTextObj(List(GrepTitle("default", "tittel12"))),
GrepTextObj(List(GrepTitle("default", ""))),
BelongsToObj("LP123", "Dette er LP123")
),
GrepKjerneelement(
"KE34",
GrepTextObj(List(GrepTitle("default", "tittel34"))),
GrepTextObj(List(GrepTitle("default", ""))),
BelongsToObj("LP123", "Dette er LP123")
)
),
kompetansemaal = List(
GrepKompetansemaal(
"KM123",
GrepTextObj(List(GrepTitle("default", "tittel123"))),
BelongsToObj("LP123", "Dette er LP123"),
BelongsToObj("KMS123", "Dette er KMS123"),
List(),
List()
)
),
tverrfagligeTemaer = List(GrepTverrfagligTema("TT2", Seq(GrepTitle("default", "tittel2"))))
)
val grepContexts = List.empty

Expand Down

0 comments on commit 1a230d9

Please sign in to comment.