feat(groupe instructeur): cannot disable default groupe instructeur

app/controllers/administrateurs/groupe_instructeurs_controller.rb

@@ -48,7 +48,10 @@ module Administrateurs
     def update
       @groupe_instructeur = groupe_instructeur
 
-      if @groupe_instructeur.update(groupe_instructeur_params)
+      if closed_params? && @groupe_instructeur.id == procedure.defaut_groupe_instructeur.id
+        redirect_to admin_procedure_groupe_instructeur_path(procedure, groupe_instructeur),
+          alert: "Il est impossible de désactiver le groupe d’instructeurs par défaut."
+      elsif @groupe_instructeur.update(groupe_instructeur_params)
         redirect_to admin_procedure_groupe_instructeur_path(procedure, groupe_instructeur),
           notice: "Le nom est à présent « #{@groupe_instructeur.label} »."
       else
@@ -260,6 +263,10 @@ module Administrateurs
 
     private
 
+    def closed_params?
+      groupe_instructeur_params[:closed] == "1"
+    end
+
     def procedure
       current_administrateur
         .procedures

spec/controllers/administrateurs/groupe_instructeurs_controller_spec.rb

@@ -181,7 +181,7 @@ describe Administrateurs::GroupeInstructeursController, type: :controller do
 
   describe '#update' do
     let(:new_name) { 'nouveau nom du groupe' }
-    let(:closed_value) { false }
+    let(:closed_value) { '0' }
     let!(:procedure_non_routee) { create(:procedure, :published, :for_individual, administrateurs: [admin]) }
     let!(:gi_1_1) { procedure_non_routee.defaut_groupe_instructeur }
 
@@ -195,26 +195,33 @@ describe Administrateurs::GroupeInstructeursController, type: :controller do
       gi_1_1.reload
     end
 
-    it { expect(response).to redirect_to(admin_procedure_groupe_instructeur_path(procedure_non_routee, gi_1_1)) }
+    it do
-    it { expect(gi_1_1.label).to eq(new_name) }
+      expect(response).to redirect_to(admin_procedure_groupe_instructeur_path(procedure_non_routee, gi_1_1))
-    it { expect(gi_1_1.closed).to eq(false) }
+      expect(gi_1_1.label).to eq(new_name)
-    it { expect(flash.notice).to be_present }
+      expect(gi_1_1.closed).to eq(false)
+      expect(flash.notice).to be_present
+    end
 
-    context 'when we try do disable the only groupe instructeur' do
+    context 'when we try do disable the default groupe instructeur' do
-      let(:closed_value) { true }
+      let(:closed_value) { '1' }
+      let!(:gi_1_2) { procedure.groupe_instructeurs.create(label: 'groupe instructeur 2') }
 
-      it { expect(response).to render_template(:show) }
+      it do
-      it { expect(gi_1_1.label).not_to eq(new_name) }
+        expect(subject).to redirect_to admin_procedure_groupe_instructeur_path(procedure_non_routee, gi_1_1)
-      it { expect(gi_1_1.closed).to eq(false) }
+        expect(gi_1_1.label).not_to eq(new_name)
-      it { expect(flash.alert).to eq(['Il doit y avoir au moins un groupe instructeur actif sur chaque démarche']) }
+        expect(gi_1_1.closed).to eq(false)
+        expect(flash.alert).to eq('Il est impossible de désactiver le groupe d’instructeurs par défaut.')
+      end
     end
 
     context 'when the name is already taken' do
       let!(:gi_1_2) { procedure_non_routee.groupe_instructeurs.create(label: 'groupe instructeur 2') }
       let(:new_name) { gi_1_2.label }
 
-      it { expect(gi_1_1.label).not_to eq(new_name) }
+      it do
-      it { expect(flash.alert).to eq(['Le libellé est déjà utilisé(e)']) }
+        expect(gi_1_1.label).not_to eq(new_name)
+        expect(flash.alert).to eq(['Le libellé est déjà utilisé(e)'])
+      end
     end
   end