AI1826

Removed old create_article activities Created new actvities for articles created on the last 8 days Create a new activity for article if activity was removed Fixed tests

AI1826
Removed old create_article activities Created new actvities for articles created on the last 8 days Create a new activity for article if activity was removed Fixed tests
Daniela Feitosa
1 parent e3f53121
Showing 6 changed files with 93 additions and 47 deletions Show diff stats
app/models/article.rb
app/models/comment.rb
db/migrate/20111228202739_remove_useless_tracked_actions.rb
db/migrate/20120228202739_adapt_create_articles_activity.rb
test/unit/article_test.rb
test/unit/comment_test.rb
@@ -561,6 +561,12 @@ class Article &lt; ActiveRecord::Base
     ActionTracker::Record.find_by_target_type_and_target_id 'Article', self.id
   end
+  def create_activity
+    if is_trackable? && !image?
+      save_action_for_verb 'create_article', [:name, :url, :lead, :first_image], Proc.new{}, :author
+    end
+  end
+
   def first_image
     img = Hpricot(self.lead.to_s).search('img[@src]').first || Hpricot(self.body.to_s).search('img').first
     img.nil? ? '' : img.attributes['src']
@@ -80,9 +80,12 @@ class Comment &lt; ActiveRecord::Base
       Comment::Notifier.deliver_mail(comment)
     end
-    if comment.source.kind_of?(Article) && comment.article.activity
-      comment.article.activity.increment!(:comments_count)
-      comment.article.activity.update_attribute(:visible, true)
+    if comment.source.kind_of?(Article)
+      comment.article.create_activity if comment.article.activity.nil?
+      if comment.article.activity
+        comment.article.activity.increment!(:comments_count)
+        comment.article.activity.update_attribute(:visible, true)
+      end
     end
   end
@@ -7,7 +7,6 @@ class RemoveUselessTrackedActions &lt; ActiveRecord::Migration
         if (activity.updated_at.to_time < Time.now.months_ago(3)) || verbs.include?(activity.verb)
           activity.destroy
         end
-      #    select_all("SELECT id, verb, updated_at FROM action_tracker WHERE verb IN ('create_article', 'update_article', 'remove_article', 'leave_comment', 'leave_community', 'remove_member_in_community')").each do |tracker|
       end
     end
   end
@@ -0,0 +1,24 @@
+class AdaptCreateArticlesActivity < ActiveRecord::Migration
+
+  # Removing 'create_article' activities that grouped articles.
+  # Creating new activities only to recent articles (not grouping)
+  def self.up
+    select_all("SELECT id FROM action_tracker WHERE verb = 'create_article'").each do |tracker|
+      activity = ActionTracker::Record.find_by_id(tracker['id'])
+      if activity
+        activity.destroy
+      end
+    end
+
+      select_all("SELECT id FROM articles").each do |art|
+      article = Article.find(art['id'])
+      if article && article.created_at >= 8.days.ago && article.author && article.author.kind_of?(Person)
+        article.create_activity
+      end
+    end
+  end
+
+  def self.down
+    say "this migration can't be reverted"
+  end
+end
@@ -976,17 +976,6 @@ assert_equal &#39;bla&#39;, profile.articles.map(&amp;:comments_count)
     assert_not_equal time, ActionTracker::Record.last.updated_at
   end
-  should 'update action when comment is created' do
-    article = create(TinyMceArticle, :profile => profile)
-    action = article.activity
-    time = action.updated_at
-
-    Time.stubs(:now).returns(time + 1.day)
-
-    comment = create(Comment, :source => article, :author => profile)
-    assert_equal time + 1.day, article.activity.updated_at
-  end
-
   should 'notifiable is false by default' do
     a = fast_create(Article)
     assert !a.notifiable?
@@ -1013,6 +1002,15 @@ assert_equal &#39;bla&#39;, profile.articles.map(&amp;:comments_count)
     assert_equal 0, ActionTracker::Record.count
   end
+  should 'create activity' do
+    a = TextileArticle.create! :name => 'bar', :profile_id => fast_create(Profile).id, :published => true
+    a.activity.destroy
+    assert_nil a.activity
+
+    a.create_activity
+    assert_not_nil a.activity
+  end
+
   should "the action_tracker_target method be defined" do
     assert Article.method_defined?(:action_tracker_target)
   end
@@ -1100,39 +1098,23 @@ assert_equal &#39;bla&#39;, profile.articles.map(&amp;:comments_count)
   end
   should 'create the notification to the friend when one friend has the notification and the other no' do
-    p1 = Person.first || fast_create(Person)
-    f1 = fast_create(Person)
-    p1.add_friend(f1)
     Article.destroy_all
     ActionTracker::Record.destroy_all
     ActionTrackerNotification.destroy_all
