Add test case for skipped volume with empty PV name but with PVC info

Co-authored-by: Cursor <cursoragent@cursor.com>
Signed-off-by: Xun Jiang <xun.jiang@broadcom.com>
This commit is contained in:
Xun Jiang
2026-09-18 17:10:07 +08:00
co-authored by Cursor
parent 5afc20d03e
commit f5982b4278
+23 -3
View File
@@ -48,9 +48,27 @@ func TestGenerateVolumeInfoForSkippedVolume(t *testing.T) {
tests := []struct {
name string
skippedVolumeName string
skippedPVCName string
skippedPVCNamespace string
pvMap map[string]pvcPvInfo
expectedVolumeInfos []*BackupVolumeInfo
}{
{
name: "Skipped volume with empty PV name but with PVC info",
skippedVolumeName: "",
skippedPVCName: "testPVC",
skippedPVCNamespace: "velero",
pvMap: map[string]pvcPvInfo{},
expectedVolumeInfos: []*BackupVolumeInfo{
{
PVName: "",
PVCName: "testPVC",
PVCNamespace: "velero",
Skipped: true,
SkippedReason: "CSI: skipped for PodVolumeBackup",
},
},
},
{
name: "Cannot find info for PV",
skippedVolumeName: "testPV",
@@ -131,11 +149,13 @@ func TestGenerateVolumeInfoForSkippedVolume(t *testing.T) {
volumesInfo := BackupVolumesInformation{}
volumesInfo.Init()
if tc.skippedVolumeName != "" {
if tc.skippedVolumeName != "" || tc.skippedPVCName != "" {
volumesInfo.SkippedVolumes = []SkippedVolume{
{
PVName: tc.skippedVolumeName,
Reasons: "CSI: skipped for PodVolumeBackup",
PVName: tc.skippedVolumeName,
PVCName: tc.skippedPVCName,
PVCNamespace: tc.skippedPVCNamespace,
Reasons: "CSI: skipped for PodVolumeBackup",
},
}
}