[Android] Add a configurable environment for devil/. (RELAND 2)

build/android/PRESUBMIT.py

Index: build/android/PRESUBMIT.py
diff --git a/build/android/PRESUBMIT.py b/build/android/PRESUBMIT.py
index 5e4c988ed05cb96411242bcdfddd38dae0fb9144..9750dc1b81c32560a01ad99065572966407ba9f2 100644
--- a/build/android/PRESUBMIT.py
+++ b/build/android/PRESUBMIT.py
@@ -12,9 +12,12 @@ details on the presubmit API built into depot_tools.
 def CommonChecks(input_api, output_api):
   output = []
 
+  build_android_dir = input_api.PresubmitLocalPath()
+  chromium_src_dir = input_api.os_path.join(build_android_dir, '..', '..')

[aiolos (Not reviewing), 2015/12/02 23:47:07]

Do you want to make sure this is using an absolute path, or can we make that
assumption about input_api.PresubmitLocalPath() ? (ie, should you add a call to
os_path.abspath here?)

[jbudorick, 2015/12/03 01:04:35]

were I keeping this, I'd probably add abspath (though AFAICT PresubmitLocalPath
should usually return one), but I think that chromium_src_dir can be removed
here since I'm just coming back down to build/android/ in the only place where I
use it.

+
   def J(*dirs):
     """Returns a path relative to presubmit directory."""
-    return input_api.os_path.join(input_api.PresubmitLocalPath(), *dirs)
+    return input_api.os_path.join(build_android_dir, *dirs)
 
   build_pys = [
       r'gyp/.*\.py$',
@@ -40,7 +43,7 @@ def CommonChecks(input_api, output_api):
 
   pylib_test_env = dict(input_api.environ)
   pylib_test_env.update({
-      'PYTHONPATH': input_api.PresubmitLocalPath(),
+      'PYTHONPATH': build_android_dir,
       'PYTHONDONTWRITEBYTECODE': '1',
   })
   output.extend(input_api.canned_checks.RunUnitTests(
@@ -48,13 +51,6 @@ def CommonChecks(input_api, output_api):
       output_api,
       unit_tests=[
           J('.', 'emma_coverage_stats_test.py'),
-          J('devil', 'android', 'fastboot_utils_test.py'),
-          J('devil', 'android', 'battery_utils_test.py'),
-          J('devil', 'android', 'device_utils_test.py'),
-          J('devil', 'android', 'md5sum_test.py'),
-          J('devil', 'android', 'logcat_monitor_test.py'),
-          J('devil', 'utils', 'cmd_helper_test.py'),
-          J('devil', 'utils', 'timeout_retry_unittest.py'),
           J('gyp', 'util', 'md5_check_test.py'),
           J('play_services', 'update_test.py'),
           J('pylib', 'base', 'test_dispatcher_unittest.py'),
@@ -64,6 +60,27 @@ def CommonChecks(input_api, output_api):
           J('pylib', 'results', 'json_results_test.py'),
       ],
       env=pylib_test_env))
+
+
+  devil_test_env = dict(pylib_test_env)
+  devil_test_env.update({
+      'DEVIL_ENV_CONFIG':
+          input_api.os_path.join(chromium_src_dir, 'build', 'android',
+                                 'devil_chromium.json')
+  })
+  output.extend(input_api.canned_checks.RunUnitTests(
+      input_api,
+      output_api,
+      unit_tests=[
+          J('devil', 'android', 'battery_utils_test.py'),
+          J('devil', 'android', 'device_utils_test.py'),
+          J('devil', 'android', 'fastboot_utils_test.py'),
+          J('devil', 'android', 'md5sum_test.py'),
+          J('devil', 'android', 'logcat_monitor_test.py'),
+          J('devil', 'utils', 'cmd_helper_test.py'),
+          J('devil', 'utils', 'timeout_retry_unittest.py'),
+      ],
+      env=devil_test_env))
   return output