Merge pull request #1555 from coolo/quote_plus
openqabot: Quote the module name
This commit is contained in:
commit
b621775b8e
@ -4,6 +4,7 @@ from datetime import date
|
|||||||
import md5
|
import md5
|
||||||
from pprint import pformat
|
from pprint import pformat
|
||||||
from urllib2 import HTTPError
|
from urllib2 import HTTPError
|
||||||
|
from urllib import quote_plus
|
||||||
from osclib.comments import CommentAPI
|
from osclib.comments import CommentAPI
|
||||||
|
|
||||||
import requests
|
import requests
|
||||||
@ -298,7 +299,7 @@ class OpenQABot(ReviewBot.ReviewBot):
|
|||||||
|
|
||||||
@staticmethod
|
@staticmethod
|
||||||
def get_step_url(testurl, modulename):
|
def get_step_url(testurl, modulename):
|
||||||
failurl = testurl + '/modules/{!s}/fails'.format(modulename)
|
failurl = testurl + '/modules/{!s}/fails'.format(quote_plus(modulename))
|
||||||
fails = requests.get(failurl).json()
|
fails = requests.get(failurl).json()
|
||||||
failed_step = fails.get('first_failed_step', 1)
|
failed_step = fails.get('first_failed_step', 1)
|
||||||
return "[{!s}]({!s}#step/{!s}/{:d})".format(OpenQABot.emd(modulename), testurl, modulename, failed_step)
|
return "[{!s}]({!s}#step/{!s}/{:d})".format(OpenQABot.emd(modulename), testurl, modulename, failed_step)
|
||||||
|
Loading…
x
Reference in New Issue
Block a user