Fix

betagouv · Nov 13, 2024 · 4c1a288 · 4c1a288
1 parent f270dfa
commit 4c1a288
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 5 deletions.
diff --git a/app/controllers/users/omniauth_callbacks_controller.rb b/app/controllers/users/omniauth_callbacks_controller.rb
@@ -127,12 +127,9 @@ def save_role(establishment, role)
     end
 
     def delete_roles!
-      # Supprime les accès qui ne sont plus présents dans KeyCloak.
-      combined_establishments = @mapper.establishments_delegated + @mapper.establishments_in_responsibility
-
       # Ne garde que les élements distincts entre les rôles d'APLyPro et ceux de KeyCloak.
       EstablishmentUserRole.where(user: @user).find_each do |access|
-        access.destroy unless combined_establishments.include?(access.establishment)
+        access.destroy unless @mapper.establishments_in_responsibility_and_delegated.include?(access.establishment)
       end
     end
 

diff --git a/features/premiere_connexion.feature b/features/premiere_connexion.feature
@@ -138,7 +138,8 @@ Fonctionnalité: Le personnel de direction se connecte
     Et que je sélectionne "ETAB2" pour "Établissement"
     Et que je clique sur "Continuez avec cet établissement"
     Et que je passe l'écran d'accueil
-    Alors la page contient "Changer d'établissement"
+    Et que je clique sur "Changer d'établissement"
+    Alors la page contient "ETAB1"
     Et la page contient "ETAB2"
     Quand je me déconnecte
     Sachant que je suis un personnel MENJ directeur de l'établissement "ETAB1" avec l'email "[email protected]"