Install support APK together with test APK

build/android/pylib/instrumentation/test_package.py

 # Copyright (c) 2013 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.
 
 """Class representing instrumentation test apk and jar."""
 
 import os
 
 from pylib.instrumentation import test_jar
 from pylib.utils import apk_helper
 
 
 class TestPackage(test_jar.TestJar):
   def __init__(self, apk_path, jar_path):
     test_jar.TestJar.__init__(self, jar_path)
 
     if not os.path.exists(apk_path):
       raise Exception('%s not found, please build it' % apk_path)
     self._apk_path = apk_path
     self._apk_name = os.path.splitext(os.path.basename(apk_path))[0]
     self._package_name = apk_helper.GetPackageName(self._apk_path)
+    self._support_apk_name = "%sSupport%s" % (

[jbudorick, 2014/06/17 20:17:08]

I don't think hardcoding this is a good idea. Can we specify the support apk
name in another way?

[aberent, 2014/06/17 20:39:42]

We could add it as an extra instrumentation option to test_runner.py , but I
don't like this because it makes the command more complicated and hence easier
to get wrong (particularly since only some groups of instrumentation tests need
this). I am not sure what other options there are.

+        os.path.splitext(self._apk_path)[0],
+        os.path.splitext(self._apk_path)[1])
 
   def GetApkPath(self):
     """Returns the absolute path to the APK."""
     return self._apk_path
 
   def GetApkName(self):
     """Returns the name of the apk without the suffix."""
     return self._apk_name
 
   def GetPackageName(self):
     """Returns the package name of this APK."""
     return self._package_name
 
   # Override.
   def Install(self, device):
     device.Install(self.GetApkPath())
+    if os.path.exists(self._support_apk_name):
+      device.Install(self._support_apk_name)