diff options
author | Richard Purdie <richard.purdie@linuxfoundation.org> | 2017-02-03 10:30:59 +0000 |
---|---|---|
committer | Richard Purdie <richard.purdie@linuxfoundation.org> | 2017-02-05 09:20:22 +0000 |
commit | b6352ff001c29f0bff10c18879b92c5618ec645c (patch) | |
tree | 50e0a69fa900e9bb9d66e2db3c2ea006b2a30c77 /meta/lib | |
parent | f61cdef0a8b2771225c6bc86881a16f8ef747983 (diff) | |
download | openembedded-core-b6352ff001c29f0bff10c18879b92c5618ec645c.tar.gz openembedded-core-b6352ff001c29f0bff10c18879b92c5618ec645c.tar.bz2 openembedded-core-b6352ff001c29f0bff10c18879b92c5618ec645c.zip |
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.
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
Diffstat (limited to 'meta/lib')
-rw-r--r-- | meta/lib/oeqa/runtime/cases/buildcvs.py | 11 | ||||
-rw-r--r-- | meta/lib/oeqa/runtime/cases/buildgalculator.py | 7 | ||||
-rw-r--r-- | meta/lib/oeqa/runtime/cases/buildiptables.py | 11 | ||||
-rw-r--r-- | meta/lib/oeqa/runtime/utils/targetbuildproject.py | 6 |
4 files changed, 13 insertions, 22 deletions
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] |