diff options
| author | Ed Santiago <santiago@redhat.com> | 2021-11-23 07:33:35 -0700 |
|---|---|---|
| committer | Ed Santiago <santiago@redhat.com> | 2021-11-23 15:39:04 -0700 |
| commit | eb3708a5243378a659ca126b5d5c457b182d16a8 (patch) | |
| tree | 60b55542ad9db7510a782712d2ef0b36cdf82f28 /test/e2e/save_test.go | |
| parent | c034147fe72a233b8c1ef33d00e9d0747053c262 (diff) | |
| download | podman-eb3708a5243378a659ca126b5d5c457b182d16a8.tar.gz podman-eb3708a5243378a659ca126b5d5c457b182d16a8.tar.bz2 podman-eb3708a5243378a659ca126b5d5c457b182d16a8.zip | |
Find and fix empty Expect()s
That previous commit made me wonder if there are any other
instances of Expect() with no assertions.
grep Expect test/e2e/*_test.go |egrep -v '\.(To|NotTo|Should)'
...finds a couple of handfuls, most of which are OK (continued
on the next line) but a few of which are bugs. Fix those.
Signed-off-by: Ed Santiago <santiago@redhat.com>
Diffstat (limited to 'test/e2e/save_test.go')
| -rw-r--r-- | test/e2e/save_test.go | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/test/e2e/save_test.go b/test/e2e/save_test.go index a8fb6c7b3..d17566e8f 100644 --- a/test/e2e/save_test.go +++ b/test/e2e/save_test.go @@ -230,7 +230,7 @@ default-docker: Expect(session).Should(Exit(0)) ids := session.OutputToStringArray() - Expect(len(RESTORE_IMAGES), len(ids)) + Expect(len(ids)).To(BeNumerically(">", 1), "We need to have *some* images to save") multiImageSave(podmanTest, ids) }) }) |