Fix test syntax

Remove the use of bash -c unless it involves a bash pipe. Also, use
run test_runner.sh inside script so that centos can be happy.

Signed-off-by: baude <bbaude@redhat.com>

Closes: #239
Approved by: baude

1 files changed, 18 insertions, 18 deletions

diff --git a/test/podman_kill.bats b/test/podman_kill.bats
index bb55ed31d..f24bd0971 100644
--- a/test/podman_kill.bats
+++ b/test/podman_kill.bats
@@ -11,61 +11,61 @@ function setup() {
 }
 
 @test "kill a bogus container" {
-    run bash -c "${PODMAN_BINARY} ${PODMAN_OPTIONS} kill foobar"
+    run ${PODMAN_BINARY} ${PODMAN_OPTIONS} kill foobar
     echo "$output"
     [ "$status" -ne 0 ]
 }
 
 @test "kill a running container by id" {
-    run bash -c "${PODMAN_BINARY} ${PODMAN_OPTIONS} run -d ${ALPINE} sleep 9999"
+    run ${PODMAN_BINARY} ${PODMAN_OPTIONS} run -d ${ALPINE} sleep 9999
     [ "$status" -eq 0 ]
     ctr_id="$output"
-    run bash -c "${PODMAN_BINARY} ${PODMAN_OPTIONS} ps"
+    run ${PODMAN_BINARY} ${PODMAN_OPTIONS} ps
     [ "$status" -eq 0 ]
-    run bash -c "${PODMAN_BINARY} ${PODMAN_OPTIONS} kill $ctr_id"
+    run ${PODMAN_BINARY} ${PODMAN_OPTIONS} kill $ctr_id
     [ "$status" -eq 0 ]
-    run bash -c "${PODMAN_BINARY} ${PODMAN_OPTIONS} ps"
+    run ${PODMAN_BINARY} ${PODMAN_OPTIONS} ps
     [ "$status" -eq 0 ]
 }
 
 @test "kill a running container by id with TERM" {
-    run bash -c "${PODMAN_BINARY} ${PODMAN_OPTIONS} run -d ${ALPINE} sleep 9999"
+    run ${PODMAN_BINARY} ${PODMAN_OPTIONS} run -d ${ALPINE} sleep 9999
     [ "$status" -eq 0 ]
     ctr_id="$output"
-    run bash -c "${PODMAN_BINARY} ${PODMAN_OPTIONS} ps"
+    run ${PODMAN_BINARY} ${PODMAN_OPTIONS} ps
     [ "$status" -eq 0 ]
-    run bash -c "${PODMAN_BINARY} ${PODMAN_OPTIONS} kill -s TERM $ctr_id"
+    run ${PODMAN_BINARY} ${PODMAN_OPTIONS} kill -s TERM $ctr_id
     [ "$status" -eq 0 ]
-    run bash -c "${PODMAN_BINARY} ${PODMAN_OPTIONS} ps --no-trunc"
+    run ${PODMAN_BINARY} ${PODMAN_OPTIONS} ps --no-trunc
     [ "$status" -eq 0 ]
 }
 
 @test "kill a running container by name" {
-    run bash -c "${PODMAN_BINARY} ${PODMAN_OPTIONS} run --name test1 -d ${ALPINE} sleep 9999"
+    run ${PODMAN_BINARY} ${PODMAN_OPTIONS} run --name test1 -d ${ALPINE} sleep 9999
     [ "$status" -eq 0 ]
-    run bash -c "${PODMAN_BINARY} ${PODMAN_OPTIONS} ps"
+    run ${PODMAN_BINARY} ${PODMAN_OPTIONS} ps
     [ "$status" -eq 0 ]
-    run bash -c "${PODMAN_BINARY} ${PODMAN_OPTIONS} kill -s TERM test1"
+    run ${PODMAN_BINARY} ${PODMAN_OPTIONS} kill -s TERM test1
     [ "$status" -eq 0 ]
-    run bash -c "${PODMAN_BINARY} ${PODMAN_OPTIONS} ps --no-trunc"
+    run ${PODMAN_BINARY} ${PODMAN_OPTIONS} ps --no-trunc
     [ "$status" -eq 0 ]
 }
 
 @test "kill a running container by id with a bogus signal" {
-    run bash -c "${PODMAN_BINARY} ${PODMAN_OPTIONS} run -d ${ALPINE} sleep 9999"
+    run ${PODMAN_BINARY} ${PODMAN_OPTIONS} run -d ${ALPINE} sleep 9999
     [ "$status" -eq 0 ]
     ctr_id="$output"
-    run bash -c "${PODMAN_BINARY} ${PODMAN_OPTIONS} ps"
+    run ${PODMAN_BINARY} ${PODMAN_OPTIONS} ps
     [ "$status" -eq 0 ]
-    run bash -c "${PODMAN_BINARY} ${PODMAN_OPTIONS} kill -s foobar $ctr_id"
+    run ${PODMAN_BINARY} ${PODMAN_OPTIONS} kill -s foobar $ctr_id
     [ "$status" -eq 125 ]
-    run bash -c "${PODMAN_BINARY} ${PODMAN_OPTIONS} ps --no-trunc"
+    run ${PODMAN_BINARY} ${PODMAN_OPTIONS} ps --no-trunc
     [ "$status" -eq 0 ]
 }
 
 @test "kill the latest container run" {
     ${PODMAN_BINARY} ${PODMAN_OPTIONS} run -d ${ALPINE} sleep 9999
-    run bash -c "${PODMAN_BINARY} ${PODMAN_OPTIONS} kill -l"
+    run ${PODMAN_BINARY} ${PODMAN_OPTIONS} kill -l
     echo "$output"
     [ "$status" -eq 0 ]
 }