[PATCH] utils: test: show test name before running it

[PATCH] utils: test: show test name before running it

[Hannes Diethelm]

This way, the running test is visible. This helps for longer running or
freezed tests.

The log is now shown after the test name.

Signed-off-by: Hannes Diethelm <hannes.diethelm@gmail.com>
---
 utils/evl-test | 13 +++++++------
 1 file changed, 7 insertions(+), 6 deletions(-)

diff --git a/utils/evl-test b/utils/evl-test
index fa3ef73..5139d09 100644
--- a/utils/evl-test
+++ b/utils/evl-test
@@ -83,22 +83,23 @@ result=0
 while : ; do
       for t in $test_list; do
 	  test \! -x $t && echo "$(basename $0): $(basename $t): no such test" && exit 2
+	  echo -n "$(basename $t):"
 	  # Swap stdout<->stderr for the test, capturing stderr into $log.
 	  log=$($t $test_arg_list 3>&1 1>&2 2>&3)
 	  status=$?
 	  if test $status = 43; then
-	      echo "$(basename $t): $log"
+	      echo " $log"
 	  else
-	      test -n "$log" && echo "$log"
 	      if test $status = 0; then
-		  echo "$(basename $t): OK"
+		  echo " OK"
 	      elif test $status = 42; then
-		  echo "$(basename $t): no kernel support"
+		  echo " no kernel support"
 	      else
-		  echo "** $(basename $t): BROKEN"
+		  echo " ** BROKEN"
 		  result=$status
-		  test x$keep_going = xfalse && break
 	      fi
+	      test -n "$log" && echo "$log"
+	      test $result -ne 0 && test x$keep_going = xfalse && break
 	  fi
       done
       test x$repeat = xfalse && break
-- 
2.47.3

Re: [PATCH] utils: test: show test name before running it

[Philippe Gerum]

Hannes Diethelm <hannes.diethelm@gmail.com> writes:

> This way, the running test is visible. This helps for longer running or
> freezed tests.
>
> The log is now shown after the test name.
>
> Signed-off-by: Hannes Diethelm <hannes.diethelm@gmail.com>
> ---
>  utils/evl-test | 13 +++++++------
>  1 file changed, 7 insertions(+), 6 deletions(-)
>
> diff --git a/utils/evl-test b/utils/evl-test
> index fa3ef73..5139d09 100644
> --- a/utils/evl-test
> +++ b/utils/evl-test
> @@ -83,22 +83,23 @@ result=0
>  while : ; do
>        for t in $test_list; do
>  	  test \! -x $t && echo "$(basename $0): $(basename $t): no such test" && exit 2
> +	  echo -n "$(basename $t):"
>  	  # Swap stdout<->stderr for the test, capturing stderr into $log.
>  	  log=$($t $test_arg_list 3>&1 1>&2 2>&3)
>  	  status=$?
>  	  if test $status = 43; then
> -	      echo "$(basename $t): $log"
> +	      echo " $log"
>  	  else
> -	      test -n "$log" && echo "$log"
>  	      if test $status = 0; then
> -		  echo "$(basename $t): OK"
> +		  echo " OK"
>  	      elif test $status = 42; then
> -		  echo "$(basename $t): no kernel support"
> +		  echo " no kernel support"
>  	      else
> -		  echo "** $(basename $t): BROKEN"
> +		  echo " ** BROKEN"
>  		  result=$status
> -		  test x$keep_going = xfalse && break
>  	      fi
> +	      test -n "$log" && echo "$log"
> +	      test $result -ne 0 && test x$keep_going = xfalse && break
>  	  fi
>        done
>        test x$repeat = xfalse && break

Merged, thanks.

-- 
Philippe.