Fix OAuth authorization page and add a spec for it

pull/6/head
Eugen Rochko 2016-11-18 23:08:52 +01:00
parent 1fd2ac774f
commit 7b7bf834e9
6 changed files with 39 additions and 5 deletions

View File

@ -2,7 +2,7 @@
= t('doorkeeper.authorizations.new.title')
.oauth-prompt
%h2= t('doorkeeper.authorizations.new.prompt', name: @pre_auth.client.name)
%h2= t('doorkeeper.authorizations.new.prompt', client_name: @pre_auth.client.name)
%p
= t('doorkeeper.authorizations.new.able_to')

View File

@ -1,6 +1,7 @@
require 'rails_helper'
RSpec.describe AboutController, type: :controller do
render_views
describe 'GET #index' do
it 'returns http success' do
@ -9,4 +10,10 @@ RSpec.describe AboutController, type: :controller do
end
end
describe 'GET #terms' do
it 'returns http success' do
get :terms
expect(response).to have_http_status(:success)
end
end
end

View File

@ -1,6 +1,8 @@
require 'rails_helper'
RSpec.describe HomeController, type: :controller do
render_views
describe 'GET #index' do
it 'redirects to about page' do
get :index

View File

@ -0,0 +1,25 @@
require 'rails_helper'
RSpec.describe Oauth::AuthorizationsController, type: :controller do
render_views
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
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)
end
it 'gives options to authorize and deny' do
expect(response.body).to match(/Authorize/)
end
end
end

View File

@ -1,6 +1,7 @@
require 'rails_helper'
RSpec.describe TagsController, type: :controller do
render_views
describe 'GET #show' do
before do
@ -12,5 +13,4 @@ RSpec.describe TagsController, type: :controller do
expect(response).to have_http_status(:success)
end
end
end

View File

@ -19,15 +19,15 @@ RSpec.describe FanOutOnWriteService do
end
it 'delivers status to home timeline' do
expect(Feed.new(:home, author).get(1).map(&:id)).to include status.id
expect(Feed.new(:home, author).get(10).map(&:id)).to include status.id
end
it 'delivers status to local followers' do
expect(Feed.new(:home, follower).get(1).map(&:id)).to include status.id
expect(Feed.new(:home, follower).get(10).map(&:id)).to include status.id
end
it 'delivers status to mentioned users' do
expect(Feed.new(:mentions, alice).get(1).map(&:id)).to include status.id
expect(Feed.new(:mentions, alice).get(10).map(&:id)).to include status.id
end
it 'delivers status to hashtag' do