IntenseWebs/discourse
3
0
Fork 0
mirror of https://github.com/discourse/discourse.git synced 2025-02-25 18:55:32 -06:00
Code Issues Packages Projects Releases Wiki Activity

FEATURE: suppress digest emails for users not seen on the site for more than (n) days.

Browse Source
This commit is contained in:
Arpit Jalan
2015-01-26 21:37:10 +05:30
parent 8ef2a111a0
commit 5c2ee8ec2a
4 changed files with 11 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files

12
spec/jobs/enqueue_digest_emails_spec.rb
View File

@@ -65,15 +65,18 @@ describe Jobs::EnqueueDigestEmails do
user = user_visited_this_week
expect(Jobs::EnqueueDigestEmails.new.target_user_ids.include?(user.id)).to eq(false)
end
end
it "does return users who have been emailed recently but have email_always set" do
user = user_visited_this_week_email_always
expect(Jobs::EnqueueDigestEmails.new.target_user_ids.include?(user.id)).to eq(true)
context 'visited the site a year ago' do
let!(:user_visited_a_year_ago) { Fabricate(:active_user, last_seen_at: 370.days.ago) }
it "doesn't return the user who have not visited the site for more than 365 days" do
expect(Jobs::EnqueueDigestEmails.new.target_user_ids.include?(user_visited_a_year_ago.id)).to eq(false)
end
end
context 'regular users' do
let!(:user) { Fabricate(:active_user) }
let!(:user) { Fabricate(:active_user, last_seen_at: 360.days.ago) }
it "returns the user" do
expect(Jobs::EnqueueDigestEmails.new.target_user_ids).to eq([user.id])
@@ -99,4 +102,3 @@ describe Jobs::EnqueueDigestEmails do
end