Re: [libvirt] [PATCH] tests: Centralize VIR_TEST_DEBUG lookup, and document it

[Cole Robinson <crobinso@xxxxxxxxxx>]

On 10/17/2009 08:03 AM, Matthias Bolte wrote:
> 2009/10/16 Cole Robinson <crobinso@xxxxxxxxxx>:
>> Provide a simple interface for other tests to lookup the testDebug variable.
>> Also remove a redundant error message in interface tests.
>>
>> If anyone feels inclined to change this env variable to match the existing
>> LIBVIRT_* format, it should now be easier to do so.
>>
>> Signed-off-by: Cole Robinson <crobinso@xxxxxxxxxx>
>> ---
>> Â HACKING Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â | Â  10 ++++++++++
>> Â tests/interfacexml2xmltest.c | Â  Â 2 --
>>  tests/statstest.c            |    4 ++--
>>  tests/testutils.c            |   38 ++++++++++++++++++++++++++------------
>>  tests/testutils.h            |    4 ++--
>>  tests/testutilsqemu.c        |    2 +-
>> Â 6 files changed, 41 insertions(+), 19 deletions(-)
>>
>> diff --git a/HACKING b/HACKING
>> index bcff8c6..fba7778 100644
>> --- a/HACKING
>> +++ b/HACKING
>> @@ -37,6 +37,16 @@ and run the tests:
>>
>> Â The latter test checks for memory leaks.
>>
>> +If you encounter any failing tests, the VIR_TEST_DEBUG environment variable
>> +may help:
>> +
>> +  VIR_TEST_DEBUG=1 make check    (or)
>> + Â VIR_TEST_DEBUG=2 make check
> 
> Maybe explain what VIR_TEST_DEBUG actually does and what's the
> difference between 1 and 2.
> 

Yes, this wording can be improved, I will fix it and repost.

>> +
>> +Also, individual tests can be run from inside the 'tests/' directory, like:
>> +
>> + Â ./qemuxml2xmltest
>> +
>> Â (6) Update tests and/or documentation, particularly if you are adding
>> Â a new feature or changing the output of a program.
> 
>> --- a/tests/testutils.c
>> +++ b/tests/testutils.c
>> @@ -45,7 +45,7 @@
>> Â  Â  ((((int) ((T)->tv_sec - (U)->tv_sec)) * 1000000.0 + Â  Â  Â  Â \
>> Â  Â  Â  ((int) ((T)->tv_usec - (U)->tv_usec))) / 1000.0)
>>
>> -unsigned int testDebug = 0;
>> +unsigned int testDebug = -1;
> 
> testDebug isn't referenced via extern anymore, declare it static like
> testOOM and testCounter.
>

Will do.

>> Â static unsigned int testOOM = 0;
>> Â static unsigned int testCounter = 0;
> 
> ACK.
>

Thanks for the review!

- Cole

--
Libvir-list mailing list
Libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list