Commit e9f6edda authored by Luke Duncalfe's avatar Luke Duncalfe Committed by Kushal Pandya

Rewrite issues show spec as a feature spec

This spec needs to run as a request-style spec in order to invoke the
Rails router.

A controller-style spec matches the wrong route, and
`session['user_return_to']` becomes incorrect.
parent 94e4d75d
......@@ -251,15 +251,13 @@ describe Projects::IssuesController do
end
end
describe 'Redirect after sign in' do
# This spec runs as a request-style spec in order to invoke the
# Rails router. A controller-style spec matches the wrong route, and
# session['user_return_to'] becomes incorrect.
describe 'Redirect after sign in', type: :request do
context 'with an AJAX request' do
it 'does not store the visited URL' do
get :show, params: {
format: :json,
namespace_id: project.namespace,
project_id: project,
id: issue.iid
}, xhr: true
get project_issue_path(project, issue), xhr: true
expect(session['user_return_to']).to be_blank
end
......@@ -267,14 +265,9 @@ describe Projects::IssuesController do
context 'without an AJAX request' do
it 'stores the visited URL' do
get :show,
params: {
namespace_id: project.namespace.to_param,
project_id: project,
id: issue.iid
}
get project_issue_path(project, issue)
expect(session['user_return_to']).to eq("/#{project.namespace.to_param}/#{project.to_param}/issues/#{issue.iid}")
expect(session['user_return_to']).to eq(project_issue_path(project, issue))
end
end
end
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment