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(a)redhat.com +1-801-349-2682
Libvirt virtualization library http://libvirt.org