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

Side by Side Diff: scripts/slave/recipe_modules/chromium_android/example.py

Issue 2439213002: Add step to process Render Test results. (Closed)
Patch Set: Addressed jbudorick's nits Created 4 years, 1 month 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 from recipe_engine.types import freeze 5 from recipe_engine.types import freeze
6 6
7 DEPS = [ 7 DEPS = [
8 'adb', 8 'adb',
9 'chromium', 9 'chromium',
10 'chromium_android', 10 'chromium_android',
(...skipping 72 matching lines...) Expand 10 before | Expand all | Expand 10 after
83 'perf_config': 'sharded_perf_tests.json', 83 'perf_config': 'sharded_perf_tests.json',
84 'last_known_devices': '.last_devices', 84 'last_known_devices': '.last_devices',
85 }, 85 },
86 'device_flags_builder': { 86 'device_flags_builder': {
87 'device_flags': 'device_flags_file', 87 'device_flags': 'device_flags_file',
88 }, 88 },
89 'no_cache_builder': { 89 'no_cache_builder': {
90 'use_git_cache': False, 90 'use_git_cache': False,
91 }, 91 },
92 'json_results_file': { 92 'json_results_file': {
93 'json_results_file': 'json_results_file' 93 'json_results_file': 'json_results_file',
94 },
95 'render_results': {
96 'render_results_dir': 'chrome/test/data/android/render_tests',
94 }, 97 },
95 'result_details': { 98 'result_details': {
96 'result_details': True, 99 'result_details': True,
97 'cs_base_url': 'cs_base_url', 100 'cs_base_url': 'cs_base_url',
98 'store_tombstones': True, 101 'store_tombstones': True,
99 }, 102 },
100 'enable_platform_mode': { 103 'enable_platform_mode': {
101 'perf_config': 'sharded_perf_tests.json', 104 'perf_config': 'sharded_perf_tests.json',
102 'enable_platform_mode': True, 105 'enable_platform_mode': True,
103 'write_buildbot_json': True, 106 'write_buildbot_json': True,
(...skipping 115 matching lines...) Expand 10 before | Expand all | Expand 10 after
219 except_annotation='FlakyTest', 222 except_annotation='FlakyTest',
220 screenshot=True, 223 screenshot=True,
221 official_build=True, 224 official_build=True,
222 timeout_scale=config.get('timeout_scale'), 225 timeout_scale=config.get('timeout_scale'),
223 strict_mode=config.get('strict_mode'), 226 strict_mode=config.get('strict_mode'),
224 additional_apks=['Additional.apk'], 227 additional_apks=['Additional.apk'],
225 device_flags=config.get('device_flags'), 228 device_flags=config.get('device_flags'),
226 json_results_file=config.get('json_results_file'), 229 json_results_file=config.get('json_results_file'),
227 result_details=config.get('result_details'), 230 result_details=config.get('result_details'),
228 cs_base_url=config.get('cs_base_url'), 231 cs_base_url=config.get('cs_base_url'),
229 store_tombstones=config.get('store_tombstones')) 232 store_tombstones=config.get('store_tombstones'),
233 render_results_dir=config.get('render_results_dir'))
ghost stip (do not use) 2016/12/08 20:16:02 ideally, we should detect the presence of these fi
mikecase (-- gone --) 2017/01/09 22:27:09 acknowledged. Going to keep it as is at first, so
230 api.chromium_android.run_test_suite( 234 api.chromium_android.run_test_suite(
231 'unittests', 235 'unittests',
232 isolate_file_path=api.path['checkout'].join('some_file.isolate'), 236 isolate_file_path=api.path['checkout'].join('some_file.isolate'),
233 gtest_filter='WebRtc*', 237 gtest_filter='WebRtc*',
234 result_details=config.get('result_details'), 238 result_details=config.get('result_details'),
235 store_tombstones=config.get('store_tombstones'), 239 store_tombstones=config.get('store_tombstones'),
236 tool='asan') 240 tool='asan')
237 if not failure: 241 if not failure:
238 api.chromium_android.run_bisect_script(extra_src='test.py', 242 api.chromium_android.run_bisect_script(extra_src='test.py',
239 path_to_config='test.py') 243 path_to_config='test.py')
(...skipping 211 matching lines...) Expand 10 before | Expand all | Expand 10 after
451 properties_for('tester') + 455 properties_for('tester') +
452 api.override_step_data('get version (2)', 456 api.override_step_data('get version (2)',
453 api.raw_io.output('MAJOR=53\nMINOR=0\nBUILD=2800\nPATCH=0\n'))) 457 api.raw_io.output('MAJOR=53\nMINOR=0\nBUILD=2800\nPATCH=0\n')))
454 458
455 yield (api.test('telemetry_browser_tests_failures') + 459 yield (api.test('telemetry_browser_tests_failures') +
456 properties_for('telemetry_browser_tests_tester') + 460 properties_for('telemetry_browser_tests_tester') +
457 api.override_step_data('Run telemetry browser_test PopularUrlsTest', 461 api.override_step_data('Run telemetry browser_test PopularUrlsTest',
458 api.json.output({'successes': ['passed_test1', 'passed_test2'], 462 api.json.output({'successes': ['passed_test1', 'passed_test2'],
459 'failures': ['failed_test_1', 'failed_test_2']}), 463 'failures': ['failed_test_1', 'failed_test_2']}),
460 retcode=1)) 464 retcode=1))
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698