Lisää testi palveluväyläkäyttäjille

Opetushallitus · Jan 3, 2024 · 705121d · 705121d
1 parent 50a427f
commit 705121d
Show file tree

Hide file tree

Showing 3 changed files with 56 additions and 4 deletions.
diff --git a/src/main/scala/fi/oph/koski/koskiuser/MockUsers.scala b/src/main/scala/fi/oph/koski/koskiuser/MockUsers.scala
@@ -396,11 +396,25 @@ object MockUsers {
     )))
   )
 
+  val hslKäyttäjä = KoskiMockUser(
+    "Palveluväylä",
+    "HSL",
+    "1.2.246.562.24.99999988899",
+    Seq(OrganisaatioJaKäyttöoikeudet(MockOrganisaatiot.hsl, List(
+      PalveluJaOikeus("KOSKI", Rooli.HSL),
+      PalveluJaOikeus("KOSKI", Rooli.GLOBAALI_LUKU_PERUSOPETUS),
+      PalveluJaOikeus("KOSKI", Rooli.GLOBAALI_LUKU_TOINEN_ASTE),
+      PalveluJaOikeus("KOSKI", Rooli.GLOBAALI_LUKU_KORKEAKOULU),
+      PalveluJaOikeus("KOSKI", Rooli.GLOBAALI_LUKU_MUU_KUIN_SAANNELTY),
+      PalveluJaOikeus("KOSKI", Rooli.GLOBAALI_LUKU_TAITEENPERUSOPETUS),
+    )))
+  )
+
   val suomiFiKäyttäjä = KoskiMockUser(
-    "Suomi",
-    "Fi",
-    "1.2.246.562.24.99999988889",
-    Seq(OrganisaatioJaKäyttöoikeudet(MockOrganisaatiot.migri, List(
+    "Palveluväylä",
+    "SuomiFi",
+    "1.2.246.562.24.99999988999",
+    Seq(OrganisaatioJaKäyttöoikeudet(MockOrganisaatiot.suomifi, List(
       PalveluJaOikeus("KOSKI", Rooli.SUOMIFI),
       PalveluJaOikeus("KOSKI", Rooli.GLOBAALI_LUKU_PERUSOPETUS),
       PalveluJaOikeus("KOSKI", Rooli.GLOBAALI_LUKU_TOINEN_ASTE),
@@ -537,6 +551,7 @@ object MockUsers {
     korkeakouluViranomainen,
     migriKäyttäjä,
     suomiFiKäyttäjä,
+    hslKäyttäjä,
     tilastokeskusKäyttäjä,
     valviraKäyttäjä,
     esiopetusTallentaja,

diff --git a/src/main/scala/fi/oph/koski/organisaatio/MockOrganisaatioRepository.scala b/src/main/scala/fi/oph/koski/organisaatio/MockOrganisaatioRepository.scala
@@ -60,6 +60,8 @@ object MockOrganisaatiot {
   val tilastokeskus = "1.2.246.562.10.35939310928"
   val migri = "1.2.246.562.10.31453145314"
   val valvira = "1.2.246.562.10.52577249361"
+  val hsl = "1.2.246.562.10.31453145314"
+  val suomifi = "1.2.246.562.10.31453145314"
   val kuopionAikuislukio = "1.2.246.562.10.42923230215"
   val kallavedenLukio = "1.2.246.562.10.63813695861"
   object EuropeanSchoolOfHelsinki {

diff --git a/src/test/scala/fi/oph/koski/mydata/MyDataAPIProxyServletTest.scala b/src/test/scala/fi/oph/koski/mydata/MyDataAPIProxyServletTest.scala
@@ -88,6 +88,41 @@ class MyDataAPIProxyServletTest extends AnyFreeSpec with KoskiHttpSpec with Matc
         }
       })
     }
+
+    "Palauttaa 200 mikäli HSL tai SUOMIFI" in {
+      KoskiApplicationForTests.mydataRepository.create(opiskelija.oid, memberId)
+
+      val palveluväyläKäyttäjät = List(
+        MockUsers.suomiFiKäyttäjä,
+        MockUsers.hslKäyttäjä
+      )
+
+      palveluväyläKäyttäjät.map(user => {
+        requestOpintoOikeudetWithoutAuthHeaders(opiskelija.hetu.get, memberHeaders(memberCode) ++ authHeaders(user)) {
+          status should equal(200)
+        }
+      })
+    }
+
+    "Palauttaa 403 mikäli ei palveluväyläkäyttäjä" in {
+      KoskiApplicationForTests.mydataRepository.create(opiskelija.oid, memberId)
+
+      val luovutuspalveluKäyttäjät = List(
+        MockUsers.ytlKäyttäjä,
+        MockUsers.valviraKäyttäjä,
+        MockUsers.kelaLaajatOikeudet,
+        MockUsers.kelaSuppeatOikeudet,
+        MockUsers.migriKäyttäjä,
+        MockUsers.tilastokeskusKäyttäjä
+      )
+
+      luovutuspalveluKäyttäjät.map(user => {
+        requestOpintoOikeudetWithoutAuthHeaders(opiskelija.hetu.get, memberHeaders(memberCode) ++ authHeaders(user)) {
+          status should equal(403)
+          body should include("Sallittu vain palveluväyläkäyttäjälle")
+        }
+      })
+    }
   }
 
   def memberHeaders(memberCode: String) = Map("X-ROAD-MEMBER" -> memberCode)