From 5c4a4580d6b0246b317cdfb041ef7f855756ad4b Mon Sep 17 00:00:00 2001 From: Jeff Casimir Date: Sun, 1 Feb 2026 19:34:01 -0700 Subject: [PATCH] Fix backup scheduler test to match actual defaults The test expected enabled=False and retention_count=0, but commit 5371519d intentionally changed these defaults to enabled=True and retention_count=3. Update test to match the intended behavior. Co-Authored-By: Claude Opus 4.5 --- apps/backups/tests.py | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/apps/backups/tests.py b/apps/backups/tests.py index dc8a5136..85076a85 100644 --- a/apps/backups/tests.py +++ b/apps/backups/tests.py @@ -765,11 +765,12 @@ class BackupSchedulerTestCase(TestCase): settings = scheduler.get_schedule_settings() - self.assertEqual(settings['enabled'], False) + # These should match the DEFAULTS in scheduler.py + self.assertEqual(settings['enabled'], True) self.assertEqual(settings['frequency'], 'daily') self.assertEqual(settings['time'], '03:00') self.assertEqual(settings['day_of_week'], 0) - self.assertEqual(settings['retention_count'], 0) + self.assertEqual(settings['retention_count'], 3) self.assertEqual(settings['cron_expression'], '') def test_update_schedule_settings_stores_values(self):