larryh/corso
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Update kopia tests for new model struct

Browse Source
This commit is contained in:
Ashlie Martinez 2023-10-04 13:38:50 -07:00
parent 7f438533f8
commit cd4905c0ad
1 changed files with 8 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

17
src/internal/kopia/cleanup_backups_test.go
View File

@ -369,13 +369,6 @@ func (suite *BackupCleanupUnitSuite) TestCleanupOrphanedData() {
return &res return &res
} }
backupWithLegacyResource := func(protectedResource string, b *backup.Backup) *backup.Backup {
res := *b
res.ResourceOwnerID = protectedResource
return &res
}
table := []struct { table := []struct {
name string name string
snapshots []*manifest.EntryMetadata snapshots []*manifest.EntryMetadata
@ -711,6 +704,9 @@ func (suite *BackupCleanupUnitSuite) TestCleanupOrphanedData() {
// not ideal, but some older versions of corso didn't even populate the // not ideal, but some older versions of corso didn't even populate the
// resource owner ID. // resource owner ID.
// //
// The old version of corso does not populated the ProtectedResourceID
// field in the backup model.
//
// Worst case, the assist base will be cleaned up when the user upgrades // Worst case, the assist base will be cleaned up when the user upgrades
// corso and generates either a new assist base or merge base with the // corso and generates either a new assist base or merge base with the
// same reason. // same reason.
@ -730,7 +726,7 @@ func (suite *BackupCleanupUnitSuite) TestCleanupOrphanedData() {
}, },
backups: []backupRes{ backups: []backupRes{
{bup: backupWithResource("ro", true, backupWithTime(baseTime, bupCurrent()))}, {bup: backupWithResource("ro", true, backupWithTime(baseTime, bupCurrent()))},
{bup: backupWithLegacyResource("ro", backupWithTime(baseTime.Add(time.Second), bupCurrent2()))}, {bup: backupWithTime(baseTime.Add(time.Second), bupCurrent2())},
}, },
time: baseTime.Add(48 * time.Hour), time: baseTime.Add(48 * time.Hour),
buffer: 24 * time.Hour, buffer: 24 * time.Hour,
@ -742,6 +738,9 @@ func (suite *BackupCleanupUnitSuite) TestCleanupOrphanedData() {
// reason and an even newer merge base from a current version of corso // reason and an even newer merge base from a current version of corso
// causes the assist base to be garbage collected. // causes the assist base to be garbage collected.
// //
// The old version of corso does not populated the ProtectedResourceID
// field in the backup model.
//
// This also tests that bases without a merge or assist tag are not // This also tests that bases without a merge or assist tag are not
// garbage collected as an assist base. // garbage collected as an assist base.
name: "AssistAndLegacyAndCurrentMergeBases NotYoungest CausesCleanup", name: "AssistAndLegacyAndCurrentMergeBases NotYoungest CausesCleanup",
@ -766,7 +765,7 @@ func (suite *BackupCleanupUnitSuite) TestCleanupOrphanedData() {
}, },
backups: []backupRes{ backups: []backupRes{
{bup: backupWithResource("ro", true, backupWithTime(baseTime, bupCurrent()))}, {bup: backupWithResource("ro", true, backupWithTime(baseTime, bupCurrent()))},
{bup: backupWithLegacyResource("ro", backupWithTime(baseTime.Add(time.Second), bupCurrent2()))}, {bup: backupWithTime(baseTime.Add(time.Second), bupCurrent2())},
{bup: backupWithResource("ro", false, backupWithTime(baseTime.Add(time.Minute), bupCurrent3()))}, {bup: backupWithResource("ro", false, backupWithTime(baseTime.Add(time.Minute), bupCurrent3()))},
}, },
time: baseTime.Add(48 * time.Hour), time: baseTime.Add(48 * time.Hour),