muck-friends 0.1.5 → 0.1.6

data/test/rails_root/test/functional/default_controller_test.rb

@@ -44,6 +44,179 @@ class Muck::DefaultControllerTest < ActionController::TestCase
       @controller.stubs(:other_user).returns(@quentin)
     end
     
+    context "friend link" do
+      setup do
+        @enable_following = GlobalConfig.enable_following
+        @enable_friending = GlobalConfig.enable_friending
+      end
+      teardown do
+        # Reset global config
+        GlobalConfig.enable_following = @enable_following
+        GlobalConfig.enable_friending = @enable_friending
+      end
+      
+      context "enable_following is true and enable_friending is true" do
+        setup do
+          GlobalConfig.enable_following = true
+          GlobalConfig.enable_friending = true
+        end
+        context "no current user" do
+          setup do
+            @controller.stubs(:current_user).returns(nil)
+            get :friend_link
+          end
+          should_respond_with :success
+          should_render_template :friend_link
+          should "have login/signup in the body" do
+            assert @response.body.include?(I18n.t("muck.friends.muck.friends.login_or_sign_up_to_follow"), :login => link_to(I18n.t('muck.friends.login'), login_path), :signup => link_to(I18n.t('muck.friends.signup'), signup_path))
+          end
+        end
+        context "current user is not friends with other user" do
+          setup do
+            @other_user = Factory(:user)
+            @controller.stubs(:other_user).returns(@other_user)
+            get :friend_link
+          end
+          should_respond_with :success
+          should_render_template :friend_link
+          should "have 'follow' in the body" do
+            assert @response.body.include?(I18n.t('muck.friends.start_following', :user => @other_user.display_name))
+          end
+        end
+        context "current user is friends with other user" do
+          setup do
+            @controller.stubs(:other_user).returns(@friend1)
+            get :friend_link
+          end
+          should_respond_with :success
+          should_render_template :friend_link
+          should "have 'stop being friends' in the body" do
+            assert @response.body.include?(I18n.t('muck.friends.stop_being_friends_with', :user => @friend1.display_name))
+          end
+        end
+        context "current user is not friend with other user, but other user follows current user" do
+          setup do
+            @controller.stubs(:other_user).returns(@follower1)
+            get :friend_link
+          end
+          should_respond_with :success
+          should_render_template :friend_link
+          should "have 'accept friend request' in the body" do
+            assert @response.body.include?(I18n.t('muck.friends.acccept_friend_request', :user => @follower1.display_name))
+          end
+        end
+      end
+      
+      context "enable_following is true and enable_friending is false" do
+        setup do
+          GlobalConfig.enable_following = true
+          GlobalConfig.enable_friending = false
+        end
+        context "no current user" do
+          setup do
+            @controller.stubs(:current_user).returns(nil)
+            get :friend_link
+          end
+          should_respond_with :success
+          should_render_template :friend_link
+          should "have login/signup in the body" do
+            assert @response.body.include?(I18n.t("muck.friends.muck.friends.login_or_sign_up_to_follow"), :login => link_to(I18n.t('muck.friends.login'), login_path), :signup => link_to(I18n.t('muck.friends.signup'), signup_path))
+          end
+        end
+        context "current user is not following other user" do
+          setup do
+            @other_user = Factory(:user)
+            @controller.stubs(:other_user).returns(@other_user)
+            get :friend_link
+          end
+          should_respond_with :success
+          should_render_template :friend_link
+          should "have 'follow' in the body" do
+            assert @response.body.include?(I18n.t('muck.friends.start_following', :user => @other_user.display_name))
+          end
+        end
+        context "current user is following the other user" do
+          setup do
+            @being_followed = Factory(:user)
+            @aaron.follow(@being_followed)
+            @controller.stubs(:other_user).returns(@being_followed)
+            get :friend_link
+          end
+          should_respond_with :success
+          should_render_template :friend_link
+          should "have 'stop following' in the body" do
+            assert @response.body.include?(I18n.t('muck.friends.stop_following', :user => @being_followed.display_name))
+          end
+        end
+        context "current user is not friend with other user, but other user follows current user" do
+          setup do
+            @controller.stubs(:other_user).returns(@follower1)
+            get :friend_link
+          end
+          should_respond_with :success
+          should_render_template :friend_link
+          should "have 'follow' in the body" do
+            assert @response.body.include?(I18n.t('muck.friends.start_following', :user => @follower1.display_name))
+          end
+        end
+      end
+      
+      context "enable_following is false and enable_friending is true" do
+        setup do
+          GlobalConfig.enable_following = false
+          GlobalConfig.enable_friending = true
+        end
+        context "no current user" do
+          setup do
+            @controller.stubs(:current_user).returns(nil)
+            get :friend_link
+          end
+          should_respond_with :success
+          should_render_template :friend_link
+          should "have login/signup in the body" do
+            assert @response.body.include?(I18n.t("muck.friends.muck.friends.login_or_sign_up_to_friend"), :login => link_to(I18n.t('muck.friends.login'), login_path), :signup => link_to(I18n.t('muck.friends.signup'), signup_path))
+          end
+        end
+        context "current user is not friends with the other user" do
+          setup do
+            @other_user = Factory(:user)
+            @controller.stubs(:other_user).returns(@other_user)
+            get :friend_link
+          end
+          should_respond_with :success
+          should_render_template :friend_link
+          should "have 'friend request' in the body" do
+            assert @response.body.include?(I18n.t('muck.friends.friend_request_prompt', :user => @other_user.display_name))
+          end
+        end
+        context "current user has sent a friend request to the other user" do
+          setup do
+            @being_followed = Factory(:user)
+            @aaron.follow(@being_followed)
+            @controller.stubs(:other_user).returns(@being_followed)
+            get :friend_link
+          end
+          should_respond_with :success
+          should_render_template :friend_link
+          should "have 'friend request pending' in the body" do
+            assert @response.body.include?(I18n.t('muck.friends.friend_request_pending', :link => ''))
+          end
+        end
+        context "current user is not friend with other user, but other user has sent a friend request" do
+          setup do
+            @controller.stubs(:other_user).returns(@follower1)
+            get :friend_link
+          end
+          should_respond_with :success
+          should_render_template :friend_link
+          should "have 'accept friend request' in the body" do
+            assert @response.body.include?(I18n.t('muck.friends.acccept_friend_request', :user => @follower1.display_name))
+          end
+        end
+      end
+      
+    end
+
     context 'all friends' do
       setup do
         get :all_friends
