diff options
author | Brent Baude <bbaude@redhat.com> | 2022-04-27 15:16:25 -0500 |
---|---|---|
committer | Matthew Heon <matthew.heon@pm.me> | 2022-05-03 13:34:25 -0400 |
commit | b1089a23bc85759d1ac9319d1816c999980d8231 (patch) | |
tree | 746cf922239f79413a8439d57cce596a486f0885 /pkg/machine/e2e/start_test.go | |
parent | 4ff6884fb85d8a95f5a90848e92c3ff9ea6bcf57 (diff) | |
download | podman-b1089a23bc85759d1ac9319d1816c999980d8231.tar.gz podman-b1089a23bc85759d1ac9319d1816c999980d8231.tar.bz2 podman-b1089a23bc85759d1ac9319d1816c999980d8231.zip |
Produce better test error messages
As Ed has pointed out, the form of Expect(session).To(Exit(0)) provides
much better error messages. Let's make Ed happy.
Signed-off-by: Brent Baude <bbaude@redhat.com>
Diffstat (limited to 'pkg/machine/e2e/start_test.go')
-rw-r--r-- | pkg/machine/e2e/start_test.go | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/pkg/machine/e2e/start_test.go b/pkg/machine/e2e/start_test.go index 1cda0e8f1..1de66eb9a 100644 --- a/pkg/machine/e2e/start_test.go +++ b/pkg/machine/e2e/start_test.go @@ -4,6 +4,7 @@ import ( "github.com/containers/podman/v4/pkg/machine" . "github.com/onsi/ginkgo" . "github.com/onsi/gomega" + . "github.com/onsi/gomega/gexec" ) var _ = Describe("podman machine start", func() { @@ -22,11 +23,11 @@ var _ = Describe("podman machine start", func() { i := new(initMachine) session, err := mb.setCmd(i.withImagePath(mb.imagePath)).run() Expect(err).To(BeNil()) - Expect(session.ExitCode()).To(Equal(0)) + Expect(session).To(Exit(0)) s := new(startMachine) startSession, err := mb.setCmd(s).run() Expect(err).To(BeNil()) - Expect(startSession.ExitCode()).To(Equal(0)) + Expect(startSession).To(Exit(0)) info, ec, err := mb.toQemuInspectInfo() Expect(err).To(BeNil()) |