OK-356 Korjattu lähetysten käyttöoikeuksien lisäys

Opetushallitus · Jan 29, 2024 · 7d00577 · 7d00577
1 parent a65761a
commit 7d00577
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/shared/src/main/scala/fi/oph/viestinvalitys/business/KantaOperaatiot.scala b/shared/src/main/scala/fi/oph/viestinvalitys/business/KantaOperaatiot.scala
@@ -255,7 +255,7 @@ class KantaOperaatiot(db: JdbcBackend.JdbcDatabaseDef) {
       DBIO.sequence(kayttooikeusRajoitukset.map(kayttooikeus => {
         sqlu"""
               WITH oikeudet AS (
-                INSERT INTO kayttooikeudet (kayttooikeus) VALUES(${kayttooikeus}) ON CONFLICT (kayttooikeus) DO NOTHING RETURNING tunniste
+                INSERT INTO kayttooikeudet (kayttooikeus) VALUES(${kayttooikeus}) ON CONFLICT (kayttooikeus) DO UPDATE SET kayttooikeus=${kayttooikeus} RETURNING tunniste
               ), viestit AS (
                 INSERT INTO viestit_kayttooikeudet SELECT ${viestiTunniste.toString}::uuid, tunniste FROM oikeudet
               )

diff --git a/shared/src/test/scala/fi/oph/viestinvalitys/business/KantaOperaatiotTest.scala b/shared/src/test/scala/fi/oph/viestinvalitys/business/KantaOperaatiotTest.scala
@@ -304,12 +304,12 @@ class KantaOperaatiotTest {
    */
   @Test def testGetViestinLahetyksenKayttooikeudet(): Unit =
     // tallennetaan viestit oikeuksilla (jolloin luodaan lähetys johon oikeudet tallennetaan)
-    val (viesti1, vastaanottajat1) = tallennaViesti(1, lahetysTunniste = null, kayttoOikeudet = Set("ROLE_JARJESTELMA_OIKEUS1"))
+    val (viesti1, vastaanottajat1) = tallennaViesti(1, lahetysTunniste = null, kayttoOikeudet = Set("ROLE_JARJESTELMA_OIKEUS1","ROLE_JARJESTELMA_OIKEUS2"))
     val (viesti2, vastaanottajat2) = tallennaViesti(1, lahetysTunniste = null, kayttoOikeudet = Set("ROLE_JARJESTELMA_OIKEUS2"))
 
     // luetut käyttöoikeudet vastaavat tallennettuja
     Assertions.assertEquals(
-      Seq(viesti1.lahetys_tunniste -> Set("ROLE_JARJESTELMA_OIKEUS1"), viesti2.lahetys_tunniste -> Set("ROLE_JARJESTELMA_OIKEUS2")).toMap,
+      Seq(viesti1.lahetys_tunniste -> Set("ROLE_JARJESTELMA_OIKEUS1","ROLE_JARJESTELMA_OIKEUS2"), viesti2.lahetys_tunniste -> Set("ROLE_JARJESTELMA_OIKEUS2")).toMap,
       kantaOperaatiot.getLahetystenKayttooikeudet(Seq(viesti1.lahetys_tunniste, viesti2.lahetys_tunniste)))
 
   /**