test_util: remove self.repo_path, generate new paths each time This solves a problem with startrev tests sporadically failing in ra.get_log() with some kind of svn corruption error. Loading each svn repository in a different place solved that, or at least prevented me from reproducing it. What is interesting is this is the same fixture being loaded each time. Also, before loading the fixture, we take care of removing an existing repository. Loading with the same options twice also failed reproducing the issue. Same thing if the first load only load the svn repository but does not convert it. So, I have absolutely no idea what was wrong, blame python subprocess, subversion or some kind of filesystem write ordering bug.

--- a/tests/test_push_renames.py
+++ b/tests/test_push_renames.py
@@ -6,8 +6,8 @@ import unittest
 class TestPushRenames(test_util.TestBase):
     def setUp(self):
         test_util.TestBase.setUp(self)
-        self._load_fixture_and_fetch('pushrenames.svndump',
-                                     stupid=True)
+        self.repo_path = self.load_and_fetch('pushrenames.svndump',
+                                             stupid=True)[1]
 
     def _debug_print_copies(self, ctx):
         w = sys.stderr.write