From dd67107ab1de0b13cad0e184ca54a6ac4c2efe37 Mon Sep 17 00:00:00 2001
From: Raphael Sofaer <raphael@joindiaspora.com>
Date: Wed, 15 Jun 2011 15:19:36 -0700
Subject: [PATCH] Make LikesController spec a bit more specific
---
spec/controllers/likes_controller_spec.rb | 37 ++++++++++++-----------
1 file changed, 19 insertions(+), 18 deletions(-)
diff --git a/spec/controllers/likes_controller_spec.rb b/spec/controllers/likes_controller_spec.rb
index 4d95f9172d..8ec331a085 100644
--- a/spec/controllers/likes_controller_spec.rb
+++ b/spec/controllers/likes_controller_spec.rb
@@ -73,27 +73,28 @@ describe LikesController do
end
describe '#destroy' do
- context 'your like' do
- before do
- @message = bob.post(:status_message, :text => "hey", :to => @aspect1.id)
- @like = alice.build_like(:positive => true, :post => @message)
- @like.save
- end
+ before do
+ @message = bob.post(:status_message, :text => "hey", :to => @aspect1.id)
+ @like = alice.build_like(:positive => true, :post => @message)
+ @like.save
+ end
- it 'lets a user destroy their like' do
- expect {
- delete :destroy, :format => "js", :post_id => @like.post_id, :id => @like.id
- }.should change(Like, :count).by(-1)
- end
+ it 'lets a user destroy their like' do
+ expect {
+ delete :destroy, :format => "js", :post_id => @like.post_id, :id => @like.id
+ }.should change(Like, :count).by(-1)
+ response.status.should == 200
+ end
- it 'does not let a user destroy other likes' do
- like2 = eve.build_like(:positive => true, :post => @message)
- like2.save
+ it 'does not let a user destroy other likes' do
+ like2 = eve.build_like(:positive => true, :post => @message)
+ like2.save
- expect {
- delete :destroy, :format => "js", :post_id => like2.post_id, :id => like2.id
- }.should_not change(Like, :count)
- end
+ expect {
+ delete :destroy, :format => "js", :post_id => like2.post_id, :id => like2.id
+ }.should_not change(Like, :count)
+
+ response.status.should == 403
end
end
end
--
GitLab