@@ -51,7 +224,7 @@ class Muck::DefaultControllerTest < ActionController::TestCase
       should_respond_with :success
       should_render_template :all_friends
     end
-  
+
     context 'mutual friends' do
       setup do
         get :mutual_friends
@@ -86,12 +259,12 @@ class Muck::DefaultControllerTest < ActionController::TestCase
     
     context 'friend requests' do
       setup do
-        @allow_following = GlobalConfig.allow_following
-        GlobalConfig.allow_following = false
+        @enable_following = GlobalConfig.enable_following
+        GlobalConfig.enable_following = false
         get :friend_requests
       end
       teardown do
-        GlobalConfig.allow_following = @allow_following
+        GlobalConfig.enable_following = @enable_following
       end
       should_respond_with :success
       should_render_template :friend_requests

data/test/rails_root/test/functional/friends_controller_test.rb

@@ -107,16 +107,16 @@ class Muck::FriendsControllerTest < ActionController::TestCase
       
       context "Following allowed" do
         setup do
-          @temp_allow_following = GlobalConfig.allow_following
-          GlobalConfig.allow_following = true
+          @temp_enable_following = GlobalConfig.enable_following
+          GlobalConfig.enable_following = true
           Friend.destroy_all
           @aaron.follow(@quentin)
           @quentin.become_friends_with(@aaron)
         end
         teardown do
-          GlobalConfig.allow_following = @temp_allow_following
+          GlobalConfig.enable_following = @temp_enable_following
         end
-        context 'js request' do
+        context 'DELETE to destroy js request' do
           setup do
             delete :destroy, { :user_id => @quentin.to_param, :id => @aaron.to_param, :format => 'js'}
           end
@@ -129,7 +129,7 @@ class Muck::FriendsControllerTest < ActionController::TestCase
             assert @quentin.followed_by?(@aaron)
           end
         end
-        context 'html request' do
+        context 'DELETE to destroy  html request' do
           setup do
             delete :destroy, { :user_id => @quentin.to_param, :id => @aaron.to_param}
           end
