From 8e580996747045da4ba2a8f8e1db30ba292b83a6 Mon Sep 17 00:00:00 2001 From: Shubham Pampattiwar Date: Fri, 5 Dec 2025 12:23:28 -0800 Subject: [PATCH] Add test for cache usage without volume policy Add test case to verify that the PVC-to-Pod cache is used even when no volume policy is configured. When defaultVolumesToFSBackup is true, the cache is used to find pods using the PVC to determine if fs-backup should be used instead of snapshot. Signed-off-by: Shubham Pampattiwar --- .../volumehelper/volume_policy_helper_test.go | 21 +++++++++++++++++++ 1 file changed, 21 insertions(+) diff --git a/internal/volumehelper/volume_policy_helper_test.go b/internal/volumehelper/volume_policy_helper_test.go index 2a02b675f..862081725 100644 --- a/internal/volumehelper/volume_policy_helper_test.go +++ b/internal/volumehelper/volume_policy_helper_test.go @@ -830,6 +830,27 @@ func TestVolumeHelperImplWithCache_ShouldPerformSnapshot(t *testing.T) { shouldSnapshot: true, expectedErr: false, }, + { + name: "No volume policy, defaultVolumesToFSBackup with cache, skips snapshot", + inputObj: builder.ForPersistentVolume("example-pv").StorageClass("gp2-csi").ClaimRef("ns", "pvc-1").Result(), + groupResource: kuberesource.PersistentVolumes, + pod: builder.ForPod("ns", "pod-1").Volumes( + &corev1api.Volume{ + Name: "volume", + VolumeSource: corev1api.VolumeSource{ + PersistentVolumeClaim: &corev1api.PersistentVolumeClaimVolumeSource{ + ClaimName: "pvc-1", + }, + }, + }, + ).Result(), + resourcePolicies: nil, + snapshotVolumesFlag: ptr.To(true), + defaultVolumesToFSBackup: true, + buildCache: true, + shouldSnapshot: false, + expectedErr: false, + }, } objs := []runtime.Object{