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

Side by Side Diff: scripts/slave/recipe_modules/chromium_tests/steps.py

Issue 1881233003: Reland Switch Android gtests to use generated wrapper scripts (Closed) Base URL: https://chromium.googlesource.com/chromium/tools/build.git@master
Patch Set: attempted fix 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 unified diff | Download patch
OLDNEW
1 # Copyright 2014 The Chromium Authors. All rights reserved. 1 # Copyright 2014 The Chromium Authors. All rights reserved.
2 # Use of this source code is governed by a BSD-style license that can be 2 # Use of this source code is governed by a BSD-style license that can be
3 # found in the LICENSE file. 3 # found in the LICENSE file.
4 4
5 import re 5 import re
6 import string 6 import string
7 7
8 8
9 class Test(object): 9 class Test(object):
10 """ 10 """
(...skipping 254 matching lines...) Expand 10 before | Expand all | Expand 10 after
265 def uses_local_devices(self): 265 def uses_local_devices(self):
266 return True # pragma: no cover 266 return True # pragma: no cover
267 267
268 def isolate_target(self, _api): 268 def isolate_target(self, _api):
269 return self.target_name # pragma: no cover 269 return self.target_name # pragma: no cover
270 270
271 def compile_targets(self, api): 271 def compile_targets(self, api):
272 if self._override_compile_targets: 272 if self._override_compile_targets:
273 return self._override_compile_targets 273 return self._override_compile_targets
274 274
275 # TODO(GYP) Remove special case once fully converted to GN.
275 if api.chromium.c.TARGET_PLATFORM == 'android': 276 if api.chromium.c.TARGET_PLATFORM == 'android':
276 return [self.target_name + '_apk'] 277 return [self.target_name, self.target_name + '_apk']
277 278
278 return [self.target_name] 279 return [self.target_name]
279 280
280 def run(self, api, suffix, test_filter=None): 281 def run(self, api, suffix, test_filter=None):
281 # Copy the list because run can be invoked multiple times and we modify 282 # Copy the list because run can be invoked multiple times and we modify
282 # the local copy. 283 # the local copy.
283 args = self._args[:] 284 args = self._args[:]
284 is_android = api.chromium.c.TARGET_PLATFORM == 'android' 285 is_android = api.chromium.c.TARGET_PLATFORM == 'android'
285 286
286 if suffix == 'without patch': 287 if suffix == 'without patch':
(...skipping 1554 matching lines...) Expand 10 before | Expand all | Expand 10 after
1841 def run(self, api, suffix, test_filter=None): 1842 def run(self, api, suffix, test_filter=None):
1842 api.chromium_android.coverage_report(upload=False) 1843 api.chromium_android.coverage_report(upload=False)
1843 api.chromium_android.get_changed_lines_for_revision() 1844 api.chromium_android.get_changed_lines_for_revision()
1844 api.chromium_android.incremental_coverage_report() 1845 api.chromium_android.incremental_coverage_report()
1845 1846
1846 1847
1847 GOMA_TESTS = [ 1848 GOMA_TESTS = [
1848 GTestTest('base_unittests'), 1849 GTestTest('base_unittests'),
1849 GTestTest('content_unittests'), 1850 GTestTest('content_unittests'),
1850 ] 1851 ]
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698