@@ -139,14 +139,14 @@ class Muck::FriendsControllerTest < ActionController::TestCase
       
       context 'Following not allowed' do
         setup do
-          @temp_allow_following = GlobalConfig.allow_following
-          GlobalConfig.allow_following = false
+          @temp_enable_following = GlobalConfig.enable_following
+          GlobalConfig.enable_following = false
           Friend.destroy_all
           @aaron.follow(@quentin)
           @quentin.become_friends_with(@aaron)          
         end
         teardown do
-          GlobalConfig.allow_following = @temp_allow_following
+          GlobalConfig.enable_following = @temp_enable_following
         end
         context 'js request' do
           setup do

data/test/rails_root/test/unit/user_test.rb

@@ -32,139 +32,176 @@ class UserTest < ActiveSupport::TestCase
       @follower_guy = Factory(:user)
     end
 
-    should "become friends" do
-      assert @aaron.follow(@quentin)
-      assert @quentin.become_friends_with(@aaron)
-      debugger
-      
-    end
-    
-    should "have friends" do
-      @aaron.follow(@quentin)
-      @quentin.become_friends_with(@aaron)
-      @friend_guy.follow(@quentin)
-      @quentin.become_friends_with(@friend_guy)
-      @quentin.reload
-      @aaron.reload
-      @friend_guy.reload
-      assert @quentin.friends.any?{|f| f.id == @aaron.id || f.id == @friend_guy.id}
-      assert @aaron.friends.any?{|f| f.id == @quentin.id}
-      assert @friend_guy.friends.any?{|f| f.id == @quentin.id}
-    end
-    
-    should "not have follower as friend" do
-      @follower_guy.follow(@quentin)
-      assert !@quentin.friends.any?{|f| f.id == @follower_guy.id}
+    context "following disabled" do
+      setup do
+        @temp_enable_following = GlobalConfig.enable_following
+        GlobalConfig.enable_following = false
+      end
+      teardown do
+        GlobalConfig.enable_following = @temp_enable_following
+      end
+      should "stop being friends" do
+        assert @quentin.follow(@aaron)
+        assert @aaron.become_friends_with(@quentin)
+        assert @quentin.drop_friend(@aaron)
+        @quentin.reload
+        @aaron.reload
+        assert !@quentin.friend_of?(@aaron)
+        assert !@quentin.following?(@aaron)
+        assert !@quentin.followed_by?(@aaron)
+        assert !@aaron.friend_of?(@quentin)
+        assert !@aaron.following?(@quentin)
+        assert !@aaron.followed_by?(@quentin)
+      end
+      context "drop_friend following not enabled" do
+        setup do
+          assert @quentin.follow(@aaron)
+          assert @aaron.become_friends_with(@quentin)
+          assert @quentin.drop_friend(@aaron)
+        end
+        should "stop being friends with the user and don't retain follow" do
+          assert !@quentin.followings.any?{|f| f.id == @aaron.id}
+          assert !@aaron.followings.any?{|f| f.id == @quentin.id}
+        end
+      end
     end
 
