From 78fa926ed560e6a9738144bec7e152fa42104139 Mon Sep 17 00:00:00 2001 From: Eugen Rochko Date: Sat, 18 Aug 2018 03:03:12 +0200 Subject: Add remote interaction dialog for toots (#8202) * Add remote interaction dialog for toots * Change AuthorizeFollow into AuthorizeInteraction, support statuses * Update brakeman.ignore * Adjust how interaction buttons are display on public pages * Fix tests --- .../authorize_follows_controller_spec.rb | 110 -------------------- .../authorize_interactions_controller_spec.rb | 111 +++++++++++++++++++++ spec/controllers/intents_controller_spec.rb | 2 +- 3 files changed, 112 insertions(+), 111 deletions(-) delete mode 100644 spec/controllers/authorize_follows_controller_spec.rb create mode 100644 spec/controllers/authorize_interactions_controller_spec.rb (limited to 'spec') diff --git a/spec/controllers/authorize_follows_controller_spec.rb b/spec/controllers/authorize_follows_controller_spec.rb deleted file mode 100644 index 52971c724..000000000 --- a/spec/controllers/authorize_follows_controller_spec.rb +++ /dev/null @@ -1,110 +0,0 @@ -# frozen_string_literal: true - -require 'rails_helper' - -describe AuthorizeFollowsController do - render_views - - describe 'GET #show' do - describe 'when signed out' do - it 'redirects to sign in page' do - get :show - - expect(response).to redirect_to(new_user_session_path) - end - end - - describe 'when signed in' do - let(:user) { Fabricate(:user) } - let(:account) { Fabricate(:account, user: user) } - - before do - sign_in(user) - end - - it 'renders error without acct param' do - get :show - - expect(response).to render_template(:error) - end - - it 'renders error when account cant be found' do - service = double - allow(ResolveAccountService).to receive(:new).and_return(service) - allow(service).to receive(:call).with('missing@hostname').and_return(nil) - - get :show, params: { acct: 'acct:missing@hostname' } - - expect(response).to render_template(:error) - expect(service).to have_received(:call).with('missing@hostname') - end - - it 'sets account from url' do - account = Account.new - service = double - allow(FetchRemoteAccountService).to receive(:new).and_return(service) - allow(service).to receive(:call).with('http://example.com').and_return(account) - - get :show, params: { acct: 'http://example.com' } - - expect(response).to have_http_status(200) - expect(assigns(:account)).to eq account - end - - it 'sets account from acct uri' do - account = Account.new - service = double - allow(ResolveAccountService).to receive(:new).and_return(service) - allow(service).to receive(:call).with('found@hostname').and_return(account) - - get :show, params: { acct: 'acct:found@hostname' } - - expect(response).to have_http_status(200) - expect(assigns(:account)).to eq account - end - end - end - - describe 'POST #create' do - describe 'when signed out' do - it 'redirects to sign in page' do - post :create - - expect(response).to redirect_to(new_user_session_path) - end - end - - describe 'when signed in' do - let(:user) { Fabricate(:user) } - let(:account) { Fabricate(:account, user: user) } - - before do - sign_in(user) - end - - it 'shows error when account not found' do - service = double - allow(FollowService).to receive(:new).and_return(service) - allow(service).to receive(:call).with(account, 'user@hostname').and_return(nil) - - post :create, params: { acct: 'acct:user@hostname' } - - expect(service).to have_received(:call).with(account, 'user@hostname') - expect(response).to render_template(:error) - end - - it 'follows account when found' do - target_account = Fabricate(:account) - result_account = double(target_account: target_account) - service = double - allow(FollowService).to receive(:new).and_return(service) - allow(service).to receive(:call).with(account, 'user@hostname').and_return(result_account) - - post :create, params: { acct: 'acct:user@hostname' } - - expect(service).to have_received(:call).with(account, 'user@hostname') - expect(response).to render_template(:success) - end - end - end -end diff --git a/spec/controllers/authorize_interactions_controller_spec.rb b/spec/controllers/authorize_interactions_controller_spec.rb new file mode 100644 index 000000000..81fd9ceb7 --- /dev/null +++ b/spec/controllers/authorize_interactions_controller_spec.rb @@ -0,0 +1,111 @@ +# frozen_string_literal: true + +require 'rails_helper' + +describe AuthorizeInteractionsController do + render_views + + describe 'GET #show' do + describe 'when signed out' do + it 'redirects to sign in page' do + get :show + + expect(response).to redirect_to(new_user_session_path) + end + end + + describe 'when signed in' do + let(:user) { Fabricate(:user) } + let(:account) { Fabricate(:account, user: user) } + + before do + sign_in(user) + end + + it 'renders error without acct param' do + get :show + + expect(response).to render_template(:error) + end + + it 'renders error when account cant be found' do + service = double + allow(ResolveAccountService).to receive(:new).and_return(service) + allow(service).to receive(:call).with('missing@hostname').and_return(nil) + + get :show, params: { acct: 'acct:missing@hostname' } + + expect(response).to render_template(:error) + expect(service).to have_received(:call).with('missing@hostname') + end + + it 'sets resource from url' do + account = Account.new + service = double + allow(ResolveURLService).to receive(:new).and_return(service) + allow(service).to receive(:call).with('http://example.com').and_return(account) + + get :show, params: { acct: 'http://example.com' } + + expect(response).to have_http_status(200) + expect(assigns(:resource)).to eq account + end + + it 'sets resource from acct uri' do + account = Account.new + service = double + allow(ResolveAccountService).to receive(:new).and_return(service) + allow(service).to receive(:call).with('found@hostname').and_return(account) + + get :show, params: { acct: 'acct:found@hostname' } + + expect(response).to have_http_status(200) + expect(assigns(:resource)).to eq account + end + end + end + + describe 'POST #create' do + describe 'when signed out' do + it 'redirects to sign in page' do + post :create + + expect(response).to redirect_to(new_user_session_path) + end + end + + describe 'when signed in' do + let!(:user) { Fabricate(:user) } + let!(:account) { user.account } + + before do + sign_in(user) + end + + it 'shows error when account not found' do + service = double + + allow(ResolveAccountService).to receive(:new).and_return(service) + allow(service).to receive(:call).with('user@hostname').and_return(nil) + + post :create, params: { acct: 'acct:user@hostname' } + + expect(response).to render_template(:error) + end + + it 'follows account when found' do + target_account = Fabricate(:account) + service = double + + allow(ResolveAccountService).to receive(:new).and_return(service) + allow(service).to receive(:call).with('user@hostname').and_return(target_account) + + post :create, params: { acct: 'acct:user@hostname' } + + expect(service).to have_received(:call).with('user@hostname') + expect(account.following?(target_account)).to be true + expect(response).to render_template(:success) + end + end + end +end diff --git a/spec/controllers/intents_controller_spec.rb b/spec/controllers/intents_controller_spec.rb index 3dde7f835..ddfd5ea36 100644 --- a/spec/controllers/intents_controller_spec.rb +++ b/spec/controllers/intents_controller_spec.rb @@ -13,7 +13,7 @@ RSpec.describe IntentsController, type: :controller do context 'when host is follow' do let(:uri) { 'web+mastodon://follow?uri=test' } - it { is_expected.to redirect_to authorize_follow_path(acct: 'test') } + it { is_expected.to redirect_to authorize_interaction_path(uri: 'test') } end context 'when host is share' do -- cgit