x
/
driverlog
Code Issues Proposed changes

Bump hacking to version 0.9.2 

Change-Id: I58677d1ed5b736a6f4395120f4dbcdb6811fdf3d
This commit is contained in:
Christian Berendt 2014-07-16 16:58:55 +02:00
parent f092e356ac
commit 416702b7b4
4 changed files with 11 additions and 15 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

16
driverlog/processor/main.py
View File

@ -29,9 +29,7 @@ LOG = logging.getLogger(__name__)
def _find_vote(review, ci_id): def _find_vote(review, ci_id):
""" """Finds vote corresponding to ci_id."""
Finds vote corresponding to ci_id
"""
for approval in (review['currentPatchSet'].get('approvals') or []): for approval in (review['currentPatchSet'].get('approvals') or []):
if approval['type'] not in ['Verified', 'VRIF']: if approval['type'] not in ['Verified', 'VRIF']:
continue continue
@ -43,9 +41,7 @@ def _find_vote(review, ci_id):
def find_ci_result(review_iterator, ci): def find_ci_result(review_iterator, ci):
""" """For a given stream of reviews finds result left by specified ci."""
For a given stream of reviews finds result left by specified ci
"""
for review in review_iterator: for review in review_iterator:
review_url = review['url'] review_url = review['url']
@ -85,9 +81,7 @@ def find_ci_result(review_iterator, ci):
def _get_release_by_branch(releases, branch): def _get_release_by_branch(releases, branch):
""" """Translates branch name into release_id."""
Translates branch name into release_id
"""
release = branch.lower() release = branch.lower()
if release.find('/') > 0: if release.find('/') > 0:
return release.split('/')[1] return release.split('/')[1]
@ -96,8 +90,8 @@ def _get_release_by_branch(releases, branch):
def update_drivers(drivers, releases): def update_drivers(drivers, releases):
""" """Iterates all drivers and searches for results produced by their CIs.
Iterates all drivers and searches for results produced by their CIs
Returns True if info was updated Returns True if info was updated
""" """
branches = [('stable/' + r['id'].lower()) for r in releases] + ['master'] branches = [('stable/' + r['id'].lower()) for r in releases] + ['master']

2
test-requirements.txt
View File

@ -1,4 +1,4 @@
hacking>=0.8.0,<0.9 hacking>=0.9.2,<0.10
coverage>=3.6 coverage>=3.6
discover discover

4
tests/unit/test_main.py
View File

@ -14,12 +14,12 @@
# limitations under the License. # limitations under the License.
import json import json
import memcache
import mock
from driverlog.processor import main from driverlog.processor import main
from driverlog.processor import utils from driverlog.processor import utils
import memcache
import mock
import testtools import testtools

4
tox.ini
View File

@ -34,7 +34,9 @@ commands = python setup.py build_sphinx
[flake8] [flake8]
# E125 continuation line does not distinguish itself from next logical line # E125 continuation line does not distinguish itself from next logical line
# H404 multi line docstring should start with a summary # H404 multi line docstring should start with a summary
ignore = E125,H404 # E251 unexpected spaces around keyword / parameter equals
# E714 test for object identity should be 'is not'
ignore = E125,H404,E251,E714
show-source = true show-source = true
builtins = _ builtins = _
exclude=.venv,.git,.tox,dist,doc,*lib/python*,*egg,tools,build exclude=.venv,.git,.tox,dist,doc,*lib/python*,*egg,tools,build