diff --git a/acts_as_follower.gemspec b/acts_as_follower.gemspec index 7975494..766e7cd 100644 --- a/acts_as_follower.gemspec +++ b/acts_as_follower.gemspec @@ -19,7 +19,7 @@ Gem::Specification.new do |s| s.require_paths = ["lib"] s.add_development_dependency "sqlite3" - s.add_development_dependency "shoulda" - s.add_development_dependency "factory_girl" + s.add_development_dependency "shoulda", "~>2.11.3" + s.add_development_dependency "factory_girl", "~>2.6.0" s.add_development_dependency "rails", "~>3.0.10" end diff --git a/lib/acts_as_follower/followable.rb b/lib/acts_as_follower/followable.rb index 7582859..178867f 100644 --- a/lib/acts_as_follower/followable.rb +++ b/lib/acts_as_follower/followable.rb @@ -89,7 +89,7 @@ def get_follow_for(follower) private def block_future_follow(follower) - follows.create(:followable => self, :follower => follower, :blocked => true) + follows.create(:followable => self, :follower => follower, :blocked => true) unless follower == self end def block_existing_follow(follower) diff --git a/test/acts_as_followable_test.rb b/test/acts_as_followable_test.rb index 8b9364d..8488821 100644 --- a/test/acts_as_followable_test.rb +++ b/test/acts_as_followable_test.rb @@ -111,6 +111,16 @@ class ActsAsFollowableTest < ActiveSupport::TestCase end end + context "blocking yourself" do + setup do + @jon.block(@jon) + end + + should "not add self to the blocked followers" do + assert_equal 0, @jon.blocked_followers_count + end + end + context "blocking a follower" do context "in my following list" do setup do