Merge pull request #4 from Opetushallitus/kouta-internal-changes

Kouta internal changes
Opetushallitus · Apr 18, 2024 · 76b4979 · 76b4979
2 parents 2dd5c16 + d10546c
commit 76b4979
Show file tree

Hide file tree

Showing 9 changed files with 57 additions and 43 deletions.
diff --git a/.env b/.env
@@ -1 +1 @@
-VIRKAILIJA_URL=https://virkailija.testiopintopolku.fi
+VIRKAILIJA_URL=https://virkailija.untuvaopintopolku.fi
diff --git a/src/app/haku/[oid]/hakukohde-list.tsx b/src/app/haku/[oid]/hakukohde-list.tsx
@@ -35,7 +35,7 @@ export const HakukohdeList = ({ oid }: { oid: string }) => {
       {hakukohteet.map((hk: Hakukohde) => (
         <StyledItem key={hk.oid}>
           <p title={hk.organisaatioOid} className="organizationLabel">
-            {getTranslation(hk.organisaatioNimi)}
+            {getTranslation(hk.jarjestyspaikkaHierarkiaNimi)}
           </p>
           <p title={hk.oid}>{getTranslation(hk.nimi)}</p>
         </StyledItem>

diff --git a/src/app/lib/configuration.ts b/src/app/lib/configuration.ts
@@ -19,7 +19,7 @@ export const configuration: Configuration = {
   serviceUrl,
   loginUrl: process.env.LOGIN_URL || `${DOMAIN}/cas/login`,
   sessionCookie: process.env.SESSION_COOKIE || 'JSESSIONID',
-  hautUrl: `${DOMAIN}/kouta-internal/haku/search?includeHakukohdeOids=true`,
+  hautUrl: `${DOMAIN}/kouta-internal/haku/search?includeHakukohdeOids=false`,
   hakuUrl: `${DOMAIN}/kouta-internal/haku`,
   hakukohteetUrl: `${DOMAIN}/kouta-internal/hakukohde/search?all=false`,
   kooditUrl: `${DOMAIN}/koodisto-service/rest/codeelement/codes/`,

diff --git a/src/app/lib/kouta-types.ts b/src/app/lib/kouta-types.ts
@@ -46,6 +46,7 @@ export type Hakukohde = {
   nimi: TranslatedName;
   organisaatioOid: string;
   organisaatioNimi: TranslatedName;
+  jarjestyspaikkaHierarkiaNimi: TranslatedName;
 };
 
 export enum Tila {

diff --git a/src/app/lib/kouta.ts b/src/app/lib/kouta.ts
@@ -15,7 +15,7 @@ export async function getHaut(active: boolean = true) {
       hakutapaKoodiUri: string;
       hakuvuosi: string;
       hakukausi: string;
-      hakukohdeOids: string[];
+      totalHakukohteet: number;
     }) => {
       const haunTila: Tila = Tila[h.tila.toUpperCase() as keyof typeof Tila];
       return {
@@ -25,7 +25,7 @@ export async function getHaut(active: boolean = true) {
         hakutapaKoodiUri: h.hakutapaKoodiUri,
         alkamisVuosi: parseInt(h.hakuvuosi),
         alkamisKausiKoodiUri: h.hakukausi,
-        hakukohteita: h?.hakukohdeOids?.length ?? 0,
+        hakukohteita: h?.totalHakukohteet ?? 0,
       };
     },
   );
@@ -47,12 +47,14 @@ export async function getHakukohteet(hakuOid: string): Promise<Hakukohde[]> {
       nimi: TranslatedName;
       organisaatioOid: string;
       organisaatioNimi: TranslatedName;
+      jarjestyspaikkaHierarkiaNimi: TranslatedName;
     }) => {
       return {
         oid: h.oid,
         nimi: h.nimi,
         organisaatioNimi: h.organisaatioNimi,
         organisaatioOid: h.organisaatioOid,
+        jarjestyspaikkaHierarkiaNimi: h.jarjestyspaikkaHierarkiaNimi,
       };
     },
   );

diff --git a/tests/e2e/fixtures/hakukohteet.json b/tests/e2e/fixtures/hakukohteet.json
@@ -17,9 +17,7 @@
     "kaytetaanHaunHakulomaketta": true,
     "aloituspaikat": 0,
     "painotetutArvosanat": [],
-    "pohjakoulutusvaatimusKoodiUrit": [
-      "pohjakoulutusvaatimuskouta_123#1"
-    ],
+    "pohjakoulutusvaatimusKoodiUrit": ["pohjakoulutusvaatimuskouta_123#1"],
     "muuPohjakoulutusvaatimus": {},
     "toinenAsteOnkoKaksoistutkinto": false,
     "kaytetaanHaunAikataulua": true,
@@ -65,9 +63,7 @@
       "fi": "Rakennetun ympäristön tiedekunta",
       "en": "Faculty of Built Environment"
     },
-    "kielivalinta": [
-      "en"
-    ],
+    "kielivalinta": ["en"],
     "modified": "2023-10-30T10:26:15",
     "jarjestaaUrheilijanAmmKoulutusta": false,
     "uudenOpiskelijanUrl": {
@@ -84,6 +80,11 @@
       "ylempiKkAste": true,
       "kkTutkinnonTaso": 3,
       "kkTutkinnonTasoSykli": 1
+    },
+    "jarjestyspaikkaHierarkiaNimi": {
+      "fi": "Tampereen yliopisto, Rakennetun ympäristön tiedekunta",
+      "sv": "Tammerfors universitet, Rakennetun ympäristön tiedekunta",
+      "en": "Tampere University, Faculty of Built Environment"
     }
   },
   {
@@ -104,9 +105,7 @@
     "kaytetaanHaunHakulomaketta": true,
     "aloituspaikat": 0,
     "painotetutArvosanat": [],
-    "pohjakoulutusvaatimusKoodiUrit": [
-      "pohjakoulutusvaatimuskouta_123#1"
-    ],
+    "pohjakoulutusvaatimusKoodiUrit": ["pohjakoulutusvaatimuskouta_123#1"],
     "muuPohjakoulutusvaatimus": {},
     "toinenAsteOnkoKaksoistutkinto": false,
     "kaytetaanHaunAikataulua": true,
@@ -152,9 +151,7 @@
       "sv": "Tekniikan ja luonnontieteiden tiedekunta",
       "en": "Faculty of Engineering and Natural Sciences"
     },
-    "kielivalinta": [
-      "en"
-    ],
+    "kielivalinta": ["en"],
     "modified": "2023-10-30T10:26:25",
     "jarjestaaUrheilijanAmmKoulutusta": false,
     "uudenOpiskelijanUrl": {
@@ -171,6 +168,11 @@
       "ylempiKkAste": true,
       "kkTutkinnonTaso": 3,
       "kkTutkinnonTasoSykli": 1
+    },
+    "jarjestyspaikkaHierarkiaNimi": {
+      "sv": "Tammerfors universitet, Tekniikan ja luonnontieteiden tiedekunta",
+      "fi": "Tampereen yliopisto, Tekniikan ja luonnontieteiden tiedekunta",
+      "en": "Tampere University, Faculty of Engineering and Natural Sciences"
     }
   },
   {
@@ -191,9 +193,7 @@
     "kaytetaanHaunHakulomaketta": true,
     "aloituspaikat": 0,
     "painotetutArvosanat": [],
-    "pohjakoulutusvaatimusKoodiUrit": [
-      "pohjakoulutusvaatimuskouta_123#1"
-    ],
+    "pohjakoulutusvaatimusKoodiUrit": ["pohjakoulutusvaatimuskouta_123#1"],
     "muuPohjakoulutusvaatimus": {},
     "toinenAsteOnkoKaksoistutkinto": false,
     "kaytetaanHaunAikataulua": true,
@@ -239,9 +239,7 @@
       "sv": "Tekniikan ja luonnontieteiden tiedekunta",
       "en": "Faculty of Engineering and Natural Sciences"
     },
