feat: varahenkilön lisäys, roolien käsittelyn muutos vaatimuksia vast…

…aavaksi
finnishtransportagency · Sep 27, 2022 · 9eee9b1 · 9eee9b1
1 parent ccd314b
commit 9eee9b1
Show file tree

Hide file tree

Showing 55 changed files with 830 additions and 546 deletions.
diff --git a/backend/integrationtest/api/__snapshots__/api.test.ts.snap b/backend/integrationtest/api/__snapshots__/api.test.ts.snap
@@ -1297,12 +1297,28 @@ Object {
       },
       Object {
         "__typename": "ProjektiKayttajaJulkinen",
-        "email": "mikko.haapamaki@cgi.com",
+        "email": "mikko.haapamaki02@cgi.com",
         "id": "1",
+        "nimi": "Hassu, A-tunnus3",
+        "organisaatio": "CGI Suomi Oy",
+        "puhelinnumero": "123",
+      },
+      Object {
+        "__typename": "ProjektiKayttajaJulkinen",
+        "email": "mikko.haapamaki@cgi.com",
+        "id": "2",
         "nimi": "Hassu, A-Tunnus",
         "organisaatio": "CGI Suomi Oy",
         "puhelinnumero": "123",
       },
+      Object {
+        "__typename": "ProjektiKayttajaJulkinen",
+        "email": "mikko.haapamaki@cgi.com",
+        "id": "3",
+        "nimi": "Hassu, Testi1",
+        "organisaatio": "CGI Suomi Oy",
+        "puhelinnumero": "123",
+      },
     ],
     "status": "ALOITUSKUULUTUS",
     "velho": Object {
@@ -2544,6 +2560,7 @@ Object {
           "nimi": "HASSU AUTOMAATTITESTIPROJEKTI1",
           "suunnittelustaVastaavaViranomainen": "VAYLAVIRASTO",
           "tyyppi": "TIE",
+          "varahenkilonEmail": "mikko.haapamaki02@cgi.com",
           "vastuuhenkilonEmail": "mikko.haapamki@cgi.com",
           "vaylamuoto": Array [
             "tie",
@@ -3144,6 +3161,7 @@ Object {
           "nimi": "HASSU AUTOMAATTITESTIPROJEKTI1",
           "suunnittelustaVastaavaViranomainen": "VAYLAVIRASTO",
           "tyyppi": "TIE",
+          "varahenkilonEmail": "mikko.haapamaki02@cgi.com",
           "vastuuhenkilonEmail": "mikko.haapamki@cgi.com",
           "vaylamuoto": Array [
             "tie",

diff --git a/backend/integrationtest/api/api.test.ts b/backend/integrationtest/api/api.test.ts
@@ -113,7 +113,7 @@ describe("Api", () => {
     const projekti = await readProjektiFromVelho();
     expect(oid).to.eq(projekti.oid);
     await cleanProjektiS3Files(oid);
-    const projektiPaallikko = await testProjektiHenkilot(projekti, oid);
+    const projektiPaallikko = await testProjektiHenkilot(projekti, oid, userFixture);
     await testProjektinTiedot(oid);
     await testAloitusKuulutusEsikatselu(oid);
     await testNullifyProjektiField(oid);

diff --git a/backend/integrationtest/api/records/HYVAKSYMISPAATOS_APPROVED.json b/backend/integrationtest/api/records/HYVAKSYMISPAATOS_APPROVED.json
@@ -21,6 +21,7 @@
         "maakunnat": [
           "Uusimaa"
         ],
+        "varahenkilonEmail": "mikko.haapamaki02@cgi.com",
         "kunnat": [
           "Helsinki",
           " Vantaa"
@@ -115,6 +116,7 @@
     "maakunnat": [
       "Uusimaa"
     ],
+    "varahenkilonEmail": "mikko.haapamaki02@cgi.com",
     "kunnat": [
       "Helsinki",
       " Vantaa"
@@ -142,6 +144,7 @@
         "maakunnat": [
           "Uusimaa"
         ],
+        "varahenkilonEmail": "mikko.haapamaki02@cgi.com",
         "kunnat": [
           "Helsinki",
           " Vantaa"
@@ -495,18 +498,37 @@
     {
       "kayttajatunnus": "A000112",
       "nimi": "Hassu, A-tunnus1",
-      "rooli": "PROJEKTIPAALLIKKO",
       "organisaatio": "CGI Suomi Oy",
+      "tyyppi": "PROJEKTIPAALLIKKO",
       "puhelinnumero": "123",
-      "email": "mikko.haapamki@cgi.com"
+      "email": "mikko.haapamki@cgi.com",
+      "muokattavissa": false
+    },
+    {
+      "kayttajatunnus": "A000114",
+      "nimi": "Hassu, A-tunnus3",
+      "organisaatio": "CGI Suomi Oy",
+      "tyyppi": "VARAHENKILO",
+      "puhelinnumero": "123",
+      "email": "mikko.haapamaki02@cgi.com",
+      "muokattavissa": false
     },
     {
       "kayttajatunnus": "A000111",
       "nimi": "Hassu, A-Tunnus",
-      "rooli": "OMISTAJA",
       "organisaatio": "CGI Suomi Oy",
+      "__typename": "ProjektiKayttaja",
       "puhelinnumero": "123",
-      "email": "mikko.haapamaki@cgi.com"
+      "email": "mikko.haapamaki@cgi.com",
+      "muokattavissa": true
+    },
+    {
+      "kayttajatunnus": "LX581241",
+      "nimi": "Hassu, Testi1",
+      "puhelinnumero": "123",
+      "organisaatio": "CGI Suomi Oy",
+      "email": "mikko.haapamaki@cgi.com",
+      "muokattavissa": true
     }
   ],
   "kasittelynTila": {
@@ -582,6 +604,7 @@
         "maakunnat": [
           "Uusimaa"
         ],
+        "varahenkilonEmail": "mikko.haapamaki02@cgi.com",
         "kunnat": [
           "Helsinki",
           " Vantaa"

diff --git a/backend/integrationtest/api/records/NAHTAVILLAOLO.json b/backend/integrationtest/api/records/NAHTAVILLAOLO.json
@@ -21,6 +21,7 @@
         "maakunnat": [
           "Uusimaa"
         ],
+        "varahenkilonEmail": "mikko.haapamaki02@cgi.com",
         "kunnat": [
           "Helsinki",
           " Vantaa"
@@ -115,6 +116,7 @@
     "maakunnat": [
       "Uusimaa"
     ],
+    "varahenkilonEmail": "mikko.haapamaki02@cgi.com",
     "kunnat": [
       "Helsinki",
       " Vantaa"
@@ -148,18 +150,37 @@
     {
       "kayttajatunnus": "A000112",
       "nimi": "Hassu, A-tunnus1",
-      "rooli": "PROJEKTIPAALLIKKO",
       "organisaatio": "CGI Suomi Oy",
+      "tyyppi": "PROJEKTIPAALLIKKO",
       "puhelinnumero": "123",
-      "email": "mikko.haapamki@cgi.com"
+      "email": "mikko.haapamki@cgi.com",
+      "muokattavissa": false
+    },
+    {
+      "kayttajatunnus": "A000114",
+      "nimi": "Hassu, A-tunnus3",
+      "organisaatio": "CGI Suomi Oy",
+      "tyyppi": "VARAHENKILO",
+      "puhelinnumero": "123",
+      "email": "mikko.haapamaki02@cgi.com",
+      "muokattavissa": false
     },
     {
       "kayttajatunnus": "A000111",
       "nimi": "Hassu, A-Tunnus",
-      "rooli": "OMISTAJA",
       "organisaatio": "CGI Suomi Oy",
+      "__typename": "ProjektiKayttaja",
       "puhelinnumero": "123",
-      "email": "mikko.haapamaki@cgi.com"
+      "email": "mikko.haapamaki@cgi.com",
+      "muokattavissa": true
+    },
+    {
+      "kayttajatunnus": "LX581241",
+      "nimi": "Hassu, Testi1",
+      "puhelinnumero": "123",
+      "organisaatio": "CGI Suomi Oy",
+      "email": "mikko.haapamaki@cgi.com",
+      "muokattavissa": true
     }
   ],
   "aloitusKuulutus": {

diff --git a/backend/integrationtest/api/testUtil/tests.ts b/backend/integrationtest/api/testUtil/tests.ts
@@ -1,12 +1,12 @@
 import {
   AineistoInput,
   AsiakirjaTyyppi,
+  KayttajaTyyppi,
   Kieli,
   Projekti,
   ProjektiJulkinen,
   ProjektiKayttaja,
   ProjektiKayttajaInput,
-  ProjektiRooli,
   Status,
   TilasiirtymaToiminto,
   TilasiirtymaTyyppi,
@@ -60,44 +60,66 @@ export async function loadProjektiJulkinenFromDatabase(oid: string, expectedStat
   return savedProjekti;
 }
 
-export async function testProjektiHenkilot(projekti: Projekti, oid: string): Promise<ProjektiKayttaja> {
+export async function testProjektiHenkilot(projekti: Projekti, oid: string, userFixture: UserFixture): Promise<ProjektiKayttaja> {
   await api.tallennaProjekti({
     oid,
-    kayttoOikeudet: projekti.kayttoOikeudet?.map(
-      (value) =>
-        // eslint-disable-next-line @typescript-eslint/consistent-type-assertions
-        ({
-          rooli: value.rooli,
-          kayttajatunnus: value.kayttajatunnus,
-          // Emulate migration where the phone number may be empty
-        } as ProjektiKayttajaInput)
-    ),
+    kayttoOikeudet: projekti.kayttoOikeudet?.map((value) => {
+      const input: ProjektiKayttajaInput = {
+        kayttajatunnus: value.kayttajatunnus,
+        // Emulate migration where the phone number may be empty
+        puhelinnumero: undefined,
+      };
+      return input;
+    }),
   });
   const p = await loadProjektiFromDatabase(oid, Status.EI_JULKAISTU_PROJEKTIN_HENKILOT);
 
   // Expect that projektipaallikko is found
   const projektiPaallikko = p.kayttoOikeudet
-    ?.filter((kayttaja) => kayttaja.rooli === ProjektiRooli.PROJEKTIPAALLIKKO && kayttaja.email)
+    ?.filter((kayttaja) => kayttaja.tyyppi === KayttajaTyyppi.PROJEKTIPAALLIKKO && kayttaja.email && kayttaja.muokattavissa === false)
     .pop();
   expect(projektiPaallikko).is.not.empty;
 
-  const kayttoOikeudet = p.kayttoOikeudet?.map((value) => ({
-    rooli: value.rooli,
-    kayttajatunnus: value.kayttajatunnus,
+  // Expect that varahenkilo from Velho is found
+  const varahenkilo = p.kayttoOikeudet
+    ?.filter((kayttaja) => kayttaja.tyyppi == KayttajaTyyppi.VARAHENKILO && kayttaja.muokattavissa === false)
+    .pop();
+  expect(varahenkilo).is.not.empty;
+
+  const kayttoOikeudet: ProjektiKayttajaInput[] = p.kayttoOikeudet?.map((value) => ({
+    ...value,
     puhelinnumero: "123",
   }));
 
+  kayttoOikeudet.push({ kayttajatunnus: UserFixture.testi1Kayttaja.uid, puhelinnumero: "123" });
+
   // Save and load projekti
   await api.tallennaProjekti({
     oid,
     kayttoOikeudet,
   });
   await loadProjektiFromDatabase(oid, Status.EI_JULKAISTU);
 
+  // Verify only omistaja can modify varahenkilo-field
+  try {
+    userFixture.loginAs(UserFixture.testi1Kayttaja);
+    const kayttoOikeudetWithVarahenkiloChanges = cloneDeep(kayttoOikeudet);
+    kayttoOikeudetWithVarahenkiloChanges
+      .filter((user) => user.kayttajatunnus == UserFixture.testi1Kayttaja.uid)
+      .forEach((user) => (user.tyyppi = KayttajaTyyppi.VARAHENKILO));
+    await api.tallennaProjekti({
+      oid,
+      kayttoOikeudet: kayttoOikeudetWithVarahenkiloChanges,
+    });
+    fail("Vain omistajan pitää pystyä muokkaamaan varahenkilöyttä");
+  } catch (e) {
+    expect(e.className).to.eq("IllegalAccessError");
+  }
+
   return { ...projektiPaallikko, puhelinnumero: "123" };
 }
 
-export async function tallennaLogo() {
+export async function tallennaLogo(): Promise<string> {
   const uploadProperties = await api.valmisteleTiedostonLataus("logo.png", "image/png");
   expect(uploadProperties).to.not.be.empty;
   expect(uploadProperties.latausLinkki).to.not.be.undefined;

diff --git a/backend/integrationtest/personSearch/personSearchClient.test.ts b/backend/integrationtest/personSearch/personSearchClient.test.ts
@@ -9,7 +9,6 @@ import log from "loglevel";
 import sinon from "sinon";
 
 export function expectNotEmptyKayttaja(kayttaja: Kayttaja): void {
-  expect(kayttaja.vaylaKayttajaTyyppi).to.not.be.empty;
   expect(kayttaja.etuNimi).to.not.be.empty;
   expect(kayttaja.sukuNimi).to.not.be.empty;
   expect(kayttaja.uid).to.not.be.empty;

diff --git a/backend/src/asiakirja/suunnittelunAloitus/Kutsu20.ts b/backend/src/asiakirja/suunnittelunAloitus/Kutsu20.ts
@@ -1,5 +1,5 @@
 import { DBProjekti, DBVaylaUser, SuunnitteluSopimus, SuunnitteluVaihe, Vuorovaikutus, VuorovaikutusTilaisuus } from "../../database/model";
-import { Kieli, ProjektiTyyppi, VuorovaikutusTilaisuusTyyppi } from "../../../../common/graphql/apiModel";
+import { KayttajaTyyppi, Kieli, ProjektiTyyppi, VuorovaikutusTilaisuusTyyppi } from "../../../../common/graphql/apiModel";
 import { formatProperNoun } from "../../../../common/util/formatProperNoun";
 import dayjs from "dayjs";
 import { linkSuunnitteluVaihe } from "../../../../common/links";
@@ -300,7 +300,7 @@ export class Kutsu20 extends CommonPdf {
             tilaisuus.esitettavatYhteystiedot?.yhteysHenkilot.forEach((kayttajatunnus) => {
               const user = this.kayttoOikeudet.filter((kayttaja) => kayttaja.kayttajatunnus == kayttajatunnus).pop();
               if (user) {
-                const role = translate("rooli." + user.rooli, this.kieli);
+                const role = user.tyyppi == KayttajaTyyppi.PROJEKTIPAALLIKKO ?  translate("rooli.PROJEKTIPAALLIKKO", this.kieli) : undefined;
                 this.doc.text(safeConcatStrings(", ", [user.nimi, role, user.puhelinnumero]));
               }
             });

diff --git a/backend/src/asiakirja/suunnittelunAloitus/KutsuAdapter.ts b/backend/src/asiakirja/suunnittelunAloitus/KutsuAdapter.ts
@@ -1,4 +1,4 @@
-import { Kieli, ProjektiRooli, ProjektiTyyppi, Viranomainen } from "../../../../common/graphql/apiModel";
+import { KayttajaTyyppi, Kieli, ProjektiTyyppi, Viranomainen } from "../../../../common/graphql/apiModel";
 import { DBVaylaUser, Kielitiedot, SuunnitteluSopimus, Velho, Vuorovaikutus, Yhteystieto } from "../../database/model";
 import { AsiakirjanMuoto } from "../asiakirjaService";
 import { translate } from "../../util/localization";
@@ -226,7 +226,7 @@ export class KutsuAdapter {
         throw new Error("BUG: Kayttöoikeudet pitää antaa jos yhteyshenkilöt on annettu.");
       }
       const yhteysHenkilotWithProjectManager = union(
-        this.kayttoOikeudet.filter((user) => user.rooli == ProjektiRooli.PROJEKTIPAALLIKKO).map((user) => user.kayttajatunnus),
+        this.kayttoOikeudet.filter((user) => user.tyyppi == KayttajaTyyppi.PROJEKTIPAALLIKKO).map((user) => user.kayttajatunnus),
         yhteysHenkilot
       );
       this.getUsersForUsernames(yhteysHenkilotWithProjectManager).forEach((user) => {
@@ -288,7 +288,7 @@ export class KutsuAdapter {
     return usernames
       ?.map((kayttajatunnus) =>
         this.kayttoOikeudet
-          .filter((kayttaja) => kayttaja.kayttajatunnus == kayttajatunnus || kayttaja.rooli == ProjektiRooli.PROJEKTIPAALLIKKO)
+          .filter((kayttaja) => kayttaja.kayttajatunnus == kayttajatunnus || kayttaja.tyyppi == KayttajaTyyppi.PROJEKTIPAALLIKKO)
           .pop()
       )
       .filter((o) => o);

diff --git a/backend/src/database/model/projekti.ts b/backend/src/database/model/projekti.ts
@@ -1,17 +1,18 @@
-import { AloitusKuulutusTila, Kieli, ProjektiRooli, ProjektiTyyppi, Viranomainen } from "../../../../common/graphql/apiModel";
+import { AloitusKuulutusTila, KayttajaTyyppi, Kieli, ProjektiTyyppi, Viranomainen } from "../../../../common/graphql/apiModel";
 import { SuunnitteluVaihe, Vuorovaikutus } from "./suunnitteluVaihe";
 import { NahtavillaoloVaihe, NahtavillaoloVaiheJulkaisu } from "./nahtavillaoloVaihe";
 import { HyvaksymisPaatosVaihe, HyvaksymisPaatosVaiheJulkaisu } from "./hyvaksymisPaatosVaihe";
-import { StandardiYhteystiedot, LocalizedMap, Yhteystieto, IlmoituksenVastaanottajat } from "./common";
+import { IlmoituksenVastaanottajat, LocalizedMap, StandardiYhteystiedot, Yhteystieto } from "./common";
 
 export type DBVaylaUser = {
-  rooli: ProjektiRooli;
   email: string;
   kayttajatunnus: string;
 
-  puhelinnumero: string;
+  puhelinnumero?: string;
   organisaatio: string;
   nimi: string;
+  tyyppi?: KayttajaTyyppi;
+  muokattavissa?: boolean;
 };
 
 export type AloitusKuulutus = {
@@ -87,8 +88,8 @@ export type Velho = {
   toteuttavaOrganisaatio?: string | null;
   vastuuhenkilonNimi?: string | null;
   vastuuhenkilonEmail?: string | null;
-  varahenkiloNimi?: string | null;
-  varahenkiloEmail?: string | null;
+  varahenkilonNimi?: string | null;
+  varahenkilonEmail?: string | null;
   maakunnat?: string[] | null;
   kunnat?: string[] | null;
   linkki?: string | null;

diff --git a/backend/src/email/lahetekirje/LahetekirjeAdapter.ts b/backend/src/email/lahetekirje/LahetekirjeAdapter.ts
@@ -1,5 +1,5 @@
 import log from "loglevel";
-import { Kieli, ProjektiRooli, ProjektiTyyppi, Viranomainen } from "../../../../common/graphql/apiModel";
+import { KayttajaTyyppi, Kieli, ProjektiTyyppi, Viranomainen } from "../../../../common/graphql/apiModel";
 import { AsiakirjanMuoto, determineAsiakirjaMuoto } from "../../asiakirja/asiakirjaService";
 import { DBProjekti, Yhteystieto } from "../../database/model";
 import { translate } from "../../util/localization";
@@ -56,8 +56,7 @@ export class LahetekirjeAdapter {
 
   protected get isElyTilaaja(): boolean {
     return (
-      this.projekti?.velho?.suunnittelustaVastaavaViranomainen &&
-      this.projekti?.velho?.suunnittelustaVastaavaViranomainen.endsWith("ELY")
+      this.projekti?.velho?.suunnittelustaVastaavaViranomainen && this.projekti?.velho?.suunnittelustaVastaavaViranomainen.endsWith("ELY")
     );
   }
 
@@ -195,9 +194,8 @@ export class LahetekirjeAdapter {
 
     kayttoOikeudet
       ?.filter(
-        ({ kayttajatunnus, rooli }) =>
-          rooli === ProjektiRooli.PROJEKTIPAALLIKKO ||
-          kuulutusYhteystiedot?.yhteysHenkilot?.find((yh) => yh === kayttajatunnus)
+        ({ kayttajatunnus, tyyppi }) =>
+          tyyppi === KayttajaTyyppi.PROJEKTIPAALLIKKO || kuulutusYhteystiedot?.yhteysHenkilot?.find((yh) => yh === kayttajatunnus)
       )
       .forEach((oikeus) => {
         yt.push(vaylaUserToYhteystieto(oikeus));