diff mbox series

[1/1] tst_test.sh: Print Summary into stderr

Message ID 20220912100603.1551-1-pvorel@suse.cz
State Accepted
Headers show
Series [1/1] tst_test.sh: Print Summary into stderr | expand

Commit Message

Petr Vorel Sept. 12, 2022, 10:06 a.m. UTC
To follow the pattern in C API, where it was changed in
8f560e5dc ("lib: Print everything to stderr").

Signed-off-by: Petr Vorel <pvorel@suse.cz>
---
NOTE: applied on the top of patchset "v5 shell: df01.sh:
$TST_ALL_FILESYSTEMS (.all_filesystems)"

https://lore.kernel.org/ltp/20220909141840.18327-1-pvorel@suse.cz/
https://patchwork.ozlabs.org/project/ltp/list/?series=317639&state=*

Kind regards,
Petr

 testcases/lib/tst_test.sh | 16 +++++++++-------
 1 file changed, 9 insertions(+), 7 deletions(-)

Comments

Cyril Hrubis Sept. 15, 2022, 11:36 a.m. UTC | #1
Hi!
Reviewed-by: Cyril Hrubis <chrubis@suse.cz>
Petr Vorel Sept. 15, 2022, 11:58 a.m. UTC | #2
Hi Cyril,

thanks for a review, merged!

Kind regards,
Petr
diff mbox series

Patch

diff --git a/testcases/lib/tst_test.sh b/testcases/lib/tst_test.sh
index de4599625..84f29af5b 100644
--- a/testcases/lib/tst_test.sh
+++ b/testcases/lib/tst_test.sh
@@ -69,13 +69,15 @@  _tst_do_exit()
 		_tst_check_security_modules
 	fi
 
-	echo
-	echo "Summary:"
-	echo "passed   $TST_PASS"
-	echo "failed   $TST_FAIL"
-	echo "broken   $TST_BROK"
-	echo "skipped  $TST_CONF"
-	echo "warnings $TST_WARN"
+	cat >&2 << EOF
+
+Summary:
+passed   $TST_PASS
+failed   $TST_FAIL
+broken   $TST_BROK
+skipped  $TST_CONF
+warnings $TST_WARN
+EOF
 
 	exit $ret
 }