[LTP] [PATCH v3] tst_test.sh: achieve TST_RETRY_FUNC function in shell
Li Wang
liwang@redhat.com
Thu May 17 08:29:19 CEST 2018
The commit c2ce4df67d(include: add an exponential backoff macro for
function retry) involves a new MACRO for function retry in C code,
here achieve it in shell lib and gives a introduction in LTP documents.
Signed-off-by: Li Wang <liwang@redhat.com>
Tested-by: Petr Vorel <pvorel@suse.cz>
---
doc/test-writing-guidelines.txt | 26 +++++++++++++++++++++++++
testcases/lib/tst_test.sh | 43 +++++++++++++++++++++++++++++++++++++++++
2 files changed, 69 insertions(+)
diff --git a/doc/test-writing-guidelines.txt b/doc/test-writing-guidelines.txt
index cbbfe6c..b2dd091 100644
--- a/doc/test-writing-guidelines.txt
+++ b/doc/test-writing-guidelines.txt
@@ -1640,6 +1640,32 @@ that can sleep for defined amount of seconds, milliseconds or microseconds.
tst_sleep 100ms
-------------------------------------------------------------------------------
+Retry a function in limited time
+++++++++++++++++++++++++++++++++
+
+Sometimes LTP test needs retrying a function for many times to get success.
+This achievement makes that possible via keeping it retrying if the return
+value of the function is NOT as we expected. After exceeding a limited time,
+test will break from the retries immediately.
+
+[source,c]
+-------------------------------------------------------------------------------
+# retry function in 1 second
+TST_RETRY_FUNC(FUNC, EXPECTED_RET)
+
+# retry function in N second
+TST_RETRY_FN_EXP_BACKOFF(FUNC, EXPECTED_RET, N)
+-------------------------------------------------------------------------------
+
+[source,sh]
+-------------------------------------------------------------------------------
+# retry function in 1 second
+TST_RETRY_FUNC "FUNC arg1 arg2 ..." "EXPECTED_RET"
+
+# retry function in N second
+TST_RETRY_FN_EXP_BACKOFF "FUNC arg1 arg2 ..." "EXPECTED_RET" "N"
+-------------------------------------------------------------------------------
+
Checking for integers
+++++++++++++++++++++
diff --git a/testcases/lib/tst_test.sh b/testcases/lib/tst_test.sh
index 8d49d34..262b9c2 100644
--- a/testcases/lib/tst_test.sh
+++ b/testcases/lib/tst_test.sh
@@ -154,6 +154,48 @@ EXPECT_FAIL()
fi
}
+TST_RETRY_FN_EXP_BACKOFF()
+{
+ local tst_fun=$1
+ local tst_exp=$2
+ local tst_sec=$(expr $3 \* 1000000)
+ local tst_delay=1
+
+ if [ $# -ne 3 ]; then
+ tst_brk TBROK "TST_RETRY_FN_EXP_BACKOFF expects 3 parameters"
+ fi
+
+ if ! tst_is_int "$tst_sec"; then
+ tst_brk TBROK "TST_RETRY_FN_EXP_BACKOFF: tst_sec must be integer ('$tst_sec')"
+ fi
+
+ while true; do
+ $tst_fun
+ if [ "$?" = "$tst_exp" ]; then
+ break
+ fi
+
+ if [ $tst_delay -lt $tst_sec ]; then
+ tst_sleep ${tst_delay}us
+ tst_delay=$((tst_delay*2))
+ else
+ tst_brk TBROK "\"$tst_fun\" timed out"
+ fi
+ done
+
+ return $tst_exp
+}
+
+TST_RETRY_FUNC()
+{
+ if [ $# -ne 2 ]; then
+ tst_brk TBROK "TST_RETRY_FN_EXP_BACKOFF expects 2 parameters"
+ fi
+
+ TST_RETRY_FN_EXP_BACKOFF "$1" "$2" 1
+ return $2
+}
+
tst_umount()
{
local device="$1"
@@ -255,6 +297,7 @@ tst_run()
OPTS|USAGE|PARSE_ARGS|POS_ARGS);;
NEEDS_ROOT|NEEDS_TMPDIR|NEEDS_DEVICE|DEVICE);;
NEEDS_CMDS|NEEDS_MODULE|MODPATH|DATAROOT);;
+ RETRY_FUNC|RETRY_FN_EXP_BACKOFF);;
IPV6);;
*) tst_res TWARN "Reserved variable TST_$tst_i used!";;
esac
--
2.9.5
More information about the ltp
mailing list