Merge pull request #124 from scarabeusiv/master

Checking only reviews for *Staging* sounds reasonable in this part of code
This commit is contained in:
Michal Hrušecký 2014-03-28 12:35:29 +01:00
commit 4bd1b661e4

View File

@ -56,6 +56,10 @@ class RequestFinder(object):
:param element: XML with list of reviews
"""
reviews = self._filter_review_by_project(element, 'new')
# First filter out reviews to contain only STAGING ones
# NOTE: this is hardcoded path to projects so usage elsewhere
# will need adjustements
reviews = [prj for prj in reviews if prj.startswith('openSUSE:Factory:Staging:')]
assert len(reviews) <= 1, 'Request "{}" have multiple review by project in new state "{}"'.format(request_id,
reviews)
return reviews[0] if reviews else None