e2e tests: clean up antihelpful BeTrue()s

Many ginkgo tests have been written to use this evil form:

    GrepString("foo")
    Expect(that to BeTrue())

...which yields horrible useless messages on failure:

    false is not true

Identify those (automatically, via script) and convert to:

    Expect(output to ContainSubstring("foo"))

...which yields:

    "this output" does not contain substring "foo"

There are still many BeTrue()s left. This is just a start.

This is commit 1 of 2. It includes the script I used, and
all changes to *.go are those computed by the script.
Commit 2 will apply some manual fixes.

Signed-off-by: Ed Santiago <santiago@redhat.com>

1 files changed, 1 insertions, 2 deletions

diff --git a/test/e2e/save_test.go b/test/e2e/save_test.go
index 0b3a88da3..9f3f750b7 100644
--- a/test/e2e/save_test.go
+++ b/test/e2e/save_test.go
@@ -253,8 +253,7 @@ func multiImageSave(podmanTest *PodmanTestIntegration, images []string) {
 	Expect(session).Should(Exit(0))
 	// Grep for each image in the `podman load` output.
 	for _, image := range images {
-		found, _ := session.GrepString(image)
-		Expect(found).Should(BeTrue())
+		Expect(session.OutputToString()).To(ContainSubstring(image))
 	}
 
 	// Make sure that each image has really been loaded.