2023-02-21 16:55:31 -08:00
|
|
|
# frozen_string_literal: true
|
|
|
|
|
2021-08-09 14:11:50 -07:00
|
|
|
require 'rails_helper'
|
|
|
|
|
|
|
|
describe Scheduler::AccountsStatusesCleanupScheduler do
|
|
|
|
subject { described_class.new }
|
|
|
|
|
|
|
|
let!(:account1) { Fabricate(:account, domain: nil) }
|
|
|
|
let!(:account2) { Fabricate(:account, domain: nil) }
|
|
|
|
let!(:account3) { Fabricate(:account, domain: nil) }
|
|
|
|
let!(:account4) { Fabricate(:account, domain: nil) }
|
2023-04-23 13:25:40 -07:00
|
|
|
let!(:account5) { Fabricate(:account, domain: nil) }
|
2021-08-09 14:11:50 -07:00
|
|
|
let!(:remote) { Fabricate(:account) }
|
|
|
|
|
|
|
|
let!(:policy1) { Fabricate(:account_statuses_cleanup_policy, account: account1) }
|
|
|
|
let!(:policy2) { Fabricate(:account_statuses_cleanup_policy, account: account3) }
|
|
|
|
let!(:policy3) { Fabricate(:account_statuses_cleanup_policy, account: account4, enabled: false) }
|
2023-04-23 13:25:40 -07:00
|
|
|
let!(:policy4) { Fabricate(:account_statuses_cleanup_policy, account: account5) }
|
2021-08-09 14:11:50 -07:00
|
|
|
|
|
|
|
let(:queue_size) { 0 }
|
|
|
|
let(:queue_latency) { 0 }
|
|
|
|
let(:process_set_stub) do
|
|
|
|
[
|
|
|
|
{
|
|
|
|
'concurrency' => 2,
|
2023-02-19 21:14:10 -08:00
|
|
|
'queues' => %w(push default),
|
2021-08-09 14:11:50 -07:00
|
|
|
},
|
|
|
|
]
|
|
|
|
end
|
|
|
|
|
|
|
|
before do
|
|
|
|
queue_stub = double
|
|
|
|
allow(queue_stub).to receive(:size).and_return(queue_size)
|
|
|
|
allow(queue_stub).to receive(:latency).and_return(queue_latency)
|
|
|
|
allow(Sidekiq::Queue).to receive(:new).and_return(queue_stub)
|
|
|
|
allow(Sidekiq::ProcessSet).to receive(:new).and_return(process_set_stub)
|
|
|
|
|
|
|
|
sidekiq_stats_stub = double
|
|
|
|
allow(Sidekiq::Stats).to receive(:new).and_return(sidekiq_stats_stub)
|
|
|
|
|
|
|
|
# Create a bunch of old statuses
|
|
|
|
10.times do
|
|
|
|
Fabricate(:status, account: account1, created_at: 3.years.ago)
|
|
|
|
Fabricate(:status, account: account2, created_at: 3.years.ago)
|
|
|
|
Fabricate(:status, account: account3, created_at: 3.years.ago)
|
|
|
|
Fabricate(:status, account: account4, created_at: 3.years.ago)
|
2023-04-23 13:25:40 -07:00
|
|
|
Fabricate(:status, account: account5, created_at: 3.years.ago)
|
2021-08-09 14:11:50 -07:00
|
|
|
Fabricate(:status, account: remote, created_at: 3.years.ago)
|
|
|
|
end
|
|
|
|
|
|
|
|
# Create a bunch of newer statuses
|
|
|
|
5.times do
|
|
|
|
Fabricate(:status, account: account1, created_at: 3.minutes.ago)
|
|
|
|
Fabricate(:status, account: account2, created_at: 3.minutes.ago)
|
|
|
|
Fabricate(:status, account: account3, created_at: 3.minutes.ago)
|
|
|
|
Fabricate(:status, account: account4, created_at: 3.minutes.ago)
|
|
|
|
Fabricate(:status, account: remote, created_at: 3.minutes.ago)
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
describe '#under_load?' do
|
|
|
|
context 'when nothing is queued' do
|
|
|
|
it 'returns false' do
|
|
|
|
expect(subject.under_load?).to be false
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'when numerous jobs are queued' do
|
|
|
|
let(:queue_size) { 5 }
|
|
|
|
let(:queue_latency) { 120 }
|
|
|
|
|
|
|
|
it 'returns true' do
|
|
|
|
expect(subject.under_load?).to be true
|
|
|
|
end
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
2023-05-03 01:31:40 -07:00
|
|
|
describe '#compute_budget' do
|
2023-05-03 20:49:08 -07:00
|
|
|
context 'with a single thread' do
|
2023-02-19 21:14:10 -08:00
|
|
|
let(:process_set_stub) { [{ 'concurrency' => 1, 'queues' => %w(push default) }] }
|
2021-08-09 14:11:50 -07:00
|
|
|
|
|
|
|
it 'returns a low value' do
|
|
|
|
expect(subject.compute_budget).to be < 10
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
2023-05-03 20:49:08 -07:00
|
|
|
context 'with a lot of threads' do
|
2021-08-09 14:11:50 -07:00
|
|
|
let(:process_set_stub) do
|
|
|
|
[
|
2023-02-19 21:14:10 -08:00
|
|
|
{ 'concurrency' => 2, 'queues' => %w(push default) },
|
2021-08-09 14:11:50 -07:00
|
|
|
{ 'concurrency' => 2, 'queues' => ['push'] },
|
|
|
|
{ 'concurrency' => 2, 'queues' => ['push'] },
|
|
|
|
{ 'concurrency' => 2, 'queues' => ['push'] },
|
|
|
|
]
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'returns a larger value' do
|
|
|
|
expect(subject.compute_budget).to be > 10
|
|
|
|
end
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
describe '#perform' do
|
|
|
|
context 'when the budget is lower than the number of toots to delete' do
|
|
|
|
it 'deletes as many statuses as the given budget' do
|
|
|
|
expect { subject.perform }.to change { Status.count }.by(-subject.compute_budget)
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'does not delete from accounts with no cleanup policy' do
|
|
|
|
expect { subject.perform }.to_not change { account2.statuses.count }
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'does not delete from accounts with disabled cleanup policies' do
|
|
|
|
expect { subject.perform }.to_not change { account4.statuses.count }
|
|
|
|
end
|
|
|
|
|
2023-04-23 13:25:40 -07:00
|
|
|
it 'eventually deletes every deletable toot given enough runs' do
|
|
|
|
stub_const 'Scheduler::AccountsStatusesCleanupScheduler::MAX_BUDGET', 4
|
|
|
|
|
|
|
|
expect { 10.times { subject.perform } }.to change { Status.count }.by(-30)
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'correctly round-trips between users across several runs' do
|
|
|
|
stub_const 'Scheduler::AccountsStatusesCleanupScheduler::MAX_BUDGET', 3
|
|
|
|
stub_const 'Scheduler::AccountsStatusesCleanupScheduler::PER_ACCOUNT_BUDGET', 2
|
|
|
|
|
|
|
|
expect { 3.times { subject.perform } }
|
|
|
|
.to change { Status.count }.by(-3 * 3)
|
|
|
|
.and change { account1.statuses.count }
|
|
|
|
.and change { account3.statuses.count }
|
|
|
|
.and change { account5.statuses.count }
|
2021-08-09 14:11:50 -07:00
|
|
|
end
|
2023-05-03 01:31:40 -07:00
|
|
|
|
|
|
|
context 'when given a big budget' do
|
|
|
|
let(:process_set_stub) { [{ 'concurrency' => 400, 'queues' => %w(push default) }] }
|
|
|
|
|
|
|
|
before do
|
|
|
|
stub_const 'Scheduler::AccountsStatusesCleanupScheduler::MAX_BUDGET', 400
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'correctly handles looping in a single run' do
|
|
|
|
expect(subject.compute_budget).to eq(400)
|
|
|
|
expect { subject.perform }.to change { Status.count }.by(-30)
|
|
|
|
end
|
|
|
|
end
|
2021-08-09 14:11:50 -07:00
|
|
|
end
|
|
|
|
end
|
|
|
|
end
|