Reduce retention period of Spree state changes and log entries from 6 to 3 months

The UK instance noticed Spree state changes and log entries seem to be filling up quite quickly.

Fixes #7123
This commit is contained in:
Cillian O'Ruanaidh
2021-03-17 12:12:45 +00:00
parent 3a5f763bf2
commit a80e1b7625
2 changed files with 11 additions and 13 deletions

View File

@@ -5,8 +5,7 @@ require 'tasks/data/remove_transient_data'
describe RemoveTransientData do
describe '#call' do
let(:medium_retention) { RemoveTransientData::MEDIUM_RETENTION }
let(:short_retention) { RemoveTransientData::SHORT_RETENTION }
let(:retention_period) { RemoveTransientData::RETENTION_PERIOD }
before do
allow(Spree::StateChange).to receive(:delete_all)
@@ -16,21 +15,21 @@ describe RemoveTransientData do
end
it 'deletes state changes older than rentention_period' do
Spree::StateChange.create(created_at: medium_retention - 1.day)
Spree::StateChange.create(created_at: retention_period - 1.day)
RemoveTransientData.new.call
expect(Spree::StateChange.all).to be_empty
end
it 'deletes log entries older than retention_period' do
Spree::LogEntry.create(created_at: medium_retention - 1.day)
Spree::LogEntry.create(created_at: retention_period - 1.day)
expect { RemoveTransientData.new.call }
.to change(Spree::LogEntry, :count).by(-1)
end
it 'deletes sessions older than retention_period' do
RemoveTransientData::Session.create(session_id: 1, updated_at: short_retention - 1.day)
RemoveTransientData::Session.create(session_id: 1, updated_at: retention_period - 1.day)
RemoveTransientData.new.call
@@ -45,12 +44,12 @@ describe RemoveTransientData do
let!(:line_item) { create(:line_item, order: cart, variant: variant) }
let!(:adjustment) { create(:adjustment, order: cart) }
let!(:old_cart) { create(:order, state: 'cart', updated_at: short_retention - 1.day) }
let!(:old_cart) { create(:order, state: 'cart', updated_at: retention_period - 1.day) }
let!(:old_line_item) { create(:line_item, order: old_cart, variant: variant) }
let!(:old_adjustment) { create(:adjustment, order: old_cart) }
before do
old_cart.update_columns(updated_at: short_retention - 1.day)
old_cart.update_columns(updated_at: retention_period - 1.day)
end
it 'deletes cart orders and related objects older than retention_period' do