cmueller/ansible
1
0
Fork 0
Code Issues Pull requests Projects Releases Wiki Activity

Fix ansible-test timeout junit output. (#53427)

Browse source 
Fix ansible-test timeout junit output.

Implemented a hack to work around not having junit-xml when running on Shippable outside of our test containers.
This commit is contained in:
Matt Clay 2019-03-06 19:46:41 -08:00 committed by GitHub
parent 6e168776db
commit 3d2c0f4973
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
1 changed files with 21 additions and 9 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

30
test/runner/lib/test.py
View file

@ -57,17 +57,15 @@ def calculate_confidence(path, line, metadata):
class TestResult(object): class TestResult(object):
"""Base class for test results.""" """Base class for test results."""
def __init__(self, command, test, python_version=None, enable_junit=False): def __init__(self, command, test, python_version=None):
""" """
:type command: str :type command: str
:type test: str :type test: str
:type python_version: str :type python_version: str
:type enable_junit: bool
""" """
self.command = command self.command = command
self.test = test self.test = test
self.python_version = python_version self.python_version = python_version
self.enable_junit = enable_junit
self.name = self.test or self.command self.name = self.test or self.command
if self.python_version: if self.python_version:
@ -90,7 +88,7 @@ class TestResult(object):
if args.lint: if args.lint:
self.write_lint() self.write_lint()
if args.junit or self.enable_junit: if args.junit:
if self.junit: if self.junit:
self.write_junit(args) self.write_junit(args)
else: else:
@ -167,11 +165,11 @@ class TestTimeout(TestResult):
""" """
:type timeout_duration: int :type timeout_duration: int
""" """
super(TestTimeout, self).__init__(command='timeout', test='', enable_junit=True) super(TestTimeout, self).__init__(command='timeout', test='')
self.timeout_duration = timeout_duration self.timeout_duration = timeout_duration
def write_junit(self, args): def write(self, args):
""" """
:type args: TestConfig :type args: TestConfig
""" """
@ -187,10 +185,24 @@ class TestTimeout(TestResult):
output += '\n\nConsult the console log for additional details on where the timeout occurred.' output += '\n\nConsult the console log for additional details on where the timeout occurred.'
test_case = self.junit.TestCase(classname=self.command, name=self.name) timestamp = datetime.datetime.utcnow().replace(microsecond=0).isoformat()
test_case.add_error_info(message, output)
self.save_junit(args, test_case) # hack to avoid requiring junit-xml, which isn't pre-installed on Shippable outside our test containers
xml = '''
<?xml version="1.0" encoding="utf-8"?>
<testsuites disabled="0" errors="1" failures="0" tests="1" time="0.0">
\t<testsuite disabled="0" errors="1" failures="0" file="None" log="None" name="ansible-test" skipped="0" tests="1" time="0" timestamp="%s" url="None">
\t\t<testcase classname="timeout" name="timeout">
\t\t\t<error message="%s" type="error">%s</error>
\t\t</testcase>
\t</testsuite>
</testsuites>
''' % (timestamp, message, output)
path = self.create_path('junit', '.xml')
with open(path, 'w') as junit_fd:
junit_fd.write(xml)
class TestSuccess(TestResult): class TestSuccess(TestResult):