From 6e79ebeaa1703c890d7054e7c5f40aef53796dc2 Mon Sep 17 00:00:00 2001 From: Chris Sosa Date: Thu, 27 Jan 2011 11:18:25 -0800 Subject: [PATCH] Actually die if a parallel test fails. Change-Id: I24b498fffb7e7ce54517945730c91fecd1d57586 BUG=chromium-os:10723 TEST=Ran it and killed a test Review URL: http://codereview.chromium.org/6260023 --- bin/cros_au_test_harness.py | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/bin/cros_au_test_harness.py b/bin/cros_au_test_harness.py index b9578a28eb..47d51ad29e 100755 --- a/bin/cros_au_test_harness.py +++ b/bin/cros_au_test_harness.py @@ -912,8 +912,9 @@ def _RunTestsInParallel(parser, options, test_class): args.append(test_case) results = _RunParallelJobs(options.jobs, threads, args, print_status=False) - if not (test_result.wasSuccessful() for test_result in results): - Die('Test harness was not successful') + for test_result in results: + if not test_result.wasSuccessful(): + Die('Test harness was not successful') def main():