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

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

Issue 954743003: Make step status EXCEPTION when test runner has infrastructure failure. (Closed) Base URL: https://chromium.googlesource.com/chromium/tools/build.git@master
Patch Set: Moved test runner exit codes to central location. Created 5 years, 9 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 1
2 DEPS = [ 2 DEPS = [
3 'amp', 3 'amp',
4 'json', 4 'json',
5 'path', 5 'path',
6 'properties', 6 'properties',
7 ] 7 ]
8 8
9 BUILDERS = { 9 BUILDERS = {
10 'normal_example': { 10 'normal_example': {
(...skipping 83 matching lines...) Expand 10 before | Expand all | Expand 10 after
94 'device_timeout': 60, 94 'device_timeout': 60,
95 'api_address': '127.0.0.1', 95 'api_address': '127.0.0.1',
96 'api_port': '80', 96 'api_port': '80',
97 'api_protocol': 'http', 97 'api_protocol': 'http',
98 }, 98 },
99 } 99 }
100 100
101 AMP_RESULTS_BUCKET = 'chrome-amp-results' 101 AMP_RESULTS_BUCKET = 'chrome-amp-results'
102 102
103 def GenSteps(api): 103 def GenSteps(api):
104 api.amp.setup()
jbudorick 2015/02/27 15:17:27 Can't say I'm a big fan of this... looking into al
mikecase (-- gone --) 2015/02/27 18:10:58 I based what I did on what recipe_modules/gpu/api.
105
104 builder = BUILDERS[api.properties['buildername']] 106 builder = BUILDERS[api.properties['buildername']]
105 api.path['checkout'] = api.path['slave_build'].join('src') 107 api.path['checkout'] = api.path['slave_build'].join('src')
106 108
107 api.amp.trigger_test_suite( 109 api.amp.trigger_test_suite(
108 'example_gtest_suite', 'gtest', 110 'example_gtest_suite', 'gtest',
109 api.amp.gtest_arguments('example_gtest_suite'), 111 api.amp.gtest_arguments('example_gtest_suite'),
110 api.amp.amp_arguments( 112 api.amp.amp_arguments(
111 device_minimum_os=builder.get('device_minimum_os', None), 113 device_minimum_os=builder.get('device_minimum_os', None),
112 device_name=builder.get('device_name', None), 114 device_name=builder.get('device_name', None),
113 device_oem=builder.get('device_oem', None), 115 device_oem=builder.get('device_oem', None),
(...skipping 50 matching lines...) Expand 10 before | Expand all | Expand 10 after
164 api.test('bad_device_data_for_collect') + 166 api.test('bad_device_data_for_collect') +
165 api.properties.generic(buildername='split_example') + 167 api.properties.generic(buildername='split_example') +
166 api.override_step_data('[collect] load example_gtest_suite data', 168 api.override_step_data('[collect] load example_gtest_suite data',
167 api.json.output({}))) 169 api.json.output({})))
168 170
169 yield ( 171 yield (
170 api.test('bad_test_id_data_for_upload') + 172 api.test('bad_test_id_data_for_upload') +
171 api.properties.generic(buildername='split_example') + 173 api.properties.generic(buildername='split_example') +
172 api.override_step_data('[upload logcat] load example_gtest_suite data', 174 api.override_step_data('[upload logcat] load example_gtest_suite data',
173 api.json.output({}))) 175 api.json.output({})))
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698