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

[Fix #554] If procedure is hidden, then don't crash

Browse source
This commit is contained in:
Mathieu Magnin 2017-07-06 14:40:06 +02:00
parent 62832f079a
commit aa929bb5b2
2 changed files with 8 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
app/controllers/users/dossiers_controller.rb
View file

@ -39,7 +39,7 @@ class Users::DossiersController < UsersController
unless params[:procedure_path].nil? unless params[:procedure_path].nil?
procedure_path = ProcedurePath.where(path: params[:procedure_path]).last procedure_path = ProcedurePath.where(path: params[:procedure_path]).last
if procedure_path.nil? if procedure_path.nil? || procedure_path.procedure.nil?
flash.alert = "Procédure inconnue" flash.alert = "Procédure inconnue"
return redirect_to root_path return redirect_to root_path
else else

8
spec/controllers/users/dossiers_controller_spec.rb
View file

@ -152,7 +152,7 @@ describe Users::DossiersController, type: :controller do
end end
describe 'GET #commencer' do describe 'GET #commencer' do
subject { get :commencer, params: {procedure_path: procedure.path} } subject { get :commencer, params: { procedure_path: procedure.path } }
it { expect(subject.status).to eq 302 } it { expect(subject.status).to eq 302 }
it { expect(subject).to redirect_to new_users_dossier_path(procedure_id: procedure.id) } it { expect(subject).to redirect_to new_users_dossier_path(procedure_id: procedure.id) }
@ -166,6 +166,12 @@ describe Users::DossiersController, type: :controller do
it { expect(subject.status).to eq 200 } it { expect(subject.status).to eq 200 }
end end
context 'when procedure is hidden' do
let(:procedure) { create(:procedure, :published, hidden_at: DateTime.now) }
it { expect(subject).to redirect_to(root_path) }
end
end end
describe 'POST #siret_informations' do describe 'POST #siret_informations' do