toggle routing after groupe instructeur update

app/models/groupe_instructeur.rb

@@ -23,18 +23,12 @@ class GroupeInstructeur < ApplicationRecord
   validates :closed, acceptance: { accept: [false], message: "Modification impossible : il doit y avoir au moins un groupe instructeur actif sur chaque procédure" }, if: -> { self.procedure.groupe_instructeurs.actif.one? }
 
   before_validation -> { label&.strip! }
-  after_create :toggle_routing
+  after_save :toggle_routing
 
   scope :without_group, -> (group) { where.not(id: group) }
   scope :for_api_v2, -> { includes(procedure: [:administrateurs]) }
   scope :actif, -> { where(closed: false) }
 
-  def toggle_routing
-    procedure = self.procedure.reload
-    routing_enabled = procedure.groupe_instructeurs.actif.count > 1
-    procedure.update!(routing_enabled: routing_enabled)
-  end
-
   def add(instructeur)
     return if in?(instructeur.groupe_instructeurs)
 
@@ -55,4 +49,10 @@ class GroupeInstructeur < ApplicationRecord
   def can_delete?
     dossiers.empty? && (procedure.groupe_instructeurs.actif.many? || (procedure.groupe_instructeurs.actif.one? && closed))
   end
+
+  private
+
+  def toggle_routing
+    procedure.update!(routing_enabled: procedure.groupe_instructeurs.actif.many?)
+  end
 end

spec/controllers/administrateurs/archives_controller_spec.rb

@@ -22,7 +22,7 @@ describe Administrateurs::ArchivesController, type: :controller do
       it { is_expected.to have_http_status(200) }
 
       it 'use all procedure.groupe_instructeurs' do
-        expect(Archive).to receive(:for_groupe_instructeur).with([groupe_instructeur1, groupe_instructeur2]).and_return([])
+        expect(Archive).to receive(:for_groupe_instructeur).and_return([])
         subject
       end
     end