Merge pull request #2847 from coolo/fix_group_bugowner
Ignore starting white space in bugowner lines
This commit is contained in:
commit
e6d11c4dc3
@ -31,10 +31,10 @@ class CheckerBugowner(ReviewBot.ReviewBot):
|
|||||||
for line in self.request.description.splitlines():
|
for line in self.request.description.splitlines():
|
||||||
matched_package = None
|
matched_package = None
|
||||||
matched_maintainer = None
|
matched_maintainer = None
|
||||||
m = re.match(r'bugowner:\s*(\S*)\s*$', line)
|
m = re.match(r'\s*bugowner:\s*(\S*)\s*$', line)
|
||||||
if m:
|
if m:
|
||||||
matched_maintainer = m.group(1)
|
matched_maintainer = m.group(1)
|
||||||
m = re.match(r'bugowner:\s(\S*)\s(\S*)\s*$', line)
|
m = re.match(r'\s*bugowner:\s(\S*)\s(\S*)\s*$', line)
|
||||||
if m:
|
if m:
|
||||||
matched_maintainer = m.group(2)
|
matched_maintainer = m.group(2)
|
||||||
matched_package = m.group(1)
|
matched_package = m.group(1)
|
||||||
|
@ -90,7 +90,7 @@ class TestCheckBugowner(OBSLocal.TestCase):
|
|||||||
|
|
||||||
self.assertReview(req_id, by_user=(self.bot_user, 'new'))
|
self.assertReview(req_id, by_user=(self.bot_user, 'new'))
|
||||||
|
|
||||||
self.review_bot.set_request_ids([req_id])
|
self.review_bot.set_request_ids_search_review()
|
||||||
self.review_bot.check_requests()
|
self.review_bot.check_requests()
|
||||||
|
|
||||||
self.assertReview(req_id, by_user=(self.bot_user, 'accepted'))
|
self.assertReview(req_id, by_user=(self.bot_user, 'accepted'))
|
||||||
|
Loading…
x
Reference in New Issue
Block a user