-    article = TinyMceArticle.create! :name => 'Tracked Article 1', :profile_id => p1.id
-    assert article.published?
-    assert_kind_of Person, article.profile
-    assert_equal 1, ActionTracker::Record.count
-    ta = ActionTracker::Record.first
-    assert_equal 'Tracked Article 1', ta.get_name.last
-    assert_equal article.url, ta.get_url.last
-    assert p1, ta.user
-    assert p1, ta.target
+
+    f1 = fast_create(Person)
+    profile.add_friend(f1)
+    article = TinyMceArticle.create! :name => 'Tracked Article 1', :profile_id => profile.id
+    assert_equal 1, ActionTracker::Record.find_all_by_verb('create_article').count
     process_delayed_job_queue
-    assert_equal 2, ActionTrackerNotification.count
+    assert_equal 2, ActionTrackerNotification.find_all_by_action_tracker_id(article.activity.id).count
     f2 = fast_create(Person)
-    p1.add_friend(f2)
-    process_delayed_job_queue
-    assert_equal 5, ActionTrackerNotification.count
-    article = TinyMceArticle.create! :name => 'Tracked Article 2', :profile_id => p1.id
-    assert article.published?
-    assert_kind_of Person, article.profile
-    assert_equal 2, ActionTracker::Record.count
-    ta = ActionTracker::Record.first
-    assert_equal 'Tracked Article 2', ta.get_name.last
-    assert_equal article.url, ta.get_url.last
-    assert_equal p1, ta.user
-    assert_equal p1, ta.target
+    profile.add_friend(f2)
+    article2 = TinyMceArticle.create! :name => 'Tracked Article 2', :profile_id => profile.id
+    assert_equal 2, ActionTracker::Record.find_all_by_verb('create_article').count
     process_delayed_job_queue
-    assert_equal 6, ActionTrackerNotification.count
+    assert_equal 3, ActionTrackerNotification.find_all_by_action_tracker_id(article2.activity.id).count
   end
   should 'found articles with published date between a range' do
@@ -65,12 +65,10 @@ class CommentTest &lt; ActiveSupport::TestCase
   should 'update counter cache in article' do
     owner = create_user('testuser').person
-    art = owner.articles.build(:name => 'ytest'); art.save!
-
+    art = create(TextileArticle, :profile_id => owner.id)
     cc = art.comments_count
-    art.comments.build(:title => 'test comment', :body => 'anything', :author => owner).save!
-    art = Article.find(art.id)
+    comment = create(Comment, :source => art, :author_id => owner.id)
     assert_equal cc + 1, Article.find(art.id).comments_count
   end
@@ -84,6 +82,18 @@ class CommentTest &lt; ActiveSupport::TestCase
     assert_equal cc + 1, ActionTracker::Record.find(action.id).comments_count
   end
+  should 'update counter cache in general activity when add a comment' do
+    person = fast_create(Person)
+    community = fast_create(Community)
+
+    activity = ActionTracker::Record.create :user => person, :target => community, :verb => 'add_member_in_community'
+
+    cc = activity.comments_count
+
+    comment = create(Comment, :source => activity, :author_id => person.id)
+    assert_equal cc + 1, ActionTracker::Record.find(activity.id).comments_count
+  end
+
   should 'provide author name for authenticated authors' do
     owner = create_user('testuser').person
     assert_equal 'testuser', Comment.new(:author => owner).author_name
@@ -329,7 +339,29 @@ class CommentTest &lt; ActiveSupport::TestCase
     assert c.rejected?
   end
-  should 'update article activity when add a comment'
   should 'update activity when add a comment'
-  should 'create a new activity when add a comment and the activity was removed'
+
+  should 'update article activity when add a comment' do
+    profile = create_user('testuser').person
+    article = create(TinyMceArticle, :profile => profile)
+    action = article.activity
+    time = action.updated_at
+
+    Time.stubs(:now).returns(time + 1.day)
+
+    comment = create(Comment, :source => article, :author => profile)
+    assert_equal time + 1.day, article.activity.updated_at
+  end
+
+  should 'create a new activity when add a comment and the activity was removed' do
+    profile = create_user('testuser').person
+    article = create(TinyMceArticle, :profile => profile)
+    article.activity.destroy
+
+    assert_nil article.activity
+
+    comment = create(Comment, :source => article, :author => profile)
+    assert_not_nil article.activity
+  end
+
 end
	@@ -561,6 +561,12 @@ class Article < ActiveRecord::Base		@@ -561,6 +561,12 @@ class Article < ActiveRecord::Base
561	ActionTracker::Record.find_by_target_type_and_target_id 'Article', self.id	561	ActionTracker::Record.find_by_target_type_and_target_id 'Article', self.id
562	end	562	end
563		563
		564	+ def create_activity
		565	+ if is_trackable? && !image?
		566	+ save_action_for_verb 'create_article', [:name, :url, :lead, :first_image], Proc.new{}, :author
		567	+ end
		568	+ end
		569	+
