On 4/15/24 8:32 PM, Nícolas F. R. A. Prado wrote:
> There are a couple uses of bash specific syntax in the script. Change
> them to the equivalent POSIX syntax. This doesn't change functionality
> and allows non-bash test scripts to make use of these helpers.
>
> Reported-by: Mike Looijmans <[email protected]>
> Closes:
> https://lore.kernel.org/all/[email protected]/
> Fixes: 2dd0b5a8fcc4 ("selftests: ktap_helpers: Add a helper to finish the
> test")
> Fixes: 14571ab1ad21 ("kselftest: Add new test for detecting unprobed
> Devicetree devices")
> Signed-off-by: Nícolas F. R. A. Prado <[email protected]>
Reviewed-by: Muhammad Usama Anjum <[email protected]>
> ---
> tools/testing/selftests/kselftest/ktap_helpers.sh | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/tools/testing/selftests/kselftest/ktap_helpers.sh
> b/tools/testing/selftests/kselftest/ktap_helpers.sh
> index f2fbb914e058..79a125eb24c2 100644
> --- a/tools/testing/selftests/kselftest/ktap_helpers.sh
> +++ b/tools/testing/selftests/kselftest/ktap_helpers.sh
> @@ -43,7 +43,7 @@ __ktap_test() {
> directive="$3" # optional
>
> local directive_str=
> - [[ ! -z "$directive" ]] && directive_str="# $directive"
> + [ ! -z "$directive" ] && directive_str="# $directive"
>
> echo $result $KTAP_TESTNO $description $directive_str
>
> @@ -99,7 +99,7 @@ ktap_exit_fail_msg() {
> ktap_finished() {
> ktap_print_totals
>
> - if [ $(("$KTAP_CNT_PASS" + "$KTAP_CNT_SKIP")) -eq "$KSFT_NUM_TESTS" ];
> then
> + if [ $((KTAP_CNT_PASS + KTAP_CNT_SKIP)) -eq "$KSFT_NUM_TESTS" ]; then
> exit "$KSFT_PASS"
> else
> exit "$KSFT_FAIL"
>
--
BR,
Muhammad Usama Anjum
