commit: 152b4d54e8637a47d8a85b157d99d950fbf5bc1a
parent: ea2ef16ea42d61b34653246a76ab2a5ba767ec15
Author: Akihiko Odaki <akihiko.odaki.4i@stu.hosei.ac.jp>
Date: Tue, 23 May 2017 23:34:28 +0900
Cover StatusesController more (#3259)
Diffstat:
1 file changed, 74 insertions(+), 4 deletions(-)
diff --git 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