From 6c277e867815db89311f0cebcab69bc4c56a7b94 Mon Sep 17 00:00:00 2001
From: Eric Leroy-Terquem <eric.leroyterquem@beta.gouv.fr>
Date: Tue, 21 Feb 2023 09:51:14 +0100
Subject: [PATCH] fix(groupe instructeur API): move condition to catch error

---
 .../mutations/groupe_instructeur_supprimer_instructeurs.rb  | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/app/graphql/mutations/groupe_instructeur_supprimer_instructeurs.rb b/app/graphql/mutations/groupe_instructeur_supprimer_instructeurs.rb
index 8a45f4719..3dc02a739 100644
--- a/app/graphql/mutations/groupe_instructeur_supprimer_instructeurs.rb
+++ b/app/graphql/mutations/groupe_instructeur_supprimer_instructeurs.rb
@@ -12,10 +12,10 @@ module Mutations
       ids, emails = partition_instructeurs_by(instructeurs)
       instructeurs = groupe_instructeur.instructeurs.find_all_by_identifier(ids:, emails:)
 
-      instructeurs.each { groupe_instructeur.remove(_1) }
-      groupe_instructeur.reload
-
       if instructeurs.present?
+        instructeurs.each { groupe_instructeur.remove(_1) }
+        groupe_instructeur.reload
+
         GroupeInstructeurMailer
           .notify_group_when_instructeurs_removed(groupe_instructeur, instructeurs, current_administrateur.email)
           .deliver_later