Android Add _incremental targets for instrumentation tests

build/android/pylib/instrumentation/instrumentation_test_instance.py

 # Copyright 2015 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 collections
 import copy
 import logging
 import os
 import pickle
 import re
@@ skipping 177 matching lines @@
   return result if result else None
 
 
 class InstrumentationTestInstance(test_instance.TestInstance):
 
   def __init__(self, args, isolate_delegate, error_func):
     super(InstrumentationTestInstance, self).__init__()
 
     self._additional_apks = []
     self._apk_under_test = None
-    self._apk_under_test_permissions = None
+    self._apk_under_test_install_script = None
     self._package_info = None
     self._suite = None
     self._test_apk = None
+    self._test_apk_install_script = None
     self._test_jar = None
     self._test_package = None
-    self._test_permissions = None
     self._test_runner = None
     self._test_support_apk = None
     self._initializeApkAttributes(args, error_func)
 
     self._data_deps = None
     self._isolate_abs_path = None
     self._isolate_delegate = None
     self._isolated_abs_path = None
     self._test_data = None
     self._initializeDataDependencyAttributes(args, isolate_delegate)
 
     self._annotations = None
     self._excluded_annotations = None
     self._test_filter = None
     self._initializeTestFilterAttributes(args)
 
     self._flags = None
     self._initializeFlagAttributes(args)
 
     self._driver_apk = None
     self._driver_package = None
     self._driver_name = None
     self._initializeDriverAttributes()
 
     self._timeout_scale = None
     self._initializeTestControlAttributes(args)
 
   def _initializeApkAttributes(self, args, error_func):
-    if args.apk_under_test.endswith('.apk'):
-      self._apk_under_test = args.apk_under_test
-    else:
-      self._apk_under_test = os.path.join(
+    apk_under_test_path = args.apk_under_test
+    if not args.apk_under_test.endswith('.apk'):
+      apk_under_test_path = os.path.join(
           constants.GetOutDirectory(), constants.SDK_BUILD_APKS_DIR,
           '%s.apk' % args.apk_under_test)
 
-    if not os.path.exists(self._apk_under_test):
-      error_func('Unable to find APK under test: %s' % self._apk_under_test)
+    if not os.path.exists(apk_under_test_path):
+      error_func('Unable to find APK under test: %s' % apk_under_test_path)
 
-    apk = apk_helper.ApkHelper(self._apk_under_test)
-    self._apk_under_test_permissions = apk.GetPermissions()
+    self._apk_under_test = apk_helper.ToHelper(apk_under_test_path)
 
     if args.test_apk.endswith('.apk'):
       self._suite = os.path.splitext(os.path.basename(args.test_apk))[0]
-      self._test_apk = args.test_apk
+      self._test_apk = apk_helper.ToHelper(args.test_apk)
     else:
       self._suite = args.test_apk
-      self._test_apk = os.path.join(
+      self._test_apk = apk_helper.ToHelper(os.path.join(
           constants.GetOutDirectory(), constants.SDK_BUILD_APKS_DIR,
-          '%s.apk' % args.test_apk)
+          '%s.apk' % args.test_apk))
+
+    self._apk_under_test_install_script = args.apk_under_test_install_script
+    self._test_apk_install_script = args.test_apk_install_script
+
+    if self._test_apk_install_script:
+      assert self._suite.endswith('_incremental')
+      self._suite = self._suite[:-12]

[jbudorick, 2016/02/09 20:08:26]

nit: self._suite[:-len('_incremental')]

[agrieve, 2016/02/09 21:43:50]

Done.

 
     self._test_jar = os.path.join(
         constants.GetOutDirectory(), constants.SDK_BUILD_TEST_JAVALIB_DIR,
         '%s.jar' % self._suite)
-    self._test_support_apk = os.path.join(
+    self._test_support_apk = apk_helper.ToHelper(os.path.join(
         constants.GetOutDirectory(), constants.SDK_BUILD_TEST_JAVALIB_DIR,
-        '%sSupport.apk' % self._suite)
+        '%sSupport.apk' % self._suite))
 
-    if not os.path.exists(self._test_apk):
-      error_func('Unable to find test APK: %s' % self._test_apk)
+    if not os.path.exists(self._test_apk.path):
+      error_func('Unable to find test APK: %s' % self._test_apk.path)
     if not os.path.exists(self._test_jar):
       error_func('Unable to find test JAR: %s' % self._test_jar)
 
-    apk = apk_helper.ApkHelper(self.test_apk)
-    self._test_package = apk.GetPackageName()
-    self._test_permissions = apk.GetPermissions()
-    self._test_runner = apk.GetInstrumentationName()
+    self._test_package = self._test_apk.GetPackageName()
+    self._test_runner = self._test_apk.GetInstrumentationName()
 
     self._package_info = None
     for package_info in constants.PACKAGE_INFO.itervalues():
       if self._test_package == package_info.test_package:
         self._package_info = package_info
     if not self._package_info:
       logging.warning('Unable to find package info for %s', self._test_package)
 
     for apk in args.additional_apks:
       if not os.path.exists(apk):
         error_func('Unable to find additional APK: %s' % apk)
-    self._additional_apks = args.additional_apks
+    self._additional_apks = (
+        [apk_helper.ToHelper(x) for x in args.additional_apks])
 
   def _initializeDataDependencyAttributes(self, args, isolate_delegate):
     self._data_deps = []
     if args.isolate_file_path:
       self._isolate_abs_path = os.path.abspath(args.isolate_file_path)
       self._isolate_delegate = isolate_delegate
       self._isolated_abs_path = os.path.join(
           constants.GetOutDirectory(), '%s.isolated' % self._test_package)
     else:
       self._isolate_delegate = None
@@ skipping 72 matching lines @@
 
   @property
   def additional_apks(self):
     return self._additional_apks
 
   @property
   def apk_under_test(self):
     return self._apk_under_test
 
   @property
-  def apk_under_test_permissions(self):
-   return self._apk_under_test_permissions
+  def apk_under_test_install_script(self):
+    return self._apk_under_test_install_script
 
   @property
   def flags(self):
     return self._flags
 
   @property
   def driver_apk(self):
     return self._driver_apk
 
   @property
@@ skipping 10 matching lines @@
 
   @property
   def suite(self):
     return self._suite
 
   @property
   def test_apk(self):
     return self._test_apk
 
   @property
+  def test_apk_install_script(self):
+    return self._test_apk_install_script
+
+  @property
   def test_jar(self):
     return self._test_jar
 
   @property
   def test_support_apk(self):
     return self._test_support_apk
 
   @property
   def test_package(self):
     return self._test_package
 
   @property
-  def test_permissions(self):
-    return self._test_permissions
-
-  @property
   def test_runner(self):
     return self._test_runner
 
   @property
   def timeout_scale(self):
     return self._timeout_scale
 
   #override
   def TestType(self):
     return 'instrumentation'
@@ skipping 190 matching lines @@
   def GenerateTestResults(
       result_code, result_bundle, statuses, start_ms, duration_ms):
     return GenerateTestResults(result_code, result_bundle, statuses,
                                start_ms, duration_ms)
 
   #override
   def TearDown(self):
     if self._isolate_delegate:
       self._isolate_delegate.Clear()