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

[Eric Blake]

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