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

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

Issue 1465343002: [Android] Add context manager to record logcat during step(s). (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/tools/build
Patch Set: Removed unused config. Created 5 years 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 | Annotate | Revision Log
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 144 matching lines...) Expand 10 before | Expand all | Expand 10 after
155 155
156 try: 156 try:
157 if config.get('perf_config'): 157 if config.get('perf_config'):
158 api.chromium_android.run_sharded_perf_tests( 158 api.chromium_android.run_sharded_perf_tests(
159 config='fake_config.json', 159 config='fake_config.json',
160 flaky_config='flake_fakes.json', 160 flaky_config='flake_fakes.json',
161 upload_archives_to_bucket='archives-bucket') 161 upload_archives_to_bucket='archives-bucket')
162 except api.step.StepFailure as f: 162 except api.step.StepFailure as f:
163 failure = f 163 failure = f
164 164
165 api.chromium_android.run_instrumentation_suite( 165 with api.chromium_android.logcat('AndroidWebViewTest'):
166 name='AndroidWebViewTest', 166 api.chromium_android.run_instrumentation_suite(
167 apk_under_test=api.chromium_android.apk_path('AndroidWebView.apk'), 167 name='AndroidWebViewTest',
168 test_apk=api.chromium_android.apk_path('AndroidWebViewTest.apk'), 168 apk_under_test=api.chromium_android.apk_path('AndroidWebView.apk'),
169 isolate_file_path='android_webview/android_webview_test_apk.isolate', 169 test_apk=api.chromium_android.apk_path('AndroidWebViewTest.apk'),
170 flakiness_dashboard='test-results.appspot.com', 170 isolate_file_path='android_webview/android_webview_test_apk.isolate',
171 annotation='SmallTest', 171 flakiness_dashboard='test-results.appspot.com',
172 except_annotation='FlakyTest', 172 annotation='SmallTest',
173 screenshot=True, 173 except_annotation='FlakyTest',
174 official_build=True, 174 screenshot=True,
175 host_driven_root=api.path['checkout'].join('chrome', 'test')) 175 official_build=True,
176 api.chromium_android.run_test_suite( 176 host_driven_root=api.path['checkout'].join('chrome', 'test'))
177 'unittests', 177 with api.chromium_android.logcat('unittests'):
178 isolate_file_path=api.path['checkout'].join('some_file.isolate'), 178 api.chromium_android.run_test_suite(
179 gtest_filter='WebRtc*', 179 'unittests',
180 tool='asan') 180 isolate_file_path=api.path['checkout'].join('some_file.isolate'),
181 gtest_filter='WebRtc*',
182 tool='asan')
181 if not failure: 183 if not failure:
182 api.chromium_android.run_bisect_script(extra_src='test.py', 184 api.chromium_android.run_bisect_script(extra_src='test.py',
183 path_to_config='test.py') 185 path_to_config='test.py')
184 api.chromium_android.logcat_dump() 186
185 api.chromium_android.stack_tool_steps() 187 logcat_file = api.chromium.output_dir.join('full_logcat')
188 api.chromium_android.logcat_dump(output_logcat_file=logcat_file)
189 api.chromium_android.stack_tool_steps(logcat_file=logcat_file)
186 if config.get('coverage', False): 190 if config.get('coverage', False):
187 api.chromium_android.coverage_report() 191 api.chromium_android.coverage_report()
188 192
189 if failure: 193 if failure:
190 raise failure 194 raise failure
191 195
192 def GenTests(api): 196 def GenTests(api):
193 def properties_for(buildername): 197 def properties_for(buildername):
194 return api.properties.generic( 198 return api.properties.generic(
195 buildername=buildername, 199 buildername=buildername,
(...skipping 31 matching lines...) Expand 10 before | Expand all | Expand 10 after
227 231
228 yield (api.test('gerrit_refs') + 232 yield (api.test('gerrit_refs') +
229 api.properties.generic( 233 api.properties.generic(
230 buildername='gerrit_try_builder', 234 buildername='gerrit_try_builder',
231 slavename='testslave', 235 slavename='testslave',
232 repo_name='src/repo', 236 repo_name='src/repo',
233 patch_url='https://the.patch.url/the.patch', 237 patch_url='https://the.patch.url/the.patch',
234 repo_url='svn://svn.chromium.org/chrome/trunk/src', 238 repo_url='svn://svn.chromium.org/chrome/trunk/src',
235 revision='4f4b02f6b7fa20a3a25682c457bbc8ad589c8a00', 239 revision='4f4b02f6b7fa20a3a25682c457bbc8ad589c8a00',
236 internal=True, **({'event.patchSet.ref':'refs/changes/50/176150/1'}))) 240 internal=True, **({'event.patchSet.ref':'refs/changes/50/176150/1'})))
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698