<div class="gmail_quote">On Mon, Nov 22, 2010 at 9:33 AM, Jay Pipes <span dir="ltr"><<a href="mailto:jaypipes@gmail.com">jaypipes@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">

I would prefer option #1 and to get rid of run_tests.py entirely.<br>
With nose, it's not useful at all.  So, I would change the<br>
run_tests.sh script to run nosetests instead of run_tests.py...this is<br></blockquote><div><br></div><div>I vote we also get rid of run_tests.sh and the unit test base class, so we aren't tempted to add dependencies required for nova tests to run.  Running tests then becomes typing "nosetests" in nova/tests.</div>

<div><br></div><div>Argument: if run_tests.sh accretes complexity, we can't just run "nosetests [some sub-package]" anymore; it's all or nothing, like run_tests.py today.  If a test base class accretes complexity, we'll end up making all our tests inherit from it out of copy-and-paste laziness, and then someone will add a feature to it that makes the base class required, and now even simple tests have a bunch of cruft attached that they didn't need.</div>

<div><br></div><div>I like how Jay has proceeded so far in ~jaypipes/nova/cleanup-unittests -- the nova.tests.utils module provides handy functions for tests that need it, but otherwise tests are just vanilla unitest.TestCases.</div>

<div><br></div><div>- Michael</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
actually what we've already done in the Glance project (which uses<br>
many common Nova modules, but not the test runner)<br>
<br>
-jay<br>
<div class="im"><br>
> Thoughts?<br>
> -Sandy<br>
><br>
> Confidentiality Notice: This e-mail message (including any attached or<br>
> embedded documents) is intended for the exclusive and confidential use of<br>
> the<br>
> individual or entity to which this message is addressed, and unless<br>
> otherwise<br>
> expressly indicated, is confidential and privileged information of<br>
> Rackspace.<br>
> Any dissemination, distribution or copying of the enclosed material is<br>
> prohibited.<br>
> If you receive this transmission in error, please notify us immediately by<br>
> e-mail<br>
> at <a href="mailto:abuse@rackspace.com">abuse@rackspace.com</a>, and delete the original message.<br>
> Your cooperation is appreciated.<br>
><br>
</div>> _______________________________________________<br>
> Mailing list: <a href="https://launchpad.net/~openstack" target="_blank">https://launchpad.net/~openstack</a><br>
> Post to     : <a href="mailto:openstack@lists.launchpad.net">openstack@lists.launchpad.net</a><br>
> Unsubscribe : <a href="https://launchpad.net/~openstack" target="_blank">https://launchpad.net/~openstack</a><br>
> More help   : <a href="https://help.launchpad.net/ListHelp" target="_blank">https://help.launchpad.net/ListHelp</a><br>
><br>
><br>
<br>
_______________________________________________<br>
Mailing list: <a href="https://launchpad.net/~openstack" target="_blank">https://launchpad.net/~openstack</a><br>
Post to     : <a href="mailto:openstack@lists.launchpad.net">openstack@lists.launchpad.net</a><br>
Unsubscribe : <a href="https://launchpad.net/~openstack" target="_blank">https://launchpad.net/~openstack</a><br>
More help   : <a href="https://help.launchpad.net/ListHelp" target="_blank">https://help.launchpad.net/ListHelp</a><br>
</blockquote></div><br>