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, 24 insertions, 24 deletions

diff --git a/test/podman_pause.bats b/test/podman_pause.bats
index b8f0a8746..4e98eb130 100644
--- a/test/podman_pause.bats
+++ b/test/podman_pause.bats
@@ -11,102 +11,102 @@ function teardown() {
 }
 
 @test "pause a bogus container" {
-    run bash -c "${PODMAN_BINARY} ${PODMAN_OPTIONS} pause foobar"
+    run ${PODMAN_BINARY} ${PODMAN_OPTIONS} pause foobar
     echo "$output"
     [ "$status" -eq 125 ]
 }
 
 @test "unpause a bogus container" {
-    run bash -c "${PODMAN_BINARY} ${PODMAN_OPTIONS} unpause foobar"
+    run ${PODMAN_BINARY} ${PODMAN_OPTIONS} unpause foobar
     echo "$output"
     [ "$status" -eq 125 ]
 }
 
 @test "pause a created container by id" {
-    run bash -c "${PODMAN_BINARY} ${PODMAN_OPTIONS} run -d $BB sleep 60"
+    run ${PODMAN_BINARY} ${PODMAN_OPTIONS} run -d $BB sleep 60
     echo "$output"
     [ "$status" -eq 0 ]
     ctr_id=`echo "$output" | tail -n 1`
-    run bash -c "${PODMAN_BINARY} ${PODMAN_OPTIONS} pause $ctr_id"
+    run ${PODMAN_BINARY} ${PODMAN_OPTIONS} pause $ctr_id
     echo "$output"
     [ "$status" -eq 0 ]
-    run bash -c "${PODMAN_BINARY} ${PODMAN_OPTIONS} unpause $ctr_id"
+    run ${PODMAN_BINARY} ${PODMAN_OPTIONS} unpause $ctr_id
     echo "$output"
     [ "$status" -eq 0 ]
-    run bash -c "${PODMAN_BINARY} ${PODMAN_OPTIONS} rm -f $ctr_id"
+    run ${PODMAN_BINARY} ${PODMAN_OPTIONS} rm -f $ctr_id
     echo "$output"
     [ "$status" -eq 0 ]
 }
 
 @test "pause a running container by id" {
-    run bash -c "${PODMAN_BINARY} ${PODMAN_OPTIONS} run -d $BB sleep 60"
+    run ${PODMAN_BINARY} ${PODMAN_OPTIONS} run -d $BB sleep 60
     echo "$output"
     [ "$status" -eq 0 ]
     ctr_id=`echo "$output" | tail -n 1`
-    run bash -c "${PODMAN_BINARY} ${PODMAN_OPTIONS} pause $ctr_id"
+    run ${PODMAN_BINARY} ${PODMAN_OPTIONS} pause $ctr_id
     echo "$output"
     [ "$status" -eq 0 ]
-    run bash -c "${PODMAN_BINARY} ${PODMAN_OPTIONS} unpause $ctr_id"
+    run ${PODMAN_BINARY} ${PODMAN_OPTIONS} unpause $ctr_id
     echo "$output"
     [ "$status" -eq 0 ]
-    run bash -c "${PODMAN_BINARY} ${PODMAN_OPTIONS} rm -f $ctr_id"
+    run ${PODMAN_BINARY} ${PODMAN_OPTIONS} rm -f $ctr_id
     echo "$output"
     [ "$status" -eq 0 ]
 }
 
 @test "unpause a running container" {
-    run bash -c "${PODMAN_BINARY} ${PODMAN_OPTIONS} run -d $BB sleep 60"
+    run ${PODMAN_BINARY} ${PODMAN_OPTIONS} run -d $BB sleep 60
     echo "$output"
     [ "$status" -eq 0 ]
     ctr_id=`echo "$output" | tail -n 1`
-    run bash -c "${PODMAN_BINARY} ${PODMAN_OPTIONS} unpause $ctr_id"
+    run ${PODMAN_BINARY} ${PODMAN_OPTIONS} unpause $ctr_id
     echo "$output"
     [ "$status" -eq 125 ]
-    run bash -c "${PODMAN_BINARY} ${PODMAN_OPTIONS} rm -f $ctr_id"
+    run ${PODMAN_BINARY} ${PODMAN_OPTIONS} rm -f $ctr_id
     echo "$output"
     [ "$status" -eq 0 ]
 }
 
 @test "remove a paused container by id" {
     skip "Test needs to wait for --force to work for podman rm"
-    run bash -c "${PODMAN_BINARY} ${PODMAN_OPTIONS} run -d $BB sleep 60"
+    run ${PODMAN_BINARY} ${PODMAN_OPTIONS} run -d $BB sleep 60
     echo "$output"
     [ "$status" -eq 0 ]
     ctr_id=`echo "$output" | tail -n 1`
-    run bash -c "${PODMAN_BINARY} ${PODMAN_OPTIONS} pause $ctr_id"
+    run ${PODMAN_BINARY} ${PODMAN_OPTIONS} pause $ctr_id
     echo "$output"
     [ "$status" -eq 0 ]
-    run bash -c "${PODMAN_BINARY} ${PODMAN_OPTIONS} rm $ctr_id"
+    run ${PODMAN_BINARY} ${PODMAN_OPTIONS} rm $ctr_id
     echo "$output"
     [ "$status" -eq 125 ]
-    run bash -c "${PODMAN_BINARY} ${PODMAN_OPTIONS} rm --force $ctr_id"
+    run ${PODMAN_BINARY} ${PODMAN_OPTIONS} rm --force $ctr_id
     echo "$output"
     [ "$status" -eq 0 ]
 }
 
 @test "stop a paused container created by id" {
-    run bash -c "${PODMAN_BINARY} ${PODMAN_OPTIONS} run -d $BB sleep 60"
+    run ${PODMAN_BINARY} ${PODMAN_OPTIONS} run -d $BB sleep 60
     echo "$output"
     [ "$status" -eq 0 ]
     ctr_id=`echo "$output" | tail -n 1`
-    run bash -c "${PODMAN_BINARY} ${PODMAN_OPTIONS} pause $ctr_id"
+    run ${PODMAN_BINARY} ${PODMAN_OPTIONS} pause $ctr_id
     echo "$output"
     [ "$status" -eq 0 ]
-    run bash -c "${PODMAN_BINARY} ${PODMAN_OPTIONS} stop $ctr_id"
+    run ${PODMAN_BINARY} ${PODMAN_OPTIONS} stop $ctr_id
     echo "$output"
     [ "$status" -eq 125 ]
-    run bash -c "${PODMAN_BINARY} ${PODMAN_OPTIONS} unpause $ctr_id"
+    run ${PODMAN_BINARY} ${PODMAN_OPTIONS} unpause $ctr_id
     echo "$output"
     [ "$status" -eq 0 ]
-    run bash -c "${PODMAN_BINARY} ${PODMAN_OPTIONS} ps -a --filter id=$ctr_id"
+    run ${PODMAN_BINARY} ${PODMAN_OPTIONS} ps -a --filter id=$ctr_id
     echo "$output"
     [ "$status" -eq 0 ]
     # Container should be running after unpause and shouldn't
     # be removable without the force flag.
-    run bash -c "${PODMAN_BINARY} ${PODMAN_OPTIONS} rm $ctr_id"
+    run ${PODMAN_BINARY} ${PODMAN_OPTIONS} rm $ctr_id
     echo "$output"
     [ "$status" -eq 125 ]
-    run bash -c "${PODMAN_BINARY} ${PODMAN_OPTIONS} rm -f $ctr_id"
+    run ${PODMAN_BINARY} ${PODMAN_OPTIONS} rm -f $ctr_id
     echo "$output"
     [ "$status" -eq 0 ]
 }