The test runner was not printing the 'pretty index'
and repeating the test names. Example:

(29/48) raw.smp2.WinXP.32.sp3.install: FAIL (0.82 s)
raw.smp2.WinXP.64.shutdown: SKIP
raw.smp2.WinXP.64.shutdown: SKIP
raw.smp2.WinXP.64.shutdown: SKIP
raw.smp2.WinXP.64.shutdown: SKIP
raw.smp2.WinXP.64.shutdown: SKIP
raw.smp2.WinXP.64.shutdown: SKIP
raw.smp2.WinXP.64.shutdown: SKIP
raw.smp2.WinXP.64.shutdown: SKIP
raw.smp2.WinXP.64.shutdown: SKIP

Fix this by moving initialization of the test object
before the 'dependencies satisfied' branch.

Signed-off-by: Lucas Meneghel Rodrigues <[email protected]>
---
 virttest/standalone_test.py | 12 +++++-------
 1 file changed, 5 insertions(+), 7 deletions(-)

diff --git a/virttest/standalone_test.py b/virttest/standalone_test.py
index dbc6889..e873555 100644
--- a/virttest/standalone_test.py
+++ b/virttest/standalone_test.py
@@ -786,13 +786,13 @@ def run_tests(parser, options):
                     break
 
         current_status = False
-        if dependencies_satisfied:
-            t = Test(dct, options)
-            t.set_debugdir(debugdir)
 
-            pretty_index = "(%d/%d)" % (index, n_tests)
-            print_stdout("%s %s:" % (pretty_index, t.tag), end=False)
+        pretty_index = "(%d/%d)" % (index, n_tests)
+        t = Test(dct, options)
+        print_stdout("%s %s:" % (pretty_index, t.tag), end=False)
 
+        if dependencies_satisfied:
+            t.set_debugdir(debugdir)
             try:
                 try:
                     t_begin = time.time()
@@ -848,8 +848,6 @@ def run_tests(parser, options):
                 t.stop_file_logging()
                 current_status = False
         else:
-            shortname = get_tag(d, tag_index)
-            print_stdout("%s:" % shortname, end=False)
             print_skip()
             status_dct[dct.get("name")] = False
             continue
-- 
1.8.1.2

_______________________________________________
Virt-test-devel mailing list
[email protected]
https://www.redhat.com/mailman/listinfo/virt-test-devel

Reply via email to