diff options
author | Jhon Honce <jhonce@redhat.com> | 2019-10-04 10:09:24 -0700 |
---|---|---|
committer | Jhon Honce <jhonce@redhat.com> | 2019-10-16 08:10:07 -0700 |
commit | 60d0be17fc55404929ad24d861f318968997458f (patch) | |
tree | e6aeecf49dd00a8e1f4a32cb3c0638b041fb0572 /test/e2e/volume_rm_test.go | |
parent | 7825c5827e039aed6656f5df65de2262591681e6 (diff) | |
download | podman-60d0be17fc55404929ad24d861f318968997458f.tar.gz podman-60d0be17fc55404929ad24d861f318968997458f.tar.bz2 podman-60d0be17fc55404929ad24d861f318968997458f.zip |
Refactor tests when checking for error exit codes
Rather than checking for non-zero, we need to check for >0 to
distinguish between timeouts and error exit codes.
Signed-off-by: Jhon Honce <jhonce@redhat.com>
Diffstat (limited to 'test/e2e/volume_rm_test.go')
-rw-r--r-- | test/e2e/volume_rm_test.go | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/test/e2e/volume_rm_test.go b/test/e2e/volume_rm_test.go index 61cf9b893..6f2020828 100644 --- a/test/e2e/volume_rm_test.go +++ b/test/e2e/volume_rm_test.go @@ -56,7 +56,7 @@ var _ = Describe("Podman volume rm", func() { session = podmanTest.Podman([]string{"volume", "rm", "myvol"}) session.WaitWithDefaultTimeout() - Expect(session.ExitCode()).To(Not(Equal(0))) + Expect(session).To(ExitWithError()) Expect(session.ErrorToString()).To(ContainSubstring(cid)) session = podmanTest.Podman([]string{"volume", "rm", "-f", "myvol"}) @@ -116,7 +116,7 @@ var _ = Describe("Podman volume rm", func() { session = podmanTest.Podman([]string{"volume", "rm", "myv"}) session.WaitWithDefaultTimeout() - Expect(session.ExitCode()).To(Not(Equal(0))) + Expect(session).To(ExitWithError()) session = podmanTest.Podman([]string{"volume", "ls"}) session.WaitWithDefaultTimeout() |