On 4/15/24 8:32 PM, Nícolas F. R. A. Prado wrote:
> There is one use of bash specific syntax in the script. Change it to the
> equivalent POSIX syntax. This doesn't change functionality and allows
> the test to be run on shells other than bash.
>
> Reported-by: Mike Looijmans <mike.looijm...@topic.nl>
> Closes:
> https://lore.kernel.org/all/efae4037-c22a-40be-8ba9-7c1c12ece...@topic.nl/
> Fixes: 4a679c5afca0 ("selftests: Add test to verify power supply properties")
> Signed-off-by: Nícolas F. R. A. Prado <nfrapr...@collabora.com>
Reviewed-by: Muhammad Usama Anjum <usama.an...@collabora.com>
> ---
> tools/testing/selftests/power_supply/test_power_supply_properties.sh | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git
> a/tools/testing/selftests/power_supply/test_power_supply_properties.sh
> b/tools/testing/selftests/power_supply/test_power_supply_properties.sh
> index df272dfe1d2a..a66b1313ed88 100755
> --- a/tools/testing/selftests/power_supply/test_power_supply_properties.sh
> +++ b/tools/testing/selftests/power_supply/test_power_supply_properties.sh
> @@ -23,7 +23,7 @@ count_tests() {
> total_tests=0
>
> for i in $SUPPLIES; do
> - total_tests=$(("$total_tests" + "$NUM_TESTS"))
> + total_tests=$((total_tests + NUM_TESTS))
> done
>
> echo "$total_tests"
>
--
BR,
Muhammad Usama Anjum
