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

Side by Side Diff: scripts/slave/recipes/findit/chromium/test.py

Issue 2146873002: ios: drop dependency on chromium_tests; move analyze to filter module (Closed) Base URL: https://chromium.googlesource.com/chromium/tools/build.git@master
Patch Set: Created 4 years, 5 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
« no previous file with comments | « scripts/slave/recipes/findit/chromium/compile.py ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 # Copyright 2015 The Chromium Authors. All rights reserved. 1 # Copyright 2015 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 from collections import defaultdict 5 from collections import defaultdict
6 import json 6 import json
7 7
8 from recipe_engine.config import Dict 8 from recipe_engine.config import Dict
9 from recipe_engine.config import List 9 from recipe_engine.config import List
10 from recipe_engine.config import Single 10 from recipe_engine.config import Single
11 from recipe_engine.recipe_api import Property 11 from recipe_engine.recipe_api import Property
12 12
13 13
14 DEPS = [ 14 DEPS = [
15 'adb', 15 'adb',
16 'buildbucket', 16 'buildbucket',
17 'depot_tools/bot_update', 17 'depot_tools/bot_update',
18 'chromium', 18 'chromium',
19 'chromium_android', 19 'chromium_android',
20 'chromium_swarming', 20 'chromium_swarming',
21 'chromium_tests', 21 'chromium_tests',
22 'commit_position', 22 'commit_position',
23 'filter',
23 'findit', 24 'findit',
24 'depot_tools/gclient', 25 'depot_tools/gclient',
25 'isolate', 26 'isolate',
26 'recipe_engine/json', 27 'recipe_engine/json',
27 'recipe_engine/path', 28 'recipe_engine/path',
28 'recipe_engine/platform', 29 'recipe_engine/platform',
29 'recipe_engine/properties', 30 'recipe_engine/properties',
30 'recipe_engine/python', 31 'recipe_engine/python',
31 'recipe_engine/raw_io', 32 'recipe_engine/raw_io',
32 'recipe_engine/step', 33 'recipe_engine/step',
(...skipping 68 matching lines...) Expand 10 before | Expand all | Expand 10 after
101 requested_test_targets.extend(test.compile_targets(api)) 102 requested_test_targets.extend(test.compile_targets(api))
102 requested_test_targets = sorted(set(requested_test_targets)) 103 requested_test_targets = sorted(set(requested_test_targets))
103 104
104 actual_tests_to_run = requested_tests_to_run 105 actual_tests_to_run = requested_tests_to_run
105 actual_compile_targets = requested_test_targets 106 actual_compile_targets = requested_test_targets
106 # Use dependency "analyze" to reduce tests to be run. 107 # Use dependency "analyze" to reduce tests to be run.
107 if use_analyze: 108 if use_analyze:
108 changed_files = api.findit.files_changed_by_revision(revision) 109 changed_files = api.findit.files_changed_by_revision(revision)
109 110
110 affected_test_targets, actual_compile_targets = ( 111 affected_test_targets, actual_compile_targets = (
111 api.chromium_tests.analyze( 112 api.filter.analyze(
112 changed_files, 113 changed_files,
113 test_targets=requested_test_targets, 114 test_targets=requested_test_targets,
114 additional_compile_targets=[], 115 additional_compile_targets=[],
115 config_file_name='trybot_analyze_config.json', 116 config_file_name='trybot_analyze_config.json',
116 mb_mastername=target_mastername, 117 mb_mastername=target_mastername,
117 mb_buildername=target_buildername, 118 mb_buildername=target_buildername,
118 additional_names=None)) 119 additional_names=None))
119 120
120 actual_tests_to_run = [] 121 actual_tests_to_run = []
121 for test in requested_tests_to_run: 122 for test in requested_tests_to_run:
(...skipping 1082 matching lines...) Expand 10 before | Expand all | Expand 10 after
1204 'git commits in range', 1205 'git commits in range',
1205 api.raw_io.stream_output( 1206 api.raw_io.stream_output(
1206 '\n'.join('r%d' % i for i in reversed(range(1, 7))))) + 1207 '\n'.join('r%d' % i for i in reversed(range(1, 7))))) +
1207 api.override_step_data( 1208 api.override_step_data(
1208 'test r2.gl_tests (r2) on Mac-10.9', 1209 'test r2.gl_tests (r2) on Mac-10.9',
1209 simulated_gtest_output(passed_test_names=['Test.One'])) + 1210 simulated_gtest_output(passed_test_names=['Test.One'])) +
1210 api.override_step_data( 1211 api.override_step_data(
1211 'test r3.gl_tests (r3) on Mac-10.9', 1212 'test r3.gl_tests (r3) on Mac-10.9',
1212 simulated_gtest_output(failed_test_names=['Test.One'])) 1213 simulated_gtest_output(failed_test_names=['Test.One']))
1213 ) 1214 )
OLDNEW
« no previous file with comments | « scripts/slave/recipes/findit/chromium/compile.py ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698