[Android] Move base tests classes to pylib/base

build/android/pylib/base/base_test_sharder.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 android_commands
 import logging
 import multiprocessing
 
-from android_commands import errors
-from forwarder import Forwarder
-from test_result import TestResults
+from pylib import android_commands
+from pylib.android_commands import errors

[craigdh, 2013/01/15 23:00:05]

Just change the 2 uses of this to android_commands.errors and get rid of this
import.

[frankf, 2013/01/16 21:34:04]

Done.

+from pylib.base.test_result import TestResults
+from pylib.forwarder import Forwarder

[craigdh, 2013/01/15 23:00:05]

Should forwarder be in utils?

[bulach, 2013/01/16 09:45:50]

if you do change this, please remember that "forwarder" is also used by
telemetry (tools/telemetry/telemetry/adb_commands.py), and iirc, webkit..

[frankf, 2013/01/16 21:34:04]

I rather do this in a separate CL.

On 2013/01/16 09:45:50, bulach wrote:

 
 
 def _ShardedTestRunnable(test):
   """Standalone function needed by multiprocessing.Pool."""
   log_format = '[' + test.device + '] # %(asctime)-15s: %(message)s'
   if logging.getLogger().handlers:
     logging.getLogger().handlers[0].setFormatter(logging.Formatter(log_format))
   else:
     logging.basicConfig(format=log_format)
   # Handle SystemExit here since python has a bug to exit current process
@@ skipping 121 matching lines @@
           self.tests += [t.name]
         if not self.tests:
           break
     else:
       # We ran out retries, possibly out of healthy devices.
       # There's no recovery at this point.
       raise Exception('Unrecoverable error while retrying test runs.')
     self.OnTestsCompleted(test_runners, final_results)
     self._KillHostForwarder()
     return final_results