summaryrefslogtreecommitdiff
path: root/HACKING.rst
diff options
context:
space:
mode:
authorMonty Taylor <mordred@inaugust.com>2013-02-23 10:35:28 +0000
committerMonty Taylor <mordred@inaugust.com>2013-03-05 18:07:14 -0500
commitcea9e39dd70e6b5c5f0e5aac5e9b7fc477ab990c (patch)
tree4dbd4ba5c937bc7cf81ddd491100c3e7f569e46b /HACKING.rst
parent4a1a37ce9f8baf2078a1a44380895d144203a876 (diff)
downloadneutron-cea9e39dd70e6b5c5f0e5aac5e9b7fc477ab990c.tar.gz
Add common test base class to hold common things.
There are several common fixtures that every test case wants. Following the pattern in Nova, add a common base test case class to hold these things. Change-Id: I2d2cd91e5051d9cbf230e6f48985d6eddcb7b58a
Diffstat (limited to 'HACKING.rst')
-rw-r--r--HACKING.rst5
1 files changed, 4 insertions, 1 deletions
diff --git a/HACKING.rst b/HACKING.rst
index 4333d25685..84e81af97d 100644
--- a/HACKING.rst
+++ b/HACKING.rst
@@ -199,7 +199,10 @@ bug that had no unit test, a new passing unit test should be added. If a
submitted bug fix does have a unit test, be sure to add a new one that fails
without the patch and passes with the patch.
-All unittest classes must ultimately inherit from testtools.TestCase.
+All unittest classes must ultimately inherit from testtools.TestCase. In the
+Quantum test suite, this should be done by inheriting from
+quantum.tests.base.BaseTestCase.
+
All setUp and tearDown methods must upcall using the super() method.
tearDown methods should be avoided and addCleanup calls should be preferred.
Never manually create tempfiles. Always use the tempfile fixtures from