Removing Spaminator from AntiSpam Plugin to start it independently

Rodrigo Souto · Daniela Feitosa
1 parent e17f85d6
Showing 2 changed files with 0 additions and 168 deletions Show diff stats
plugins/anti_spam/lib/anti_spam_plugin/spaminator.rb
plugins/anti_spam/test/unit/anti_spam_plugin/spaminator_test.rb
@@ -1,115 +0,0 @@
-require 'benchmark'
-
-class AntiSpamPlugin::Spaminator
-
-  class << self
-    def run(environment)
-      instance = new(environment)
-      instance.run
-    end
-
-    def benchmark(environment)
-      puts Benchmark.measure { run(environment) }
-    end
-  end
-
-
-  def initialize(environment)
-    @environment = environment
-  end
-
-  def run
-    start_time = Time.now
-
-    process_all_comments
-    process_all_people
-    process_people_without_network
-
-    finish(start_time)
-  end
-
-  protected
-
-  def finish(start_time)
-    @environment.settings[:spaminator_last_run] = start_time
-    @environment.save!
-  end
-
-  def conditions(table)
-    last_run = @environment.settings[:spaminator_last_run]
-    if last_run
-      ["profiles.environment_id = ? AND #{table}.created_at > ?", @environment.id, last_run]
-    else
-      [ "profiles.environment_id = ?", @environment.id]
-    end
-  end
-
-  def process_all_comments
-    puts 'Processing comments ...'
-    i = 0
-    comments = Comment.joins("JOIN articles ON (comments.source_id = articles.id AND comments.source_type = 'Article') JOIN profiles ON (profiles.id = articles.profile_id)").where(conditions(:comments))
-    total = comments.count
-    comments.find_each do |comment|
-      puts "Comment #{i += 1}/#{total} (#{100*i/total}%)"
-      process_comment(comment)
-    end
-  end
-
-  def process_all_people
-    puts 'Processing people ...'
-    i = 0
-    people = Person.where(conditions(:profiles))
-    total = people.count
-    people.find_each do |person|
-      puts "Person #{i += 1}/#{total} (#{100*i/total}%)"
-      process_person(person)
-    end
-  end
-
-  def process_comment(comment)
-    comment.check_for_spam
-
-    # TODO several comments with the same content:
-    #   → disable author
-    #   → mark all of them as spam
-
-    # TODO check comments that contains URL's
-  end
-
-  def process_person(person)
-    # person is author of more than 2 comments marked as spam
-    #   → burn
-    #
-    number_of_spam_comments = Comment.spam.where(author_id => person.id).count
-    if number_of_spam_comments > 2
-      mark_as_spammer(person)
-    end
-  end
-
-  def process_people_without_network
-    # people who signed up more than one month ago, have no friends and <= 1
-    # communities
-    #
-    #   → burn
-    #   → mark their comments as spam
-    #
-    Person.where(:environment_id => @environment.id).where(['created_at < ?', Time.now - 1.month]).find_each do |person|
-      # TODO progress indicator - see process_all_people above
-      number_of_friends = person.friends.count
-      number_of_communities = person.communities.count
-      if number_of_friends == 0 && number_of_communities <= 1
-        mark_as_spammer(person)
-        Comment.where(:author_id => person.id).find_each do |comment|
-          comment.spam!
-        end
-      end
-    end
-  end
-
-  def mark_as_spammer(person)
-    # FIXME create an AbuseComplaint and finish instead of calling
-    # Person#disable directly
-    person.disable
-  end
-
-end
@@ -1,53 +0,0 @@
-require 'test_helper'
-
-class AntiSpamPluginSpaminatorTest < ActiveSupport::TestCase
-
-  def setup
-    @environment = Environment.new
-    @environment.id = 99
-    @spaminator = AntiSpamPlugin::Spaminator.new(@environment)
-    @spaminator.stubs(:puts)
-    @now = Time.now
-    Time.stubs(:now).returns(@now)
-  end
-
-  should 'search everything in the first run' do
-    assert_equal(['profiles.environment_id = ?',99], @spaminator.send(:conditions, nil))
-  end
-
-  should 'search using recorded last date' do
-    @environment.settings[:spaminator_last_run] = @now
-    assert_equal(['profiles.environment_id = ? AND table.created_at > ?', 99, @now], @spaminator.send(:conditions, 'table'))
-  end
-
-  should 'record time of last run in environment' do
-    @spaminator.expects(:process_all_comments)
-    @spaminator.expects(:process_all_people)
-    @environment.stubs(:save!)
-    @spaminator.run
-    assert_equal @now, @environment.settings[:spaminator_last_run]
-  end
-
-  should 'find all comments' do
-    @spaminator.stubs(:process_comment)
-    @spaminator.send :process_all_comments
-  end
-
-  should 'find all people' do
-    @spaminator.stubs(:process_person)
-    @spaminator.send :process_all_people
-  end
-
-  should 'find all comments newer than a date' do
-    @environment.settings[:spaminator_last_run] = Time.now - 1.month
-    @spaminator.stubs(:process_comment)
-    @spaminator.send :process_all_comments
-  end
-
-  should 'find all people newer than a date' do
-    @environment.settings[:spaminator_last_run] = Time.now - 1.month
-    @spaminator.stubs(:process_person)
-    @spaminator.send :process_all_people
-  end
-
-end
...	...	@@ -1,115 +0,0 @@
1		-require 'benchmark'
2		-
3		-class AntiSpamPlugin::Spaminator
4		-
5		- class << self
6		- def run(environment)
7		- instance = new(environment)
8		- instance.run
9		- end
10		-
11		- def benchmark(environment)
12		- puts Benchmark.measure { run(environment) }
13		- end
14		- end
15		-
16		-
17		- def initialize(environment)
18		- @environment = environment
19		- end
20		-
21		- def run
22		- start_time = Time.now
23		-
24		- process_all_comments
25		- process_all_people
26		- process_people_without_network
27		-
28		- finish(start_time)
29		- end
30		-
31		- protected
32		-
33		- def finish(start_time)
34		- @environment.settings[:spaminator_last_run] = start_time
35		- @environment.save!
36		- end
37		-
38		- def conditions(table)
39		- last_run = @environment.settings[:spaminator_last_run]
40		- if last_run
41		- ["profiles.environment_id = ? AND #{table}.created_at > ?", @environment.id, last_run]
42		- else
43		- [ "profiles.environment_id = ?", @environment.id]
44		- end
45		- end
46		-
47		- def process_all_comments
48		- puts 'Processing comments ...'
49		- i = 0
50		- comments = Comment.joins("JOIN articles ON (comments.source_id = articles.id AND comments.source_type = 'Article') JOIN profiles ON (profiles.id = articles.profile_id)").where(conditions(:comments))
51		- total = comments.count
52		- comments.find_each do \|comment\|
53		- puts "Comment #{i += 1}/#{total} (#{100*i/total}%)"
54		- process_comment(comment)
55		- end
56		- end
57		-
58		- def process_all_people
59		- puts 'Processing people ...'
60		- i = 0
61		- people = Person.where(conditions(:profiles))
62		- total = people.count
63		- people.find_each do \|person\|
64		- puts "Person #{i += 1}/#{total} (#{100*i/total}%)"
65		- process_person(person)
66		- end
67		- end
68		-
69		- def process_comment(comment)
70		- comment.check_for_spam
71		-
72		- # TODO several comments with the same content:
73		- # → disable author
74		- # → mark all of them as spam
75		-
76		- # TODO check comments that contains URL's
77		- end
78		-
79		- def process_person(person)
80		- # person is author of more than 2 comments marked as spam
81		- # → burn
82		- #
83		- number_of_spam_comments = Comment.spam.where(author_id => person.id).count
84		- if number_of_spam_comments > 2
85		- mark_as_spammer(person)
86		- end
87		- end
88		-
89		- def process_people_without_network
90		- # people who signed up more than one month ago, have no friends and <= 1
91		- # communities
92		- #
93		- # → burn
94		- # → mark their comments as spam
95		- #
96		- Person.where(:environment_id => @environment.id).where(['created_at < ?', Time.now - 1.month]).find_each do \|person\|
97		- # TODO progress indicator - see process_all_people above
98		- number_of_friends = person.friends.count
99		- number_of_communities = person.communities.count
100		- if number_of_friends == 0 && number_of_communities <= 1
101		- mark_as_spammer(person)
102		- Comment.where(:author_id => person.id).find_each do \|comment\|
103		- comment.spam!
104		- end
105		- end
106		- end
107		- end
108		-
109		- def mark_as_spammer(person)
110		- # FIXME create an AbuseComplaint and finish instead of calling
111		- # Person#disable directly
112		- person.disable
113		- end
114		-
115		-end
...	...	@@ -1,53 +0,0 @@
1		-require 'test_helper'
2		-
3		-class AntiSpamPluginSpaminatorTest < ActiveSupport::TestCase
4		-
5		- def setup
6		- @environment = Environment.new
7		- @environment.id = 99
8		- @spaminator = AntiSpamPlugin::Spaminator.new(@environment)
9		- @spaminator.stubs(:puts)
10		- @now = Time.now
11		- Time.stubs(:now).returns(@now)
12		- end
13		-
14		- should 'search everything in the first run' do
15		- assert_equal(['profiles.environment_id = ?',99], @spaminator.send(:conditions, nil))
16		- end
17		-
18		- should 'search using recorded last date' do
19		- @environment.settings[:spaminator_last_run] = @now
20		- assert_equal(['profiles.environment_id = ? AND table.created_at > ?', 99, @now], @spaminator.send(:conditions, 'table'))
21		- end
22		-
23		- should 'record time of last run in environment' do
24		- @spaminator.expects(:process_all_comments)
25		- @spaminator.expects(:process_all_people)
26		- @environment.stubs(:save!)
27		- @spaminator.run
28		- assert_equal @now, @environment.settings[:spaminator_last_run]
29		- end
30		-
31		- should 'find all comments' do
32		- @spaminator.stubs(:process_comment)
33		- @spaminator.send :process_all_comments
34		- end
35		-
36		- should 'find all people' do
37		- @spaminator.stubs(:process_person)
38		- @spaminator.send :process_all_people
39		- end
40		-
41		- should 'find all comments newer than a date' do
42		- @environment.settings[:spaminator_last_run] = Time.now - 1.month
43		- @spaminator.stubs(:process_comment)
44		- @spaminator.send :process_all_comments
45		- end
46		-
47		- should 'find all people newer than a date' do
48		- @environment.settings[:spaminator_last_run] = Time.now - 1.month
49		- @spaminator.stubs(:process_person)
50		- @spaminator.send :process_all_people
51		- end
52		-
53		-end