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 committed Jun 24, 2021
1 parent 068bb8d commit 5ffdb53
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 5ffdb53

Please sign in to comment.