[libvirt] [PATCH 2/3] tests: Only use privileged mode if Qemu user and group exists

[Eric Blake]

On 04/08/2014 07:06 AM, Guido Günther wrote:

>>> What about passing 'false' to ConfigNew() but then manually
>>> set  'cfg->privileged = true' on the object we get back.
>>>
>>
>> That could work.  All tests passed on my setup like that.  And it
>> doesn't seem weird since we're playing with the config a lot in the
>> tests.
> 
> I do wonder if my approach wouldn't be cleaner since it doesn't poke
> into the objects internals.

Your approach skips the test, which means less testsuite coverage.  I'd
rather poke into internals to make the testsuite independent of the
environment while still maximizing coverage.

-- 
Eric Blake   eblake redhat com    +1-919-301-3266
Libvirt virtualization library http://libvirt.org

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 604 bytes
Desc: OpenPGP digital signature
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20140408/f02a8c07/attachment-0001.sig>