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

Side by Side Diff: scripts/slave/recipe_modules/chromium/api.py

Issue 2247373002: Refactor stages 1, 2 and test_api overhaul. (Closed) Base URL: https://chromium.googlesource.com/chromium/tools/build.git@master
Patch Set: Got full coverage for new and changed code. Created 4 years, 3 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 re 5 import re
6 6
7 from recipe_engine import recipe_api 7 from recipe_engine import recipe_api
8 from recipe_engine import util as recipe_util 8 from recipe_engine import util as recipe_util
9 9
10 class TestLauncherFilterFileInputPlaceholder(recipe_util.InputPlaceholder): 10 class TestLauncherFilterFileInputPlaceholder(recipe_util.InputPlaceholder):
(...skipping 295 matching lines...) Expand 10 before | Expand all | Expand 10 after
306 full_args.append('--perf-dashboard-id=%s' % perf_dashboard_id) 306 full_args.append('--perf-dashboard-id=%s' % perf_dashboard_id)
307 if perf_id: 307 if perf_id:
308 full_args.append('--perf-id=%s' % perf_id) 308 full_args.append('--perf-id=%s' % perf_id)
309 if perf_config: 309 if perf_config:
310 full_args.extend(['--perf-config', perf_config]) 310 full_args.extend(['--perf-config', perf_config])
311 # This replaces the step_name that used to be sent via factory_properties. 311 # This replaces the step_name that used to be sent via factory_properties.
312 if test_type: 312 if test_type:
313 full_args.append('--test-type=%s' % test_type) 313 full_args.append('--test-type=%s' % test_type)
314 step_name = name or t_name 314 step_name = name or t_name
315 full_args.append('--step-name=%s' % step_name) 315 full_args.append('--step-name=%s' % step_name)
316 if chartjson_file: 316 if chartjson_file and not 'step_test_data' in kwargs:
317 full_args.append('--chartjson-file') 317 full_args.append('--chartjson-file')
318 full_args.append(self.m.json.output()) 318 full_args.append(self.m.json.output())
319 kwargs['step_test_data'] = lambda: self.m.json.test_api.output([]) 319 kwargs['step_test_data'] = lambda: self.m.json.test_api.output([])
320 if test_launcher_summary_output: 320 if test_launcher_summary_output:
321 full_args.extend([ 321 full_args.extend([
322 '--test-launcher-summary-output', 322 '--test-launcher-summary-output',
323 test_launcher_summary_output 323 test_launcher_summary_output
324 ]) 324 ])
325 if flakiness_dash: 325 if flakiness_dash:
326 full_args.extend([ 326 full_args.extend([
(...skipping 468 matching lines...) Expand 10 before | Expand all | Expand 10 after
795 })) 795 }))
796 796
797 def get_annotate_by_test_name(self, test_name): 797 def get_annotate_by_test_name(self, test_name):
798 return 'graphing' 798 return 'graphing'
799 799
800 def download_lto_plugin(self): 800 def download_lto_plugin(self):
801 return self.m.python( 801 return self.m.python(
802 name='download LTO plugin', 802 name='download LTO plugin',
803 script=self.m.path['checkout'].join( 803 script=self.m.path['checkout'].join(
804 'build', 'download_gold_plugin.py')) 804 'build', 'download_gold_plugin.py'))
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698