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>
This commit is contained in:
Richard Purdie 2018-07-14 14:50:48 +00:00
parent eacc12c2ec
commit 6b219f64c4
6 changed files with 1 additions and 13 deletions

View File

@ -59,7 +59,7 @@ class OETestContext(object):
self.suites = self.loader.discover()
def runTests(self, skips=[]):
self.runner = self.runnerClass(self, descriptions=False, verbosity=2)
self.runner = self.runnerClass(self, descriptions=False, verbosity=2, buffer=True)
# Dinamically skip those tests specified though arguments
self.skipTests(skips)

View File

@ -48,11 +48,6 @@ class OETestResult(_TestResult):
self._tc_map_results()
def startTest(self, test):
# Allow us to trigger the testcase buffer mode on a per test basis
# so stdout/stderr are only printed upon failure. Enables debugging
# but clean output
if hasattr(test, "buffer"):
self.buffer = test.buffer
super(OETestResult, self).startTest(test)
def _tc_map_results(self):

View File

@ -13,8 +13,6 @@ from oeqa.core.decorator.oeid import OETestID
class DevtoolBase(OESelftestTestCase):
buffer = True
def _test_recipe_contents(self, recipefile, checkvars, checkinherits):
with open(recipefile, 'r') as f:
invar = None

View File

@ -11,7 +11,6 @@ from oeqa.utils.commands import bitbake, runqemu, get_bb_var
class GenericEFITest(OESelftestTestCase):
"""EFI booting test class"""
buffer = True
cmd_common = "runqemu nographic serial wic ovmf"
efi_provider = "systemd-boot"
image = "core-image-minimal"

View File

@ -10,8 +10,6 @@ class ImageFeatures(OESelftestTestCase):
test_user = 'tester'
root_user = 'root'
buffer = True
@OETestID(1107)
def test_non_root_user_can_connect_via_ssh_without_password(self):
"""

View File

@ -14,8 +14,6 @@ class RunqemuTests(OESelftestTestCase):
image_is_ready = False
deploy_dir_image = ''
# We only want to print runqemu stdout/stderr if there is a test case failure
buffer = True
def setUpLocal(self):
super(RunqemuTests, self).setUpLocal()