maverick/docbrown
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Create less data for fast_destroy_old_notifications spec (#5256) [deploy]

Browse source
This commit is contained in:
rhymes 2019-12-27 16:55:42 +01:00 • committed by Molly Struve
parent b45f35d044
commit dbaf37e712
1 changed files with 6 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

10
spec/models/notification_spec.rb
View file

@ -519,15 +519,17 @@ RSpec.describe Notification, type: :model do
end
describe "#fast_destroy_old_notifications" do
let_it_be(:notifications) { create_list(:notification, 3) }
it "bulk deletes notifications older than 4 months by default" do
create_list :notification, 5
described_class.last.update(created_at: 5.months.ago)
notifications.last.update(created_at: 5.months.ago)
expect { described_class.fast_destroy_old_notifications }.to change(described_class, :count).by(-1)
end
it "bulk deletes notifications older than a given timestamp" do
create_list :notification, 5
expect { described_class.fast_destroy_old_notifications(Time.zone.now) }.to change(described_class, :count).by(-5)
expect do
described_class.fast_destroy_old_notifications(Time.current)
end.to change(described_class, :count).by(-notifications.size)
end
end
end