fix test

Author: Benoit-MINT

spec/controllers/administrateurs/groupe_instructeurs_controller_spec.rb

@@ -949,7 +949,7 @@ def remove_instructeur(instructeur)
       end
 
       let!(:drop_down_tdc) { procedure3.draft_revision.types_de_champ.first }
-      let!(:dossier) { create(:dossier, procedure: procedure3, state: :en_construction) }
+      let!(:dossier) { create(:dossier, :en_construction, procedure: procedure3) }
 
       before { post :create_simple_routing, params: { procedure_id: procedure3.id, create_simple_routing: { stable_id: drop_down_tdc.stable_id } } }
 
@@ -1137,9 +1137,9 @@ def remove_instructeur(instructeur)
     end
 
     let!(:drop_down_tdc) { procedure.draft_revision.types_de_champ.first }
-    let!(:dossier1) { create(:dossier, :with_populated_champs, procedure: procedure, state: :en_construction) }
-    let!(:dossier2) { create(:dossier, :with_populated_champs, procedure: procedure, state: :en_construction) }
-    let!(:dossier3) { create(:dossier, :with_populated_champs, procedure: procedure, state: :accepte) }
+    let!(:dossier1) { create(:dossier, :en_construction, :with_populated_champs, procedure: procedure) }
+    let!(:dossier2) { create(:dossier, :en_construction, :with_populated_champs, procedure: procedure) }
+    let!(:dossier3) { create(:dossier, :accepte, :with_populated_champs, procedure: procedure) }
 
     before do
       dossier1.champs.first.update(value: 'Paris')

spec/jobs/bulk_route_job_spec.rb

@@ -14,9 +14,9 @@
     end
 
     let!(:drop_down_tdc) { procedure.draft_revision.types_de_champ.first }
-    let!(:dossier1) { create(:dossier, :with_populated_champs, procedure: procedure, state: :en_construction) }
-    let!(:dossier2) { create(:dossier, :with_populated_champs, procedure: procedure, state: :en_construction) }
-    let!(:dossier3) { create(:dossier, :with_populated_champs, procedure: procedure, state: :accepte) }
+    let!(:dossier1) { create(:dossier, :en_construction, :with_populated_champs, procedure: procedure) }
+    let!(:dossier2) { create(:dossier, :en_construction, :with_populated_champs, procedure: procedure) }
+    let!(:dossier3) { create(:dossier, :accepte, :with_populated_champs, procedure: procedure) }
     let!(:groupe_instructeur_paris) { create(:groupe_instructeur, procedure: procedure, label: 'Paris', routing_rule: ds_eq(champ_value(drop_down_tdc.stable_id), constant('Paris'))) }
     let!(:groupe_instructeur_lyon) { create(:groupe_instructeur, procedure: procedure, label: 'Lyon', routing_rule: ds_eq(champ_value(drop_down_tdc.stable_id), constant('Lyon'))) }
     let!(:groupe_instructeur_marseille) { create(:groupe_instructeur, procedure: procedure, label: 'Marseille', routing_rule: ds_eq(champ_value(drop_down_tdc.stable_id), constant('Marseille'))) }

spec/models/dossier_notification_spec.rb

@@ -106,19 +106,15 @@
     let!(:other_groupe_instructeur) { create(:groupe_instructeur, instructeurs: [instructeur]) }
     let!(:dossier) { create(:dossier, :accepte, groupe_instructeur:) }
     let!(:other_dossier) { create(:dossier, :en_construction, groupe_instructeur: other_groupe_instructeur) }
-    let!(:notification_instructeur) { create(:dossier_notification, :for_instructeur, dossier:, instructeur:) }
-    let!(:notification_grp_instructeur) { create(:dossier_notification, :for_groupe_instructeur, dossier:, groupe_instructeur:) }
-    let!(:other_notification_instructeur) { create(:dossier_notification, :for_instructeur, dossier: other_dossier, instructeur:) }
-    let!(:other_notification_grp_instructeur) { create(:dossier_notification, :for_groupe_instructeur, dossier: other_dossier, groupe_instructeur: other_groupe_instructeur) }
+    let!(:notification_instructeur) { create(:dossier_notification, :for_instructeur, dossier:, instructeur:, notification_type: :dossier_modifie) }
+    let!(:other_notification_instructeur) { create(:dossier_notification, :for_instructeur, dossier: other_dossier, instructeur:, notification_type: :dossier_modifie) }
 
     context 'a given instructeur and one dossier' do
       subject { DossierNotification.notifications_for_instructeur_dossier(instructeur, dossier) }
 
       it 'includes correct notifications and excludes the others' do
         is_expected.to include(notification_instructeur)
-        is_expected.to include(notification_grp_instructeur)
         is_expected.not_to include(other_notification_instructeur)
-        is_expected.not_to include(other_notification_grp_instructeur)
       end
     end
 
@@ -129,9 +125,7 @@
 
       it 'includes correct notifications and excludes the others' do
         expect(subject[dossier.id]).to include(notification_instructeur)
-        expect(subject[dossier.id]).to include(notification_grp_instructeur)
         expect(subject[other_dossier.id]).to include(other_notification_instructeur)
-        expect(subject[other_dossier.id]).to include(other_notification_grp_instructeur)
       end
     end
 
@@ -141,10 +135,8 @@
       subject { DossierNotification.notifications_for_instructeur_procedure(groupe_instructeur_ids, instructeur) }
 
       it 'includes correct notifications and excludes the others' do
-        expect(subject['traites']['dossier_depose']).to include(notification_instructeur)
-        expect(subject['traites']['dossier_depose']).to include(notification_grp_instructeur)
-        expect(subject['a-suivre']['dossier_depose']).to include(other_notification_instructeur)
-        expect(subject['a-suivre']['dossier_depose']).to include(other_notification_grp_instructeur)
+        expect(subject['traites']['dossier_modifie']).to include(notification_instructeur)
+        expect(subject['a-suivre']['dossier_modifie']).to include(other_notification_instructeur)
       end
     end
   end