564	def first_image	570	def first_image
565	img = Hpricot(self.lead.to_s).search('img[@src]').first \|\| Hpricot(self.body.to_s).search('img').first	571	img = Hpricot(self.lead.to_s).search('img[@src]').first \|\| Hpricot(self.body.to_s).search('img').first
566	img.nil? ? '' : img.attributes['src']	572	img.nil? ? '' : img.attributes['src']
@@ -0,0 +1,24 @@		@@ -0,0 +1,24 @@
	1	+class AdaptCreateArticlesActivity < ActiveRecord::Migration
	2	+
	3	+ # Removing 'create_article' activities that grouped articles.
	4	+ # Creating new activities only to recent articles (not grouping)
	5	+ def self.up
	6	+ select_all("SELECT id FROM action_tracker WHERE verb = 'create_article'").each do \|tracker\|
	7	+ activity = ActionTracker::Record.find_by_id(tracker['id'])
	8	+ if activity
	9	+ activity.destroy
	10	+ end
	11	+ end
	12	+
	13	+ select_all("SELECT id FROM articles").each do \|art\|
	14	+ article = Article.find(art['id'])
	15	+ if article && article.created_at >= 8.days.ago && article.author && article.author.kind_of?(Person)
	16	+ article.create_activity
	17	+ end
	18	+ end
	19	+ end
	20	+
	21	+ def self.down
	22	+ say "this migration can't be reverted"
	23	+ end
	24	+end
	@@ -65,12 +65,10 @@ class CommentTest < ActiveSupport::TestCase		@@ -65,12 +65,10 @@ class CommentTest < ActiveSupport::TestCase
65		65
66	should 'update counter cache in article' do	66	should 'update counter cache in article' do
67	owner = create_user('testuser').person	67	owner = create_user('testuser').person
68	- art = owner.articles.build(:name => 'ytest'); art.save!
69	-	68	+ art = create(TextileArticle, :profile_id => owner.id)
70	cc = art.comments_count	69	cc = art.comments_count
71	- art.comments.build(:title => 'test comment', :body => 'anything', :author => owner).save!
72	- art = Article.find(art.id)
73		70
		71	+ comment = create(Comment, :source => art, :author_id => owner.id)
74	assert_equal cc + 1, Article.find(art.id).comments_count	72	assert_equal cc + 1, Article.find(art.id).comments_count
75	end	73	end
76		74
	@@ -84,6 +82,18 @@ class CommentTest < ActiveSupport::TestCase		@@ -84,6 +82,18 @@ class CommentTest < ActiveSupport::TestCase
84	assert_equal cc + 1, ActionTracker::Record.find(action.id).comments_count	82	assert_equal cc + 1, ActionTracker::Record.find(action.id).comments_count
85	end	83	end
86		84
		85	+ should 'update counter cache in general activity when add a comment' do
		86	+ person = fast_create(Person)
		87	+ community = fast_create(Community)
		88	+
		89	+ activity = ActionTracker::Record.create :user => person, :target => community, :verb => 'add_member_in_community'
		90	+
		91	+ cc = activity.comments_count
		92	+
		93	+ comment = create(Comment, :source => activity, :author_id => person.id)
		94	+ assert_equal cc + 1, ActionTracker::Record.find(activity.id).comments_count
		95	+ end
		96	+
87	should 'provide author name for authenticated authors' do	97	should 'provide author name for authenticated authors' do
88	owner = create_user('testuser').person	98	owner = create_user('testuser').person
89	assert_equal 'testuser', Comment.new(:author => owner).author_name	99	assert_equal 'testuser', Comment.new(:author => owner).author_name
	@@ -329,7 +339,29 @@ class CommentTest < ActiveSupport::TestCase		@@ -329,7 +339,29 @@ class CommentTest < ActiveSupport::TestCase
329	assert c.rejected?	339	assert c.rejected?
330	end	340	end
331		341
332	- should 'update article activity when add a comment'
333	should 'update activity when add a comment'	342	should 'update activity when add a comment'
334	- should 'create a new activity when add a comment and the activity was removed'	343	+
		344	+ should 'update article activity when add a comment' do
		345	+ profile = create_user('testuser').person
		346	+ article = create(TinyMceArticle, :profile => profile)
		347	+ action = article.activity
		348	+ time = action.updated_at
		349	+
		350	+ Time.stubs(:now).returns(time + 1.day)
		351	+
		352	+ comment = create(Comment, :source => article, :author => profile)
		353	+ assert_equal time + 1.day, article.activity.updated_at
		354	+ end
		355	+
		356	+ should 'create a new activity when add a comment and the activity was removed' do
		357	+ profile = create_user('testuser').person
		358	+ article = create(TinyMceArticle, :profile => profile)
		359	+ article.activity.destroy
		360	+
		361	+ assert_nil article.activity
		362	+
		363	+ comment = create(Comment, :source => article, :author => profile)
		364	+ assert_not_nil article.activity
		365	+ end
		366	+
335	end	367	end