3 files changed, 79 insertions, 0 deletions
diff --git a/test/e2e/play_kube_test.go b/test/e2e/play_kube_test.go
index 331412a39..b0a9f2ead 100644
--- a/test/e2e/play_kube_test.go
+++ b/test/e2e/play_kube_test.go
@@ -140,6 +140,30 @@ var _ = Describe("Podman generate kube", func() {
 		Expect(inspect.OutputToString()).To(ContainSubstring(ctrCmd[0]))
 	})
 
+	It("podman play kube test correct output", func() {
+		ctrName := "testCtr"
+		ctrCmd := []string{"echo", "hello"}
+		testContainer := Container{ctrCmd, ALPINE, ctrName, false, nil, nil}
+		tempFile := filepath.Join(podmanTest.TempDir, "kube.yaml")
+
+		err := generateKubeYaml([]Container{testContainer}, tempFile)
+		Expect(err).To(BeNil())
+
+		kube := podmanTest.Podman([]string{"play", "kube", tempFile})
+		kube.WaitWithDefaultTimeout()
+		Expect(kube.ExitCode()).To(Equal(0))
+
+		logs := podmanTest.Podman([]string{"logs", ctrName})
+		logs.WaitWithDefaultTimeout()
+		Expect(logs.ExitCode()).To(Equal(0))
+		Expect(logs.OutputToString()).To(ContainSubstring("hello"))
+
+		inspect := podmanTest.Podman([]string{"inspect", ctrName, "--format", "'{{ .Config.Cmd }}'"})
+		inspect.WaitWithDefaultTimeout()
+		Expect(inspect.ExitCode()).To(Equal(0))
+		Expect(inspect.OutputToString()).To(ContainSubstring("hello"))
+	})
+
 	It("podman play kube cap add", func() {
 		ctrName := "testCtr"
 		ctrCmd := []string{"cat", "/proc/self/status"}
diff --git a/test/system/030-run.bats b/test/system/030-run.bats
index cefff0e2c..9e609b434 100644
--- a/test/system/030-run.bats
+++ b/test/system/030-run.bats
@@ -43,4 +43,17 @@ echo $rand        |   0 | $rand
     is "$output" "" "unwanted /sys/kernel in 'mount' output (with --net=host)"
 }
 
+# 'run --rm' goes through different code paths and may lose exit status.
+# See https://github.com/containers/libpod/issues/3795
+@test "podman run --rm" {
+    skip_if_remote "podman-remote does not handle exit codes"
+
+    run_podman 0 run --rm $IMAGE /bin/true
+    run_podman 1 run --rm $IMAGE /bin/false
+
+    # Believe it or not, 'sh -c' resulted in different behavior
+    run_podman 0 run --rm $IMAGE sh -c /bin/true
+    run_podman 1 run --rm $IMAGE sh -c /bin/false
+}
+
 # vim: filetype=sh
diff --git a/test/system/055-rm.bats b/test/system/055-rm.bats
new file mode 100644
index 000000000..c13c8c52e
--- /dev/null
+++ b/test/system/055-rm.bats
@@ -0,0 +1,42 @@
+#!/usr/bin/env bats   -*- bats -*-
+#
+# tests for podman rm
+#
+
+load helpers
+
+@test "podman rm" {
+    rand=$(random_string 30)
+    run_podman run --name $rand $IMAGE /bin/true
+
+    # Don't care about output, just check exit status (it should exist)
+    run_podman 0 inspect $rand
+
+    # container should be in output of 'ps -a'
+    run_podman ps -a
+    is "$output" ".* $IMAGE .*/true .* $rand" "Container present in 'ps -a'"
+
+    # Remove container; now 'inspect' should fail
+    run_podman rm $rand
+    run_podman 125 inspect $rand
+}
+
+# I'm sorry! This test takes 13 seconds. There's not much I can do about it,
+# please know that I think it's justified: podman 1.5.0 had a strange bug
+# in with exit status was not preserved on some code paths with 'rm -f'
+# or 'podman run --rm' (see also 030-run.bats). The test below is a bit
+# kludgy: what we care about is the exit status of the killed container,
+# not 'podman rm', but BATS has no provision (that I know of) for forking,
+# so what we do is start the 'rm' beforehand and monitor the exit status
+# of the 'sleep' container.
+#
+# See https://github.com/containers/libpod/issues/3795
+@test "podman rm -f" {
+    skip_if_remote "podman-remote does not handle exit codes"
+
+    rand=$(random_string 30)
+    ( sleep 3; run_podman rm -f $rand ) &
+    run_podman 137 run --name $rand $IMAGE sleep 30
+}
+
+# vim: filetype=sh