Move admin/assigns to the new UI like groupe_instructeurs

spec/controllers/new_administrateur/groupe_instructeurs_controller_spec.rb

@@ -31,6 +31,22 @@ describe NewAdministrateur::GroupeInstructeursController, type: :controller do
 
       it { expect(response).to have_http_status(:ok) }
     end
+
+    context 'when the routage is not activated on the procedure' do
+      let(:procedure) { create :procedure, administrateur: admin, instructeurs: [instructeur_assigned_1, instructeur_assigned_2] }
+      let!(:instructeur_assigned_1) { create :instructeur, email: 'instructeur_1@ministere_a.gouv.fr', administrateurs: [admin] }
+      let!(:instructeur_assigned_2) { create :instructeur, email: 'instructeur_2@ministere_b.gouv.fr', administrateurs: [admin] }
+      let!(:instructeur_not_assigned_1) { create :instructeur, email: 'instructeur_3@ministere_a.gouv.fr', administrateurs: [admin] }
+      let!(:instructeur_not_assigned_2) { create :instructeur, email: 'instructeur_4@ministere_b.gouv.fr', administrateurs: [admin] }
+      subject! { get :show, params: { procedure_id: procedure.id, id: gi_1_1.id } }
+
+      it { expect(response.status).to eq(200) }
+
+      it 'sets the assigned and not assigned instructeurs' do
+        expect(assigns(:instructeurs)).to match_array([instructeur_assigned_1, instructeur_assigned_2])
+        expect(assigns(:available_instructeur_emails)).to match_array(['instructeur_3@ministere_a.gouv.fr', 'instructeur_4@ministere_b.gouv.fr'])
+      end
+    end
   end
 
   describe '#create' do
@@ -189,10 +205,40 @@ describe NewAdministrateur::GroupeInstructeursController, type: :controller do
     end
   end
 
+  describe '#add_instructeur_procedure_non_routee' do
+    let(:procedure) { create :procedure, administrateur: admin }
+    let(:emails) { ['instructeur_3@ministere_a.gouv.fr', 'instructeur_4@ministere_b.gouv.fr'] }
+    subject { post :add_instructeur, params: { emails: emails, procedure_id: procedure.id, id: gi_1_1.id } }
+
+    context 'when all emails are valid' do
+      let(:emails) { ['test@b.gouv.fr', 'test2@b.gouv.fr'] }
+      it { expect(response.status).to eq(200) }
+      it { expect(subject.request.flash[:alert]).to be_nil }
+      it { expect(subject.request.flash[:notice]).to be_present }
+      it { expect(subject).to redirect_to procedure_groupe_instructeur_path(procedure, procedure.defaut_groupe_instructeur) }
+    end
+
+    context 'when there is at least one bad email' do
+      let(:emails) { ['badmail', 'instructeur2@gmail.com'] }
+      it { expect(response.status).to eq(200) }
+      it { expect(subject.request.flash[:alert]).to be_present }
+      it { expect(subject.request.flash[:notice]).to be_present }
+      it { expect(subject).to redirect_to procedure_groupe_instructeur_path(procedure, procedure.defaut_groupe_instructeur) }
+    end
+
+    context 'when the admin wants to assign an instructor who is already assigned on this procedure' do
+      let(:emails) { ['instructeur_1@ministere_a.gouv.fr'] }
+      it { expect(subject.request.flash[:alert]).to be_present }
+      it { expect(subject).to redirect_to procedure_groupe_instructeur_path(procedure, procedure.defaut_groupe_instructeur) }
+    end
+  end
+
   describe '#add_instructeur' do
     let!(:instructeur) { create(:instructeur) }
+    let(:gi_1_2) { procedure.groupe_instructeurs.create(label: 'groupe instructeur 2') }
+
     before do
-      gi_1_1.instructeurs << instructeur
+      gi_1_2.instructeurs << instructeur
 
       allow(GroupeInstructeurMailer).to receive(:add_instructeurs)
         .and_return(double(deliver_later: true))
@@ -200,7 +246,7 @@ describe NewAdministrateur::GroupeInstructeursController, type: :controller do
       post :add_instructeur,
         params: {
           procedure_id: procedure.id,
-          id: gi_1_1.id,
+          id: gi_1_2.id,
           emails: new_instructeur_emails
         }
     end
@@ -208,12 +254,13 @@ describe NewAdministrateur::GroupeInstructeursController, type: :controller do
     context 'of a news instructeurs' do
       let(:new_instructeur_emails) { ['new_i1@mail.com', 'new_i2@mail.com'] }
 