-    should "have followers" do
-      @follower_guy.follow(@quentin)
-      assert @quentin.followers.any?{|f| f.id == @follower_guy.id}
-    end
-    
-    should "have people user follows (followings)" do
-      @follower_guy.follow(@quentin)
-      assert @follower_guy.followings.any?{|f| f.id == @quentin.id}
-    end
-    
-    should "be a friend" do
-      @quentin.follow(@aaron)
-      @quentin.follow(@friend_guy)
-      @aaron.become_friends_with(@quentin)
-      @friend_guy.become_friends_with(@quentin)
-      assert_equal @quentin.occurances_as_friend.count, 2
-    end
-    
-    should "find friendships initiated by me" do
-      @quentin.follow(@aaron)
-      @quentin.follow(@friend_guy)
-      assert_equal 2, @quentin.friendships_initiated_by_me.count
-      assert @quentin.friendships_initiated_by_me.any?{|f| f.id == @aaron.id || f.id == @friend_guy.id}
-    end
-    
-    should "find friendships not initiated by me" do
-      @aaron.follow(@quentin)
-      @friend_guy.follow(@quentin)
-      assert_equal 2, @quentin.friendships_not_initiated_by_me.count
-      assert @quentin.friendships_not_initiated_by_me.any?{|f| f.id == @aaron.id || f.id == @friend_guy.id}
-    end
-    
-    should "follow the user" do
-      @quentin.follow(@aaron)
-      assert @quentin.followings.any?{|f| f.id == @aaron.id}
-    end
-    
-    should "stop following the user" do
-      @quentin.follow(@aaron)
-      @quentin.stop_following(@aaron)
-      assert !@quentin.followings.any?{|f| f.id == @aaron.id}
-    end
-    
-    should "become friends" do
-      @quentin.follow(@aaron)
-      @aaron.become_friends_with(@quentin)
-      @quentin.reload
-      @aaron.reload
-      assert @aaron.friends.any?{|f| f.id == @quentin.id}
-      assert @quentin.friends.any?{|f| f.id == @aaron.id}
-    end
+    context "following enabled" do
+      setup do
+        @temp_enable_following = GlobalConfig.enable_following
+        GlobalConfig.enable_following = true
+      end
+      teardown do
+        GlobalConfig.enable_following = @temp_enable_following
+      end
+      should "stop being friends but still allow follow" do
+        assert @quentin.follow(@aaron)
+        assert @aaron.become_friends_with(@quentin)
+        assert @quentin.drop_friend(@aaron)
+        @quentin.reload
+        @aaron.reload
+        assert !@quentin.friend_of?(@aaron)
+        assert !@quentin.following?(@aaron)
+        assert @quentin.followed_by?(@aaron)
+        assert !@aaron.friend_of?(@quentin)
+        assert @aaron.following?(@quentin)
+        assert !@aaron.followed_by?(@quentin)
+      end
+      context "drop_friend" do
+        setup do
+          assert @quentin.follow(@aaron)
+        end
+        should "stop following the user" do
+          assert @aaron.become_friends_with(@quentin)
+          assert @quentin.drop_friend(@aaron)
+          assert !@quentin.followings.any?{|f| f.id == @aaron.id}
+          assert @aaron.followings.any?{|f| f.id == @quentin.id}
+        end
+        should "stop being friends with the user but retain follow" do
+          assert @quentin.drop_friend(@aaron)
+          assert !@quentin.followings.any?{|f| f.id == @aaron.id}
+        end
+      end
+      should "become friends" do
+        assert @aaron.follow(@quentin)
+        assert @quentin.become_friends_with(@aaron)
+      end
 
-    should "stop being friends" do
-      @temp_allow_following = GlobalConfig.allow_following
-      GlobalConfig.allow_following = false
-      @quentin.follow(@aaron)
-      @aaron.become_friends_with(@quentin)
-      @quentin.drop_friend(@aaron)
-      assert !@quentin.friend_of?(@aaron)
-      assert !@quentin.following?(@aaron)
-      assert !@quentin.followed_by?(@aaron)
-      assert !@aaron.friend_of?(@quentin)
-      assert !@aaron.following?(@quentin)
-      assert !@aaron.followed_by?(@quentin)
-      GlobalConfig.allow_following = @temp_allow_following
-    end
+      should "have friends" do
+        assert @aaron.follow(@quentin)
+        assert @quentin.become_friends_with(@aaron)
+        assert @friend_guy.follow(@quentin)
+        assert @quentin.become_friends_with(@friend_guy)
+        @quentin.reload
+        @aaron.reload
+        @friend_guy.reload
+        assert @quentin.friends.any?{|f| f.id == @aaron.id || f.id == @friend_guy.id}
+        assert @aaron.friends.any?{|f| f.id == @quentin.id}
+        assert @friend_guy.friends.any?{|f| f.id == @quentin.id}
+      end
 
