about summary refs log tree commit diff
path: root/spec
diff options
context:
space:
mode:
authorAkihiko Odaki <akihiko.odaki.4i@stu.hosei.ac.jp>2017-05-22 23:29:48 +0900
committerEugen Rochko <eugen@zeonfederated.com>2017-05-22 16:29:48 +0200
commit422e4d897bf77bcf2f30e04f7b537608e4674002 (patch)
treef0c4a592557858932f7c98a6c0b02f06aa3a898f /spec
parentcb2707776f799022e42a71a140b3db64f2b59046 (diff)
Cover AccountFollowController more in spec (#3227)
Diffstat (limited to 'spec')
-rw-r--r--spec/controllers/account_follow_controller_spec.rb19
1 files changed, 13 insertions, 6 deletions
diff --git a/spec/controllers/account_follow_controller_spec.rb b/spec/controllers/account_follow_controller_spec.rb
index b0e646c1f..ac15499be 100644
--- a/spec/controllers/account_follow_controller_spec.rb
+++ b/spec/controllers/account_follow_controller_spec.rb
@@ -7,16 +7,23 @@ describe AccountFollowController do
   let(:alice) { Fabricate(:account, username: 'alice') }
 
   describe 'POST #create' do
-    before do
-      sign_in(user)
-    end
+    let(:service) { double }
 
-    it 'redirects to account path' do
-      service = double
+    subject { post :create, params: { account_username: alice.username } }
+
+    before do
       allow(FollowService).to receive(:new).and_return(service)
       allow(service).to receive(:call)
+    end
+
+    it 'does not create for user who is not signed in' do
+      subject
+      expect(FollowService).not_to receive(:new)
+    end
 
-      post :create, params: { account_username: alice.username }
+    it 'redirects to account path' do
+      sign_in(user)
+      subject
 
       expect(service).to have_received(:call).with(user.account, 'alice')
       expect(response).to redirect_to(account_path(alice))