about summary refs log tree commit diff
diff options
context:
space:
mode:
authorAkihiko Odaki <akihiko.odaki.4i@stu.hosei.ac.jp>2017-05-23 23:34:28 +0900
committerEugen Rochko <eugen@zeonfederated.com>2017-05-23 16:34:28 +0200
commit152b4d54e8637a47d8a85b157d99d950fbf5bc1a (patch)
treec872830b816aaecfafb013d48a955e88a99c3753
parentea2ef16ea42d61b34653246a76ab2a5ba767ec15 (diff)
Cover StatusesController more (#3259)
-rw-r--r--spec/controllers/statuses_controller_spec.rb78
1 files 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