Merge pull request #3226 from Opetushallitus/tor-2210-esimerkit-kaiki…

…sta-paluuarvotyypeista TOR-2210: OmaData OAuth2: Lisää testiapplikaatioon testerit kaikille scopeille
Opetushallitus · Nov 28, 2024 · 861633e · 861633e
2 parents b80ba9a + 730ea25
commit 861633e
Show file tree

Hide file tree

Showing 3 changed files with 34 additions and 4 deletions.
diff --git a/omadata-oauth2-sample/client/src/App.tsx b/omadata-oauth2-sample/client/src/App.tsx
@@ -8,7 +8,31 @@ function App() {
         <p>OmaDataOAuth2 Sample app</p>
       </header>
       <p>
-        <a href={"/api/openid-api-test"}>Test whole authorization code flow</a>
+        <a
+          href={
+            "/api/openid-api-test?scope=HENKILOTIEDOT_KAIKKI_TIEDOT+OPISKELUOIKEUDET_SUORITETUT_TUTKINNOT"
+          }
+        >
+          Test whole authorization code flow - suoritetut tutkinnot
+        </a>
+      </p>
+      <p>
+        <a
+          href={
+            "/api/openid-api-test?scope=HENKILOTIEDOT_KAIKKI_TIEDOT+OPISKELUOIKEUDET_AKTIIVISET_JA_PAATTYNEET_OPINNOT"
+          }
+        >
+          Test whole authorization code flow - aktiiviset ja päättyneet opinnot
+        </a>
+      </p>
+      <p>
+        <a
+          href={
+            "/api/openid-api-test?scope=HENKILOTIEDOT_KAIKKI_TIEDOT+OPISKELUOIKEUDET_KAIKKI_TIEDOT"
+          }
+        >
+          Test whole authorization code flow - kaikki tiedot
+        </a>
       </p>
       <p>
         <a href={"/api/openid-api-test/invalid-redirect-uri"}>

diff --git a/omadata-oauth2-sample/server/src/apiroutes/openid-api-test.ts b/omadata-oauth2-sample/server/src/apiroutes/openid-api-test.ts
@@ -18,11 +18,13 @@ const router: Router = express.Router()
 // TODO: TOR-2210: Toistaiseksi vain muistinvarainen map
 let verifiers: Map<string, string> = new Map()
 
-const scope: string =
+const defaultScope: string =
   'HENKILOTIEDOT_NIMI HENKILOTIEDOT_SYNTYMAAIKA HENKILOTIEDOT_HETU OPISKELUOIKEUDET_SUORITETUT_TUTKINNOT'
 
 router.get('/', async (req: Request, res: Response, next: NextFunction) => {
   try {
+    const scope = req.query.scope ? (req.query.scope as string) : defaultScope
+
     const state = uuidv4()
     const code_verifier = client.randomPKCECodeVerifier()
     verifiers.set(state, code_verifier)
@@ -45,7 +47,7 @@ router.get(
       const redirectTo = await buildAuthorizationUrl(
         code_verifier,
         state,
-        scope,
+        defaultScope,
         'http://localhost:9999/'
       )
 

diff --git a/src/main/scala/fi/oph/koski/koskiuser/MockUsers.scala b/src/main/scala/fi/oph/koski/koskiuser/MockUsers.scala
@@ -57,10 +57,14 @@ object MockUsers {
     "omadataoauth2sample",
     "1.2.246.562.24.99999984735",
     Seq(OrganisaatioJaKäyttöoikeudet(MockOrganisaatiot.dvv, List(
-      PalveluJaOikeus("KOSKI", Rooli.OMADATAOAUTH2_OPISKELUOIKEUDET_SUORITETUT_TUTKINNOT),
       PalveluJaOikeus("KOSKI", Rooli.OMADATAOAUTH2_HENKILOTIEDOT_NIMI),
       PalveluJaOikeus("KOSKI", Rooli.OMADATAOAUTH2_HENKILOTIEDOT_SYNTYMAAIKA),
       PalveluJaOikeus("KOSKI", Rooli.OMADATAOAUTH2_HENKILOTIEDOT_HETU),
+      PalveluJaOikeus("KOSKI", Rooli.OMADATAOAUTH2_HENKILOTIEDOT_OPPIJANUMERO),
+      PalveluJaOikeus("KOSKI", Rooli.OMADATAOAUTH2_HENKILOTIEDOT_KAIKKI_TIEDOT),
+      PalveluJaOikeus("KOSKI", Rooli.OMADATAOAUTH2_OPISKELUOIKEUDET_SUORITETUT_TUTKINNOT),
+      PalveluJaOikeus("KOSKI", Rooli.OMADATAOAUTH2_OPISKELUOIKEUDET_AKTIIVISET_JA_PAATTYNEET_OPINNOT),
+      PalveluJaOikeus("KOSKI", Rooli.OMADATAOAUTH2_OPISKELUOIKEUDET_KAIKKI_TIEDOT)
     )))
   )