Implement repo_exists for all VCS backends

readthedocs/projects/tasks.py

@@ -249,11 +249,11 @@ def sync_versions(self, version_repo):
         version_post_data = {'repo': version_repo.repo_url}
         tags = None
         branches = None
-        if all([
-                version_repo.supports_lsremote,
-                not version_repo.repo_exists(),
-                self.project.has_feature(Feature.VCS_REMOTE_LISTING),
-        ]):
+        if (
+            version_repo.supports_lsremote and
+            not version_repo.repo_exists() and
+            self.project.has_feature(Feature.VCS_REMOTE_LISTING)
+        ):
             # Do not use ``ls-remote`` if the VCS does not support it or if we
             # have already cloned the repository locally. The latter happens
             # when triggering a normal build.

readthedocs/vcs_support/backends/bzr.py

@@ -19,11 +19,14 @@ class Backend(BaseVCS):
 
     def update(self):
         super().update()
-        retcode = self.run('bzr', 'status', record=False)[0]
-        if retcode == 0:
+        if self.repo_exists():
             return self.up()
         return self.clone()
 
+    def repo_exists(self):
+        retcode = self.run('bzr', 'status', record=False)[0]
+        return retcode == 0
+
     def up(self):
         retcode = self.run('bzr', 'revert')[0]
         if retcode != 0:

readthedocs/vcs_support/backends/hg.py

@@ -15,11 +15,14 @@ class Backend(BaseVCS):
 
     def update(self):
         super().update()
-        retcode = self.run('hg', 'status', record=False)[0]
-        if retcode == 0:
+        if self.repo_exists():
             return self.pull()
         return self.clone()
 
+    def repo_exists(self):
+        retcode = self.run('hg', 'status', record=False)[0]
+        return retcode == 0
+
     def pull(self):
         (pull_retcode, _, _) = self.run('hg', 'pull')
         if pull_retcode != 0:

readthedocs/vcs_support/backends/svn.py

@@ -29,13 +29,16 @@ def __init__(self, *args, **kwargs):
 
     def update(self):
         super().update()
-        # For some reason `svn status` gives me retcode 0 in non-svn
-        # directories that's why I use `svn info` here.
-        retcode, _, _ = self.run('svn', 'info', record=False)
-        if retcode == 0:
+        if self.repo_exists():
             return self.up()
         return self.co()
 
+    def repo_exists(self):
+        # For some reason `svn status` gives me retcode 0 in non-svn
+        # directories that's why I use `svn info` here.
+        retcode, *_ = self.run('svn', 'info', record=False)
+        return retcode == 0
+
     def up(self):
         retcode = self.run('svn', 'revert', '--recursive', '.')[0]
         if retcode != 0:

readthedocs/vcs_support/base.py

@@ -160,3 +160,6 @@ def update_submodules(self, config):
         :type config: readthedocs.config.BuildConfigBase
         """
         raise NotImplementedError
+
+    def repo_exists(self):
+        raise NotImplementedError