From a8736aab7a27fd09cf30df13400becfe39a15ba1 Mon Sep 17 00:00:00 2001
From: Akihiko Odaki <akihiko.odaki.4i@stu.hosei.ac.jp>
Date: Thu, 25 May 2017 23:07:38 +0900
Subject: [PATCH] Cover TagsController more (#3306)

---
 spec/controllers/tags_controller_spec.rb | 62 ++++++++++++++++++++----
 1 file changed, 52 insertions(+), 10 deletions(-)

diff --git a/spec/controllers/tags_controller_spec.rb b/spec/controllers/tags_controller_spec.rb
index f1119ebce2f..3f46c14c0df 100644
--- a/spec/controllers/tags_controller_spec.rb
+++ b/spec/controllers/tags_controller_spec.rb
@@ -4,19 +4,61 @@ RSpec.describe TagsController, type: :controller do
   render_views
 
   describe 'GET #show' do
-    before do
-      Fabricate(:tag, name: 'test')
+    let!(:tag)     { Fabricate(:tag, name: 'test') }
+    let!(:local)  { Fabricate(:status, tags: [ tag ], text: 'local #test') }
+    let!(:remote) { Fabricate(:status, tags: [ tag ], text: 'remote #test', account: Fabricate(:account, domain: 'remote')) }
+    let!(:late)  { Fabricate(:status, tags: [ tag ], text: 'late #test') }
+
+    context 'when tag exists' do
+      it 'returns http success' do
+        get :show, params: { id: 'test', max_id: late.id }
+        expect(response).to have_http_status(:success)
+      end
+
+      it 'renders public layout' do
+        get :show, params: { id: 'test', max_id: late.id }
+        expect(response).to render_template layout: 'public'
+      end
+
+      it 'renders only local statuses if local parameter is specified' do
+        get :show, params: { id: 'test', local: true, max_id: late.id }
+
+        expect(assigns(:tag)).to eq tag
+        statuses = assigns(:statuses).to_a
+        expect(statuses.size).to eq 1
+        expect(statuses[0]).to eq local
+      end
+
+      it 'renders local and remote statuses if local parameter is not specified' do
+        get :show, params: { id: 'test', max_id: late.id }
+
+        expect(assigns(:tag)).to eq tag
+        statuses = assigns(:statuses).to_a
+        expect(statuses.size).to eq 2
+        expect(statuses[0]).to eq remote
+        expect(statuses[1]).to eq local
+      end
+
+      it 'filters statuses by the current account' do
+        user = Fabricate(:user)
+        user.account.block!(remote.account)
+
+        sign_in(user)
+        get :show, params: { id: 'test', max_id: late.id }
+
+        expect(assigns(:tag)).to eq tag
+        statuses = assigns(:statuses).to_a
+        expect(statuses.size).to eq 1
+        expect(statuses[0]).to eq local
+      end
     end
 
-    it 'returns http success' do
-      get :show, params: { id: 'test' }
-      expect(response).to have_http_status(:success)
-    end
+    context 'when tag does not exist' do
+      it 'returns http missing for non-existent tag' do
+        get :show, params: { id: 'none' }
 
-    it 'returns http missing for non-existent tag' do
-      get :show, params: { id: 'none' }
-
-      expect(response).to have_http_status(:missing)
+        expect(response).to have_http_status(:missing)
+      end
     end
   end
 end