Rev 4664: Fixed as per Andrew's review. in file:///home/vila/src/bzr/experimental/test-isolation/

[Vincent Ladeuil]

At file:///home/vila/src/bzr/experimental/test-isolation/

------------------------------------------------------------
revno: 4664
revision-id: v.ladeuil+lp at free.fr-20090903081023-jxa59liaax1k5ba4
parent: v.ladeuil+lp at free.fr-20090902090724-b3rekpd6jd7u1gae
committer: Vincent Ladeuil <v.ladeuil+lp at free.fr>
branch nick: test-isolation
timestamp: Thu 2009-09-03 10:10:23 +0200
message:
  Fixed as per Andrew's review.
  
  * bzrlib/tests/test_selftest.py:
  (TestTestCase.test_trace_nesting): Call addCleanup asap.
-------------- next part --------------
=== modified file 'bzrlib/tests/test_selftest.py'
--- a/bzrlib/tests/test_selftest.py	2009-09-02 08:43:44 +0000
+++ b/bzrlib/tests/test_selftest.py	2009-09-03 08:10:23 +0000
@@ -1493,8 +1493,8 @@
         outer_test = TestTestCase("outer_child")
         result = self.make_test_result()
         outer_test.run(result)
+        self.addCleanup(osutils.delete_any, outer_test._log_file_name)
         self.assertEqual(original_trace, bzrlib.trace._trace_file)
-        self.addCleanup(osutils.delete_any, outer_test._log_file_name)
 
     def method_that_times_a_bit_twice(self):
         # call self.time twice to ensure it aggregates