-    "kielivalinta": [
-      "en"
-    ],
+    "kielivalinta": ["en"],
     "modified": "2023-10-30T10:26:20",
     "jarjestaaUrheilijanAmmKoulutusta": false,
     "uudenOpiskelijanUrl": {
@@ -258,6 +256,11 @@
       "ylempiKkAste": true,
       "kkTutkinnonTaso": 3,
       "kkTutkinnonTasoSykli": 1
+    },
+    "jarjestyspaikkaHierarkiaNimi": {
+      "sv": "Tammerfors universitet, Tekniikan ja luonnontieteiden tiedekunta",
+      "fi": "Tampereen yliopisto, Tekniikan ja luonnontieteiden tiedekunta",
+      "en": "Tampere University, Faculty of Engineering and Natural Sciences"
     }
   }
-]
+]
diff --git a/tests/e2e/fixtures/haut.json b/tests/e2e/fixtures/haut.json
@@ -64,11 +64,10 @@
       }
     ],
     "hakukohdeOids": [],
+    "totalHakukohteet": 0,
     "valintakokeet": [],
     "muokkaaja": "1.2.246.562.24.10327979532",
-    "kielivalinta": [
-      "fi"
-    ],
+    "kielivalinta": ["fi"],
     "modified": "2023-11-02T12:27:41"
   },
   {
@@ -140,11 +139,10 @@
       }
     ],
     "hakukohdeOids": [],
