Added user follow testing

hcarreras · hcarreras · commit 5170c66917c3 · 2014-02-09T18:22:06.000+02:00
diff --git a/app/models/user.rb b/app/models/user.rb
@@ -79,7 +79,7 @@ def unfollow! followee
   # this method returns true if we're following the given Hacker, and
   # false otherwise
   def following? hacker
-    following.include? hacker
+    following_ids.include? hacker.id
   end
 
   # Everyone should have at least one follower. And steve would like to follow
diff --git a/spec/following_policy_spec.rb b/spec/following_policy_spec.rb
@@ -1,28 +1,28 @@
 require_relative '../app/models/following_policy'
 
 describe FollowingPolicy do
-  before(:each) do 
+  before(:each) do
     @followee = double 'followee'
     @follower = double 'follower'
   end
-  
+
   it 'can follow another' do
     @follower.should_receive(:following?).with(@followee).and_return(false)
     policy = FollowingPolicy.new(@follower)
     policy.can_follow?(@followee).should be_true
   end
-  
+
   it "cannot follow self" do
     policy = FollowingPolicy.new(@follower)
     policy.following_self?(@follower).should be_true
     policy.can_follow?(@follower).should be_false
   end
-  
+
   it "cannot follow twice" do
     @follower.should_receive(:following?).twice.with(@followee).and_return(true)
     policy = FollowingPolicy.new(@follower)
     policy.already_following?(@followee).should be_true
     policy.can_follow?(@followee).should be_false
   end
-  
+
 end
diff --git a/spec/models/user_spec.rb b/spec/models/user_spec.rb
@@ -0,0 +1,19 @@
+require 'spec_helper'
+
+describe User do
+
+  let(:bob){Fabricate(:user)}
+  let(:mozart){Fabricate(:user)}
+
+  it 'can be created validly' do
+    bob.should be_valid
+    mozart.should be_valid
+  end
+
+  it 'Users can follow and unfollow' do
+    bob.follow!(mozart)
+    bob.following?(mozart).should be_true
+    bob.unfollow!(mozart)
+    bob.following?(mozart).should be_false
+  end
+end
