aboutsummaryrefslogtreecommitdiffstats
path: root/libbe/bugdir.py
diff options
context:
space:
mode:
authorW. Trevor King <wking@drexel.edu>2008-11-22 19:45:37 -0500
committerW. Trevor King <wking@drexel.edu>2008-11-22 19:45:37 -0500
commit4a626e67b3f401b8e242a55571a802147123a196 (patch)
treee5907e6e16e9110a092d723e43f34571a4cac378 /libbe/bugdir.py
parent9524d061aa93451a282f45150944878d6b5cf065 (diff)
downloadbugseverywhere-4a626e67b3f401b8e242a55571a802147123a196.tar.gz
Explicit rcs.cleanup() in bugdir test.
Don't use del(rcs), because if there was an error, there is still a reference to rcs in the traceback, so it is never cleaned up. This can leave the external archive cluttering up your Arch install if you're using the Arch backend. See the __del__ documentation http://python.active-venture.com/ref/customization.html#l2h-175 for details. Also fixed some out-of-date method names in libbe.diff
Diffstat (limited to 'libbe/bugdir.py')
-rw-r--r--libbe/bugdir.py6
1 files changed, 3 insertions, 3 deletions
diff --git a/libbe/bugdir.py b/libbe/bugdir.py
index a552b0f..cc21878 100644
--- a/libbe/bugdir.py
+++ b/libbe/bugdir.py
@@ -363,8 +363,8 @@ class BugDirTestCase(unittest.TestCase):
self.bugdir = BugDir(self.dir.path, sink_to_existing_root=False, allow_rcs_init=True)
self.rcs = self.bugdir.rcs
def tearDown(self):
- del(self.rcs)
- del(self.dir)
+ self.rcs.cleanup()
+ self.dir.cleanup()
def fullPath(self, path):
return os.path.join(self.dir.path, path)
def assertPathExists(self, path):
@@ -410,4 +410,4 @@ class BugDirTestCase(unittest.TestCase):
self.versionTest()
unitsuite = unittest.TestLoader().loadTestsFromTestCase(BugDirTestCase)
-suite = unittest.TestSuite([unitsuite, doctest.DocTestSuite()])
+suite = unittest.TestSuite([unitsuite])#, doctest.DocTestSuite()])