+    "totalHakukohteet": 0,
     "valintakokeet": [],
     "muokkaaja": "1.2.246.562.24.61168703071",
-    "kielivalinta": [
-      "fi"
-    ],
+    "kielivalinta": ["fi"],
     "modified": "2022-01-05T16:32:06"
   },
   {
@@ -169,11 +167,10 @@
     "organisaatioOid": "1.2.246.562.10.135945357610",
     "hakuajat": [],
     "hakukohdeOids": [],
+    "totalHakukohteet": 0,
     "valintakokeet": [],
     "muokkaaja": "1.2.246.562.24.78053256082",
-    "kielivalinta": [
-      "fi"
-    ],
+    "kielivalinta": ["fi"],
     "modified": "2022-06-27T13:47:36"
   },
   {
@@ -236,11 +233,10 @@
       }
     ],
     "hakukohdeOids": [],
+    "totalHakukohteet": 0,
     "valintakokeet": [],
     "muokkaaja": "1.2.246.562.24.65467286208",
-    "kielivalinta": [
-      "fi"
-    ],
+    "kielivalinta": ["fi"],
     "modified": "2023-01-07T12:56:02"
   },
   {
@@ -274,9 +270,7 @@
     "hakukohdeOids": [],
     "valintakokeet": [],
     "muokkaaja": "1.2.246.562.24.80913193985",
-    "kielivalinta": [
-      "fi"
-    ],
+    "kielivalinta": ["fi"],
     "modified": "2023-11-14T13:23:41"
   },
   {
@@ -286,6 +280,7 @@
       "1.2.246.562.20.00000000000000045104",
       "1.2.246.562.20.00000000000000045105"
     ],
+    "totalHakukohteet": 3,
     "tila": "julkaistu",
     "nimi": {
       "en": "Tampere University Separate Admission/ Finnish MAOL Competition Route 2024"
@@ -324,9 +319,7 @@
     ],
     "valintakokeet": [],
     "muokkaaja": "1.2.246.562.24.57549617509",
-    "kielivalinta": [
-      "en"
-    ],
+    "kielivalinta": ["en"],
     "modified": "2023-10-30T10:34:31"
   }
 ]
diff --git a/tests/e2e/haku.spec.ts → tests/e2e/haut.spec.ts b/tests/e2e/haku.spec.ts → tests/e2e/haut.spec.ts
@@ -79,4 +79,16 @@ test('navigates to haku page', async ({ page }) => {
   await expect(page.locator('h1')).toHaveText(
     '> Tampere University Separate Admission/ Finnish MAOL Competition Route 2024',
   );
+  await expect(page.locator('.organizationLabel')).toHaveCount(3);
+});
+
+test('navigates to haku page with no hakukohde', async ({ page }) => {
+  await page.locator('tbody tr:first-child td:first-child a').click();
+  await expect(page).toHaveURL(
+    '/valintojen-toteuttaminen/haku/1.2.246.562.29.00000000000000046872',
+  );
+  await expect(page.locator('h1')).toHaveText(
+    '> Hausjärven lukio jatkuva haku',
+  );
+  await expect(page.locator('.organizationLabel')).toHaveCount(0);
 });
diff --git a/tests/e2e/playwright.setup.js b/tests/e2e/playwright.setup.js
@@ -26,7 +26,10 @@ export default async function playwrightSetup() {
       response.end();
       return;
     } else if (request.url.includes('kouta-internal/hakukohde/search')) {
-      response.write(JSON.stringify(HAKUKOHTEET));
+      const hakuId = request.url.split('&haku=')[1];
+      response.write(
+        JSON.stringify(HAKUKOHTEET.filter((hk) => hk.hakuOid === hakuId)),
+      );
       response.end();
       return;
     } else if (request.url.includes(`kouta-internal/haku/`)) {
Original file line number	Diff line number	Diff line change
		@@ -1 +1 @@
		VIRKAILIJA_URL=https://virkailija.testiopintopolku.fi
		VIRKAILIJA_URL=https://virkailija.untuvaopintopolku.fi