summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorYuri Pankov <yuri.pankov@nexenta.com>2017-05-16 11:23:08 -0700
committerPrakash Surya <prakash.surya@delphix.com>2017-05-16 15:28:24 -0700
commitf0e8a3d8e67418155c5316a52ebd6aacaf493b09 (patch)
tree860a76b971219b762f129ee8431776e7bb7eb5f2
parent7da74b7620d65183bc0949a426dc2cad658f19c3 (diff)
downloadillumos-joyent-f0e8a3d8e67418155c5316a52ebd6aacaf493b09.tar.gz
8072 zfs-tests: several test cases incorrectly spell TESTPOOL
Reviewed by: Matthew Ahrens <mahrens@delphix.com> Reviewed by: John Kennedy <jwk404@gmail.com> Approved by: Richard Lowe <richlowe@richlowe.net>
-rw-r--r--usr/src/test/zfs-tests/tests/functional/cli_root/zfs_promote/zfs_promote_006_neg.ksh2
-rw-r--r--usr/src/test/zfs-tests/tests/functional/cli_user/misc/misc.cfg8
-rw-r--r--usr/src/test/zfs-tests/tests/functional/cli_user/misc/zfs_get_001_neg.ksh7
-rw-r--r--usr/src/test/zfs-tests/tests/functional/cli_user/misc/zfs_promote_001_neg.ksh2
-rw-r--r--usr/src/test/zfs-tests/tests/functional/cli_user/misc/zfs_set_001_neg.ksh7
5 files changed, 12 insertions, 14 deletions
diff --git a/usr/src/test/zfs-tests/tests/functional/cli_root/zfs_promote/zfs_promote_006_neg.ksh b/usr/src/test/zfs-tests/tests/functional/cli_root/zfs_promote/zfs_promote_006_neg.ksh
index ffe2c43561..cedfa676a1 100644
--- a/usr/src/test/zfs-tests/tests/functional/cli_root/zfs_promote/zfs_promote_006_neg.ksh
+++ b/usr/src/test/zfs-tests/tests/functional/cli_root/zfs_promote/zfs_promote_006_neg.ksh
@@ -53,7 +53,7 @@ snap=$TESTPOOL/$TESTFS@$TESTSNAP
set -A args "" \
"$TESTPOOL/blah" \
"$TESTPOOL" "$TESTPOOL/$TESTFS" "$snap" \
- "$TESTPOOL/$TESTVOL" "$TESTPOL $TESTPOOL/$TESTFS" \
+ "$TESTPOOL/$TESTVOL" "$TESTPOOL $TESTPOOL/$TESTFS" \
"$clone $TESTPOOL/$TESTFS" "- $clone" "-? $clone"
function cleanup
diff --git a/usr/src/test/zfs-tests/tests/functional/cli_user/misc/misc.cfg b/usr/src/test/zfs-tests/tests/functional/cli_user/misc/misc.cfg
index 55d38e824f..595201fa3f 100644
--- a/usr/src/test/zfs-tests/tests/functional/cli_user/misc/misc.cfg
+++ b/usr/src/test/zfs-tests/tests/functional/cli_user/misc/misc.cfg
@@ -40,17 +40,17 @@ PROP_NAMES="\
# zfs get/set subcommands - ordered as per the list above so we
# can iterate over both sets in an array
PROP_VALS="\
- secure discard on \
+ passthrough discard on \
fletcher2 on on \
on legacy none on \
- 128k none on on \
+ 128K none on on \
visible"
# these are an alternate set of property values
PROP_ALTVALS="\
- noallow groupmask off \
+ passthrough-x groupmask off \
fletcher4 lzjb off \
- off /tmp/zfstest 100m off \
+ off /tmp/zfstest 100M off \
512 10m off off \
hidden"
diff --git a/usr/src/test/zfs-tests/tests/functional/cli_user/misc/zfs_get_001_neg.ksh b/usr/src/test/zfs-tests/tests/functional/cli_user/misc/zfs_get_001_neg.ksh
index 5ec603dfea..422dc5848e 100644
--- a/usr/src/test/zfs-tests/tests/functional/cli_user/misc/zfs_get_001_neg.ksh
+++ b/usr/src/test/zfs-tests/tests/functional/cli_user/misc/zfs_get_001_neg.ksh
@@ -41,7 +41,6 @@
# 1. Run zfs get with an array of different arguments
# 2. Verify for each property, we get the value that's expected
#
-#
log_assert "zfs get works when run as a user"
@@ -50,16 +49,16 @@ typeset -i i=0
set -A props $PROP_NAMES
set -A prop_vals $PROP_VALS
-while [[ $i -lt ${#args[*]} ]]
+while [[ $i -lt ${#props[*]} ]]
do
PROP=${props[$i]}
EXPECTED=${prop_vals[$i]}
- ACTUAL=$( zfs get $PROP -o value -H snapdir $TESTPOOl/$TESTFS/prop )
+ ACTUAL=$(zfs get -H -o value $PROP $TESTPOOL/$TESTFS/prop)
if [ "$ACTUAL" != "$EXPECTED" ]
then
log_fail "Property $PROP value was $ACTUAL, expected $EXPECTED"
fi
- i=$(( $i + 1 ))
+ i=$(( $i + 1 ))
done
log_pass "zfs get works when run as a user"
diff --git a/usr/src/test/zfs-tests/tests/functional/cli_user/misc/zfs_promote_001_neg.ksh b/usr/src/test/zfs-tests/tests/functional/cli_user/misc/zfs_promote_001_neg.ksh
index 6a4798b8bd..9534f0d3ef 100644
--- a/usr/src/test/zfs-tests/tests/functional/cli_user/misc/zfs_promote_001_neg.ksh
+++ b/usr/src/test/zfs-tests/tests/functional/cli_user/misc/zfs_promote_001_neg.ksh
@@ -50,7 +50,7 @@ log_mustnot zfs promote $TESTPOOL/$TESTFS/clone
# Now verify that the above command didn't do anything
if datasetexists $TESTPOOL/$TESTFS/clone@snap
then
- log_fail "Clone $TESTPOOl/$TESTFS/clone was actually promoted!"
+ log_fail "Clone $TESTPOOL/$TESTFS/clone was actually promoted!"
fi
log_pass "zfs promote returns an error when run as a user"
diff --git a/usr/src/test/zfs-tests/tests/functional/cli_user/misc/zfs_set_001_neg.ksh b/usr/src/test/zfs-tests/tests/functional/cli_user/misc/zfs_set_001_neg.ksh
index aac9554d37..5110708b0f 100644
--- a/usr/src/test/zfs-tests/tests/functional/cli_user/misc/zfs_set_001_neg.ksh
+++ b/usr/src/test/zfs-tests/tests/functional/cli_user/misc/zfs_set_001_neg.ksh
@@ -41,7 +41,6 @@
# 1. Attempt to set an array of properties on a dataset
# 2. Verify that those properties were not set and retain their original values.
#
-#
log_assert "zfs set returns an error when run as a user"
@@ -51,7 +50,7 @@ set -A props $PROP_NAMES
set -A prop_vals $PROP_VALS
set -A prop_new $PROP_ALTVALS
-while [[ $i -lt ${#args[*]} ]]
+while [[ $i -lt ${#props[*]} ]]
do
PROP=${props[$i]}
EXPECTED=${prop_vals[$i]}
@@ -59,12 +58,12 @@ do
log_mustnot zfs set $PROP=$NEW $TESTPOOL/$TESTFS/prop
# Now verify that the above command did nothing
- ACTUAL=$(zfs get $PROP -o value -H snapdir $TESTPOOl/$TESTFS/prop )
+ ACTUAL=$(zfs get -H -o value $PROP $TESTPOOL/$TESTFS/prop)
if [ "$ACTUAL" != "$EXPECTED" ]
then
log_fail "Property $PROP was set to $ACTUAL, expected $EXPECTED"
fi
- i=$(( $i + 1 ))
+ i=$(( $i + 1 ))
done
log_pass "zfs set returns an error when run as a user"