commit 7d49f869de664a5171d611a8a0b71edcb74c7fda
author Po-Hsien Wang <pwang@chromium.org> Tue Jul 17 10:33:10 2018 -0700
committer android-build-merger <android-build-merger@google.com> Tue Jul 17 10:33:10 2018 -0700
tree 6f4953bac29882a45189f083ead7a1766952a44e
parent c64fa041dda89a0e2d412eae57773e226f46ebce
parent fedc65750e40cc16cb4b402e5c56231e7ef78307

GraphicsTest: Fix missing subtest entries.
am: fedc65750e

Change-Id: Ia3cd8474b697add9e641a9e01a432638d6e16a44

client/cros/graphics/graphics_utils.py

diff --git a/client/cros/graphics/graphics_utils.py b/client/cros/graphics/graphics_utils.py
index 35611f3..2a14e55 100644
--- a/client/cros/graphics/graphics_utils.py
+++ b/client/cros/graphics/graphics_utils.py
@@ -199,8 +199,6 @@
         target = self._get_failure(name, subtest=subtest)
         if name in target['names']:
             target['names'].remove(name)
-            if len(target['names']) == 0:
-                self._failures.remove(target)
 
 
     def _output_perf(self):
@@ -219,8 +217,9 @@
         total_failures = 0
         # Report subtests failures
         for failure in self._failures:
-            logging.debug('GraphicsTest failure: %s' % failure['names'])
-            total_failures += len(failure['names'])
+            if len(failure['names']) > 0:
+                logging.debug('GraphicsTest failure: %s' % failure['names'])
+                total_failures += len(failure['names'])
 
             if not self._test_failure_report_subtest:
                 continue