Merge lp:~jelmer/brz/more-git-errors into lp:brz

[Jelmer Vernooij]

Handle the Launchpad way of saying "Repository not found".

Handle another way of formatting "Repository not found".

This is the format used by Launchpad.

[Martin Packman (gz)]

Looks good, though see comment.

review: Approve

=== modified file 'breezy/git/remote.py'
--- breezy/git/remote.py	2018-09-11 19:38:39 +0000
+++ breezy/git/remote.py	2018-09-12 17:23:34 +0000
@@ -194,7 +194,8 @@
     """
     message = str(message).strip()
     if (message.startswith("Could not find Repository ") or
-        message == 'Repository not found.'):
+        message == 'Repository not found.' or
+        (message.startswith('Repository ') and message.endswith(' not found.'))):
         return NotBranchError(url, message)
     if message == "HEAD failed to update":
         base_url, _ = urlutils.split_segment_parameters(url)

=== modified file 'breezy/git/tests/test_remote.py'
--- breezy/git/tests/test_remote.py	2018-09-11 19:36:00 +0000
+++ breezy/git/tests/test_remote.py	2018-09-12 17:23:34 +0000
@@ -88,6 +88,10 @@
         e = parse_git_error("url", "\n Could not find Repository foo/bar")
         self.assertIsInstance(e, NotBranchError)
 
+    def test_notbrancherror_launchpad(self):
+        e = parse_git_error("url", "Repository 'foo/bar' not found.")
+        self.assertIsInstance(e, NotBranchError)
+
     def test_notbrancherror_github(self):
         e = parse_git_error("url", "Repository not found.\n")
         self.assertIsInstance(e, NotBranchError)