Merge lp:~jelmer/brz/fork-disabled into lp:brz/3.1

[Jelmer Vernooij]

Raise proper exception when forking isn't available for a project.

[Jelmer Vernooij (jelmer)]

review: Approve

=== modified file 'breezy/plugins/gitlab/hoster.py'
--- breezy/plugins/gitlab/hoster.py	2020-06-22 02:21:47 +0000
+++ breezy/plugins/gitlab/hoster.py	2020-06-25 00:10:31 +0000
@@ -100,6 +100,14 @@
         self.error = error
 
 
+class ForkingDisabled(errors.BzrError):
+
+    _fmt = ("Forking on project %(project)s is disabled.")
+
+    def __init__(self, project):
+        self.project = project
+
+
 class MergeRequestExists(Exception):
     """Raised when a merge requests already exists."""
 
@@ -356,6 +364,8 @@
     def _fork_project(self, project_name, timeout=50, interval=5):
         path = 'projects/%s/fork' % urlutils.quote(str(project_name), '')
         response = self._api_request('POST', path)
+        if response.status == 404:
+            raise ForkingDisabled(project_name)
         if response.status not in (200, 201):
             raise errors.InvalidHttpResponse(path, response.text)
         # The response should be valid JSON, but let's ignore it