Skip to content

Commit

Permalink
test: fix deletepod testcase
Browse files Browse the repository at this point in the history
Trigger the function that was intended to be tested, ie. DeletePod.
We are not sure what is the behaviour we want when a nil pod reaches
this function, if that's possible at all. Further analysis and work is
needed.

Also, note that Create and Update functions show a different behaviour
introduced as quick fix to race condition:
#598 (comment). To
re-evaluate.
  • Loading branch information
fprojetto authored and Filippo Projetto (Rebase PR Action) committed Jun 24, 2021
1 parent 7b1f831 commit 45cc4fd
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions pkg/virtualKubelet/provider/pods_test.go
Expand Up @@ -133,8 +133,8 @@ var _ = Describe("Pods", func() {
})

It("delete pod", func() {
err := provider.CreatePod(context.TODO(), nil)
Expect(err).NotTo(HaveOccurred())
err := provider.DeletePod(context.TODO(), nil)
Expect(err).To(HaveOccurred())
})
})
})

0 comments on commit 45cc4fd

Please sign in to comment.