From a7d0ac15d77832f36bb46d6c0e6321f24f608544 Mon Sep 17 00:00:00 2001 From: fengluodb Date: Tue, 24 Oct 2023 10:43:28 +0800 Subject: [PATCH] chore: refine the comment --- controllers/dataprotection/backup_controller_test.go | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/controllers/dataprotection/backup_controller_test.go b/controllers/dataprotection/backup_controller_test.go index 5a4e45ce308..de4280d0d30 100644 --- a/controllers/dataprotection/backup_controller_test.go +++ b/controllers/dataprotection/backup_controller_test.go @@ -196,7 +196,7 @@ var _ = Describe("Backup Controller test", func() { }) backupKey := client.ObjectKeyFromObject(backup) - By("check backup failed and its expiration is set") + By("check backup failed and its expiration when retentionPeriod is not set") Eventually(testapps.CheckObj(&testCtx, backupKey, func(g Gomega, fetched *dpv1alpha1.Backup) { g.Expect(fetched.Status.Phase).To(Equal(dpv1alpha1.BackupPhaseFailed)) g.Expect(fetched.Status.Expiration).Should(BeNil()) @@ -459,7 +459,7 @@ var _ = Describe("Backup Controller test", func() { }) }) - It("should fail because of invalid backup method", func() { + It("should fail because backup method doesn't specify snapshotVolumes with empty actionSet", func() { backup := testdp.NewFakeBackup(&testCtx, func(backup *dpv1alpha1.Backup) { backup.Spec.BackupPolicyName = backupPolicy.Name backup.Spec.BackupMethod = "invalid"