DGNum/demarches-normaliennes
15
1
Fork 0
Code Issues 1 Pull requests Projects Releases Packages Wiki Activity

Fix test in new_admin/groupe_instructeur_controller

Browse source
This commit is contained in:
simon lehericey 2019-10-25 10:19:58 +02:00
parent def9c67c56
commit 07aefdd40b
1 changed files with 5 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

10
spec/controllers/new_administrateur/groupe_instructeurs_controller_spec.rb
View file

@ -118,17 +118,17 @@ describe NewAdministrateur::GroupeInstructeursController, type: :controller do
before { gi_1_1.instructeurs << admin.instructeur << instructeur } before { gi_1_1.instructeurs << admin.instructeur << instructeur }
def remove_instructeur(email) def remove_instructeur(instructeur)
delete :remove_instructeur, delete :remove_instructeur,
params: { params: {
procedure_id: procedure.id, procedure_id: procedure.id,
id: gi_1_1.id, id: gi_1_1.id,
instructeur: { id: admin.instructeur.id } instructeur: { id: instructeur.id }
} }
end end
context 'when there are many instructeurs' do context 'when there are many instructeurs' do
before { remove_instructeur(admin.user.email) } before { remove_instructeur(admin.instructeur) }
it { expect(gi_1_1.instructeurs).to include(instructeur) } it { expect(gi_1_1.instructeurs).to include(instructeur) }
it { expect(gi_1_1.reload.instructeurs.count).to eq(1) } it { expect(gi_1_1.reload.instructeurs.count).to eq(1) }
@ -137,8 +137,8 @@ describe NewAdministrateur::GroupeInstructeursController, type: :controller do
context 'when there is only one instructeur' do context 'when there is only one instructeur' do
before do before do
remove_instructeur(admin.user.email) remove_instructeur(admin.instructeur)
remove_instructeur(instructeur.email) remove_instructeur(instructeur)
end end
it { expect(gi_1_1.instructeurs).to include(instructeur) } it { expect(gi_1_1.instructeurs).to include(instructeur) }