Re: [PATCH] openvz: Add simple test for openvzReadNetworkConf

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

 



On 05/31/2011 07:11 AM, Matthias Bolte wrote:
> Convert openvzLocateConfFile to a replaceable callback to allow
> testing the config file parsing without rewriting the whole OpenVZ
> config parsing to a more testable structure.
> 
> Also ignore the openvzutilstest binary.

Serves me right for pushing that .gitignore fix before finishing reading
my inbox...

> ---
>  src/openvz/openvz_conf.c   |   28 +++++++++++----
>  src/openvz/openvz_conf.h   |    6 +++
>  tests/.gitignore           |    1 +
>  tests/openvzutilstest.c    |   78 ++++++++++++++++++++++++++++++++++++++++++++
>  tests/openvzutilstest.conf |    1 +
>  5 files changed, 106 insertions(+), 8 deletions(-)
> 
> +++ b/tests/.gitignore
> @@ -16,6 +16,7 @@ object-locking
>  object-locking-files.txt
>  object-locking.cmi
>  object-locking.cmx
> +openvzutilstest

I'd rather move towards having more exceptions in the top-level
.gitignore, and fewer lines here in the directory-specific
tests/.gitignore (for that matter, since .gitignore is hierarchical, I
would eventually like to get rid of all the child .gitignore files and
put everything in the top level).  But that's a cleanup for another day.
 Meanwhile, this hunk is no longer necessary, thanks to commit ad962bc.

ACK to the rest of the patch; more testsuite exposure is always okay,
even after feature freeze.

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

Attachment: signature.asc
Description: OpenPGP digital signature

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

[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]