From e255bb9929f2f81892cca3c60a1538f8a19a1a43 Mon Sep 17 00:00:00 2001
From: Eric Leroy-Terquem <eric.leroyterquem@beta.gouv.fr>
Date: Fri, 8 Sep 2023 15:39:02 +0200
Subject: [PATCH] refactor(routing): extract methods to create groups from tdcs

---
 .../groupe_instructeurs_controller.rb         | 44 ++++++++++---------
 1 file changed, 23 insertions(+), 21 deletions(-)

diff --git a/app/controllers/administrateurs/groupe_instructeurs_controller.rb b/app/controllers/administrateurs/groupe_instructeurs_controller.rb
index 5f5d79e19..3238cb719 100644
--- a/app/controllers/administrateurs/groupe_instructeurs_controller.rb
+++ b/app/controllers/administrateurs/groupe_instructeurs_controller.rb
@@ -43,31 +43,13 @@ module Administrateurs
       case tdc.type_champ
       when TypeDeChamp.type_champs.fetch(:departements)
         tdc_options = APIGeoService.departements.map { ["#{_1[:code]} – #{_1[:name]}", _1[:code]] }
-        tdc_options.each do |code_and_name, code|
-          routing_rule = ds_eq(champ_value(stable_id), constant(code))
-          @procedure
-            .groupe_instructeurs
-            .find_or_create_by(label: code_and_name)
-            .update(instructeurs: [current_administrateur.instructeur], routing_rule:)
-        end
+        create_groups_from_territorial_tdc(tdc_options, stable_id)
       when TypeDeChamp.type_champs.fetch(:regions)
         tdc_options = APIGeoService.regions.map { ["#{_1[:code]} – #{_1[:name]}", _1[:code]] }
-        tdc_options.each do |code_and_name, code|
-          routing_rule = ds_eq(champ_value(stable_id), constant(code))
-          @procedure
-            .groupe_instructeurs
-            .find_or_create_by(label: code_and_name)
-            .update(instructeurs: [current_administrateur.instructeur], routing_rule:)
-        end
+        create_groups_from_territorial_tdc(tdc_options, stable_id)
       when TypeDeChamp.type_champs.fetch(:drop_down_list)
         tdc_options = tdc.drop_down_options.reject(&:empty?)
-        tdc_options.each do |option_label|
-          routing_rule = ds_eq(champ_value(stable_id), constant(option_label))
-          @procedure
-            .groupe_instructeurs
-            .find_or_create_by(label: option_label)
-            .update(instructeurs: [current_administrateur.instructeur], routing_rule:)
-        end
+        create_groups_from_drop_down_list_tdc(tdc_options, stable_id)
       end
 
       if tdc.drop_down_other?
@@ -468,5 +450,25 @@ module Administrateurs
     def flash_message_for_invalid_csv
       flash[:alert] = "Importation impossible, veuillez importer un csv suivant #{view_context.link_to('ce modèle', "/csv/import-instructeurs-test.csv")} pour une procédure sans routage ou #{view_context.link_to('celui-ci', "/csv/#{I18n.locale}/import-groupe-test.csv")} pour une procédure routée"
     end
+
+    def create_groups_from_territorial_tdc(tdc_options, stable_id)
+      tdc_options.each do |label, code|
+        routing_rule = ds_eq(champ_value(stable_id), constant(code))
+        @procedure
+          .groupe_instructeurs
+          .find_or_create_by(label: label)
+          .update(instructeurs: [current_administrateur.instructeur], routing_rule:)
+      end
+    end
+
+    def create_groups_from_drop_down_list_tdc(tdc_options, stable_id)
+      tdc_options.each do |label|
+        routing_rule = ds_eq(champ_value(stable_id), constant(label))
+        @procedure
+          .groupe_instructeurs
+          .find_or_create_by(label: label)
+          .update(instructeurs: [current_administrateur.instructeur], routing_rule:)
+      end
+    end
   end
 end