btrfs-progs: tests: add variable quotation to cli-tests
authorDavid Sterba <dsterba@suse.com>
Tue, 24 Jan 2017 16:59:47 +0000 (17:59 +0100)
committerDavid Sterba <dsterba@suse.com>
Fri, 27 Jan 2017 11:20:43 +0000 (12:20 +0100)
Signed-off-by: David Sterba <dsterba@suse.com>
tests/cli-tests/001-btrfs/test.sh
tests/cli-tests/002-balance-full-no-filters/test.sh
tests/cli-tests/003-fi-resize-args/test.sh
tests/cli-tests/004-send-parent-multi-subvol/test.sh
tests/cli-tests/005-qgroup-show/test.sh

index 560e489..c680604 100755 (executable)
@@ -1,18 +1,18 @@
 #!/bin/bash
 # test commands of btrfs
 
-source $TOP/tests/common
+source "$TOP/tests/common"
 
 check_prereq btrfs
 
 # returns 1
 run_mayfail $TOP/btrfs || true
-run_check $TOP/btrfs version
-run_check $TOP/btrfs version --
-run_check $TOP/btrfs help
-run_check $TOP/btrfs help --
-run_check $TOP/btrfs help --full
-run_check $TOP/btrfs --help
-run_check $TOP/btrfs --help --full
-run_check $TOP/btrfs --version
-run_check $TOP/btrfs --version --help
+run_check "$TOP/btrfs" version
+run_check "$TOP/btrfs" version --
+run_check "$TOP/btrfs" help
+run_check "$TOP/btrfs" help --
+run_check "$TOP/btrfs" help --full
+run_check "$TOP/btrfs" --help
+run_check "$TOP/btrfs" --help --full
+run_check "$TOP/btrfs" --version
+run_check "$TOP/btrfs" --version --help
index c2757f2..81a719e 100755 (executable)
@@ -2,7 +2,7 @@
 #
 # coverage of balance --full-balance
 
-source $TOP/tests/common
+source "$TOP/tests/common"
 
 check_prereq mkfs.btrfs
 check_prereq btrfs
@@ -10,12 +10,12 @@ check_prereq btrfs
 setup_root_helper
 prepare_test_dev 2g
 
-run_check $TOP/mkfs.btrfs -f $IMAGE
+run_check "$TOP/mkfs.btrfs" -f "$IMAGE"
 run_check_mount_test_dev
 
-run_check $SUDO_HELPER $TOP/btrfs balance start --full-balance $TEST_MNT
-run_check $SUDO_HELPER $TOP/btrfs balance start $TEST_MNT
-run_check $SUDO_HELPER $TOP/btrfs balance --full-balance $TEST_MNT
-run_check $SUDO_HELPER $TOP/btrfs balance $TEST_MNT
+run_check $SUDO_HELPER "$TOP/btrfs" balance start --full-balance "$TEST_MNT"
+run_check $SUDO_HELPER "$TOP/btrfs" balance start "$TEST_MNT"
+run_check $SUDO_HELPER "$TOP/btrfs" balance --full-balance "$TEST_MNT"
+run_check $SUDO_HELPER "$TOP/btrfs" balance "$TEST_MNT"
 
 run_check_umount_test_dev
index 2f136fa..b835e07 100755 (executable)
@@ -2,7 +2,7 @@
 #
 # test parsing of various resize arguments
 
-source $TOP/tests/common
+source "$TOP/tests/common"
 
 check_prereq mkfs.btrfs
 check_prereq btrfs
@@ -10,37 +10,37 @@ check_prereq btrfs
 setup_root_helper
 prepare_test_dev 2g
 
-run_check $TOP/mkfs.btrfs -f $IMAGE
+run_check "$TOP/mkfs.btrfs" -f "$IMAGE"
 run_check_mount_test_dev
 
 # missing the one of the required arguments
 for sep in '' '--'; do
-       run_check_stdout $TOP/btrfs filesystem resize $sep |
+       run_check_stdout "$TOP/btrfs" filesystem resize $sep |
                grep -q "btrfs filesystem resize: too few arguments"
-       run_check_stdout $TOP/btrfs filesystem resize $sep $TEST_MNT |
+       run_check_stdout "$TOP/btrfs" filesystem resize $sep "$TEST_MNT" |
                grep -q "btrfs filesystem resize: too few arguments"
-       run_check_stdout $TOP/btrfs filesystem resize $sep -128M |
+       run_check_stdout "$TOP/btrfs" filesystem resize $sep -128M |
                grep -q "btrfs filesystem resize: too few arguments"
-       run_check_stdout $TOP/btrfs filesystem resize $sep +128M |
+       run_check_stdout "$TOP/btrfs" filesystem resize $sep +128M |
                grep -q "btrfs filesystem resize: too few arguments"
-       run_check_stdout $TOP/btrfs filesystem resize $sep 512M |
+       run_check_stdout "$TOP/btrfs" filesystem resize $sep 512M |
                grep -q "btrfs filesystem resize: too few arguments"
-       run_check_stdout $TOP/btrfs filesystem resize $sep 1:-128M |
+       run_check_stdout "$TOP/btrfs" filesystem resize $sep 1:-128M |
                grep -q "btrfs filesystem resize: too few arguments"
