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

Side by Side Diff: scripts/slave/recipes/chromium_trybot.py

Issue 2243853002: Drop unwanted dependencies on chromium_tests recipe module (Closed) Base URL: https://chromium.googlesource.com/chromium/tools/build.git@master
Patch Set: coverage Created 4 years, 4 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_checkout',
13 'chromium_swarming', 14 'chromium_swarming',
14 'chromium_tests', 15 'chromium_tests',
15 'commit_position', 16 'commit_position',
16 'file', 17 'file',
17 'filter', 18 'filter',
18 'depot_tools/gclient', 19 'depot_tools/gclient',
19 'isolate', 20 'isolate',
20 'recipe_engine/json', 21 'recipe_engine/json',
21 'recipe_engine/path', 22 'recipe_engine/path',
22 'recipe_engine/platform', 23 'recipe_engine/platform',
(...skipping 85 matching lines...) Expand 10 before | Expand all | Expand 10 after
108 bot_update_step, bot_db = api.chromium_tests.prepare_checkout( 109 bot_update_step, bot_db = api.chromium_tests.prepare_checkout(
109 bot_config_object) 110 bot_config_object)
110 111
111 tests, tests_including_triggered = api.chromium_tests.get_tests( 112 tests, tests_including_triggered = api.chromium_tests.get_tests(
112 bot_config_object, bot_db) 113 bot_config_object, bot_db)
113 114
114 def add_tests(additional_tests): 115 def add_tests(additional_tests):
115 tests.extend(additional_tests) 116 tests.extend(additional_tests)
116 tests_including_triggered.extend(additional_tests) 117 tests_including_triggered.extend(additional_tests)
117 118
118 affected_files = api.chromium_tests.get_files_affected_by_patch('src/') 119 affected_files = api.chromium_checkout.get_files_affected_by_patch('src/')
119 120
120 affects_blink_paths = any( 121 affects_blink_paths = any(
121 f.startswith(path) for f in affected_files 122 f.startswith(path) for f in affected_files
122 for path in CHROMIUM_BLINK_TESTS_PATHS) 123 for path in CHROMIUM_BLINK_TESTS_PATHS)
123 affects_blink = any( 124 affects_blink = any(
124 f.startswith('third_party/WebKit') for f in affected_files) 125 f.startswith('third_party/WebKit') for f in affected_files)
125 126
126 if affects_blink: 127 if affects_blink:
127 subproject_tag = 'blink' 128 subproject_tag = 'blink'
128 elif affects_blink_paths: 129 elif affects_blink_paths:
(...skipping 793 matching lines...) Expand 10 before | Expand all | Expand 10 after
922 props(mastername='tryserver.v8', 923 props(mastername='tryserver.v8',
923 buildername='v8_linux_blink_rel', 924 buildername='v8_linux_blink_rel',
924 patch_project='v8') + 925 patch_project='v8') +
925 api.platform.name('linux') + 926 api.platform.name('linux') +
926 suppress_analyze(more_exclusions=['v8/f.*']) + 927 suppress_analyze(more_exclusions=['v8/f.*']) +
927 api.override_step_data('webkit_tests (with patch)', 928 api.override_step_data('webkit_tests (with patch)',
928 api.test_utils.canned_test_output(passing=False)) + 929 api.test_utils.canned_test_output(passing=False)) +
929 api.override_step_data('webkit_tests (without patch)', 930 api.override_step_data('webkit_tests (without patch)',
930 api.test_utils.canned_test_output(passing=True, minimal=True)) 931 api.test_utils.canned_test_output(passing=True, minimal=True))
931 ) 932 )
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698