Skip to content

Commit

Permalink
Merge pull request #11077 from flouthoc/healthcheck-nit
Browse files Browse the repository at this point in the history
`ci-fix`: healthcheck tests should use `.Should()` instead of `.To()`.
  • Loading branch information
openshift-merge-robot authored Jul 29, 2021
2 parents 9c4b7f9 + 2b5d9cd commit f17b810
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions test/e2e/healthcheck_run_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -177,11 +177,11 @@ var _ = Describe("Podman healthcheck run", func() {
It("podman healthcheck unhealthy but valid arguments check", func() {
session := podmanTest.Podman([]string{"run", "-dt", "--name", "hc", "--health-retries", "2", "--health-cmd", "[\"ls\", \"/foo\"]", ALPINE, "top"})
session.WaitWithDefaultTimeout()
Expect(session.ExitCode()).To(Equal(0))
Expect(session).Should(Exit(0))

hc := podmanTest.Podman([]string{"healthcheck", "run", "hc"})
hc.WaitWithDefaultTimeout()
Expect(hc.ExitCode()).To(Equal(1))
Expect(hc).Should(Exit(1))
})

It("podman healthcheck single healthy result changes failed to healthy", func() {
Expand Down

0 comments on commit f17b810

Please sign in to comment.