diff options
author | Zdenek Kabelac <zkabelac@redhat.com> | 2017-01-21 19:21:08 +0100 |
---|---|---|
committer | Zdenek Kabelac <zkabelac@redhat.com> | 2017-01-21 19:28:06 +0100 |
commit | e2fa90bf38461a5de63cd11ffcc01fb984c8193b (patch) | |
tree | c21a89cd260aa613f931892e9434413ce48d33aa | |
parent | 1a2b88516bd213a912ba6c5133a977a5003f3a16 (diff) | |
download | lvm2-e2fa90bf38461a5de63cd11ffcc01fb984c8193b.tar.gz |
tests: properly quote heredoc
Prepend \$ for vars which should remain in script.
Also drop --lazy umount.
Move inittest call up, so mntdir and mntusedir have proper full path.
-rw-r--r-- | test/shell/thin-autoumount-dmeventd.sh | 16 |
1 files changed, 9 insertions, 7 deletions
diff --git a/test/shell/thin-autoumount-dmeventd.sh b/test/shell/thin-autoumount-dmeventd.sh index 363efe229..eacad0a9c 100644 --- a/test/shell/thin-autoumount-dmeventd.sh +++ b/test/shell/thin-autoumount-dmeventd.sh @@ -16,6 +16,8 @@ SKIP_WITH_LVMPOLLD=1 export LVM_TEST_THIN_REPAIR_CMD=${LVM_TEST_THIN_REPAIR_CMD-/bin/false} +. lib/inittest + mntdir="${PREFIX}mnt with space" mntusedir="${PREFIX}mntuse" @@ -32,8 +34,6 @@ is_lv_opened_() test $(get lv_field "$1" lv_device_open --binary) = "1" } -. lib/inittest - # # Main # @@ -48,15 +48,17 @@ aux lvmconf "dmeventd/thin_command = \"$PWD/testcmd.sh\"" cat <<- EOF >testcmd.sh #!/bin/sh -echo "Data $DMEVENTD_THIN_POOL_DATA" -echo "Metadata $DMEVENTD_THIN_POOL_METADATA" +echo "Data: \$DMEVENTD_THIN_POOL_DATA" +echo "Metadata: \$DMEVENTD_THIN_POOL_METADATA" -lvextend --use-policies $1 || { - umount --lazy "$mntdir" || true - umount --lazy "$mntusedir" || true +$TESTDIR/lib/lvextend --use-policies \$1 || { + umount "$mntdir" || true + umount "$mntusedir" || true } EOF chmod +x testcmd.sh +# Show prepared script +cat testcmd.sh aux prepare_dmeventd |