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

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

Issue 873403002: Add support for cc_perftests and other non-telemetry gtest based tests. (Closed) Base URL: https://chromium.googlesource.com/chromium/tools/build.git@master
Patch Set: Created 5 years, 10 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 os 5 import os
6 import urllib 6 import urllib
7 7
8 from infra.libs.infra_types import freeze 8 from infra.libs.infra_types import freeze
9 from slave import recipe_api 9 from slave import recipe_api
10 10
(...skipping 453 matching lines...) Expand 10 before | Expand all | Expand 10 after
464 results_url='https://chromeperf.appspot.com', 464 results_url='https://chromeperf.appspot.com',
465 perf_id=perf_id, 465 perf_id=perf_id,
466 env=self.m.chromium.get_env(), 466 env=self.m.chromium.get_env(),
467 chartjson_file=chartjson_file) 467 chartjson_file=chartjson_file)
468 except self.m.step.StepFailure as f: 468 except self.m.step.StepFailure as f:
469 failures.append(f) 469 failures.append(f)
470 470
471 if failures: 471 if failures:
472 raise self.m.step.StepFailure('sharded perf tests failed %s' % failures) 472 raise self.m.step.StepFailure('sharded perf tests failed %s' % failures)
473 473
474 def run_other_gtests(self, perf_tests, perf_id=None):
475 failures = []
476 for test_name, test_data in sorted(perf_tests.iteritems()):
477 test_name = str(test_name) # un-unicode
478 test_suite = str(test_data['cmd']) # un-unicode
ghost stip (do not use) 2015/01/27 01:02:02 nit: # Comments are a capitalized statement ending
479 dashboard_id = str(test_data['perf_dashboard_id'])
480 annotate = self.m.chromium.get_annotate_by_test_name(test_name)
481
482 try:
483 self.m.chromium.runtest(
484 self.c.test_runner,
485 ['gtest', '--release', '--suite', test_suite, '--verbose'],
486 name=test_name,
487 perf_dashboard_id=dashboard_id,
488 annotate=annotate,
489 results_url='https://chromeperf.appspot.com',
490 perf_id=perf_id,
491 env=self.m.chromium.get_env())
492 except self.m.step.StepFailure as f:
493 failures.append(f)
494 if failures:
495 raise self.m.step.StepFailure('non-telemetry tests failed %s' % failures)
496
474 @recipe_api.non_step 497 @recipe_api.non_step
475 def get_instrumentation_suite(self, suite_name): 498 def get_instrumentation_suite(self, suite_name):
476 for suite in INSTRUMENTATION_TESTS: 499 for suite in INSTRUMENTATION_TESTS:
477 if suite['test'] == suite_name: 500 if suite['test'] == suite_name:
478 return suite 501 return suite
479 return None 502 return None
480 503
481 def run_instrumentation_suite(self, test_apk, test_data=None, 504 def run_instrumentation_suite(self, test_apk, test_data=None,
482 isolate_file_path=None, 505 isolate_file_path=None,
483 flakiness_dashboard=None, 506 flakiness_dashboard=None,
(...skipping 209 matching lines...) Expand 10 before | Expand all | Expand 10 after
693 716
694 self.m.gsutil.upload( 717 self.m.gsutil.upload(
695 source=self.coverage_dir.join('coverage_html'), 718 source=self.coverage_dir.join('coverage_html'),
696 bucket='chrome-code-coverage', 719 bucket='chrome-code-coverage',
697 dest=gs_dest, 720 dest=gs_dest,
698 args=['-R'], 721 args=['-R'],
699 name='upload coverage report', 722 name='upload coverage report',
700 link_name='Coverage report', 723 link_name='Coverage report',
701 version='4.7', 724 version='4.7',
702 **kwargs) 725 **kwargs)
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698