Commit 7ce408c3 authored by Oswaldo Ferreira's avatar Oswaldo Ferreira Committed by Robert Speicher

Remove unnecessary stub for testing authorization being skipped

parent c1fa5e6d
...@@ -34,22 +34,6 @@ describe Oauth::AuthorizationsController do ...@@ -34,22 +34,6 @@ describe Oauth::AuthorizationsController do
end end
context 'with valid params' do context 'with valid params' do
context 'when trusted application' do
before do
doorkeeper.update(trusted: true)
end
it 'deletes session.user_return_to and redirects' do
request.session['user_return_to'] = 'http://example.com'
allow(controller).to receive(:skip_authorization?).and_return(true)
get :new, params
expect(request.session['user_return_to']).to be_nil
expect(response).to have_http_status(302)
end
end
it 'returns 200 code and renders view' do it 'returns 200 code and renders view' do
get :new, params get :new, params
...@@ -58,8 +42,8 @@ describe Oauth::AuthorizationsController do ...@@ -58,8 +42,8 @@ describe Oauth::AuthorizationsController do
end end
it 'deletes session.user_return_to and redirects when skip authorization' do it 'deletes session.user_return_to and redirects when skip authorization' do
doorkeeper.update(trusted: true)
request.session['user_return_to'] = 'http://example.com' request.session['user_return_to'] = 'http://example.com'
allow(controller).to receive(:skip_authorization?).and_return(true)
get :new, params get :new, params
......
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