-      it { expect(gi_1_1.instructeurs.pluck(:email)).to include(*new_instructeur_emails) }
+      it { expect(gi_1_2.instructeurs.pluck(:email)).to include(*new_instructeur_emails) }
       it { expect(flash.notice).to be_present }
-      it { expect(response).to redirect_to(procedure_groupe_instructeur_path(procedure, gi_1_1)) }
+      it { expect(response).to redirect_to(procedure_groupe_instructeur_path(procedure, gi_1_2)) }
+      it { expect(procedure.routee?).to be_truthy }
       it "calls GroupeInstructeurMailer with the right groupe and instructeurs" do
         expect(GroupeInstructeurMailer).to have_received(:add_instructeurs).with(
-          gi_1_1,
+          gi_1_2,
           satisfy { |instructeurs| instructeurs.all? { |i| new_instructeur_emails.include?(i.email) } },
           admin.email
         )
@@ -223,20 +270,20 @@ describe NewAdministrateur::GroupeInstructeursController, type: :controller do
     context 'of an instructeur already in the group' do
       let(:new_instructeur_emails) { [instructeur.email] }
 
-      it { expect(response).to redirect_to(procedure_groupe_instructeur_path(procedure, procedure.defaut_groupe_instructeur)) }
+      it { expect(response).to redirect_to(procedure_groupe_instructeur_path(procedure, gi_1_2)) }
     end
 
     context 'of badly formed email' do
       let(:new_instructeur_emails) { ['badly_formed_email'] }
 
       it { expect(flash.alert).to be_present }
-      it { expect(response).to redirect_to(procedure_groupe_instructeur_path(procedure, procedure.defaut_groupe_instructeur)) }
+      it { expect(response).to redirect_to(procedure_groupe_instructeur_path(procedure, gi_1_2)) }
     end
 
     context 'of an empty string' do
       let(:new_instructeur_emails) { '' }
 
-      it { expect(response).to redirect_to(procedure_groupe_instructeur_path(procedure, procedure.defaut_groupe_instructeur)) }
+      it { expect(response).to redirect_to(procedure_groupe_instructeur_path(procedure, gi_1_2)) }
     end
   end
 
@@ -275,6 +322,33 @@ describe NewAdministrateur::GroupeInstructeursController, type: :controller do
     end
   end
 
+  describe '#remove_instructeur_procedure_non_routee' do
+    let(:procedure) { create :procedure, administrateur: admin, instructeurs: [instructeur_assigned_1, instructeur_assigned_2] }
+    let!(:instructeur_assigned_1) { create :instructeur, email: 'instructeur_1@ministere_a.gouv.fr', administrateurs: [admin] }
+    let!(:instructeur_assigned_2) { create :instructeur, email: 'instructeur_2@ministere_b.gouv.fr', administrateurs: [admin] }
+    let!(:instructeur_assigned_3) { create :instructeur, email: 'instructeur_3@ministere_a.gouv.fr', administrateurs: [admin] }
+    subject! { get :show, params: { procedure_id: procedure.id, id: gi_1_1.id } }
+    it 'sets the assigned instructeurs' do
+      expect(assigns(:instructeurs)).to match_array([instructeur_assigned_1, instructeur_assigned_2])
+    end
+
+    context 'when the instructor is assigned to the procedure' do
+      subject { delete :remove_instructeur, params: { instructeur: { id: instructeur_assigned_1.id }, procedure_id: procedure.id, id: gi_1_1.id } }
+      it { expect(subject.request.flash[:notice]).to be_present }
+      it { expect(subject.request.flash[:alert]).to be_nil }
+      it { expect(response.status).to eq(302) }
+      it { expect(subject).to redirect_to procedure_groupe_instructeur_path(procedure, gi_1_1) }
+    end
+
+    context 'when the instructor is not assigned to the procedure' do
+      subject { delete :remove_instructeur, params: { instructeur: { id: instructeur_assigned_3.id }, procedure_id: procedure.id, id: gi_1_1.id } }
+      it { expect(subject.request.flash[:alert]).to be_present }
+      it { expect(subject.request.flash[:notice]).to be_nil }
+      it { expect(response.status).to eq(302) }
+      it { expect(subject).to redirect_to procedure_groupe_instructeur_path(procedure, gi_1_1) }
+    end
+  end
+
   describe '#update_routing_criteria_name' do
     before do
       patch :update_routing_criteria_name,