diff options
| author | Ed Santiago <santiago@redhat.com> | 2022-09-15 16:18:46 -0600 |
|---|---|---|
| committer | Ed Santiago <santiago@redhat.com> | 2022-09-15 20:10:34 -0600 |
| commit | cfbc4aaeb50b5fb12ec9363328bb08fc459a1067 (patch) | |
| tree | e396257e102a885561638fe6702d6c0a6e917743 /test/compose | |
| parent | 750726e62c13941a739ef70040a1ec0d745cdb43 (diff) | |
| download | podman-cfbc4aaeb50b5fb12ec9363328bb08fc459a1067.tar.gz podman-cfbc4aaeb50b5fb12ec9363328bb08fc459a1067.tar.bz2 podman-cfbc4aaeb50b5fb12ec9363328bb08fc459a1067.zip | |
Cleanup: fix problems reported by shell lint
Followup to #15616, which is not usable as it is (way, way, way
too much noise) but actually found a few real nits that should
be fixed.
Signed-off-by: Ed Santiago <santiago@redhat.com>
Diffstat (limited to 'test/compose')
| -rwxr-xr-x | test/compose/test-compose | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/test/compose/test-compose b/test/compose/test-compose index 99d063c25..fe2da9532 100755 --- a/test/compose/test-compose +++ b/test/compose/test-compose @@ -64,7 +64,7 @@ function is() { local expect=$2 local testname=$3 - if [[ $actual = $expect ]]; then + if [[ "$actual" = "$expect" ]]; then # On success, include expected value; this helps readers understand _show_ok 1 "$testname=$expect" return @@ -303,12 +303,12 @@ n_tests=0 # We aren't really TAP 13; this helps logformatter recognize our output as BATS echo "TAP version 13" -for t in ${tests_to_run[@]}; do +for t in "${tests_to_run[@]}"; do testdir="$(dirname $t)" testname="$(basename $testdir)" if [ -e $test_dir/SKIP ]; then - local reason="$(<$test_dir/SKIP)" + reason="$(<$test_dir/SKIP)" if [ -n "$reason" ]; then reason=" - $reason" fi |