From d813498d030c4ad346088579f4f6b488ff494d34 Mon Sep 17 00:00:00 2001 From: "frankf@chromium.org" Date: Sat, 19 Jan 2013 03:55:22 +0000 Subject: [PATCH] [Android] Uploaded instrumentation tests to prod flakiness dashboard. BUG=150801 Review URL: https://codereview.chromium.org/11943018 git-svn-id: http://src.chromium.org/svn/trunk/src/build@177822 4ff67af0-8c30-449e-8e8b-ad334ec8d88c --- android/buildbot/bb_device_steps.py | 4 ++-- android/pylib/base/test_result.py | 9 ++------- android/pylib/constants.py | 2 ++ 3 files changed, 6 insertions(+), 9 deletions(-) diff --git a/android/buildbot/bb_device_steps.py b/android/buildbot/bb_device_steps.py index b5d8310e8..d512a855e 100755 --- a/android/buildbot/bb_device_steps.py +++ b/android/buildbot/bb_device_steps.py @@ -169,8 +169,8 @@ def RunInstrumentationSuite(options, test): if options.asan: args.append('--tool=asan') if options.upload_to_flakiness_server: - args.append('--flakiness-dashboard-server=' - 'chrome-android-staging-results.appspot.com') + args.append('--flakiness-dashboard-server=%s' % + constants.UPSTREAM_FLAKINESS_SERVER) RunCmd(['build/android/run_instrumentation_tests.py'] + args) diff --git a/android/pylib/base/test_result.py b/android/pylib/base/test_result.py index 772d3d5de..ee2421e46 100644 --- a/android/pylib/base/test_result.py +++ b/android/pylib/base/test_result.py @@ -15,9 +15,6 @@ from pylib import constants from pylib.utils import flakiness_dashboard_results_uploader -_STAGING_SERVER = 'chrome-android-staging' - - class BaseTestResult(object): """A single result from a unit test.""" @@ -188,15 +185,13 @@ class TestResults(object): return try: - # TODO(frankf): Temp server for initial testing upstream. - # Use http://test-results.appspot.com once we're confident this works. - if _STAGING_SERVER in flakiness_server: + if flakiness_server == constants.UPSTREAM_FLAKINESS_SERVER: assert test_package in ['ContentShellTest', 'ChromiumTestShellTest', 'AndroidWebViewTest'] dashboard_test_type = ('%s_instrumentation_tests' % test_package.lower().rstrip('test')) - # Downstream prod server. + # Downstream server. else: dashboard_test_type = 'Chromium_Android_Instrumentation' diff --git a/android/pylib/constants.py b/android/pylib/constants.py index 013ab7ff5..a7845f1fd 100644 --- a/android/pylib/constants.py +++ b/android/pylib/constants.py @@ -51,3 +51,5 @@ SCREENSHOTS_DIR = os.path.join(CHROME_DIR, 'out_screenshots') ANDROID_SDK_VERSION = 17 ANDROID_SDK_ROOT = os.path.join(CHROME_DIR, 'third_party/android_tools/sdk') ANDROID_NDK_ROOT = os.path.join(CHROME_DIR, 'third_party/android_tools/ndk') + +UPSTREAM_FLAKINESS_SERVER = 'test-results.appspot.com'