From 6840618f1a718d3aa8fd9e7e4fd3ffde01559e7d Mon Sep 17 00:00:00 2001 From: Richard Purdie Date: Fri, 3 Feb 2017 10:30:59 +0000 Subject: oeqa/runtime: Improve failure log output Printing a message which says "configure failed" without the log output is effectively useless. If a command fails, print the output by default and simplify the calling code which makes debugging any of these failures much easier. (From OE-Core rev: b6352ff001c29f0bff10c18879b92c5618ec645c) Signed-off-by: Richard Purdie --- meta/lib/oeqa/runtime/cases/buildcvs.py | 11 +++-------- meta/lib/oeqa/runtime/cases/buildgalculator.py | 7 ++----- meta/lib/oeqa/runtime/cases/buildiptables.py | 11 +++-------- meta/lib/oeqa/runtime/utils/targetbuildproject.py | 6 +++++- 4 files changed, 13 insertions(+), 22 deletions(-) (limited to 'meta') diff --git a/meta/lib/oeqa/runtime/cases/buildcvs.py b/meta/lib/oeqa/runtime/cases/buildcvs.py index 341eb4959d..c3f3acc736 100644 --- a/meta/lib/oeqa/runtime/cases/buildcvs.py +++ b/meta/lib/oeqa/runtime/cases/buildcvs.py @@ -25,11 +25,6 @@ class BuildCvsTest(OERuntimeTestCase): 'Test requires tools-sdk to be in IMAGE_FEATURES') @OETestDepends(['ssh.SSHTest.test_ssh']) def test_cvs(self): - self.assertEqual(self.project.run_configure(), 0, - msg="Running configure failed") - - self.assertEqual(self.project.run_make(), 0, - msg="Running make failed") - - self.assertEqual(self.project.run_install(), 0, - msg="Running make install failed") + self.project.run_configure() + self.project.run_make() + self.project.run_install() diff --git a/meta/lib/oeqa/runtime/cases/buildgalculator.py b/meta/lib/oeqa/runtime/cases/buildgalculator.py index 0bd76f9ea2..7c9d4a392b 100644 --- a/meta/lib/oeqa/runtime/cases/buildgalculator.py +++ b/meta/lib/oeqa/runtime/cases/buildgalculator.py @@ -24,8 +24,5 @@ class GalculatorTest(OERuntimeTestCase): 'Test requires tools-sdk to be in IMAGE_FEATURES') @OETestDepends(['ssh.SSHTest.test_ssh']) def test_galculator(self): - self.assertEqual(self.project.run_configure(), 0, - msg="Running configure failed") - - self.assertEqual(self.project.run_make(), 0, - msg="Running make failed") + self.project.run_configure() + self.project.run_make() diff --git a/meta/lib/oeqa/runtime/cases/buildiptables.py b/meta/lib/oeqa/runtime/cases/buildiptables.py index bae80392d9..002b16c483 100644 --- a/meta/lib/oeqa/runtime/cases/buildiptables.py +++ b/meta/lib/oeqa/runtime/cases/buildiptables.py @@ -25,14 +25,9 @@ class BuildIptablesTest(OERuntimeTestCase): 'Test requires tools-sdk to be in IMAGE_FEATURES') @OETestDepends(['ssh.SSHTest.test_ssh']) def test_iptables(self): - self.assertEqual(self.project.run_configure(), 0, - msg="Running configure failed") - - self.assertEqual(self.project.run_make(), 0, - msg="Running make failed") - - self.assertEqual(self.project.run_install(), 0, - msg="Running make install failed") + self.project.run_configure() + self.project.run_make() + self.project.run_install() @classmethod def tearDownClass(self): diff --git a/meta/lib/oeqa/runtime/utils/targetbuildproject.py b/meta/lib/oeqa/runtime/utils/targetbuildproject.py index 1ed5789ae5..551b0b6f27 100644 --- a/meta/lib/oeqa/runtime/utils/targetbuildproject.py +++ b/meta/lib/oeqa/runtime/utils/targetbuildproject.py @@ -33,4 +33,8 @@ class TargetBuildProject(BuildProject): # The timeout parameter of target.run is set to 0 # to make the ssh command run with no timeout. def _run(self, cmd): - return self.target.run(cmd, 0)[0] + ret = self.target.run(cmd, 0) + msg = "Command %s failed with exit code %s: %s" % (cmd, ret[0], ret[1]) + if ret[0] != 0: + raise Exception(msg) + return ret[0] -- cgit 1.2.3-korg