about summary refs log tree commit diff
path: root/spec
diff options
context:
space:
mode:
authorAkihiko Odaki <akihiko.odaki.4i@stu.hosei.ac.jp>2017-05-30 01:07:39 +0900
committerEugen Rochko <eugen@zeonfederated.com>2017-05-29 18:07:39 +0200
commitaecce5694b1763b9f63341dfe1c469fc14e5a775 (patch)
tree782809d1dd03255ff5822baf1b3f7d79ee4c189d /spec
parent0e4ca51951c4cd3ea51e4983bb5c502850d3056f (diff)
Cover Oauth::AuthorizationsController more (#3361)
Diffstat (limited to 'spec')
-rw-r--r--spec/controllers/oauth/authorizations_controller_spec.rb40
1 files changed, 31 insertions, 9 deletions
diff --git a/spec/controllers/oauth/authorizations_controller_spec.rb b/spec/controllers/oauth/authorizations_controller_spec.rb
index a5997bba3..5c2a62b48 100644
--- a/spec/controllers/oauth/authorizations_controller_spec.rb
+++ b/spec/controllers/oauth/authorizations_controller_spec.rb
@@ -7,21 +7,43 @@ RSpec.describe Oauth::AuthorizationsController, type: :controller do
 
   let(:app) { Doorkeeper::Application.create!(name: 'test', redirect_uri: 'http://localhost/') }
 
-  before do
-    sign_in Fabricate(:user), scope: :user
-  end
-
   describe 'GET #new' do
-    before do
+    subject do
       get :new, params: { client_id: app.uid, response_type: 'code', redirect_uri: 'http://localhost/' }
     end
 
-    it 'returns http success' do
-      expect(response).to have_http_status(:success)
+    shared_examples 'stores location for user' do
+      it 'stores location for user' do
+        subject
+        expect(controller.stored_location_for(:user)).to eq "/oauth/authorize?client_id=#{app.uid}&redirect_uri=http%3A%2F%2Flocalhost%2F&response_type=code"
+      end
+    end
+
+    context 'when signed in' do
+      before do
+        sign_in Fabricate(:user), scope: :user
+      end
+
+      it 'returns http success' do
+        subject
+        expect(response).to have_http_status(:success)
+      end
+
+      it 'gives options to authorize and deny' do
+        subject
+        expect(response.body).to match(/Authorize/)
+      end
+
+      include_examples 'stores location for user'
     end
 
-    it 'gives options to authorize and deny' do
-      expect(response.body).to match(/Authorize/)
+    context 'when not signed in' do
+      it 'redirects' do
+        subject
+        expect(response).to redirect_to '/auth/sign_in'
+      end
+
+      include_examples 'stores location for user'
     end
   end
 end