Some tasks don't have to run on every tenant. The task to calculate the TSV is only done for records which were present before we added the TSV column, and that isn't going to happen in any tenants because we added the TSV column before adding the tenants table. Similarly, the migration needed for existing polls isn't necessary because there weren't any tenants before we allowed to set the starting/ending time to polls. We aren't adding any tests for these tasks because tests for rake tasks are slow and tests creating tenants are also slow, making the combination of the two even slower, particularly if we add tests for every single task we're changing. We're adding tests for the `.run_on_each` method instead.
11 lines
363 B
Ruby
11 lines
363 B
Ruby
namespace :files do
|
|
desc "Removes cached attachments which weren't deleted for some reason"
|
|
task remove_old_cached_attachments: :environment do
|
|
Tenant.run_on_each do
|
|
ActiveStorage::Blob.unattached
|
|
.where("active_storage_blobs.created_at <= ?", 1.day.ago)
|
|
.find_each(&:purge_later)
|
|
end
|
|
end
|
|
end
|