[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]

Re: [libvirt] [PATCH] Improve error reporting in test suites



On 10/13/2010 08:45 AM, Eric Blake wrote:
On 10/13/2010 05:11 AM, Daniel P. Berrange wrote:
Before running each test case clear the thread local error
indicator. After running each test case, dispatch any error
that was reported

* tests/testutils.c: Fix error reporting in test suites
---
tests/testutils.c | 4 ++++
1 files changed, 4 insertions(+), 0 deletions(-)

ACK.

Hmm, this adds a lot of noise to 'make check':

TEST: qemuxml2argvtest
..libvir: QEMU error : unsupported configuration: the QEMU binary /usr/bin/kvm does not support kvm ......libvir: QEMU error : unsupported configuration: the QEMU binary /usr/bin/xenner does not support kvm
................................ 40
.....libvir: QEMU error : unsupported configuration: the QEMU binary /usr/bin/xenner does not support kvm

...
TEST: nwfilterxml2xmltest
libvir: Network Filtererror : internal error protocolid has illegal value 15
.libvir: Network Filtererror : internal error opcode has illegal value 65536
.libvir: Network Filtererror : internal error opcode has illegal value 65536
.libvir: Network Filtererror : internal error dscp has illegal value 64
...


It doesn't add any additional failures, but looks ugly. We should probably do something similar to 9e3525df8, by using virtTestLogContentAndReset() in appropriate places to silence tests where we expect an error to be logged, rather than leaking the errors out the testsuite.

--
Eric Blake   eblake redhat com    +1-801-349-2682
Libvirt virtualization library http://libvirt.org


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]