-       run_check_stdout $TOP/btrfs filesystem resize $sep 1:512M |
+       run_check_stdout "$TOP/btrfs" filesystem resize $sep 1:512M |
                grep -q "btrfs filesystem resize: too few arguments"
-       run_check_stdout $TOP/btrfs filesystem resize $sep 1:+128M |
+       run_check_stdout "$TOP/btrfs" filesystem resize $sep 1:+128M |
                grep -q "btrfs filesystem resize: too few arguments"
 done
 
 # valid resize
 for sep in '' '--'; do
-       run_check $SUDO_HELPER $TOP/btrfs filesystem resize $sep -128M $TEST_MNT
-       run_check $SUDO_HELPER $TOP/btrfs filesystem resize $sep +128M $TEST_MNT
-       run_check $SUDO_HELPER $TOP/btrfs filesystem resize $sep 512M $TEST_MNT
-       run_check $SUDO_HELPER $TOP/btrfs filesystem resize $sep 1:-128M $TEST_MNT
-       run_check $SUDO_HELPER $TOP/btrfs filesystem resize $sep 1:512M $TEST_MNT
-       run_check $SUDO_HELPER $TOP/btrfs filesystem resize $sep 1:+128M $TEST_MNT
+       run_check $SUDO_HELPER "$TOP/btrfs" filesystem resize $sep -128M "$TEST_MNT"
+       run_check $SUDO_HELPER "$TOP/btrfs" filesystem resize $sep +128M "$TEST_MNT"
+       run_check $SUDO_HELPER "$TOP/btrfs" filesystem resize $sep 512M "$TEST_MNT"
+       run_check $SUDO_HELPER "$TOP/btrfs" filesystem resize $sep 1:-128M "$TEST_MNT"
+       run_check $SUDO_HELPER "$TOP/btrfs" filesystem resize $sep 1:512M "$TEST_MNT"
+       run_check $SUDO_HELPER "$TOP/btrfs" filesystem resize $sep 1:+128M "$TEST_MNT"
 done
 
 run_check_umount_test_dev
index e9ccf22..860f657 100755 (executable)
@@ -2,7 +2,7 @@
 #
 # minimal test for the following syntax: btrfs send -p parent subvol1 subvol2
 
-source $TOP/tests/common
+source "$TOP/tests/common"
 
 check_prereq mkfs.btrfs
 check_prereq btrfs
@@ -10,7 +10,7 @@ check_prereq btrfs
 setup_root_helper
 prepare_test_dev 2g
 
-run_check $TOP/mkfs.btrfs -f $IMAGE
+run_check "$TOP/mkfs.btrfs" -f "$IMAGE"
 run_check_mount_test_dev
 
 here=`pwd`
@@ -26,7 +26,7 @@ run_check $SUDO_HELPER btrfs subvolume snapshot -r subv-parent subv-snap3
 
 run_check truncate -s0 "$here"/send.stream
 run_check chmod a+w "$here"/send.stream
-run_check $SUDO_HELPER $TOP/btrfs send -f "$here"/send.stream -p subv-snap1 subv-snap2 subv-snap3
+run_check $SUDO_HELPER "$TOP/btrfs" send -f "$here"/send.stream -p subv-snap1 subv-snap2 subv-snap3
 
 cd "$here" || _fail "cannot chdir back to test directory"
 rm send.stream
index 28c22c6..46d3c3a 100755 (executable)
@@ -2,7 +2,7 @@
 #
 # qgroup show behaviour when quotas are not enabled
 
-source $TOP/tests/common
+source "$TOP/tests/common"
 
 check_prereq mkfs.btrfs
 check_prereq btrfs
@@ -10,12 +10,12 @@ check_prereq btrfs
 setup_root_helper
 prepare_test_dev 2g
 
-run_check $TOP/mkfs.btrfs -f $IMAGE
+run_check "$TOP/mkfs.btrfs" -f "$IMAGE"
 run_check_mount_test_dev
-run_mayfail $TOP/btrfs qgroup show "$TEST_MNT"
-run_mayfail $SUDO_HELPER $TOP/btrfs qgroup show "$TEST_MNT"
-run_check $SUDO_HELPER $TOP/btrfs quota enable "$TEST_MNT"
-run_mayfail $TOP/btrfs qgroup show "$TEST_MNT"
-run_check $SUDO_HELPER $TOP/btrfs qgroup show "$TEST_MNT"
-run_check $SUDO_HELPER $TOP/btrfs quota disable "$TEST_MNT"
+run_mayfail "$TOP/btrfs" qgroup show "$TEST_MNT"
+run_mayfail $SUDO_HELPER "$TOP/btrfs" qgroup show "$TEST_MNT"
+run_check $SUDO_HELPER "$TOP/btrfs" quota enable "$TEST_MNT"
+run_mayfail "$TOP/btrfs" qgroup show "$TEST_MNT"
+run_check $SUDO_HELPER "$TOP/btrfs" qgroup show "$TEST_MNT"
+run_check $SUDO_HELPER "$TOP/btrfs" quota disable "$TEST_MNT"
 run_check_umount_test_dev