perf(bulk_routing): make less requests when bulk_routing

demarches-simplifiees · Mar 4, 2025 · 6216f3c · 6216f3c
1 parent a0dc47e
commit 6216f3c
Show file tree

Hide file tree

Showing 3 changed files with 14 additions and 6 deletions.
diff --git a/app/controllers/administrateurs/groupe_instructeurs_controller.rb b/app/controllers/administrateurs/groupe_instructeurs_controller.rb
@@ -398,14 +398,15 @@ def export_groupe_instructeurs
     end
 
     def bulk_route
-      dossiers = procedure.dossiers.state_not_termine
+      dossiers = procedure.dossiers.includes(:procedure, :groupe_instructeur, :champs, revision: [:types_de_champ]).state_not_termine
 
       dossiers.update_all(forced_groupe_instructeur: false)
 
-      dossiers.each do |dossier|
-        RoutingEngine.compute(dossier, assignment_mode: DossierAssignment.modes.fetch(:bulk_routing))
+      dossiers.in_batches do |relation|
+        relation.each do |dossier|
+          RoutingEngine.compute(dossier, assignment_mode: DossierAssignment.modes.fetch(:bulk_routing))
+        end
       end
-
       procedure.update!(routing_alert: false)
 
       flash[:notice] = "Les dossiers ont étés routés vers les groupes d’instructeurs"

diff --git a/app/models/dossier.rb b/app/models/dossier.rb
@@ -680,7 +680,7 @@ def show_procedure_state_warning?
 
   def assign_to_groupe_instructeur(groupe_instructeur, mode, author = nil)
     return if groupe_instructeur.present? && groupe_instructeur.procedure != procedure
-    return if self.groupe_instructeur == groupe_instructeur
+    return if self.groupe_instructeur == groupe_instructeur && mode == self.dossier_assignment&.mode
 
     previous_groupe_instructeur = self.groupe_instructeur
 

diff --git a/spec/controllers/administrateurs/groupe_instructeurs_controller_spec.rb b/spec/controllers/administrateurs/groupe_instructeurs_controller_spec.rb
@@ -1110,13 +1110,20 @@ def remove_instructeur(instructeur)
       dossier2.champs.first.update(value: 'Lyon')
       dossier3.champs.first.update(value: 'Marseille')
       post :create_simple_routing, params: { procedure_id: procedure.id, create_simple_routing: { stable_id: drop_down_tdc.stable_id } }
-      post :bulk_route, params: { procedure_id: procedure.id }
     end
 
     it 'routes only dossiers en construction or en instruction' do
+      query_count = 0
+      ActiveSupport::Notifications.subscribed(lambda { |*_args| query_count += 1 }, "sql.active_record") do
+        post :bulk_route, params: { procedure_id: procedure.id }
+      end
+      expect(query_count).to be_between(60, 90)
       expect(dossier1.reload.groupe_instructeur.label).to eq 'Paris'
+      expect(dossier1.reload.dossier_assignment.mode).to eq 'bulk_routing'
       expect(dossier2.reload.groupe_instructeur.label).to eq 'Lyon'
+      expect(dossier2.reload.dossier_assignment.mode).to eq 'bulk_routing'
       expect(dossier3.reload.groupe_instructeur.label).to eq 'Lyon'
+      expect(dossier3.reload.dossier_assignment.mode).to eq 'auto'
       # Lyon is default group
       expect(procedure.reload.routing_alert).to be_falsey
     end