Fix backup rotation test for weekly backups

nickprotop · nickprotop · commit 698758daff8a · 2026-01-05T08:06:30.000+02:00
Adjust test to create backups starting from 1 day ago instead of 'now'
to ensure all backups fall within the weekly retention window when
KeepAllForHours and KeepDailyForDays are both 0.
diff --git a/LocalizationManager.Tests/UnitTests/Backup/BackupRotationPolicyTests.cs b/LocalizationManager.Tests/UnitTests/Backup/BackupRotationPolicyTests.cs
@@ -175,9 +175,10 @@ public async Task ApplyRotationAsync_KeepsWeeklyBackups()
         var backups = new List<BackupMetadata>();
 
         // Create backups for 4 weeks (one per week)
+        // Start from 1 day ago to ensure all backups are clearly in the past
         for (int i = 0; i < 4; i++)
         {
-            backups.Add(CreateBackup(i + 1, now.AddDays(-i * 7)));
+            backups.Add(CreateBackup(i + 1, now.AddDays(-1 - i * 7)));
         }
 
         var manifest = new BackupManifest

Original file line number	Diff line number	Diff line change
`@@ -175,9 +175,10 @@ public async Task ApplyRotationAsync_KeepsWeeklyBackups()`
`175`	`175`	`var backups = new List<BackupMetadata>();`
`176`	`176`
`177`	`177`	`// Create backups for 4 weeks (one per week)`
	`178`	`+ // Start from 1 day ago to ensure all backups are clearly in the past`
`178`	`179`	`for (int i = 0; i < 4; i++)`
`179`	`180`	`{`
`180`		`- backups.Add(CreateBackup(i + 1, now.AddDays(-i * 7)));`
	`181`	`+ backups.Add(CreateBackup(i + 1, now.AddDays(-1 - i * 7)));`
`181`	`182`	`}`
`182`	`183`
`183`	`184`	`var manifest = new BackupManifest`