| OLD | NEW |
| 1 # Copyright (c) 2012 The Chromium Authors. All rights reserved. | 1 # Copyright (c) 2012 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 """Uploads the results to the flakiness dashboard server.""" | 5 """Uploads the results to the flakiness dashboard server.""" |
| 6 # pylint: disable=E1002,R0201 | 6 # pylint: disable=E1002,R0201 |
| 7 | 7 |
| 8 import logging | 8 import logging |
| 9 import os | 9 import os |
| 10 import shutil | 10 import shutil |
| 11 import tempfile | 11 import tempfile |
| 12 import xml | 12 import xml |
| 13 | 13 |
| 14 | 14 |
| 15 from devil.utils import cmd_helper | 15 from devil.utils import cmd_helper |
| 16 from pylib import constants | 16 from pylib.constants import host_paths |
| 17 from pylib.results.flakiness_dashboard import json_results_generator | 17 from pylib.results.flakiness_dashboard import json_results_generator |
| 18 from pylib.utils import repo_utils | 18 from pylib.utils import repo_utils |
| 19 | 19 |
| 20 | 20 |
| 21 | 21 |
| 22 class JSONResultsGenerator(json_results_generator.JSONResultsGeneratorBase): | 22 class JSONResultsGenerator(json_results_generator.JSONResultsGeneratorBase): |
| 23 """Writes test results to a JSON file and handles uploading that file to | 23 """Writes test results to a JSON file and handles uploading that file to |
| 24 the test results server. | 24 the test results server. |
| 25 """ | 25 """ |
| 26 def __init__(self, builder_name, build_name, build_number, tmp_folder, | 26 def __init__(self, builder_name, build_name, build_number, tmp_folder, |
| (...skipping 27 matching lines...) Expand all Loading... |
| 54 """ | 54 """ |
| 55 def _is_git_directory(in_directory): | 55 def _is_git_directory(in_directory): |
| 56 """Returns true if the given directory is in a git repository. | 56 """Returns true if the given directory is in a git repository. |
| 57 | 57 |
| 58 Args: | 58 Args: |
| 59 in_directory: The directory path to be tested. | 59 in_directory: The directory path to be tested. |
| 60 """ | 60 """ |
| 61 if os.path.exists(os.path.join(in_directory, '.git')): | 61 if os.path.exists(os.path.join(in_directory, '.git')): |
| 62 return True | 62 return True |
| 63 parent = os.path.dirname(in_directory) | 63 parent = os.path.dirname(in_directory) |
| 64 if parent == constants.DIR_SOURCE_ROOT or parent == in_directory: | 64 if parent == host_paths.DIR_SOURCE_ROOT or parent == in_directory: |
| 65 return False | 65 return False |
| 66 return _is_git_directory(parent) | 66 return _is_git_directory(parent) |
| 67 | 67 |
| 68 in_directory = os.path.join(constants.DIR_SOURCE_ROOT, in_directory) | 68 in_directory = os.path.join(host_paths.DIR_SOURCE_ROOT, in_directory) |
| 69 | 69 |
| 70 if not os.path.exists(os.path.join(in_directory, '.svn')): | 70 if not os.path.exists(os.path.join(in_directory, '.svn')): |
| 71 if _is_git_directory(in_directory): | 71 if _is_git_directory(in_directory): |
| 72 return repo_utils.GetGitHeadSHA1(in_directory) | 72 return repo_utils.GetGitHeadSHA1(in_directory) |
| 73 else: | 73 else: |
| 74 return '' | 74 return '' |
| 75 | 75 |
| 76 output = cmd_helper.GetCmdOutput(['svn', 'info', '--xml'], cwd=in_directory) | 76 output = cmd_helper.GetCmdOutput(['svn', 'info', '--xml'], cwd=in_directory) |
| 77 try: | 77 try: |
| 78 dom = xml.dom.minidom.parseString(output) | 78 dom = xml.dom.minidom.parseString(output) |
| (...skipping 13 matching lines...) Expand all Loading... |
| 92 if not self._build_number or not self._builder_name: | 92 if not self._build_number or not self._builder_name: |
| 93 raise Exception('You should not be uploading tests results to the server' | 93 raise Exception('You should not be uploading tests results to the server' |
| 94 'from your local machine.') | 94 'from your local machine.') |
| 95 | 95 |
| 96 upstream = (tests_type != 'Chromium_Android_Instrumentation') | 96 upstream = (tests_type != 'Chromium_Android_Instrumentation') |
| 97 if upstream: | 97 if upstream: |
| 98 # TODO(frankf): Use factory properties (see buildbot/bb_device_steps.py) | 98 # TODO(frankf): Use factory properties (see buildbot/bb_device_steps.py) |
| 99 # This requires passing the actual master name (e.g. 'ChromiumFYI' not | 99 # This requires passing the actual master name (e.g. 'ChromiumFYI' not |
| 100 # 'chromium.fyi'). | 100 # 'chromium.fyi'). |
| 101 from slave import slave_utils # pylint: disable=F0401 | 101 from slave import slave_utils # pylint: disable=F0401 |
| 102 self._build_name = slave_utils.SlaveBuildName(constants.DIR_SOURCE_ROOT) | 102 self._build_name = slave_utils.SlaveBuildName(host_paths.DIR_SOURCE_ROOT) |
| 103 self._master_name = slave_utils.GetActiveMaster() | 103 self._master_name = slave_utils.GetActiveMaster() |
| 104 else: | 104 else: |
| 105 self._build_name = 'chromium-android' | 105 self._build_name = 'chromium-android' |
| 106 buildbot_branch = os.environ.get('BUILDBOT_BRANCH') | 106 buildbot_branch = os.environ.get('BUILDBOT_BRANCH') |
| 107 if not buildbot_branch: | 107 if not buildbot_branch: |
| 108 buildbot_branch = 'master' | 108 buildbot_branch = 'master' |
| 109 else: | 109 else: |
| 110 # Ensure there's no leading "origin/" | 110 # Ensure there's no leading "origin/" |
| 111 buildbot_branch = buildbot_branch[buildbot_branch.find('/') + 1:] | 111 buildbot_branch = buildbot_branch[buildbot_branch.find('/') + 1:] |
| 112 self._master_name = '%s-%s' % (self._build_name, buildbot_branch) | 112 self._master_name = '%s-%s' % (self._build_name, buildbot_branch) |
| (...skipping 59 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... |
| 172 """Reports test results to the flakiness dashboard for Chrome for Android. | 172 """Reports test results to the flakiness dashboard for Chrome for Android. |
| 173 | 173 |
| 174 Args: | 174 Args: |
| 175 results: test results. | 175 results: test results. |
| 176 flakiness_dashboard_server: the server to upload the results to. | 176 flakiness_dashboard_server: the server to upload the results to. |
| 177 test_type: the type of the tests (as displayed by the flakiness dashboard). | 177 test_type: the type of the tests (as displayed by the flakiness dashboard). |
| 178 """ | 178 """ |
| 179 uploader = ResultsUploader(test_type) | 179 uploader = ResultsUploader(test_type) |
| 180 uploader.AddResults(results) | 180 uploader.AddResults(results) |
| 181 uploader.Upload(flakiness_dashboard_server) | 181 uploader.Upload(flakiness_dashboard_server) |
| OLD | NEW |