[et-mgmt-tools] FYI: Changes to virtinst test suite
Cole Robinson
crobinso at redhat.com
Mon Nov 24 22:24:45 UTC 2008
Hi all,
I've added some changes to the virtinst test suite that will affect
anyone writing patches. Two commands have been added to the setup.py script:
python setup.py check : runs a pylint wrapper against the
codebase
python setup.py test_urls: runs a url fetching test suite
A patch shouldn't change the output of 'check'. If you hit something
which you think is a false positive or an unnecessary check, bring it up
on the list (though I've tried to remove all the blatantly useless
checks already). 'check' uses the pylint utility.
'test_urls' only needs to be run if changes are against the url fetching
code (virtinst/OSDistro and some virtinst/Distromanager). It takes a
while to run and requires a network connection, so I didn't include it
as part of the regular test suite ('setup.py test', which should also be
run with any changes).
All the above info was added to a HACKING file as well. I'll also be
adding a similar pylint script to virt-manager in the near future.
Thanks,
Cole
More information about the et-mgmt-tools
mailing list