From eda695ae3a6401bb95026c6e6f9ca2cf4e96270a Mon Sep 17 00:00:00 2001 From: Shubham Pampattiwar Date: Tue, 21 Jul 2026 15:34:43 -0700 Subject: [PATCH] Fix linter issues: gofmt alignment and require.Error Signed-off-by: Shubham Pampattiwar --- pkg/exposer/csi_snapshot_test.go | 4 ++-- pkg/util/kube/secrets_copy_test.go | 20 ++++++++++---------- 2 files changed, 12 insertions(+), 12 deletions(-) diff --git a/pkg/exposer/csi_snapshot_test.go b/pkg/exposer/csi_snapshot_test.go index e8e947913..fe9d81b0d 100644 --- a/pkg/exposer/csi_snapshot_test.go +++ b/pkg/exposer/csi_snapshot_test.go @@ -2366,10 +2366,10 @@ func TestCleanUp_SecretsAndConfigMaps(t *testing.T) { exposer.CleanUp(t.Context(), ownerObject, "", "app-ns") _, err := fakeKubeClient.CoreV1().Secrets("velero").Get(t.Context(), "kms-token", metav1.GetOptions{}) - assert.Error(t, err, "owned secret should be deleted") + require.Error(t, err, "owned secret should be deleted") _, err = fakeKubeClient.CoreV1().ConfigMaps("velero").Get(t.Context(), "kms-config", metav1.GetOptions{}) - assert.Error(t, err, "owned configmap should be deleted") + require.Error(t, err, "owned configmap should be deleted") _, err = fakeKubeClient.CoreV1().Secrets("velero").Get(t.Context(), "other-secret", metav1.GetOptions{}) assert.NoError(t, err, "unrelated secret should not be deleted") diff --git a/pkg/util/kube/secrets_copy_test.go b/pkg/util/kube/secrets_copy_test.go index 4071e7910..690c57273 100644 --- a/pkg/util/kube/secrets_copy_test.go +++ b/pkg/util/kube/secrets_copy_test.go @@ -89,11 +89,11 @@ func TestCopySecret(t *testing.T) { }, }, { - name: "returns collision when same data but different owner", - secretName: "ceph-csi-kms-token", - sourceNS: "app-ns", - targetNS: "velero", - ownerName: "du-456", + name: "returns collision when same data but different owner", + secretName: "ceph-csi-kms-token", + sourceNS: "app-ns", + targetNS: "velero", + ownerName: "du-456", objects: []k8sruntime.Object{ &corev1api.Secret{ ObjectMeta: metav1.ObjectMeta{Name: "ceph-csi-kms-token", Namespace: "app-ns"}, @@ -269,11 +269,11 @@ func TestCopyConfigMap(t *testing.T) { }, }, { - name: "returns collision when same data but different owner", - cmName: "ceph-csi-kms-config", - sourceNS: "app-ns", - targetNS: "velero", - ownerName: "du-456", + name: "returns collision when same data but different owner", + cmName: "ceph-csi-kms-config", + sourceNS: "app-ns", + targetNS: "velero", + ownerName: "du-456", objects: []k8sruntime.Object{ &corev1api.ConfigMap{ ObjectMeta: metav1.ObjectMeta{Name: "ceph-csi-kms-config", Namespace: "app-ns"},