[Lldb-commits] [lldb] r238771 - Dump error if dotest itself fails and no tests were run.

[Chaoren Lin]

Author: chaoren
Date: Mon Jun  1 14:06:01 2015
New Revision: 238771

URL: http://llvm.org/viewvc/llvm-project?rev=238771&view=rev
Log:
Dump error if dotest itself fails and no tests were run.

Reviewers: zturner, clayborg

Subscribers: lldb-commits

Differential Revision: http://reviews.llvm.org/D10163

Modified:
    lldb/trunk/test/dosep.py

Modified: lldb/trunk/test/dosep.py
URL: http://llvm.org/viewvc/llvm-project/lldb/trunk/test/dosep.py?rev=238771&r1=238770&r2=238771&view=diff
==============================================================================
--- lldb/trunk/test/dosep.py (original)
+++ lldb/trunk/test/dosep.py Mon Jun  1 14:06:01 2015
@@ -125,7 +125,7 @@ def call_with_timeout(command, timeout,
     output = process.communicate()
     exit_status = process.returncode
     passes, failures = parse_test_results(output)
-    update_status(name, output if failures > 0 else None)
+    update_status(name, output if exit_status != 0 else None)
     return exit_status, passes, failures
 
 def process_dir(root, files, test_root, dotest_argv):
@@ -388,8 +388,10 @@ Run lldb test suite using a separate pro
             touch(os.path.join(session_dir, "{}-{}".format(result, test_name)))
 
     print
-    print "Ran %d test suites (%d failed) (%f%%)" % (num_test_files, len(failed), 100.0*len(failed)/num_test_files)
-    print "Ran %d test cases (%d failed) (%f%%)" % (num_tests, all_fails, 100.0*all_fails/num_tests)
+    print "Ran %d test suites (%d failed) (%f%%)" % (num_test_files, len(failed),
+            (100.0 * len(failed) / num_test_files) if num_test_files > 0 else float('NaN'))
+    print "Ran %d test cases (%d failed) (%f%%)" % (num_tests, all_fails,
+            (100.0 * all_fails / num_tests) if num_tests > 0 else float('NaN'))
     if len(failed) > 0:
         failed.sort()
         print "Failing Tests (%d)" % len(failed)