From ad5279f4ab6edb5869f7ef9c2151eac3e66ab1df Mon Sep 17 00:00:00 2001 From: Paul Chavard Date: Wed, 15 Sep 2021 12:55:54 +0200 Subject: [PATCH] test(groupe_instructeur): fix groupe_instructeur controller spec --- .../groupe_instructeurs_controller_spec.rb | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/spec/controllers/new_administrateur/groupe_instructeurs_controller_spec.rb b/spec/controllers/new_administrateur/groupe_instructeurs_controller_spec.rb index ddd292612..3444b7361 100644 --- a/spec/controllers/new_administrateur/groupe_instructeurs_controller_spec.rb +++ b/spec/controllers/new_administrateur/groupe_instructeurs_controller_spec.rb @@ -2,7 +2,7 @@ describe NewAdministrateur::GroupeInstructeursController, type: :controller do render_views let(:admin) { create(:administrateur) } - let(:procedure) { create(:procedure, :published, :for_individual, administrateurs: [admin]) } + let(:procedure) { create(:procedure, :published, :for_individual, administrateurs: [admin], routing_enabled: true) } let!(:gi_1_1) { procedure.defaut_groupe_instructeur } let(:procedure2) { create(:procedure, :published) } @@ -222,7 +222,7 @@ describe NewAdministrateur::GroupeInstructeursController, type: :controller do 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 admin_procedure_groupe_instructeur_path(procedure, procedure.defaut_groupe_instructeur) } + it { expect(subject).to redirect_to admin_procedure_groupe_instructeurs_path(procedure) } end context 'when there is at least one bad email' do @@ -230,13 +230,13 @@ describe NewAdministrateur::GroupeInstructeursController, type: :controller do 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 admin_procedure_groupe_instructeur_path(procedure, procedure.defaut_groupe_instructeur) } + it { expect(subject).to redirect_to admin_procedure_groupe_instructeurs_path(procedure) } 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'].to_json } it { expect(subject.request.flash[:alert]).to be_present } - it { expect(subject).to redirect_to admin_procedure_groupe_instructeur_path(procedure, procedure.defaut_groupe_instructeur) } + it { expect(subject).to redirect_to admin_procedure_groupe_instructeurs_path(procedure) } end end @@ -344,7 +344,7 @@ describe NewAdministrateur::GroupeInstructeursController, type: :controller do 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 admin_procedure_groupe_instructeur_path(procedure, gi_1_1) } + it { expect(subject).to redirect_to admin_procedure_groupe_instructeurs_path(procedure) } end context 'when the instructor is not assigned to the procedure' do @@ -352,7 +352,7 @@ describe NewAdministrateur::GroupeInstructeursController, type: :controller do 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 admin_procedure_groupe_instructeur_path(procedure, gi_1_1) } + it { expect(subject).to redirect_to admin_procedure_groupe_instructeurs_path(procedure) } end end