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

Side by Side Diff: scripts/slave/recipes/chromium_trybot.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
OLDNEW
1 # Copyright 2013 The Chromium Authors. All rights reserved. 1 # Copyright 2013 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 collections 5 import collections
6 6
7 from recipe_engine.types import freeze 7 from recipe_engine.types import freeze
8 8
9 DEPS = [ 9 DEPS = [
10 'depot_tools/bot_update', 10 'depot_tools/bot_update',
11 'chromium', 11 'chromium',
12 'chromium_android', 12 'chromium_android',
13 'chromium_swarming', 13 'chromium_swarming',
14 'chromium_tests', 14 'chromium_tests',
15 'commit_position', 15 'commit_position',
16 'file', 16 'file',
17 'filter',
17 'depot_tools/gclient', 18 'depot_tools/gclient',
18 'isolate', 19 'isolate',
19 'recipe_engine/json', 20 'recipe_engine/json',
20 'recipe_engine/path', 21 'recipe_engine/path',
21 'recipe_engine/platform', 22 'recipe_engine/platform',
22 'recipe_engine/properties', 23 'recipe_engine/properties',
23 'recipe_engine/python', 24 'recipe_engine/python',
24 'recipe_engine/raw_io', 25 'recipe_engine/raw_io',
25 'recipe_engine/step', 26 'recipe_engine/step',
26 'swarming', 27 'swarming',
(...skipping 134 matching lines...) Expand 10 before | Expand all | Expand 10 after
161 162
162 compile_targets = api.chromium_tests.get_compile_targets( 163 compile_targets = api.chromium_tests.get_compile_targets(
163 bot_config_object, 164 bot_config_object,
164 bot_db, 165 bot_db,
165 tests_including_triggered) 166 tests_including_triggered)
166 167
167 test_targets = sorted(set( 168 test_targets = sorted(set(
168 all_compile_targets(api, tests + tests_including_triggered))) 169 all_compile_targets(api, tests + tests_including_triggered)))
169 additional_compile_targets = sorted(set(compile_targets) - 170 additional_compile_targets = sorted(set(compile_targets) -
170 set(test_targets)) 171 set(test_targets))
171 test_targets, compile_targets = \ 172 test_targets, compile_targets = api.filter.analyze(
172 api.chromium_tests.analyze(affected_files, 173 affected_files, test_targets, additional_compile_targets,
173 test_targets, 174 'trybot_analyze_config.json')
174 additional_compile_targets,
175 'trybot_analyze_config.json')
176 175
177 if bot_config.get('analyze_mode') == 'compile': 176 if bot_config.get('analyze_mode') == 'compile':
178 tests = [] 177 tests = []
179 tests_including_triggered = [] 178 tests_including_triggered = []
180 179
181 # Blink tests have to bypass "analyze", see below. 180 # Blink tests have to bypass "analyze", see below.
182 if compile_targets or add_blink_tests: 181 if compile_targets or add_blink_tests:
183 tests = tests_in_compile_targets(api, test_targets, tests) 182 tests = tests_in_compile_targets(api, test_targets, tests)
184 tests_including_triggered = tests_in_compile_targets( 183 tests_including_triggered = tests_in_compile_targets(
185 api, test_targets, tests_including_triggered) 184 api, test_targets, tests_including_triggered)
(...skipping 758 matching lines...) Expand 10 before | Expand all | Expand 10 after
944 props(mastername='tryserver.v8', 943 props(mastername='tryserver.v8',
945 buildername='v8_linux_blink_rel', 944 buildername='v8_linux_blink_rel',
946 patch_project='v8') + 945 patch_project='v8') +
947 api.platform.name('linux') + 946 api.platform.name('linux') +
948 suppress_analyze(more_exclusions=['v8/f.*']) + 947 suppress_analyze(more_exclusions=['v8/f.*']) +
949 api.override_step_data('webkit_tests (with patch)', 948 api.override_step_data('webkit_tests (with patch)',
950 api.test_utils.canned_test_output(passing=False)) + 949 api.test_utils.canned_test_output(passing=False)) +
951 api.override_step_data('webkit_tests (without patch)', 950 api.override_step_data('webkit_tests (without patch)',
952 api.test_utils.canned_test_output(passing=True, minimal=True)) 951 api.test_utils.canned_test_output(passing=True, minimal=True))
953 ) 952 )
OLDNEW
« no previous file with comments | « scripts/slave/recipe_modules/ios/api.py ('k') | scripts/slave/recipes/findit/chromium/compile.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698