mirror of
https://github.com/discourse/discourse.git
synced 2024-12-14 00:53:51 +08:00
4e716e9ce5
* FIX: Reduce the time_read threshold to one minute. Five minutes is too much and could fill the queue with false positives. * Update spec/jobs/enqueue_suspect_users_spec.rb Co-authored-by: Arpit Jalan <arpit@techapj.com> Co-authored-by: Arpit Jalan <arpit@techapj.com>
109 lines
3.0 KiB
Ruby
109 lines
3.0 KiB
Ruby
# frozen_string_literal: true
|
|
|
|
require 'rails_helper'
|
|
|
|
describe Jobs::EnqueueSuspectUsers do
|
|
before { SiteSetting.approve_suspect_users = true }
|
|
|
|
it 'does nothing when there are no suspect users' do
|
|
subject.execute({})
|
|
|
|
expect(ReviewableUser.count).to be_zero
|
|
end
|
|
|
|
context 'with suspect users' do
|
|
let!(:suspect_user) { Fabricate(:active_user, created_at: 1.day.ago) }
|
|
|
|
it 'creates a reviewable when there is a suspect user' do
|
|
subject.execute({})
|
|
|
|
expect(ReviewableUser.count).to eq(1)
|
|
end
|
|
|
|
it 'only creates one reviewable per user' do
|
|
review_user = ReviewableUser.needs_review!(
|
|
target: suspect_user,
|
|
created_by: Discourse.system_user,
|
|
reviewable_by_moderator: true
|
|
)
|
|
|
|
subject.execute({})
|
|
|
|
expect(ReviewableUser.count).to eq(1)
|
|
expect(ReviewableUser.last).to eq(review_user)
|
|
end
|
|
|
|
it 'adds a score' do
|
|
subject.execute({})
|
|
score = ReviewableScore.last
|
|
|
|
expect(score.reason).to eq('suspect_user')
|
|
end
|
|
|
|
it 'only enqueues non-approved users' do
|
|
suspect_user.update!(approved: true)
|
|
|
|
subject.execute({})
|
|
|
|
expect(ReviewableUser.where(target: suspect_user).exists?).to eq(false)
|
|
end
|
|
|
|
it 'does nothing if must_approve_users is set to true' do
|
|
SiteSetting.must_approve_users = true
|
|
suspect_user.update!(approved: false)
|
|
|
|
subject.execute({})
|
|
|
|
expect(ReviewableUser.where(target: suspect_user).exists?).to eq(false)
|
|
end
|
|
|
|
it 'ignores users created more than six months ago' do
|
|
suspect_user.update!(created_at: 1.year.ago)
|
|
|
|
subject.execute({})
|
|
|
|
expect(ReviewableUser.where(target: suspect_user).exists?).to eq(false)
|
|
end
|
|
|
|
it 'ignores users that were imported from another site' do
|
|
suspect_user.upsert_custom_fields({ import_id: 'fake_id' })
|
|
|
|
subject.execute({})
|
|
|
|
expect(ReviewableUser.where(target: suspect_user).exists?).to eq(false)
|
|
end
|
|
|
|
it 'enqueues a suspect users with custom fields' do
|
|
suspect_user.upsert_custom_fields({ field_a: 'value', field_b: 'value' })
|
|
|
|
subject.execute({})
|
|
|
|
expect(ReviewableUser.where(target: suspect_user).exists?).to eq(true)
|
|
end
|
|
|
|
it 'ignores imported users even if they have multiple custom fields' do
|
|
suspect_user.upsert_custom_fields({ field_a: 'value', field_b: 'value', import_id: 'fake_id' })
|
|
|
|
subject.execute({})
|
|
|
|
expect(ReviewableUser.where(target: suspect_user).exists?).to eq(false)
|
|
end
|
|
|
|
it 'enqueues a suspect user with not enough time read' do
|
|
suspect_user.user_stat.update!(posts_read_count: 2, topics_entered: 2, time_read: 30.seconds.to_i)
|
|
|
|
subject.execute({})
|
|
|
|
expect(ReviewableUser.count).to eq(1)
|
|
end
|
|
|
|
it 'ignores users if their time read is higher than one minute' do
|
|
suspect_user.user_stat.update!(posts_read_count: 2, topics_entered: 2, time_read: 2.minutes.to_i)
|
|
|
|
subject.execute({})
|
|
|
|
expect(ReviewableUser.count).to eq(0)
|
|
end
|
|
end
|
|
end
|