From 5e23a0681dc69bdc6d74617ffd84dab8ecf8f9c0 Mon Sep 17 00:00:00 2001
From: Sam <sam.saffron@gmail.com>
Date: Fri, 2 Aug 2013 09:59:12 +1000
Subject: [PATCH] work in progress, will finalize soon

---
 app/models/user_action.rb       |  5 +++++
 spec/models/user_action_spec.rb | 21 ++++++++++++++++++---
 2 files changed, 23 insertions(+), 3 deletions(-)

diff --git a/app/models/user_action.rb b/app/models/user_action.rb
index 1a6c5da4270..23c89a89df9 100644
--- a/app/models/user_action.rb
+++ b/app/models/user_action.rb
@@ -248,8 +248,13 @@ SQL
     builder.exec
   end
 
+  def self.synchronize_favorites
+    
+  end
+
   def self.ensure_consistency!
     self.synchronize_target_topic_ids
+    self.synchronize_favorites
   end
 
   protected
diff --git a/spec/models/user_action_spec.rb b/spec/models/user_action_spec.rb
index e845e854429..cdddf7b9613 100644
--- a/spec/models/user_action_spec.rb
+++ b/spec/models/user_action_spec.rb
@@ -261,7 +261,23 @@ describe UserAction do
     end
   end
 
-  describe 'ensure_consistency!' do
+  describe 'synchronize_favorites' do
+    pending 'corrects out of sync favs' do
+      post = Fabricate(:post)
+      post.topic.toggle_star(post.user, true)
+
+      action1 = UserAction.log_action!(
+        action_type: UserAction::STAR,
+        user_id: post.user.id,
+        acting_user_id: post.user.id,
+        target_topic_id: -1,
+        target_post_id: post.id,
+      )
+
+    end
+  end
+
+  describe 'synchronize_target_topic_ids' do
     it 'correct target_topic_id' do
       post = Fabricate(:post)
 
@@ -281,11 +297,10 @@ describe UserAction do
         target_post_id: post.id,
       )
 
-      UserAction.ensure_consistency!
+      UserAction.synchronize_target_topic_ids
 
       action.reload
       action.target_topic_id.should == post.topic_id
-
     end
   end
 end