Code review comment for lp:~termie/nova/move_tests

Revision history for this message
Soren Hansen (soren) wrote :

To elaborate a bit: Tests are code. It should be just as pretty, neat, and wonderful as any other piece of code.

I want them to be installed so that people can run tests on the installed code as well. There might be environmental things that factor into the tests, so I think it's very valuable to be able to tell people who report bugs to run the test suite, for instance.

In fact, I think I'd much rather move the tests for e.g. the objectstore into nova/objectstore/tests. It feels perfectly natural to me that tests pertaining to nova.objectstore are in nova.objectstore.tests.

« Back to merge proposal