-    should "stop being friends but still allow follow" do
-      @temp_allow_following = GlobalConfig.allow_following
-      GlobalConfig.allow_following = true
-      @quentin.follow(@aaron)
-      @aaron.become_friends_with(@quentin)
-      @quentin.drop_friend(@aaron)
-      @quentin.reload
-      @aaron.reload
-      assert !@quentin.friend_of?(@aaron)
-      assert !@quentin.following?(@aaron)
-      assert @quentin.followed_by?(@aaron)
-      assert !@aaron.friend_of?(@quentin)
-      assert @aaron.following?(@quentin)
-      assert !@aaron.followed_by?(@quentin)
-      GlobalConfig.allow_following = @temp_allow_following
-    end
-    
-    should "not have a network" do
-      assert !@quentin.has_network?
-    end
-    
-    should "have a network" do
-      @quentin.follow(@aaron)
-      assert @quentin.has_network?
-    end
-    
-    should "block user" do
-      @quentin.follow(@aaron)
-      @aaron.block_user(@quentin)
-      assert !@aaron.friends.any?{|f| f.id == @quentin.id}
-      assert !@aaron.followers.any?{|f| f.id == @quentin.id}
-      assert !@quentin.followings.any?{|f| f.id == @aaron.id}
+      should "not have follower as friend" do
+        assert @follower_guy.follow(@quentin)
+        assert !@quentin.friends.any?{|f| f.id == @follower_guy.id}
+      end
+
+      should "have followers" do
+        assert @follower_guy.follow(@quentin)
+        assert @quentin.followers.any?{|f| f.id == @follower_guy.id}
+      end
+
+      should "have people user follows (followings)" do
+        assert @follower_guy.follow(@quentin)
+        assert @follower_guy.followings.any?{|f| f.id == @quentin.id}
+      end
+
+      should "be a friend" do
+        assert @quentin.follow(@aaron)
+        assert @quentin.follow(@friend_guy)
+        assert @aaron.become_friends_with(@quentin)
+        assert @friend_guy.become_friends_with(@quentin)
+        assert_equal @quentin.occurances_as_friend.count, 2
+      end
+
+      should "find friendships initiated by me" do
+        assert @quentin.follow(@aaron)
+        assert @quentin.follow(@friend_guy)
+        assert_equal 2, @quentin.friendships_initiated_by_me.count
+        assert @quentin.friendships_initiated_by_me.any?{|f| f.id == @aaron.id || f.id == @friend_guy.id}
+      end
+
+      should "find friendships not initiated by me" do
+        assert @aaron.follow(@quentin)
+        assert @friend_guy.follow(@quentin)
+        assert_equal 2, @quentin.friendships_not_initiated_by_me.count
+        assert @quentin.friendships_not_initiated_by_me.any?{|f| f.id == @aaron.id || f.id == @friend_guy.id}
+      end
+
+      should "follow the user" do
+        assert @quentin.follow(@aaron)
+        assert @quentin.followings.any?{|f| f.id == @aaron.id}
+      end
+
+      should "stop following the user" do
+        assert @quentin.follow(@aaron)
+        assert @quentin.stop_following(@aaron)
+        assert !@quentin.followings.any?{|f| f.id == @aaron.id}
+      end
+
+      should "become friends" do
+        assert @quentin.follow(@aaron)
+        assert @aaron.become_friends_with(@quentin)
+        @quentin.reload
+        @aaron.reload
+        assert @aaron.friends.any?{|f| f.id == @quentin.id}
+        assert @quentin.friends.any?{|f| f.id == @aaron.id}
+      end
+      should "not have a network" do
+        assert !@quentin.has_network?
+      end
+
+      should "have a network" do
+        assert @quentin.follow(@aaron)
+        assert @quentin.has_network?
+      end
+
+      should "block user" do
+        assert @quentin.follow(@aaron)
+        assert @aaron.block_user(@quentin)
+        assert !@aaron.friends.any?{|f| f.id == @quentin.id}
+        assert !@aaron.followers.any?{|f| f.id == @quentin.id}
+        assert !@quentin.followings.any?{|f| f.id == @aaron.id}
+      end
+
+      should "unblock user" do
+        assert @quentin.follow(@aaron)
+        assert @aaron.block_user(@quentin)
+        assert @aaron.unblock_user(@quentin)
+        assert @aaron.followers.any?{|f| f.id == @quentin.id}
+        assert @quentin.followings.any?{|f| f.id == @aaron.id}
+      end
     end
     
-    should "unblock user" do
-      @quentin.follow(@aaron)
-      @aaron.block_user(@quentin)
-      @aaron.unblock_user(@quentin)
-      assert @aaron.followers.any?{|f| f.id == @quentin.id}
-      assert @quentin.followings.any?{|f| f.id == @aaron.id}
-    end
   end
 
 end