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, 1 deletions

diff --git a/test/e2e/image_scp_test.go b/test/e2e/image_scp_test.go
index 3e7e8da48..63276e57f 100644
--- a/test/e2e/image_scp_test.go
+++ b/test/e2e/image_scp_test.go
@@ -77,7 +77,7 @@ var _ = Describe("podman image scp", func() {
 		list := podmanTest.Podman([]string{"image", "list"}) // our image should now contain alpine loaded in from root
 		list.WaitWithDefaultTimeout()
 		Expect(list).To(Exit(0))
-		Expect(list.LineInOutputStartsWith("quay.io/libpod/alpine")).To(BeTrue())
+		Expect(list.OutputToStringArray()).To(ContainElement(HavePrefix("quay.io/libpod/alpine")))
 
 		scp = podmanTest.PodmanAsUser([]string{"image", "scp", "root@localhost::" + ALPINE}, 0, 0, "", env) //transfer from root to rootless (us)
 		scp.WaitWithDefaultTimeout()