From 152b4d54e8637a47d8a85b157d99d950fbf5bc1a Mon Sep 17 00:00:00 2001 From: Akihiko Odaki Date: Tue, 23 May 2017 23:34:28 +0900 Subject: [PATCH] Cover StatusesController more (#3259) --- spec/controllers/statuses_controller_spec.rb | 78 +++++++++++++++++++- 1 file changed, 74 insertions(+), 4 deletions(-) diff --git a/spec/controllers/statuses_controller_spec.rb b/spec/controllers/statuses_controller_spec.rb index 06ee23f3d..c329996c8 100644 --- a/spec/controllers/statuses_controller_spec.rb +++ b/spec/controllers/statuses_controller_spec.rb @@ -6,11 +6,81 @@ describe StatusesController do render_views describe '#show' do - it 'returns a success' do - status = Fabricate(:status) - get :show, params: { account_username: status.account.username, id: status.id } + context 'account is suspended' do + it 'returns gone' do + account = Fabricate(:account, suspended: true) + status = Fabricate(:status, account: account) - expect(response).to have_http_status(:success) + get :show, params: { account_username: account.username, id: status.id } + + expect(response).to have_http_status(410) + end + end + + context 'status is not permitted' do + it 'raises ActiveRecord::RecordNotFound' do + user = Fabricate(:user) + status = Fabricate(:status) + status.account.block!(user.account) + + sign_in(user) + get :show, params: { account_username: status.account.username, id: status.id } + + expect(response).to have_http_status(404) + end + end + + context 'account is not suspended and status is permitted' do + it 'assigns @account' do + status = Fabricate(:status) + get :show, params: { account_username: status.account.username, id: status.id } + expect(assigns(:account)).to eq status.account + end + + it 'assigns @status' do + status = Fabricate(:status) + get :show, params: { account_username: status.account.username, id: status.id } + expect(assigns(:status)).to eq status + end + + it 'assigns @stream_entry' do + status = Fabricate(:status) + get :show, params: { account_username: status.account.username, id: status.id } + expect(assigns(:stream_entry)).to eq status.stream_entry + end + + it 'assigns @type' do + status = Fabricate(:status) + get :show, params: { account_username: status.account.username, id: status.id } + expect(assigns(:type)).to eq 'status' + end + + it 'assigns @ancestors for ancestors of the status if it is a reply' do + ancestor = Fabricate(:status) + status = Fabricate(:status, in_reply_to_id: ancestor.id) + + get :show, params: { account_username: status.account.username, id: status.id } + + expect(assigns(:ancestors)).to match_array([ancestor]) + end + + it 'assigns @ancestors for [] if it is not a reply' do + status = Fabricate(:status) + get :show, params: { account_username: status.account.username, id: status.id } + expect(assigns(:ancestors)).to eq [] + end + + it 'returns a success' do + status = Fabricate(:status) + get :show, params: { account_username: status.account.username, id: status.id } + expect(response).to have_http_status(:success) + end + + it 'renders stream_entries/show' do + status = Fabricate(:status) + get :show, params: { account_username: status.account.username, id: status.id } + expect(response).to render_template 'stream_entries/show' + end end end end