Skip to content

Commit 8ed5e0b

Browse files
committed
Fix failing dry-run test.
The test was failing because the dry-run delete returns no resource. Signed-off-by: Kevin McDermott <bigkevmcd@gmail.com>
1 parent 68cdf84 commit 8ed5e0b

File tree

1 file changed

+2
-1
lines changed

1 file changed

+2
-1
lines changed

pkg/client/dryrun_test.go

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -182,7 +182,8 @@ var _ = Describe("DryRunClient", func() {
182182
})
183183

184184
It("should not delete objects", func() {
185-
Expect(getClient().Delete(ctx, dep)).NotTo(HaveOccurred())
185+
changedDep := dep.DeepCopy()
186+
Expect(getClient().Delete(ctx, changedDep)).NotTo(HaveOccurred())
186187

187188
actual, err := clientset.AppsV1().Deployments(ns).Get(ctx, dep.Name, metav1.GetOptions{})
188189
Expect(err).NotTo(HaveOccurred())

0 commit comments

Comments
 (0)