commit 1f271dc47847c265175d5a9a056fd07c31decad2
author Simon Glass <sjg@chromium.org> Mon Jan 20 14:26:01 2025 -0700
committer Tom Rini <trini@konsulko.com> Fri Jan 24 14:34:41 2025 -0600
tree 2c050dc91a50176da754fabc9580e92436056f7a
parent 4f621e577b573b54de20b1d6a8f9e6fc8a475581

test: Move stat-printing into its own function

Add a function to show the stats, so we can decide when to print it.

This slightly adjusts the output, so that any 'test not found' message
appears on its own line after all other output.

The 'failures' message now appears in lower case so update pytest
accordingly.

Signed-off-by: Simon Glass <sjg@chromium.org>

test/py/tests/test_spl.py

diff --git a/test/py/tests/test_spl.py b/test/py/tests/test_spl.py
index 42e4c43..474f430 100644
--- a/test/py/tests/test_spl.py
+++ b/test/py/tests/test_spl.py
@@ -36,7 +36,7 @@
         cons = u_boot_console
         cons.restart_uboot_with_flags(['-u', '-k', ut_spl_subtest.split()[1]])
         output = cons.get_spawn_output().replace('\r', '')
-        assert 'Failures: 0' in output
+        assert 'failures: 0' in output
     finally:
         # Restart afterward in case a non-SPL test is run next. This should not
         # happen since SPL tests are run in their own invocation of test.py, but

test/py/tests/test_upl.py

diff --git a/test/py/tests/test_upl.py b/test/py/tests/test_upl.py
index 3164bda..d94359d 100644
--- a/test/py/tests/test_upl.py
+++ b/test/py/tests/test_upl.py
@@ -35,4 +35,4 @@
 
     # Check the FIT offsets look correct
     output = cons.run_command('ut upl -f upl_test_info_norun')
-    assert 'Failures: 0' in output
+    assert 'failures: 0' in output

test/py/tests/test_ut.py

diff --git a/test/py/tests/test_ut.py b/test/py/tests/test_ut.py
index cacf11f..d2d8ce1 100644
--- a/test/py/tests/test_ut.py
+++ b/test/py/tests/test_ut.py
@@ -607,4 +607,4 @@
         assert 'Unknown command \'quux\' - try \'help\'' in output
     else:
         output = u_boot_console.run_command('ut ' + ut_subtest)
-    assert output.endswith('Failures: 0')
+    assert output.endswith('failures: 0')

test/py/tests/test_vbe.py

diff --git a/test/py/tests/test_vbe.py b/test/py/tests/test_vbe.py
index 50b6c1c..861df3f 100644
--- a/test/py/tests/test_vbe.py
+++ b/test/py/tests/test_vbe.py
@@ -117,4 +117,4 @@
     with cons.log.section('Kernel load'):
         output = cons.run_command_list(cmd.splitlines())
 
-    assert 'Failures: 0' in output[-1]
+    assert 'failures: 0' in output[-1]

test/py/tests/test_vpl.py

diff --git a/test/py/tests/test_vpl.py b/test/py/tests/test_vpl.py
index 4af578b..8c472ca 100644
--- a/test/py/tests/test_vpl.py
+++ b/test/py/tests/test_vpl.py
@@ -26,7 +26,7 @@
         cons = u_boot_console
         cons.restart_uboot_with_flags(['-u', '-k', ut_vpl_subtest.split()[1]])
         output = cons.get_spawn_output().replace('\r', '')
-        assert 'Failures: 0' in output
+        assert 'failures: 0' in output
     finally:
         # Restart afterward in case a non-VPL test is run next. This should not
         # happen since VPL tests are run in their own invocation of test.py, but