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

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

Issue 1902983002: Revert of Switch Android gtests to use generated wrapper scripts (Closed) Base URL: https://chromium.googlesource.com/chromium/tools/build.git@master
Patch Set: 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 255 matching lines...) Expand 10 before | Expand all | Expand 10 after
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 if api.chromium.c.TARGET_PLATFORM == 'android': 275 if api.chromium.c.TARGET_PLATFORM == 'android':
276 # TODO(agrieve): Remove _apk suffix in favour of bin/run_${target} once
277 # GYP is gone. http://crbug.com/599919
278 return [self.target_name + '_apk'] 276 return [self.target_name + '_apk']
279 277
280 return [self.target_name] 278 return [self.target_name]
281 279
282 def run(self, api, suffix, test_filter=None): 280 def run(self, api, suffix, test_filter=None):
283 # Copy the list because run can be invoked multiple times and we modify 281 # Copy the list because run can be invoked multiple times and we modify
284 # the local copy. 282 # the local copy.
285 args = self._args[:] 283 args = self._args[:]
286 is_android = api.chromium.c.TARGET_PLATFORM == 'android' 284 is_android = api.chromium.c.TARGET_PLATFORM == 'android'
287 285
(...skipping 531 matching lines...) Expand 10 before | Expand all | Expand 10 after
819 return self._override_compile_targets 817 return self._override_compile_targets
820 818
821 if api.chromium.c.TARGET_PLATFORM == 'android': 819 if api.chromium.c.TARGET_PLATFORM == 'android':
822 # Not all _apk_runs have a corresponding _apk, so we only return the 820 # Not all _apk_runs have a corresponding _apk, so we only return the
823 # _apk_run here. 821 # _apk_run here.
824 return [self.target_name + '_apk_run'] 822 return [self.target_name + '_apk_run']
825 823
826 return [self.target_name, self.target_name + '_run'] 824 return [self.target_name, self.target_name + '_run']
827 825
828 def isolate_target(self, api): 826 def isolate_target(self, api):
829 # TODO(agrieve): Remove _apk suffix in favour of bin/run_${target} once
830 # GYP is gone. http://crbug.com/599919
831 if api.chromium.c.TARGET_PLATFORM == 'android': 827 if api.chromium.c.TARGET_PLATFORM == 'android':
832 return self.target_name + '_apk' 828 return self.target_name + '_apk'
833 return self.target_name 829 return self.target_name
834 830
835 def create_task(self, api, suffix, isolated_hash, test_filter=None): 831 def create_task(self, api, suffix, isolated_hash, test_filter=None):
836 # For local tests test_args are added inside api.chromium.runtest. 832 # For local tests test_args are added inside api.chromium.runtest.
837 args = self._args[:] 833 args = self._args[:]
838 args.extend(api.chromium.c.runtests.test_args) 834 args.extend(api.chromium.c.runtests.test_args)
839 835
840 if suffix == 'without patch': 836 if suffix == 'without patch':
(...skipping 1021 matching lines...) Expand 10 before | Expand all | Expand 10 after
1862 def run(self, api, suffix, test_filter=None): 1858 def run(self, api, suffix, test_filter=None):
1863 api.chromium_android.coverage_report(upload=False) 1859 api.chromium_android.coverage_report(upload=False)
1864 api.chromium_android.get_changed_lines_for_revision() 1860 api.chromium_android.get_changed_lines_for_revision()
1865 api.chromium_android.incremental_coverage_report() 1861 api.chromium_android.incremental_coverage_report()
1866 1862
1867 1863
1868 GOMA_TESTS = [ 1864 GOMA_TESTS = [
1869 GTestTest('base_unittests'), 1865 GTestTest('base_unittests'),
1870 GTestTest('content_unittests'), 1866 GTestTest('content_unittests'),
1871 ] 1867 ]
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698