diff options
| author | Richard Purdie <richard.purdie@linuxfoundation.org> | 2018-07-13 13:13:10 +0100 |
|---|---|---|
| committer | Richard Purdie <richard.purdie@linuxfoundation.org> | 2018-07-18 10:18:41 +0100 |
| commit | f5975278e3d35b02983217b832975cfde3ad9a9b (patch) | |
| tree | 0ae28f9f114037b908ee1885170e4eabc9a23351 | |
| parent | d9de34ce9906399ae9845d0434e2b2b1642bf46a (diff) | |
| download | poky-f5975278e3d35b02983217b832975cfde3ad9a9b.tar.gz | |
oeqa: Remove xmlrunner
This isn't present on modern distros by default and doesn't work with
testtools, needing multiple code paths in the code. Remove it in favour
of finding a better replacement for results collection/analysis.
(From OE-Core rev: 8001d933a8dc86004db014777f094d718086687d)
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
| -rw-r--r-- | meta/lib/oeqa/core/runner.py | 54 |
1 files changed, 9 insertions, 45 deletions
diff --git a/meta/lib/oeqa/core/runner.py b/meta/lib/oeqa/core/runner.py index 0f84a1d280..6650a63ce7 100644 --- a/meta/lib/oeqa/core/runner.py +++ b/meta/lib/oeqa/core/runner.py | |||
| @@ -7,16 +7,8 @@ import unittest | |||
| 7 | import logging | 7 | import logging |
| 8 | import re | 8 | import re |
| 9 | 9 | ||
| 10 | xmlEnabled = False | 10 | from unittest import TextTestResult as _TestResult |
| 11 | try: | 11 | from unittest import TextTestRunner as _TestRunner |
| 12 | import xmlrunner | ||
| 13 | from xmlrunner.result import _XMLTestResult as _TestResult | ||
| 14 | from xmlrunner.runner import XMLTestRunner as _TestRunner | ||
| 15 | xmlEnabled = True | ||
| 16 | except ImportError: | ||
| 17 | # use the base runner instead | ||
| 18 | from unittest import TextTestResult as _TestResult | ||
| 19 | from unittest import TextTestRunner as _TestRunner | ||
| 20 | 12 | ||
| 21 | class OEStreamLogger(object): | 13 | class OEStreamLogger(object): |
| 22 | def __init__(self, logger): | 14 | def __init__(self, logger): |
| @@ -84,19 +76,10 @@ class OETestResult(_TestResult): | |||
| 84 | found = False | 76 | found = False |
| 85 | 77 | ||
| 86 | for (scase, msg) in getattr(self, type): | 78 | for (scase, msg) in getattr(self, type): |
| 87 | # XXX: When XML reporting is enabled scase is | 79 | if case.id() == scase.id(): |
| 88 | # xmlrunner.result._TestInfo instance instead of | 80 | found = True |
| 89 | # string. | 81 | break |
| 90 | if xmlEnabled: | 82 | scase_str = str(scase.id()) |
| 91 | if case.id() == scase.test_id: | ||
| 92 | found = True | ||
| 93 | break | ||
| 94 | scase_str = scase.test_id | ||
| 95 | else: | ||
| 96 | if case.id() == scase.id(): | ||
| 97 | found = True | ||
| 98 | break | ||
| 99 | scase_str = str(scase.id()) | ||
| 100 | 83 | ||
| 101 | # When fails at module or class level the class name is passed as string | 84 | # When fails at module or class level the class name is passed as string |
| 102 | # so figure out to see if match | 85 | # so figure out to see if match |
| @@ -167,33 +150,14 @@ class OETestRunner(_TestRunner): | |||
| 167 | streamLoggerClass = OEStreamLogger | 150 | streamLoggerClass = OEStreamLogger |
| 168 | 151 | ||
| 169 | def __init__(self, tc, *args, **kwargs): | 152 | def __init__(self, tc, *args, **kwargs): |
| 170 | if xmlEnabled: | ||
| 171 | if not kwargs.get('output'): | ||
| 172 | kwargs['output'] = os.path.join(os.getcwd(), | ||
| 173 | 'TestResults_%s_%s' % (time.strftime("%Y%m%d%H%M%S"), os.getpid())) | ||
| 174 | |||
| 175 | kwargs['stream'] = self.streamLoggerClass(tc.logger) | 153 | kwargs['stream'] = self.streamLoggerClass(tc.logger) |
| 176 | super(OETestRunner, self).__init__(*args, **kwargs) | 154 | super(OETestRunner, self).__init__(*args, **kwargs) |
| 177 | self.tc = tc | 155 | self.tc = tc |
| 178 | self.resultclass = OETestResult | 156 | self.resultclass = OETestResult |
| 179 | 157 | ||
| 180 | # XXX: The unittest-xml-reporting package defines _make_result method instead | 158 | def _makeResult(self): |
| 181 | # of _makeResult standard on unittest. | 159 | return self.resultclass(self.tc, self.stream, self.descriptions, |
| 182 | if xmlEnabled: | 160 | self.verbosity) |
| 183 | def _make_result(self): | ||
| 184 | """ | ||
| 185 | Creates a TestResult object which will be used to store | ||
| 186 | information about the executed tests. | ||
| 187 | """ | ||
| 188 | # override in subclasses if necessary. | ||
| 189 | return self.resultclass(self.tc, | ||
| 190 | self.stream, self.descriptions, self.verbosity, self.elapsed_times | ||
| 191 | ) | ||
| 192 | else: | ||
| 193 | def _makeResult(self): | ||
| 194 | return self.resultclass(self.tc, self.stream, self.descriptions, | ||
| 195 | self.verbosity) | ||
| 196 | |||
| 197 | 161 | ||
| 198 | def _walk_suite(self, suite, func): | 162 | def _walk_suite(self, suite, func): |
| 199 | for obj in suite: | 163 | for obj in suite: |
