summaryrefslogtreecommitdiff
path: root/test/e2e
diff options
context:
space:
mode:
authorMatthew Heon <matthew.heon@pm.me>2020-02-12 17:06:53 -0500
committerMatthew Heon <matthew.heon@pm.me>2020-02-12 20:14:20 -0500
commitd2100cd47359ef0afa1a25f6f3a6a7708cd855f7 (patch)
treea6af1821b09d5bf7b5511813ec6214c79f13f7b4 /test/e2e
parent2281cbdd6d5b1b7bca99a605ffc8625b6bee7eb3 (diff)
downloadpodman-d2100cd47359ef0afa1a25f6f3a6a7708cd855f7.tar.gz
podman-d2100cd47359ef0afa1a25f6f3a6a7708cd855f7.tar.bz2
podman-d2100cd47359ef0afa1a25f6f3a6a7708cd855f7.zip
Only set --all when a status filter is given to ps
The changes in #5075 turn out to be too aggressive; we should only be setting --all if a status= filter is given. Otherwise only running containers are filtered. Signed-off-by: Matthew Heon <matthew.heon@pm.me>
Diffstat (limited to 'test/e2e')
-rw-r--r--test/e2e/ps_test.go13
1 files changed, 13 insertions, 0 deletions
diff --git a/test/e2e/ps_test.go b/test/e2e/ps_test.go
index fccc5c93b..48dd186e2 100644
--- a/test/e2e/ps_test.go
+++ b/test/e2e/ps_test.go
@@ -243,6 +243,19 @@ var _ = Describe("Podman ps", func() {
Expect(psAll.OutputToString()).To(Equal(psFilter.OutputToString()))
})
+ It("podman filter without status does not find non-running", func() {
+ ctrName := "aContainerName"
+ ctr := podmanTest.Podman([]string{"create", "--name", ctrName, "-t", "-i", ALPINE, "ls", "/"})
+ ctr.WaitWithDefaultTimeout()
+ Expect(ctr.ExitCode()).To(Equal(0))
+
+ psFilter := podmanTest.Podman([]string{"ps", "--no-trunc", "--quiet", "--format", "{{.Names}}", "--filter", fmt.Sprintf("name=%s", ctrName)})
+ psFilter.WaitWithDefaultTimeout()
+ Expect(psFilter.ExitCode()).To(Equal(0))
+
+ Expect(strings.Contains(psFilter.OutputToString(), ctrName)).To(BeFalse())
+ })
+
It("podman ps mutually exclusive flags", func() {
session := podmanTest.Podman([]string{"ps", "-aqs"})
session.WaitWithDefaultTimeout()