*: fix unit test case

Signed-off-by: BornChanger <[email protected]>

pkg/fedvolumebackup/backupschedule/backup_schedule_manager.go

@@ -345,11 +345,9 @@ func (bm *backupScheduleManager) backupGCByMaxBackups(vbs *v1alpha1.VolumeBackup
 		return
 	}
 
-	//sort.Sort(byCreateTimeDesc(backupsList))
-	ascBackups := sortSnapshotBackups(backupsList)
-
+	sort.Sort(byCreateTimeDesc(backupsList))
 	var deleteCount int
-	for i, backup := range ascBackups {
+	for i, backup := range backupsList {
 		if i < int(*vbs.Spec.MaxBackups) {
 			continue
 		}
@@ -362,7 +360,7 @@ func (bm *backupScheduleManager) backupGCByMaxBackups(vbs *v1alpha1.VolumeBackup
 		klog.Infof("backup schedule %s/%s gc backup %s success", ns, bsName, backup.GetName())
 	}
 
-	if deleteCount == len(ascBackups) && deleteCount > 0 {
+	if deleteCount == len(backupsList) && deleteCount > 0 {
 		// All backups have been deleted, so the last backup information in the backupSchedule should be reset
 		bm.resetLastBackup(vbs)
 	}
@@ -374,6 +372,14 @@ func (bm *backupScheduleManager) resetLastBackup(vbs *v1alpha1.VolumeBackupSched
 	vbs.Status.AllBackupCleanTime = &metav1.Time{Time: bm.now()}
 }
 
+type byCreateTimeDesc []*v1alpha1.VolumeBackup
+
+func (b byCreateTimeDesc) Len() int      { return len(b) }
+func (b byCreateTimeDesc) Swap(i, j int) { b[i], b[j] = b[j], b[i] }
+func (b byCreateTimeDesc) Less(i, j int) bool {
+	return b[j].ObjectMeta.CreationTimestamp.Before(&b[i].ObjectMeta.CreationTimestamp)
+}
+
 var _ fedvolumebackup.BackupScheduleManager = &backupScheduleManager{}
 
 type FakeBackupScheduleManager struct {