changeset 7215:8a1caf0f2592

tests: vcs: recreate_repo_per_test is really optional While the documentation says this attribute is required, most tests do not set it when the default value is fine. Follow-through that convention and update the documentation.
author Thomas De Schampheleire <thomas.de_schampheleire@nokia.com>
date Fri, 16 Mar 2018 22:08:46 +0100
parents 22e80164295f
children c0ff58abe049
files kallithea/tests/vcs/base.py kallithea/tests/vcs/test_changesets.py kallithea/tests/vcs/test_inmemchangesets.py
diffstat 3 files changed, 1 insertions(+), 4 deletions(-) [+]
line wrap: on
line diff
--- a/kallithea/tests/vcs/base.py	Fri Mar 16 21:44:08 2018 +0100
+++ b/kallithea/tests/vcs/base.py	Fri Mar 16 22:08:46 2018 +0100
@@ -18,7 +18,7 @@
     This is a backend independent test case class which should be created
     with ``type`` method.
 
-    It is required to set following attributes at subclass:
+    It is possible to set following attributes at subclass:
 
     - ``backend_alias``: alias of used backend (see ``vcs.BACKENDS``)
     - ``recreate_repo_per_test``: If set to ``False``, repo would NOT be created
--- a/kallithea/tests/vcs/test_changesets.py	Fri Mar 16 21:44:08 2018 +0100
+++ b/kallithea/tests/vcs/test_changesets.py	Fri Mar 16 22:08:46 2018 +0100
@@ -53,7 +53,6 @@
 
 
 class _ChangesetsWithCommitsTestCaseixin(_BackendTestMixin):
-    recreate_repo_per_test = True
 
     @classmethod
     def _get_commits(cls):
--- a/kallithea/tests/vcs/test_inmemchangesets.py	Fri Mar 16 21:44:08 2018 +0100
+++ b/kallithea/tests/vcs/test_inmemchangesets.py	Fri Mar 16 22:08:46 2018 +0100
@@ -25,8 +25,6 @@
 
 class InMemoryChangesetTestMixin(_BackendTestMixin):
 
-    recreate_repo_per_test = True
-
     @classmethod
     def _get_commits(cls):
         # Note: this is slightly different than the regular _get_commits methods