describe NewAdministrateur::GroupeInstructeursController, type: :controller do render_views let(:admin) { create(:administrateur) } let(:procedure) { create(:procedure, :published, administrateurs: [admin]) } let!(:gi_1_1) { procedure.defaut_groupe_instructeur } let(:procedure2) { create(:procedure, :published) } let!(:gi_2_2) { procedure2.groupe_instructeurs.create(label: 'groupe instructeur 2 2') } before { sign_in(admin.user) } describe '#index' do context 'of a procedure I own' do let!(:gi_1_2) { procedure.groupe_instructeurs.create(label: 'groupe instructeur 2') } before { get :index, params: { procedure_id: procedure.id } } context 'when a procedure has multiple groups' do it { expect(response).to have_http_status(:ok) } it { expect(response.body).to include(gi_1_1.label) } it { expect(response.body).to include(gi_1_2.label) } it { expect(response.body).not_to include(gi_2_2.label) } end end end describe '#show' do context 'of a group I belong to' do before { get :show, params: { procedure_id: procedure.id, id: gi_1_1.id } } 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 before do post :create, params: { procedure_id: procedure.id, groupe_instructeur: { label: label } } end context 'with a valid name' do let(:label) { "nouveau_groupe" } it { expect(flash.notice).to be_present } it { expect(response).to redirect_to(procedure_groupe_instructeur_path(procedure, procedure.groupe_instructeurs.last)) } it { expect(procedure.groupe_instructeurs.count).to eq(2) } end context 'with an invalid group name' do let(:label) { gi_1_1.label } it { expect(response).to render_template(:index) } it { expect(procedure.groupe_instructeurs.count).to eq(1) } it { expect(flash.alert).to be_present } end end describe '#destroy' do def delete_group(group) delete :destroy, params: { procedure_id: procedure.id, id: group.id } end context 'with only one group' do before { delete_group gi_1_1 } it { expect(flash.alert).to be_present } it { expect(response).to redirect_to(procedure_groupe_instructeurs_path(procedure)) } it { expect(procedure.groupe_instructeurs.count).to eq(1) } end context 'with many groups' do let!(:gi_1_2) { procedure.groupe_instructeurs.create(label: 'groupe instructeur 2') } context 'of a group that can be deleted' do before { delete_group gi_1_2 } it { expect(flash.notice).to be_present } it { expect(procedure.groupe_instructeurs.count).to eq(1) } it { expect(response).to redirect_to(procedure_groupe_instructeurs_path(procedure)) } end context 'of a group with dossiers, that cannot be deleted' do let!(:dossier12) { create(:dossier, procedure: procedure, state: Dossier.states.fetch(:en_construction), groupe_instructeur: gi_1_2) } before { delete_group gi_1_2 } it { expect(flash.alert).to be_present } it { expect(procedure.groupe_instructeurs.count).to eq(2) } it { expect(response).to redirect_to(procedure_groupe_instructeurs_path(procedure)) } end end end describe '#reaffecter_dossiers' do let!(:gi_1_2) { procedure.groupe_instructeurs.create(label: 'groupe instructeur 2') } let!(:gi_1_3) { procedure.groupe_instructeurs.create(label: 'groupe instructeur 3') } before do get :reaffecter_dossiers, params: { procedure_id: procedure.id, id: gi_1_2.id } end def reaffecter_url(group) reaffecter_procedure_groupe_instructeur_path(:id => gi_1_2, :target_group => group) end it { expect(response).to have_http_status(:ok) } it { expect(response.body).to include(reaffecter_url(procedure.defaut_groupe_instructeur)) } it { expect(response.body).not_to include(reaffecter_url(gi_1_2)) } it { expect(response.body).to include(reaffecter_url(gi_1_3)) } end describe '#reaffecter' do let!(:gi_1_2) { procedure.groupe_instructeurs.create(label: 'groupe instructeur 2') } let!(:dossier12) { create(:dossier, :en_construction, procedure: procedure, groupe_instructeur: gi_1_1) } let!(:dossier_discarded) do dossier = create(:dossier, :en_construction, procedure: procedure, groupe_instructeur: gi_1_1) dossier.discard! dossier end describe 'when the new group is a group of the procedure' do before do post :reaffecter, params: { procedure_id: procedure.id, id: gi_1_1.id, target_group: gi_1_2.id } dossier12.reload end it { expect(response).to redirect_to(procedure_groupe_instructeurs_path(procedure)) } it { expect(gi_1_1.dossiers.with_discarded.count).to be(0) } it { expect(gi_1_2.dossiers.with_discarded.count).to be(2) } it { expect(gi_1_2.dossiers.last.id).to be(dossier12.id) } it { expect(dossier12.groupe_instructeur.id).to be(gi_1_2.id) } end describe 'when the target group is not a possible group' do subject { post :reaffecter, params: { procedure_id: procedure.id, id: gi_1_1.id, target_group: gi_2_2.id } } before do dossier12.reload end it { expect { subject }.to raise_error(ActiveRecord::RecordNotFound) } end end describe '#update' do let(:new_name) { 'nouveau nom du groupe' } before do patch :update, params: { procedure_id: procedure.id, id: gi_1_1.id, groupe_instructeur: { label: new_name } } end it { expect(gi_1_1.reload.label).to eq(new_name) } it { expect(response).to redirect_to(procedure_groupe_instructeur_path(procedure, gi_1_1)) } it { expect(flash.notice).to be_present } context 'when the name is already taken' do let!(:gi_1_2) { procedure.groupe_instructeurs.create(label: 'groupe instructeur 2') } let(:new_name) { gi_1_2.label } it { expect(gi_1_1.reload.label).not_to eq(new_name) } it { expect(flash.alert).to be_present } 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_2.instructeurs << instructeur allow(GroupeInstructeurMailer).to receive(:add_instructeurs) .and_return(double(deliver_later: true)) post :add_instructeur, params: { procedure_id: procedure.id, id: gi_1_2.id, emails: new_instructeur_emails } end context 'of a news instructeurs' do let(:new_instructeur_emails) { ['new_i1@mail.com', 'new_i2@mail.com'] } 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_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_2, satisfy { |instructeurs| instructeurs.all? { |i| new_instructeur_emails.include?(i.email) } }, admin.email ) end end 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, 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, 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, gi_1_2)) } end end describe '#remove_instructeur' do let!(:instructeur) { create(:instructeur) } before { gi_1_1.instructeurs << admin.instructeur << instructeur } def remove_instructeur(instructeur) delete :remove_instructeur, params: { procedure_id: procedure.id, id: gi_1_1.id, instructeur: { id: instructeur.id } } end context 'when there are many instructeurs' do before { remove_instructeur(admin.instructeur) } it { expect(gi_1_1.instructeurs).to include(instructeur) } it { expect(gi_1_1.reload.instructeurs.count).to eq(1) } it { expect(response).to redirect_to(procedure_groupe_instructeur_path(procedure, gi_1_1)) } end context 'when there is only one instructeur' do before do remove_instructeur(admin.instructeur) remove_instructeur(instructeur) end it { expect(gi_1_1.instructeurs).to include(instructeur) } it { expect(gi_1_1.instructeurs.count).to eq(1) } it { expect(flash.alert).to eq('Suppression impossible : il doit y avoir au moins un instructeur dans le groupe') } it { expect(response).to redirect_to(procedure_groupe_instructeur_path(procedure, gi_1_1)) } 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, params: { procedure_id: procedure.id, procedure: { routing_criteria_name: 'new name !' } } end it { expect(procedure.reload.routing_criteria_name).to eq('new name !') } end end