From 30f96a79c27a1017ee391b027e235568c164f452 Mon Sep 17 00:00:00 2001
From: Eric Leroy-Terquem <eric.leroyterquem@beta.gouv.fr>
Date: Tue, 5 Mar 2024 17:22:15 +0100
Subject: [PATCH] chore(bulk messages): update associations

---
 app/controllers/instructeurs/procedures_controller.rb | 1 -
 app/models/bulk_message.rb                            | 1 -
 app/models/groupe_instructeur.rb                      | 1 -
 app/models/procedure.rb                               | 2 ++
 spec/factories/bulk_message.rb                        | 1 -
 spec/factories/groupe_instructeur.rb                  | 4 ----
 spec/factories/procedure.rb                           | 2 +-
 7 files changed, 3 insertions(+), 9 deletions(-)

diff --git a/app/controllers/instructeurs/procedures_controller.rb b/app/controllers/instructeurs/procedures_controller.rb
index 7efeb7f6a..8caf08fee 100644
--- a/app/controllers/instructeurs/procedures_controller.rb
+++ b/app/controllers/instructeurs/procedures_controller.rb
@@ -283,7 +283,6 @@ module Instructeurs
         dossier_state: Dossier.states.fetch(:brouillon),
         sent_at: Time.zone.now,
         instructeur_id: current_instructeur.id,
-        groupe_instructeurs: GroupeInstructeur.for_dossiers(dossiers),
         procedure_id: @procedure.id
       )
       bulk_message.save!
diff --git a/app/models/bulk_message.rb b/app/models/bulk_message.rb
index ca343d701..5ec93da52 100644
--- a/app/models/bulk_message.rb
+++ b/app/models/bulk_message.rb
@@ -1,5 +1,4 @@
 class BulkMessage < ApplicationRecord
   belongs_to :instructeur
   belongs_to :procedure
-  has_and_belongs_to_many :groupe_instructeurs, -> { order(:label) }
 end
diff --git a/app/models/groupe_instructeur.rb b/app/models/groupe_instructeur.rb
index f3bb7bc6d..b0b165b1e 100644
--- a/app/models/groupe_instructeur.rb
+++ b/app/models/groupe_instructeur.rb
@@ -10,7 +10,6 @@ class GroupeInstructeur < ApplicationRecord
   has_many :assignments, class_name: 'DossierAssignment', dependent: :nullify, inverse_of: :groupe_instructeur
   has_many :previous_assignments, class_name: 'DossierAssignment', dependent: :nullify, inverse_of: :previous_groupe_instructeur
   has_and_belongs_to_many :exports, dependent: :destroy
-  has_and_belongs_to_many :bulk_messages, dependent: :destroy
 
   has_one :defaut_procedure, -> { with_discarded }, class_name: 'Procedure', foreign_key: :defaut_groupe_instructeur_id, dependent: :nullify, inverse_of: :defaut_groupe_instructeur
   has_one :contact_information, dependent: :destroy
diff --git a/app/models/procedure.rb b/app/models/procedure.rb
index ad992d2c6..735b8be08 100644
--- a/app/models/procedure.rb
+++ b/app/models/procedure.rb
@@ -60,6 +60,8 @@ class Procedure < ApplicationRecord
   belongs_to :zone, optional: true
   has_and_belongs_to_many :zones
 
+  has_many :bulk_messages, dependent: :destroy
+
   def active_dossier_submitted_message
     published_dossier_submitted_message || draft_dossier_submitted_message
   end
diff --git a/spec/factories/bulk_message.rb b/spec/factories/bulk_message.rb
index d1d0cff7b..627460477 100644
--- a/spec/factories/bulk_message.rb
+++ b/spec/factories/bulk_message.rb
@@ -5,7 +5,6 @@ FactoryBot.define do
     dossier_state { Dossier.states.fetch(:brouillon) }
     sent_at { 1.day.ago }
     instructeur { association :instructeur }
-    groupe_instructeurs { [association(:groupe_instructeur, strategy: :build)] }
     procedure { association :procedure }
   end
 end
diff --git a/spec/factories/groupe_instructeur.rb b/spec/factories/groupe_instructeur.rb
index 18ed1c58d..9d0a2eafb 100644
--- a/spec/factories/groupe_instructeur.rb
+++ b/spec/factories/groupe_instructeur.rb
@@ -12,9 +12,5 @@ FactoryBot.define do
     trait :default do
       label { GroupeInstructeur::DEFAUT_LABEL }
     end
-
-    trait :with_bulk_message do
-      bulk_messages { [association(:bulk_message, strategy: :build)] }
-    end
   end
 end
diff --git a/spec/factories/procedure.rb b/spec/factories/procedure.rb
index e2c96c34a..74785ebfa 100644
--- a/spec/factories/procedure.rb
+++ b/spec/factories/procedure.rb
@@ -94,7 +94,7 @@ FactoryBot.define do
     end
 
     trait :with_bulk_message do
-      groupe_instructeurs { [association(:groupe_instructeur, :default, :with_bulk_message, procedure: instance, strategy: :build)] }
+      bulk_messages { [create(:bulk_message)] }
     end
 
     trait :with_logo do