diff options
| author | Ed Santiago <santiago@redhat.com> | 2021-11-30 13:14:56 -0700 |
|---|---|---|
| committer | Ed Santiago <santiago@redhat.com> | 2021-11-30 15:19:57 -0700 |
| commit | 9ce7ade8c8dedb6a082614e45aa10d0e37c725cc (patch) | |
| tree | aeefee1ec3a9bac7ee92c7d8d88f3d85c1364cef /test/e2e/commit_test.go | |
| parent | 295a6f7dd086731448a1168a349f62d3035258ca (diff) | |
| download | podman-9ce7ade8c8dedb6a082614e45aa10d0e37c725cc.tar.gz podman-9ce7ade8c8dedb6a082614e45aa10d0e37c725cc.tar.bz2 podman-9ce7ade8c8dedb6a082614e45aa10d0e37c725cc.zip | |
e2e: yet more cleanup of BeTrue/BeFalse
Thanks to Paul for teaching me about HaveKey()
Signed-off-by: Ed Santiago <santiago@redhat.com>
Diffstat (limited to 'test/e2e/commit_test.go')
| -rw-r--r-- | test/e2e/commit_test.go | 8 |
1 files changed, 3 insertions, 5 deletions
diff --git a/test/e2e/commit_test.go b/test/e2e/commit_test.go index b751c338f..a09666468 100644 --- a/test/e2e/commit_test.go +++ b/test/e2e/commit_test.go @@ -204,8 +204,7 @@ var _ = Describe("Podman commit", func() { inspect.WaitWithDefaultTimeout() Expect(inspect).Should(Exit(0)) image := inspect.InspectImageJSON() - _, ok := image[0].Config.Volumes["/foo"] - Expect(ok).To(BeFalse()) + Expect(image[0].Config.Volumes).To(Not(HaveKey("/foo"))) }) It("podman commit with volume mounts and --include-volumes", func() { @@ -224,8 +223,7 @@ var _ = Describe("Podman commit", func() { inspect.WaitWithDefaultTimeout() Expect(inspect).Should(Exit(0)) image := inspect.InspectImageJSON() - _, ok := image[0].Config.Volumes["/foo"] - Expect(ok).To(BeTrue(), ".Config.Volumes[/foo]") + Expect(image[0].Config.Volumes).To(HaveKey("/foo")) r := podmanTest.Podman([]string{"run", "newimage"}) r.WaitWithDefaultTimeout() @@ -250,7 +248,7 @@ var _ = Describe("Podman commit", func() { for _, v := range image[0].Config.Env { envMap[v] = true } - Expect(envMap["TEST=1=1-01=9.01"]).To(BeTrue(), "envMap[TEST=1=1-01=9.01]") + Expect(envMap).To(HaveKey("TEST=1=1-01=9.01")) }) It("podman commit container and print id to external file", func() { |