OpenZFS 8071 - zfs-tests: 7290 missed some cases

Authored by: Yuri Pankov <yuri.pankov@nexenta.com>
Reviewed by: Matthew Ahrens <mahrens@delphix.com>
Reviewed by: John Kennedy <jwk404@gmail.com>
Approved by: Richard Lowe <richlowe@richlowe.net>
Reviewed-by: Giuseppe Di Natale <dinatale2@llnl.gov>
Reviewed-by: Brian Behlendorf <behlendorf1@llnl.gov>
Ported-by: bunder2015 <omfgbunder@gmail.com>
OpenZFS-issue: https://www.illumos.org/issues/8071
OpenZFS-commit: https://github.com/openzfs/openzfs/commit/e84991e
Closes #6161

diff --git a/tests/zfs-tests/tests/functional/cli_root/zpool_destroy/zpool_destroy_002_pos.ksh b/tests/zfs-tests/tests/functional/cli_root/zpool_destroy/zpool_destroy_002_pos.ksh
index 863e7d368ee98d165386c432d2f394a3d0d3ca6a..ad9425795c20c54442a186c553049850e1e3570b 100755 (executable)
--- a/tests/zfs-tests/tests/functional/cli_root/zpool_destroy/zpool_destroy_002_pos.ksh
+++ b/tests/zfs-tests/tests/functional/cli_root/zpool_destroy/zpool_destroy_002_pos.ksh
@@ -52,9 +52,7 @@ function cleanup
        [[ -n $cwd ]] && log_must cd $cwd
 
        if [[ -d $TESTDIR ]]; then
-               ismounted $TESTDIR
-               ((  $? == 0 )) && \
-                       log_must $UNMOUNT $TESTDIR
+               ismounted $TESTDIR && log_must umount $TESTDIR
                log_must rm -rf $TESTDIR
        fi
 

diff --git a/tests/zfs-tests/tests/functional/xattr/xattr_009_neg.ksh b/tests/zfs-tests/tests/functional/xattr/xattr_009_neg.ksh
index ea5b40eec97f40bdd3e40a7007d725d0f34bcbde..f64f7e42b4750b9b62ac898103d814d9d9030d14 100755 (executable)
--- a/tests/zfs-tests/tests/functional/xattr/xattr_009_neg.ksh
+++ b/tests/zfs-tests/tests/functional/xattr/xattr_009_neg.ksh
@@ -58,9 +58,9 @@ log_must touch $TESTDIR/myfile.$$
 create_xattr $TESTDIR/myfile.$$ passwd /etc/passwd
 
 # Try to create a soft link from the xattr namespace to the default namespace
-log_mustnot runat $TESTDIR/myfile.$$ $LN -s /etc/passwd foo
+log_mustnot runat $TESTDIR/myfile.$$ ln -s /etc/passwd foo
 
 # Try to create a hard link from the xattr namespace to the default namespace
-log_mustnot runat $TESTDIR/myfile.$$ $LN /etc/passwd foo
+log_mustnot runat $TESTDIR/myfile.$$ ln /etc/passwd foo
 
 log_pass "links between xattr and normal file namespace fail"

diff --git a/tests/zfs-tests/tests/functional/zvol/zvol_misc/zvol_misc_002_pos.ksh b/tests/zfs-tests/tests/functional/zvol/zvol_misc/zvol_misc_002_pos.ksh
index c749756fd59d5e2441a6056dc9142144eb07d179..00a7ac166316d67304acbc4fe3f42af936ff4c1c 100755 (executable)
--- a/tests/zfs-tests/tests/functional/zvol/zvol_misc/zvol_misc_002_pos.ksh
+++ b/tests/zfs-tests/tests/functional/zvol/zvol_misc/zvol_misc_002_pos.ksh
@@ -77,14 +77,13 @@ typeset -i fn=0
 typeset -i retval=0
 
 while (( 1 )); do
-        file_write -o create -f $TESTDIR/testfile$$.$fn \
-            -b $BLOCKSZ -c $NUM_WRITES
-        retval=$?
-        if (( $retval != 0 )); then
-                break
-        fi
-
-        (( fn = fn + 1 ))
+       file_write -o create -f $TESTDIR/testfile$$.$fn \
+           -b $BLOCKSZ -c $NUM_WRITES
+       retval=$?
+       if (( $retval != 0 )); then
+               break
+       fi
+       (( fn = fn + 1 ))
 done
 
 if is_linux; then