feat: Automaatio aineistojen kategorisoinnin lisäykseen (#536)

* siivoa lintterin huomautukset * jonkinlaista kategorisointia * päivitä hakutunnisteita * korjaa kategorisointia määrityksen mukaisiksi * wip * wip * päivitä yva regexpiä * korjaa testit * lisää testausta muokatulle aineistoKategoriat toiminnallisuudelle * muokkaa virheelliset AineistoKategoriat termit Toimeksiannoiksi kuten ne on Velhossakin * korvaa ylakategoroiden aineistojen lisääminen yksittäisellä painikkeella * lisää testejä aineistokategorisoinnille * estä seuraaville kuulutuksen tiedot -sivuille meno ennen kuin aineisto on kunnossa * siirrä kuulutus-sivut omalle polulleen vaiheen alle; päivitä bäkkäritestit * korjaa PaatosIndexPage muokkausTila tarkistusta ja routerointia * päivitä integration testit * lisää testejä backendiin * update ingtegration test * lisää testausta hyväksymispäätösvaiheen validoinnille * korjaa suunnitelmaluonnoksien lisäämistoiminnallisuus
finnishtransportagency · Jan 25, 2023 · 9919ffb · 9919ffb
1 parent f10380f
commit 9919ffb
Show file tree

Hide file tree

Showing 60 changed files with 2,216 additions and 843 deletions.
diff --git a/backend/integrationtest/api/__snapshots__/api.test.ts.snap b/backend/integrationtest/api/__snapshots__/api.test.ts.snap
diff --git a/backend/integrationtest/api/__snapshots__/jatkopaatos.test.ts.snap b/backend/integrationtest/api/__snapshots__/jatkopaatos.test.ts.snap
@@ -9,10 +9,10 @@ Object {
       "aineistoNahtavilla": Array [
         Object {
           "__typename": "Aineisto",
-          "dokumenttiOid": "1.2.246.578.5.100.2160495081.1282714556",
+          "dokumenttiOid": "1.2.246.578.5.100.2147637429.4251089044",
           "jarjestys": 1,
-          "kategoriaId": "T1xx",
-          "nimi": "T113 TS Esite.txt",
+          "kategoriaId": "osa_a",
+          "nimi": "TYHJÄ.txt",
           "tila": "ODOTTAA_TUONTIA",
         },
       ],
@@ -107,8 +107,8 @@ Object {
 exports[`Jatkopäätökset should go through jatkopäätös1, epäaktiivinen, jatkopäätös2, and epäaktiivinen states successfully 2`] = `
 Object {
   "yllapito S3 files jatkopäätös1 created": Object {
-    "/1/T113 TS Esite.txt": FileMetadata {
-      "ContentDisposition": "inline; filename*=UTF-8''T113%20TS%20Esite.txt",
+    "/1/TYHJÄ.txt": FileMetadata {
+      "ContentDisposition": "inline; filename*=UTF-8''TYHJ%C3%84.txt",
       "ContentType": "text/plain",
       "checksum": "ABC123",
     },
@@ -130,11 +130,11 @@ Object {
       "aineistoNahtavilla": Array [
         Object {
           "__typename": "Aineisto",
-          "dokumenttiOid": "1.2.246.578.5.100.2160495081.1282714556",
+          "dokumenttiOid": "1.2.246.578.5.100.2147637429.4251089044",
           "jarjestys": 1,
-          "kategoriaId": "T1xx",
-          "nimi": "T113 TS Esite.txt",
-          "tiedosto": "/yllapito/tiedostot/projekti/1.2.246.578.5.1.2978288874.2711575506/jatkopaatos1/1/T113 TS Esite.txt",
+          "kategoriaId": "osa_a",
+          "nimi": "TYHJÄ.txt",
+          "tiedosto": "/yllapito/tiedostot/projekti/1.2.246.578.5.1.2978288874.2711575506/jatkopaatos1/1/TYHJÄ.txt",
           "tila": "VALMIS",
           "tuotu": "***unittest***",
         },
@@ -215,11 +215,11 @@ Object {
       "aineistoNahtavilla": Array [
         Object {
           "__typename": "Aineisto",
-          "dokumenttiOid": "1.2.246.578.5.100.2160495081.1282714556",
+          "dokumenttiOid": "1.2.246.578.5.100.2147637429.4251089044",
           "jarjestys": 1,
-          "kategoriaId": "T1xx",
-          "nimi": "T113 TS Esite.txt",
-          "tiedosto": "/yllapito/tiedostot/projekti/1.2.246.578.5.1.2978288874.2711575506/jatkopaatos1/1/T113 TS Esite.txt",
+          "kategoriaId": "osa_a",
+          "nimi": "TYHJÄ.txt",
+          "tiedosto": "/yllapito/tiedostot/projekti/1.2.246.578.5.1.2978288874.2711575506/jatkopaatos1/1/TYHJÄ.txt",
           "tila": "VALMIS",
           "tuotu": "***unittest***",
         },
@@ -348,11 +348,11 @@ Object {
     "aineistoNahtavilla": Array [
       Object {
         "__typename": "Aineisto",
-        "dokumenttiOid": "1.2.246.578.5.100.2160495081.1282714556",
+        "dokumenttiOid": "1.2.246.578.5.100.2147637429.4251089044",
         "jarjestys": 1,
-        "kategoriaId": "T1xx",
-        "nimi": "T113 TS Esite.txt",
-        "tiedosto": "/tiedostot/suunnitelma/1.2.246.578.5.1.2978288874.2711575506/jatkopaatos1/T113 TS Esite.txt",
+        "kategoriaId": "osa_a",
+        "nimi": "TYHJÄ.txt",
+        "tiedosto": "/tiedostot/suunnitelma/1.2.246.578.5.1.2978288874.2711575506/jatkopaatos1/TYHJÄ.txt",
         "tuotu": "***unittest***",
       },
     ],
@@ -453,10 +453,10 @@ Object {
       "aineistoNahtavilla": Array [
         Object {
           "__typename": "Aineisto",
-          "dokumenttiOid": "1.2.246.578.5.100.2160495081.1282714556",
+          "dokumenttiOid": "1.2.246.578.5.100.2147637429.4251089044",
           "jarjestys": 1,
-          "kategoriaId": "T1xx",
-          "nimi": "T113 TS Esite.txt",
+          "kategoriaId": "osa_a",
+          "nimi": "TYHJÄ.txt",
           "tila": "ODOTTAA_TUONTIA",
         },
       ],
@@ -555,8 +555,8 @@ Object {
 exports[`Jatkopäätökset should go through jatkopäätös1, epäaktiivinen, jatkopäätös2, and epäaktiivinen states successfully 7`] = `
 Object {
   "yllapito S3 files jatkopäätös2 created": Object {
-    "/1/T113 TS Esite.txt": FileMetadata {
-      "ContentDisposition": "inline; filename*=UTF-8''T113%20TS%20Esite.txt",
+    "/1/TYHJÄ.txt": FileMetadata {
+      "ContentDisposition": "inline; filename*=UTF-8''TYHJ%C3%84.txt",
       "ContentType": "text/plain",
       "checksum": "ABC123",
     },
@@ -578,11 +578,11 @@ Object {
       "aineistoNahtavilla": Array [
         Object {
           "__typename": "Aineisto",
-          "dokumenttiOid": "1.2.246.578.5.100.2160495081.1282714556",
+          "dokumenttiOid": "1.2.246.578.5.100.2147637429.4251089044",
           "jarjestys": 1,
-          "kategoriaId": "T1xx",
-          "nimi": "T113 TS Esite.txt",
-          "tiedosto": "/yllapito/tiedostot/projekti/1.2.246.578.5.1.2978288874.2711575506/jatkopaatos1/1/T113 TS Esite.txt",
+          "kategoriaId": "osa_a",
+          "nimi": "TYHJÄ.txt",
+          "tiedosto": "/yllapito/tiedostot/projekti/1.2.246.578.5.1.2978288874.2711575506/jatkopaatos1/1/TYHJÄ.txt",
           "tila": "VALMIS",
           "tuotu": "***unittest***",
         },
@@ -663,11 +663,11 @@ Object {
       "aineistoNahtavilla": Array [
         Object {
           "__typename": "Aineisto",
-          "dokumenttiOid": "1.2.246.578.5.100.2160495081.1282714556",
+          "dokumenttiOid": "1.2.246.578.5.100.2147637429.4251089044",
           "jarjestys": 1,
-          "kategoriaId": "T1xx",
-          "nimi": "T113 TS Esite.txt",
-          "tiedosto": "/yllapito/tiedostot/projekti/1.2.246.578.5.1.2978288874.2711575506/jatkopaatos2/1/T113 TS Esite.txt",
+          "kategoriaId": "osa_a",
+          "nimi": "TYHJÄ.txt",
+          "tiedosto": "/yllapito/tiedostot/projekti/1.2.246.578.5.1.2978288874.2711575506/jatkopaatos2/1/TYHJÄ.txt",
           "tila": "VALMIS",
           "tuotu": "***unittest***",
         },
@@ -796,11 +796,11 @@ Object {
     "aineistoNahtavilla": Array [
       Object {
         "__typename": "Aineisto",
-        "dokumenttiOid": "1.2.246.578.5.100.2160495081.1282714556",
+        "dokumenttiOid": "1.2.246.578.5.100.2147637429.4251089044",
         "jarjestys": 1,
-        "kategoriaId": "T1xx",
-        "nimi": "T113 TS Esite.txt",
-        "tiedosto": "/tiedostot/suunnitelma/1.2.246.578.5.1.2978288874.2711575506/jatkopaatos2/T113 TS Esite.txt",
+        "kategoriaId": "osa_a",
+        "nimi": "TYHJÄ.txt",
+        "tiedosto": "/tiedostot/suunnitelma/1.2.246.578.5.1.2978288874.2711575506/jatkopaatos2/TYHJÄ.txt",
         "tuotu": "***unittest***",
       },
     ],

diff --git a/backend/integrationtest/api/api.test.ts b/backend/integrationtest/api/api.test.ts
@@ -117,8 +117,8 @@ describe("Api", () => {
 
     projekti = await testSuunnitteluvaihePerustiedot(oid);
     await testSuunnitteluvaiheVuorovaikutus(projekti, projektiPaallikko.kayttajatunnus);
-    const velhoAineistoKategorias = await testListDocumentsToImport(oid); // testaa sitä kun käyttäjä avaa aineistodialogin ja valkkaa sieltä tiedostoja
-    await testImportAineistot(oid, velhoAineistoKategorias); // vastaa sitä kun käyttäjä on valinnut tiedostot ja tallentaa
+    const velhoToimeksiannot = await testListDocumentsToImport(oid); // testaa sitä kun käyttäjä avaa aineistodialogin ja valkkaa sieltä tiedostoja
+    await testImportAineistot(oid, velhoToimeksiannot); // vastaa sitä kun käyttäjä on valinnut tiedostot ja tallentaa
     await importAineistoMock.processQueue();
     await verifyVuorovaikutusSnapshot(oid, userFixture);
 
@@ -129,7 +129,7 @@ describe("Api", () => {
     emailClientStub.verifyEmailsSent();
     await importAineistoMock.processQueue();
     userFixture.loginAs(UserFixture.mattiMeikalainen);
-    await loadProjektiFromDatabase(oid, Status.NAHTAVILLAOLO);
+    await loadProjektiFromDatabase(oid, Status.NAHTAVILLAOLO_AINEISTOT);
     await recordProjektiTestFixture(FixtureName.NAHTAVILLAOLO, oid);
     // TODO: test päivitä suunnitteluvaiheen perustietoja
     // TODO: test päivitä vuorovaikutustilaisuuksia
@@ -143,7 +143,7 @@ describe("Api", () => {
 
     userFixture.loginAs(UserFixture.mattiMeikalainen);
     projekti = await testNahtavillaolo(oid, projektiPaallikko.kayttajatunnus);
-    const nahtavillaoloVaihe = await testImportNahtavillaoloAineistot(projekti, velhoAineistoKategorias);
+    const nahtavillaoloVaihe = await testImportNahtavillaoloAineistot(projekti, velhoToimeksiannot);
     await importAineistoMock.processQueue();
     await testNahtavillaoloLisaAineisto(oid, nahtavillaoloVaihe.lisaAineistoParametrit!);
     await testNahtavillaoloApproval(oid, projektiPaallikko, userFixture);
@@ -156,7 +156,7 @@ describe("Api", () => {
     await testCreateHyvaksymisPaatosWithAineistot(
       oid,
       "hyvaksymisPaatosVaihe",
-      velhoAineistoKategorias,
+      velhoToimeksiannot,
       projektiPaallikko.kayttajatunnus,
       Status.HYVAKSYTTY
     );

diff --git a/backend/integrationtest/api/apiTestFixture.ts b/backend/integrationtest/api/apiTestFixture.ts
@@ -230,6 +230,10 @@ class ApiTestFixture {
     ],
   });
 
+  nahtavillaoloVaiheAineisto = (): NahtavillaoloVaiheInput => ({
+    aineistoNahtavilla: [{ dokumenttiOid: "123", nimi: "tiedosto.pdf", kategoriaId: "osa_a" }],
+  });
+
   nahtavillaoloVaihe = (kuulutusYhteysHenkilot: string[]): NahtavillaoloVaiheInput => ({
     hankkeenKuvaus: {
       SUOMI: "Lorem Ipsum nahtavillaoloVaihe",

diff --git a/backend/integrationtest/api/hyvaksymisPaatosHyvaksytty.test.ts b/backend/integrationtest/api/hyvaksymisPaatosHyvaksytty.test.ts
@@ -96,7 +96,7 @@ describe("Hyväksytyn hyväksymispäätöskuulutuksen jälkeen", () => {
     // Tallennuksen tulos:
     // * jatkopäätöksen tallennus resetoi projektin henkilöt TODO: suunnittelusopimuksen henkilöviittaus pitäisi toteuttaa pois, jotta ko. henkilön voi poistaa
     // * Projekti on jatkopäätösvaiheessa
-    let jatkopaatosProjekti = await expectYllapitoProjektiStatus(Status.JATKOPAATOS_1);
+    let jatkopaatosProjekti = await expectYllapitoProjektiStatus(Status.JATKOPAATOS_1_AINEISTOT);
     jatkopaatosProjekti.paivitetty = "***unit test***";
     expectToMatchSnapshot("jatkopaatosProjekti käyttöoikeudet resetoinnin jälkeen", jatkopaatosProjekti.kayttoOikeudet);
     await expectJulkinenNotFound(oid, userFixture);

diff --git a/backend/integrationtest/api/jatkopaatos.test.ts b/backend/integrationtest/api/jatkopaatos.test.ts
@@ -66,11 +66,11 @@ describe("Jatkopäätökset", () => {
 
   async function addJatkopaatos1WithAineistot() {
     // Lisää aineistot
-    const velhoAineistoKategorias = await api.listaaVelhoProjektiAineistot(oid);
+    const velhoToimeksiannot = await api.listaaVelhoProjektiAineistot(oid);
     await testCreateHyvaksymisPaatosWithAineistot(
       oid,
       "jatkoPaatos1Vaihe",
-      velhoAineistoKategorias,
+      velhoToimeksiannot,
       UserFixture.mattiMeikalainen.uid!,
       Status.JATKOPAATOS_1
     );
@@ -80,11 +80,11 @@ describe("Jatkopäätökset", () => {
 
   async function addJatkopaatos2WithAineistot() {
     // Lisää aineistot
-    const velhoAineistoKategorias = await api.listaaVelhoProjektiAineistot(oid);
+    const velhoToimeksiannot = await api.listaaVelhoProjektiAineistot(oid);
     await testCreateHyvaksymisPaatosWithAineistot(
       oid,
       "jatkoPaatos2Vaihe",
-      velhoAineistoKategorias,
+      velhoToimeksiannot,
       UserFixture.mattiMeikalainen.uid!,
       Status.JATKOPAATOS_2
     );
@@ -94,7 +94,7 @@ describe("Jatkopäätökset", () => {
 
   it("should go through jatkopäätös1, epäaktiivinen, jatkopäätös2, and epäaktiivinen states successfully", async () => {
     userFixture.loginAs(UserFixture.projari112);
-    let projekti = await loadProjektiFromDatabase(oid, Status.JATKOPAATOS_1);
+    let projekti = await loadProjektiFromDatabase(oid, Status.JATKOPAATOS_1_AINEISTOT);
     const projektiPaallikko = projekti.kayttoOikeudet?.filter((user) => user.tyyppi == KayttajaTyyppi.PROJEKTIPAALLIKKO).pop()!;
 
     await addJatkopaatos1WithAineistot();