Allow None as argument to Repository.has_revision().

diff --git a/repository.py b/repository.py
index 9618c351b30310499d2a8067dffc7fce7a395a7b..e3ff37633f9a72f9f9c55468cf20f2678aebf0ae 100644 (file)
--- a/repository.py
+++ b/repository.py
@@ -295,6 +295,9 @@ See http://bazaar-vcs.org/BzrSvn for details.
         return ancestry
 
     def has_revision(self, revision_id):
+        if revision_id is None:
+            return True
+
         try:
             (path, revnum) = self.parse_revision_id(revision_id)
         except NoSuchRevision:

diff --git a/tests/test_repos.py b/tests/test_repos.py
index 3350cc9e92a0a4fb5b69be5544089de869f2fc75..e9fbbefb4003b35cb67f78d4777f7c5a5cbb76a6 100644 (file)
--- a/tests/test_repos.py
+++ b/tests/test_repos.py
@@ -66,6 +66,11 @@ class TestSubversionRepositoryWorks(TestCaseWithSubversionRepository):
         self.assertTrue(repository.has_revision("svn-v1:1@%s-" % repository.uuid))
         self.assertFalse(repository.has_revision("some-other-revision"))
 
+    def test_has_revision_none(self):
+        bzrdir = self.make_client_and_bzrdir('d', 'dc')
+        repository = bzrdir.open_repository()
+        self.assertTrue(repository.has_revision(None))
+
     def test_revision_parents(self):
         repos_url = self.make_client('d', 'dc')
         self.build_tree({'dc/foo': "data"})