When a merge request is created (confirmed_by_user), each reviewer will have a favorite created.

- Support creation of favorites without creating an event.

2 files changed, 41 insertions, 0 deletions

diff --git a/test/unit/favorite_test.rb b/test/unit/favorite_test.rb
index 837095e..6970c3c 100644
--- a/test/unit/favorite_test.rb
+++ b/test/unit/favorite_test.rb
@@ -83,6 +83,14 @@ class FavoriteTest < ActiveSupport::TestCase
         assert new_favorite.save
       end
     end
+
+    should "support a skip_events setter" do
+      favorite = @user.favorites.build(:skip_events => true, :watchable => @repo)
+      assert favorite.skip_events?
+      assert !favorite.event_should_be_created?
+      assert favorite.save
+      assert !favorite.event_exists?
+    end
   end
 
   context "Watching merge requests" do
diff --git a/test/unit/merge_request_test.rb b/test/unit/merge_request_test.rb
index f037c7b..60d5c5a 100644
--- a/test/unit/merge_request_test.rb
+++ b/test/unit/merge_request_test.rb
@@ -794,4 +794,37 @@ class MergeRequestTest < ActiveSupport::TestCase
     end
   end
 
+  context "Reviewers" do
+    setup do
+      @source_repository = repositories(:johans)
+      @user = @source_repository.user
+      @target_repository = repositories(:moes)
+      @merge_request = @target_repository.merge_requests.build(
+        :source_repository => @source_repository,
+        :summary => "Please merge",
+        :sha_snapshot => "ffac",
+        :ending_commit => "caff",
+        :user => @user
+        )
+    end
+
+    should "be accessible from the merge request" do
+      assert_equal(@merge_request.target_repository.reviewers.uniq.reject{|r|r == @merge_request.user},
+        @merge_request.reviewers)
+    end
+    
+    should "add a favorite for each reviewer" do
+      @merge_request.expects(:add_to_reviewers_favorites).times(@merge_request.reviewers.size)
+      @merge_request.notify_subscribers_about_creation
+    end
+
+    should "add self to reviewer's favorites" do
+      reviewer = users(:johan)
+      assert_incremented_by(reviewer.favorites, :size, 1) do
+        @merge_request.add_to_reviewers_favorites(reviewer)
+      end
+    end
+    
+  end
+
 end