Readability cleanups
This commit is contained in:
parent
8f1b936795
commit
3e23f6dee2
@ -424,17 +424,16 @@ class ToTestBase(object):
|
|||||||
url = self.api.makeurl(['build', project, '_result'], query)
|
url = self.api.makeurl(['build', project, '_result'], query)
|
||||||
f = self.api.retried_GET(url)
|
f = self.api.retried_GET(url)
|
||||||
root = ET.parse(f).getroot()
|
root = ET.parse(f).getroot()
|
||||||
succeeded = root.findall('result/status[@code="succeeded"]')
|
|
||||||
# [@code!='succeeded'] is not supported by ET
|
# [@code!='succeeded'] is not supported by ET
|
||||||
failed = [status for status in root.findall("result/status") if status.get('code') != 'succeeded']
|
failed = [status for status in root.findall("result/status") if status.get('code') != 'succeeded']
|
||||||
|
|
||||||
if len(failed) > 0:
|
if any(failed):
|
||||||
logger.info(
|
logger.info(
|
||||||
'%s %s %s %s -> %s' % (project, package, repository, arch, failed[0].get('code')))
|
'%s %s %s %s -> %s' % (project, package, repository, arch, failed[0].get('code')))
|
||||||
return False
|
return False
|
||||||
|
|
||||||
if len(succeeded) == 0:
|
if not len(root.findall('result/status[@code="succeeded"]')):
|
||||||
logger.info('No results for %s %s %s %s' % (project, package, repository, arch))
|
logger.info('No "succeeded" for %s %s %s %s' % (project, package, repository, arch))
|
||||||
return False
|
return False
|
||||||
|
|
||||||
maxsize = self.maxsize_for_package(package)
|
maxsize = self.maxsize_for_package(package)
|
||||||
|
Loading…
x
Reference in New Issue
Block a user