Skip to content

Commit

Permalink
remove sanity check in replacement test
Browse files Browse the repository at this point in the history
In replacement test, it spawns a process to truncate a file background
and make sure that the process exists 1 second later. However, the
process may have finished its work and exited therefore it has the
chance to report a false alarm.

This patch just removed those sanity check.

Signed-off-by: Jinshan Xiong <[email protected]>
Signed-off-by: Brian Behlendorf <[email protected]>
Closes #4516
  • Loading branch information
Jinshan Xiong authored and behlendorf committed Apr 13, 2016
1 parent e612379 commit edd770e
Show file tree
Hide file tree
Showing 3 changed files with 0 additions and 9 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -99,9 +99,6 @@ function replace_test
typeset pid=$!

$SLEEP 1
if ! $PS -p $pid > /dev/null 2>&1; then
log_fail "$FILE_TRUNC $options $TESTDIR/$TESTFILE.$i"
fi

child_pids="$child_pids $pid"
((i = i + 1))
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -99,9 +99,6 @@ function attach_test
typeset pid=$!

$SLEEP 1
if ! $PS -p $pid > /dev/null 2>&1; then
log_fail "$FILE_TRUNC $options $TESTDIR/$TESTFILE.$i"
fi

child_pids="$child_pids $pid"
((i = i + 1))
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -97,9 +97,6 @@ function detach_test
typeset pid=$!

$SLEEP 1
if ! $PS -p $pid > /dev/null 2>&1; then
log_fail "$FILE_TRUNC $options $TESTDIR/$TESTFILE.$i"
fi

child_pids="$child_pids $pid"
((i = i + 1))
Expand Down

0 comments on commit edd770e

Please sign in to comment.