Remove unnecessary nolint directives from test file

The nolint:staticcheck directives are not needed in the test file because
it calls NewVolumeHelperImpl within the same package, which doesn't
trigger deprecation warnings. Only cross-package calls need the directive.

Signed-off-by: Shubham Pampattiwar <spampatt@redhat.com>
This commit is contained in:
Shubham Pampattiwar
2025-12-17 11:01:00 -08:00
parent 4ba2effaac
commit 78e9470028

View File

@@ -322,7 +322,6 @@ func TestVolumeHelperImpl_ShouldPerformSnapshot(t *testing.T) {
t.Fatalf("failed to build policy with error %v", err)
}
}
//nolint:staticcheck // Testing deprecated function for backwards compatibility
vh := NewVolumeHelperImpl(
p,
tc.snapshotVolumesFlag,
@@ -688,7 +687,6 @@ func TestVolumeHelperImpl_ShouldPerformFSBackup(t *testing.T) {
t.Fatalf("failed to build policy with error %v", err)
}
}
//nolint:staticcheck // Testing deprecated function for backwards compatibility
vh := NewVolumeHelperImpl(
p,
tc.snapshotVolumesFlag,