Skip to content

Commit acb39fc

Browse files
AttilaFueloepbehlendorf
authored andcommitted
ZTS: three small follow up fixes for #11167
Follow up fix for 0cb40fa. Remove unused variables, don't source unused libs and add missed cleanup. Reviewed-by: Ryan Moeller <[email protected]> Reviewed-by: Brian Behlendorf <[email protected]> Signed-off-by: Attila Fülöp <[email protected]> Closes #11311
1 parent 6539bf7 commit acb39fc

File tree

3 files changed

+4
-8
lines changed

3 files changed

+4
-8
lines changed

tests/zfs-tests/tests/functional/cli_root/zpool_add/zpool_add_dryrun_output.ksh

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -25,9 +25,7 @@
2525
#
2626

2727
. $STF_SUITE/include/libtest.shlib
28-
. $STF_SUITE/tests/functional/cli_root/zpool_add/zpool_add.kshlib
2928

30-
typeset TMPFILE_PREFIX="$TEST_BASE_DIR/zpool_add_dryrun_output"
3129
typeset STR_DRYRUN="would update '$TESTPOOL' to the following configuration:"
3230
typeset VDEV_PREFIX="$TEST_BASE_DIR/filedev"
3331

@@ -136,7 +134,7 @@ verify_runnable "global"
136134
function cleanup
137135
{
138136
destroy_pool "$TESTPOOL"
139-
rm -f "$TMPFILE_PREFIX"* "$VDEV_PREFIX"*
137+
rm -f "$VDEV_PREFIX"*
140138
}
141139

142140
log_assert "'zpool add -n <pool> <vdev> ...' can display the configuration"

tests/zfs-tests/tests/functional/cli_root/zpool_create/zpool_create_dryrun_output.ksh

Lines changed: 2 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -25,9 +25,7 @@
2525
#
2626

2727
. $STF_SUITE/include/libtest.shlib
28-
. $STF_SUITE/tests/functional/cli_root/zpool_create/zpool_create.shlib
2928

30-
typeset TMPFILE_PREFIX="$TEST_BASE_DIR/zpool_create_dryrun_output"
3129
typeset STR_DRYRUN="would create '$TESTPOOL' with the following layout:"
3230
typeset VDEV_PREFIX="$TEST_BASE_DIR/filedev"
3331

@@ -36,7 +34,7 @@ typeset VDEV_PREFIX="$TEST_BASE_DIR/filedev"
3634
# 'zpool create -n <pool> <vdev> ...' can display the correct configuration
3735
#
3836
# STRATEGY:
39-
# 1. Create a storage pool
37+
# 1. Create -n a storage pool and verify the output is as expected.
4038
#
4139

4240
typeset -a dev=(
@@ -112,13 +110,12 @@ verify_runnable "global"
112110

113111
function cleanup
114112
{
115-
rm -f "$TMPFILE_PREFIX"* "$VDEV_PREFIX"*
113+
rm -f "$VDEV_PREFIX"*
116114
}
117115

118116
log_assert "'zpool add -n <pool> <vdev> ...' can display the configuration"
119117

120118
log_onexit cleanup
121-
typeset disk1=$(create_blockfile $FILESIZE)
122119

123120
# Create needed file vdevs.
124121
for (( i=0; i < ${#dev[@]}; i+=1 )); do

tests/zfs-tests/tests/functional/cli_root/zpool_split/zpool_split_dryrun_output.ksh

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -118,6 +118,7 @@ verify_runnable "global"
118118
function cleanup
119119
{
120120
destroy_pool "$TESTPOOL"
121+
rm -f "$VDEV_PREFIX"*
121122
}
122123

123124
log_assert \

0 commit comments

Comments
 (0)