selftests/vsock: add check_result() for pass/fail counting
Add check_result() function to reuse logic for incrementing the pass/fail counters. This function will get used by different callers as we add different types of tests in future patches (namely, namespace and non-namespace tests will be called at different places, and re-use this function). Reviewed-by: Simon Horman <horms@kernel.org> Signed-off-by: Bobby Eshleman <bobbyeshleman@meta.com> Reviewed-by: Stefano Garzarella <sgarzare@redhat.com> Link: https://patch.msgid.link/20251108-vsock-selftests-fixes-and-improvements-v4-7-d5e8d6c87289@meta.com Signed-off-by: Jakub Kicinski <kuba@kernel.org>
This commit is contained in:
committed by
Jakub Kicinski
parent
9e2ad0bc36
commit
7fea50dff9
@@ -79,6 +79,26 @@ die() {
|
||||
exit "${KSFT_FAIL}"
|
||||
}
|
||||
|
||||
check_result() {
|
||||
local rc arg
|
||||
|
||||
rc=$1
|
||||
arg=$2
|
||||
|
||||
cnt_total=$(( cnt_total + 1 ))
|
||||
|
||||
if [[ ${rc} -eq ${KSFT_PASS} ]]; then
|
||||
cnt_pass=$(( cnt_pass + 1 ))
|
||||
echo "ok ${cnt_total} ${arg}"
|
||||
elif [[ ${rc} -eq ${KSFT_SKIP} ]]; then
|
||||
cnt_skip=$(( cnt_skip + 1 ))
|
||||
echo "ok ${cnt_total} ${arg} # SKIP"
|
||||
elif [[ ${rc} -eq ${KSFT_FAIL} ]]; then
|
||||
cnt_fail=$(( cnt_fail + 1 ))
|
||||
echo "not ok ${cnt_total} ${arg} # exit=${rc}"
|
||||
fi
|
||||
}
|
||||
|
||||
vm_ssh() {
|
||||
ssh -q -o UserKnownHostsFile=/dev/null -p ${SSH_HOST_PORT} localhost "$@"
|
||||
return $?
|
||||
@@ -530,17 +550,7 @@ cnt_total=0
|
||||
for arg in "${ARGS[@]}"; do
|
||||
run_test "${arg}"
|
||||
rc=$?
|
||||
if [[ ${rc} -eq $KSFT_PASS ]]; then
|
||||
cnt_pass=$(( cnt_pass + 1 ))
|
||||
echo "ok ${cnt_total} ${arg}"
|
||||
elif [[ ${rc} -eq $KSFT_SKIP ]]; then
|
||||
cnt_skip=$(( cnt_skip + 1 ))
|
||||
echo "ok ${cnt_total} ${arg} # SKIP"
|
||||
elif [[ ${rc} -eq $KSFT_FAIL ]]; then
|
||||
cnt_fail=$(( cnt_fail + 1 ))
|
||||
echo "not ok ${cnt_total} ${arg} # exit=$rc"
|
||||
fi
|
||||
cnt_total=$(( cnt_total + 1 ))
|
||||
check_result "${rc}" "${arg}"
|
||||
done
|
||||
|
||||
terminate_pidfiles "${pidfile}"
|
||||
|
||||
Reference in New Issue
Block a user