diff options
author | Richard Purdie <richard.purdie@linuxfoundation.org> | 2018-07-14 14:50:48 +0000 |
---|---|---|
committer | Richard Purdie <richard.purdie@linuxfoundation.org> | 2018-07-16 13:21:35 +0100 |
commit | 6b219f64c403361926a464925f0a32b2b3d27ffc (patch) | |
tree | 535e1d443737f3fc94cbbd1ec0c48806d3e23c5f /meta/lib/oeqa/core | |
parent | eacc12c2ec6d778c70f91f8c754f64c6c097f35a (diff) | |
download | poky-6b219f64c403361926a464925f0a32b2b3d27ffc.tar.gz |
oeqa: Default to buffer mode for tests
Currently some tests run in buffer mode and some don't. Those that don't can
corrupt stdout/stderr. Switch to using buffer mode everywhere so we're consistent.
If there is useful output on stdout/stderr, it will be displayed if the test
fails.
(From OE-Core rev: 978548c0abde2cb94c2782538552f39bdf2bf630)
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
Diffstat (limited to 'meta/lib/oeqa/core')
-rw-r--r-- | meta/lib/oeqa/core/context.py | 2 | ||||
-rw-r--r-- | meta/lib/oeqa/core/runner.py | 5 |
2 files changed, 1 insertions, 6 deletions
diff --git a/meta/lib/oeqa/core/context.py b/meta/lib/oeqa/core/context.py index acd547416f..10481b44b6 100644 --- a/meta/lib/oeqa/core/context.py +++ b/meta/lib/oeqa/core/context.py | |||
@@ -59,7 +59,7 @@ class OETestContext(object): | |||
59 | self.suites = self.loader.discover() | 59 | self.suites = self.loader.discover() |
60 | 60 | ||
61 | def runTests(self, skips=[]): | 61 | def runTests(self, skips=[]): |
62 | self.runner = self.runnerClass(self, descriptions=False, verbosity=2) | 62 | self.runner = self.runnerClass(self, descriptions=False, verbosity=2, buffer=True) |
63 | 63 | ||
64 | # Dinamically skip those tests specified though arguments | 64 | # Dinamically skip those tests specified though arguments |
65 | self.skipTests(skips) | 65 | self.skipTests(skips) |
diff --git a/meta/lib/oeqa/core/runner.py b/meta/lib/oeqa/core/runner.py index 374d30cc38..219102c6b0 100644 --- a/meta/lib/oeqa/core/runner.py +++ b/meta/lib/oeqa/core/runner.py | |||
@@ -48,11 +48,6 @@ class OETestResult(_TestResult): | |||
48 | self._tc_map_results() | 48 | self._tc_map_results() |
49 | 49 | ||
50 | def startTest(self, test): | 50 | def startTest(self, test): |
51 | # Allow us to trigger the testcase buffer mode on a per test basis | ||
52 | # so stdout/stderr are only printed upon failure. Enables debugging | ||
53 | # but clean output | ||
54 | if hasattr(test, "buffer"): | ||
55 | self.buffer = test.buffer | ||
56 | super(OETestResult, self).startTest(test) | 51 | super(OETestResult, self).startTest(test) |
57 | 52 | ||
58 | def _tc_map_results(self): | 53 | def _tc_map_results(self): |