Re: [PATCH v2] test: Display count of skipped tests in status and summary

[Laurent Vivier <lvivier@redhat.com>]

Sorry this version is not correct (forgot the commit amend...)

Correct v2 follows.

Laurent

On 14/05/2025 09:16, Laurent Vivier wrote:
> This commit enhances test reporting by tracking and displaying the
> number of skipped tests.
> 
> The skipped test count is now visible in the tmux status bar during
> execution and included in the final test summary log. This provides
> a more complete overview of test suite results.
> 
> Signed-off-by: Laurent Vivier <lvivier@redhat.com>
> ---
> 
> Notes:
>      v2: add missing comma before SKIPPED in report log
> 
>   test/lib/term | 7 +++++--
>   test/run      | 4 ++--
>   2 files changed, 7 insertions(+), 4 deletions(-)
> 
> diff --git a/test/lib/term b/test/lib/term
> index ed690de82c20..089364c67f85 100755
> --- a/test/lib/term
> +++ b/test/lib/term
> @@ -19,6 +19,7 @@ STATUS_FILE_INDEX=0
>   STATUS_COLS=
>   STATUS_PASS=0
>   STATUS_FAIL=0
> +STATUS_SKIPPED=0
>   
>   PR_RED='\033[1;31m'
>   PR_GREEN='\033[1;32m'
> @@ -439,19 +440,21 @@ info_layout() {
>   # status_test_ok() - Update counter of passed tests, log and display message
>   status_test_ok() {
>   	STATUS_PASS=$((STATUS_PASS + 1))
> -	tmux set status-right "PASS: ${STATUS_PASS} | FAIL: ${STATUS_FAIL} | #(TZ="UTC" date -Iseconds)"
> +	tmux set status-right "PASS: ${STATUS_PASS} | FAIL: ${STATUS_FAIL} | SKIPPED: ${STATUS_SKIPPED} | #(TZ="UTC" date -Iseconds)"
>   	info_passed
>   }
>   
>   # status_test_fail() - Update counter of failed tests, log and display message
>   status_test_fail() {
>   	STATUS_FAIL=$((STATUS_FAIL + 1))
> -	tmux set status-right "PASS: ${STATUS_PASS} | FAIL: ${STATUS_FAIL} | #(TZ="UTC" date -Iseconds)"
> +	tmux set status-right "PASS: ${STATUS_PASS} | FAIL: ${STATUS_FAIL} | SKIPPED: ${STATUS_SKIPPED} | #(TZ="UTC" date -Iseconds)"
>   	info_failed
>   }
>   
>   # status_test_fail() - Update counter of failed tests, log and display message
>   status_test_skip() {
> +	STATUS_SKIPPED=$((STATUS_SKIPPED + 1))
> +	tmux set status-right "PASS: ${STATUS_PASS} | FAIL: ${STATUS_FAIL} | SKIPPED: ${STATUS_SKIPPED} | #(TZ="UTC" date -Iseconds)"
>   	info_skipped
>   }
>   
> diff --git a/test/run b/test/run
> index 4e86f30fb750..a6070d28e8c4 100755
> --- a/test/run
> +++ b/test/run
> @@ -202,7 +202,7 @@ skip_distro() {
>   	perf_finish
>   	[ ${CI} -eq 1 ] && video_stop
>   
> -	log "PASS: ${STATUS_PASS}, FAIL: ${STATUS_FAIL}"
> +	log "PASS: ${STATUS_PASS}, FAIL: ${STATUS_FAIL} SKIPPED: ${STATUS_SKIPPED}"
>   
>   	pause_continue \
>   		"Press any key to keep test session open"	\
> @@ -236,7 +236,7 @@ run_selected() {
>   	done
>   	teardown "${__setup}"
>   
> -	log "PASS: ${STATUS_PASS}, FAIL: ${STATUS_FAIL}"
> +	log "PASS: ${STATUS_PASS}, FAIL: ${STATUS_FAIL} SKIPPED: ${STATUS_SKIPPED}"
>   
>   	pause_continue \
>   		"Press any key to keep test session open"	\