feat: add scheduled archive of units

Author: macite

app/sidekiq/archive_old_units_job.rb

@@ -0,0 +1,26 @@
+# frozen_string_literal: true
+
+# Move old units to archive folder
+class ArchiveOldUnitsJob
+  include Sidekiq::Job
+
+  def perform
+    archive_period = Doubtfire::Application.config.unit_archive_after_period
+
+    archive_period = 1.year if archive_period < 1.year
+
+    units = Unit.where(archived: false).where('end_date < :archive_before', archive_before: DateTime.now - archive_period)
+
+    units.find_each(&:move_files_to_archive)
+  rescue StandardError => e
+    begin
+      # Notify system admin
+      mail = ErrorLogMailer.error_message('Archive Units', "Failed to move old units to archive", e)
+      mail.deliver if mail.present?
+
+      logger.error e
+    rescue StandardError => e
+      logger.error "Failed to send error log to admin"
+    end
+  end
+end

config/schedule.yml

@@ -11,3 +11,7 @@ progress_turn_it_in_jobs:
 clean_up_auth_tokens:
   cron: "every 30 minutes"
   class: "ClearAccessTokensJob"
+
+archive_old_units:
+  cron: "every 6 months"
+  class: "ArchiveOldUnitsJob"

test/models/unit_model_test.rb

@@ -874,4 +874,22 @@ def test_archive_unit
     assert_not File.exist?(p.portfolio_path), "New portfolio exists after delete - #{p.portfolio_path}"
   end
 
+  def test_archive_unit_job
+    unit = FactoryBot.create :unit, with_students: false, task_count: 0
+
+    unit.end_date = Time.zone.now - Doubtfire::Application.config.unit_archive_after_period - 1.day
+    unit.start_date = unit.end_date - 14.weeks
+    unit.save!
+
+    unit2 = FactoryBot.create :unit, with_students: false, task_count: 0
+
+    assert_not unit.archived
+    assert_not unit2.archived
+    ArchiveOldUnitsJob.new.perform
+    unit.reload
+
+    assert unit.archived
+    assert_not unit2.archived
+  end
+
 end