Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(21)

Unified Diff: build/android/pylib/local/device/local_device_gtest_run.py

Issue 1854233002: Reland 2 of GN: Make breakpad_unittests & sandbox_linux_unittests use test() (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@test-minor-renames
Patch Set: Fix PIE errors, fix component mode. Created 4 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « build/android/pylib/gtest/gtest_test_instance.py ('k') | build/android/test_runner.py » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: build/android/pylib/local/device/local_device_gtest_run.py
diff --git a/build/android/pylib/local/device/local_device_gtest_run.py b/build/android/pylib/local/device/local_device_gtest_run.py
index a745751d72c830b793eda4eb1781581447f2d958..8172790fb379450bffa7215b873a680135ca6716 100644
--- a/build/android/pylib/local/device/local_device_gtest_run.py
+++ b/build/android/pylib/local/device/local_device_gtest_run.py
@@ -161,26 +161,18 @@ class _ApkDelegate(object):
class _ExeDelegate(object):
- def __init__(self, tr, exe):
- self._exe_host_path = exe
- self._exe_file_name = os.path.split(exe)[-1]
- self._exe_device_path = '%s/%s' % (
- constants.TEST_EXECUTABLE_DIR, self._exe_file_name)
- deps_host_path = self._exe_host_path + '_deps'
- if os.path.exists(deps_host_path):
- self._deps_host_path = deps_host_path
- self._deps_device_path = self._exe_device_path + '_deps'
- else:
- self._deps_host_path = None
+ def __init__(self, tr, dist_dir):
+ self._host_dist_dir = dist_dir
+ self._exe_file_name = os.path.basename(dist_dir)[:-len('__dist')]
+ self._device_dist_dir = posixpath.join(
+ constants.TEST_EXECUTABLE_DIR, os.path.basename(dist_dir))
self._test_run = tr
def Install(self, device):
# TODO(jbudorick): Look into merging this with normal data deps pushing if
# executables become supported on nonlocal environments.
- host_device_tuples = [(self._exe_host_path, self._exe_device_path)]
- if self._deps_host_path:
- host_device_tuples.append((self._deps_host_path, self._deps_device_path))
- device.PushChangedFiles(host_device_tuples)
+ device.PushChangedFiles([(self._host_dist_dir, self._device_dist_dir)],
+ delete_device_stale=True)
def Run(self, test, device, flags=None, **kwargs):
tool = self._test_run.GetTool(device).GetTestWrapper()
@@ -188,17 +180,17 @@ class _ExeDelegate(object):
cmd = [tool]
else:
cmd = []
- cmd.append(self._exe_device_path)
+ cmd.append(posixpath.join(self._device_dist_dir, self._exe_file_name))
if test:
cmd.append('--gtest_filter=%s' % ':'.join(test))
if flags:
+ # TODO(agrieve): This won't work if multiple flags are passed.
cmd.append(flags)
cwd = constants.TEST_EXECUTABLE_DIR
env = {
- 'LD_LIBRARY_PATH':
- '%s/%s_deps' % (constants.TEST_EXECUTABLE_DIR, self._exe_file_name),
+ 'LD_LIBRARY_PATH': self._device_dist_dir
}
try:
gcov_strip_depth = os.environ['NATIVE_COVERAGE_DEPTH_STRIP']
@@ -228,8 +220,8 @@ class LocalDeviceGtestRun(local_device_test_run.LocalDeviceTestRun):
if self._test_instance.apk:
self._delegate = _ApkDelegate(self._test_instance)
- elif self._test_instance.exe:
- self._delegate = _ExeDelegate(self, self._test_instance.exe)
+ elif self._test_instance.exe_dist_dir:
+ self._delegate = _ExeDelegate(self, self._test_instance.exe_dist_dir)
self._crashes = set()
self._servers = collections.defaultdict(list)
« no previous file with comments | « build/android/pylib/gtest/gtest_test_instance.py ('k') | build/android/test_runner.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698