Retry tests on other bots if the device is unresponsive/offline

build/android/pylib/test_result.py

 # Copyright (c) 2012 The Chromium Authors. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
 
 import json
 import logging
 import os
 import time
 import traceback
@@ skipping 36 matching lines @@
 class TestResults(object):
   """Results of a test run."""
 
   def __init__(self):
     self.ok = []
     self.failed = []
     self.crashed = []
     self.unknown = []
     self.timed_out = False
     self.overall_fail = False
+    self.device_except = False

[bulach, 2012/10/23 09:30:56]

if you agree with the above:

self.device_exception = None

 
   @staticmethod
   def FromRun(ok=None, failed=None, crashed=None, timed_out=False,
-              overall_fail=False):
+              overall_fail=False, device_except=False):

[bulach, 2012/10/23 09:30:56]

nit: device_exception=None

     ret = TestResults()
     ret.ok = ok or []
     ret.failed = failed or []
     ret.crashed = crashed or []
     ret.timed_out = timed_out
     ret.overall_fail = overall_fail
+    ret.device_except = device_except
     return ret
 
   @staticmethod
   def FromTestResults(results):
     """Combines a list of results in a single TestResults object."""
     ret = TestResults()
     for t in results:
       ret.ok += t.ok
       ret.failed += t.failed
       ret.crashed += t.crashed
       ret.unknown += t.unknown
       if t.timed_out:
         ret.timed_out = True
       if t.overall_fail:
         ret.overall_fail = True
     return ret
 
   @staticmethod
+  def HasDeviceExcept(results):
+    for t in results:
+      if t.device_except:
+          return True
+
+    return False

[bulach, 2012/10/23 09:30:56]

I think this could be:

return any(filter(lambda t: t.device_exception, results))

[yongsheng, 2012/10/23 12:58:14]

good.

+
+  @staticmethod
   def FromPythonException(test_name, start_date_ms, exc_info):
     """Constructs a TestResults with exception information for the given test.
 
     Args:
       test_name: name of the test which raised an exception.
       start_date_ms: the starting time for the test.
       exc_info: exception info, ostensibly from sys.exc_info().
 
     Returns:
       A TestResults object with a SingleTestResult in the failed list.
@@ skipping 89 matching lines @@
     return summary_string
 
   def PrintAnnotation(self):
     """Print buildbot annotations for test results."""
     if self.timed_out:
       buildbot_report.PrintWarning()
     elif self.failed or self.crashed or self.overall_fail:
       buildbot_report.PrintError()
     else:
       print 'Step success!'  # No annotation needed