From: Jelmer Vernooij <jelmer@samba.org>
Date: Sun, 22 Jun 2008 07:34:26 +0000 (+0200)
Subject: Merge 0.4.
X-Git-Tag: subvertpy-0.6.0~267^2~24
X-Git-Url: http://git.samba.org/samba.git/?p=jelmer%2Fsubvertpy.git;a=commitdiff_plain;h=eafd90a48a8134b09b46626334996709c4ee14b5

Merge 0.4.
---

eafd90a48a8134b09b46626334996709c4ee14b5
diff --cc __init__.py
index 12206ce2,2a1100c5..e03c0528
--- a/__init__.py
+++ b/__init__.py
@@@ -64,7 -64,18 +64,6 @@@ def check_bzrlib_version(desired)
          if not (bzrlib_version[0], bzrlib_version[1]-1) in desired:
              raise BzrError('Version mismatch')
  
 -def check_subversion_version():
 -    """Check that Subversion is compatible.
 -
 -    """
 -    from bzrlib.plugins.svn.ra import version
 -    ra_version = version()
 -    if (ra_version[0] >= 5 and getattr(ra, 'SVN_REVISION', None) and 27729 <= ra.SVN_REVISION < 31470):
 -        warning('Installed Subversion has buggy svn.ra.get_log() implementation, please install newer.')
 -
 -    mutter("bzr-svn: using Subversion %d.%d.%d (%s)" % ra_version)
 -
--
  def check_rebase_version(min_version):
      """Check what version of bzr-rebase is installed.
  
@@@ -83,11 -94,7 +82,23 @@@
          raise RebaseNotPresent(e)
  
  
 -check_subversion_version()
 +def check_subversion_version():
++    """Check that Subversion is compatible.
++
++    """
 +    try:
 +        from bzrlib.plugins.svn import core
 +    except:
 +        warning("Unable to load bzr-svn extensions - did you build it?")
++    from bzrlib.plugins.svn.ra import version
++    ra_version = version()
++    if (ra_version[0] >= 5 and getattr(ra, 'SVN_REVISION', None) and 27729 <= ra.SVN_REVISION < 31470):
++        warning('Installed Subversion has buggy svn.ra.get_log() implementation, please install newer.')
++
++    mutter("bzr-svn: using Subversion %d.%d.%d (%s)" % ra_version)
++
++
++
  
  register_transport_proto('svn+ssh://', 
      help="Access using the Subversion smart server tunneled over SSH.")
diff --cc branch.py
index 471d1134,0e26cafb..c848b5b6
--- a/branch.py
+++ b/branch.py
@@@ -24,7 -24,11 +24,9 @@@ from bzrlib.inventory import (Inventory
  from bzrlib.revision import is_null, ensure_null, NULL_REVISION
  from bzrlib.workingtree import WorkingTree
  
 -import svn.core
 -
 -from bzrlib.plugins.svn import core
 +from bzrlib.plugins.svn import core, wc
+ from bzrlib.plugins.svn.auth import create_auth_baton
+ from bzrlib.plugins.svn.client import Client, get_config
  from bzrlib.plugins.svn.commit import push
  from bzrlib.plugins.svn.config import BranchConfig
  from bzrlib.plugins.svn.core import SubversionException
diff --cc commit.py
index 5aa0ccc2,da49bfe6..76ddf28a
--- a/commit.py
+++ b/commit.py
@@@ -397,17 -395,14 +395,12 @@@ class SvnCommitBuilder(RootCommitBuilde
          """Finish the commit.
  
          """
-         def done(revision, author, date):
+         def done(*args):
              """Callback that is called by the Subversion commit editor 
              once the commit finishes.
 -
 -            :param revision_data: Revision metadata
              """
-             class CommitResult:
-                 def __init__(self, revision, author, date):
-                     self.revision = revision
-                     self.author = author
-                     self.date = date
-             self.revision_metadata = CommitResult(revision, author, date)
- 
+             self.revision_metadata = args
+         
          bp_parts = self.branch.get_branch_path().split("/")
          repository_latest_revnum = self.repository.get_latest_revnum()
          lock = self.repository.transport.lock_write(".")
diff --cc logwalker.py
index 35c557a9,488e5e51..ea7ad0bb
--- a/logwalker.py
+++ b/logwalker.py
@@@ -20,14 -20,14 +20,13 @@@ from bzrlib.errors import NoSuchRevisio
  from bzrlib.trace import mutter
  import bzrlib.ui as ui
  
 -import svn.delta
 -
  from bzrlib.plugins.svn import changes, core
 -from bzrlib.plugins.svn.cache import CacheTable
  from bzrlib.plugins.svn.core import SubversionException
 +from bzrlib.plugins.svn.cache import CacheTable
  from bzrlib.plugins.svn.errors import ERR_FS_NO_SUCH_REVISION, ERR_FS_NOT_FOUND
 +from bzrlib.plugins.svn.ra import DIRENT_KIND
  from bzrlib.plugins.svn.transport import SvnRaTransport
  
- 
  class lazy_dict(object):
      def __init__(self, initial, create_fn, *args):
          self.initial = initial
diff --cc tests/__init__.py
index 48204ad9,2776a0d9..e18a1b55
--- a/tests/__init__.py
+++ b/tests/__init__.py
@@@ -26,10 -26,12 +26,11 @@@ from bzrlib import osutils, urlutil
  from bzrlib.bzrdir import BzrDir
  from bzrlib.tests import TestCaseInTempDir, TestSkipped
  from bzrlib.trace import mutter
 -from bzrlib.urlutils import local_path_to_url
  from bzrlib.workingtree import WorkingTree
  
- from bzrlib.plugins.svn import repos, wc, client, ra, properties
- from bzrlib.plugins.svn.ra import RemoteAccess, txdelta_send_stream
+ from bzrlib.plugins.svn import properties, ra, repos
+ from bzrlib.plugins.svn.client import Client
+ from bzrlib.plugins.svn.ra import Auth, RemoteAccess, txdelta_send_stream
  
  class TestCaseWithSubversionRepository(TestCaseInTempDir):
      """A test case that provides the ability to build Subversion 
diff --cc tests/test_repository.py
index 28d9bec1,00000000..913edba4
mode 100644,000000..100644
--- a/tests/test_repository.py
+++ b/tests/test_repository.py
@@@ -1,1232 -1,0 +1,1245 @@@
 +# -*- coding: utf-8 -*-
 +
 +# Copyright (C) 2006-2007 Jelmer Vernooij <jelmer@samba.org>
 +
 +# This program is free software; you can redistribute it and/or modify
 +# it under the terms of the GNU General Public License as published by
 +# the Free Software Foundation; either version 3 of the License, or
 +# (at your option) any later version.
 +
 +# This program is distributed in the hope that it will be useful,
 +# but WITHOUT ANY WARRANTY; without even the implied warranty of
 +# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
 +# GNU General Public License for more details.
 +
 +# You should have received a copy of the GNU General Public License
 +# along with this program; if not, write to the Free Software
 +# Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307  USA
 +
 +"""Subversion repository tests."""
 +
 +from bzrlib import urlutils
 +from bzrlib.branch import Branch
 +from bzrlib.bzrdir import BzrDir, format_registry
 +from bzrlib.config import GlobalConfig
 +from bzrlib.errors import NoSuchRevision, UninitializableFormat, BzrError
 +from bzrlib.inventory import Inventory
 +from bzrlib.osutils import has_symlinks
 +from bzrlib.repository import Repository
 +from bzrlib.revision import NULL_REVISION, Revision
 +from bzrlib.tests import TestCase
 +
 +import os, sys
 +
- from bzrlib.plugins.svn import format
++from bzrlib.plugins.svn import format, ra
 +from bzrlib.plugins.svn.mapping import (escape_svn_path, unescape_svn_path, 
 +                     SVN_PROP_BZR_REVISION_ID)
 +from bzrlib.plugins.svn.mapping3 import (SVN_PROP_BZR_BRANCHING_SCHEME, set_branching_scheme,
 +                      set_property_scheme, BzrSvnMappingv3)
 +from bzrlib.plugins.svn.mapping3.scheme import (TrunkBranchingScheme, NoBranchingScheme, 
 +                    ListBranchingScheme, SingleBranchingScheme)
 +from bzrlib.plugins.svn.transport import SvnRaTransport
 +from bzrlib.plugins.svn.tests import TestCaseWithSubversionRepository
 +from bzrlib.plugins.svn.repository import SvnRepositoryFormat
 +
 +
 +class TestSubversionRepositoryWorks(TestCaseWithSubversionRepository):
 +    def test_format(self):
 +        """ Test repository format is correct """
 +        bzrdir = self.make_local_bzrdir('a', 'ac')
 +        self.assertEqual(bzrdir._format.get_format_string(), \
 +                "Subversion Local Checkout")
 +        
 +        self.assertEqual(bzrdir._format.get_format_description(), \
 +                "Subversion Local Checkout")
 +
 +    def test_get_branch_log(self):
 +        repos_url = self.make_repository("a")
 +        cb = self.commit_editor(repos_url)
 +        cb.add_file("foo")
 +        cb.done()
 +
 +        repos = Repository.open(repos_url)
 +
 +        self.assertEqual([
 +            ('', {'foo': ('A', None, -1)}, 1), 
 +            ('', {'': ('A', None, -1)}, 0)],
 +            [(l.branch_path, l.paths, l.revnum) for l in repos.iter_reverse_branch_changes("", 1, NoBranchingScheme())])
 +
 +    def test_make_working_trees(self):
 +        repos_url = self.make_repository("a")
 +        repos = Repository.open(repos_url)
 +        self.assertFalse(repos.make_working_trees())
 +
 +    def test_get_config_global_set(self):
 +        repos_url = self.make_repository("a")
 +        cfg = GlobalConfig()
 +        cfg.set_user_option("foo", "Still Life")
 +
 +        repos = Repository.open(repos_url)
 +        self.assertEquals("Still Life", 
 +                repos.get_config().get_user_option("foo"))
 +
 +    def test_get_config(self):
 +        repos_url = self.make_repository("a")
 +        repos = Repository.open(repos_url)
 +        repos.get_config().set_user_option("foo", "Van Der Graaf Generator")
 +
 +        repos = Repository.open(repos_url)
 +        self.assertEquals("Van Der Graaf Generator", 
 +                repos.get_config().get_user_option("foo"))
 +
 +
 +    def test_get_physical_lock_status(self):
 +        repos_url = self.make_repository("a")
 +        repos = Repository.open(repos_url)
 +        self.assertFalse(repos.get_physical_lock_status())
 +
 +    def test_iter_changes_parent_rename(self):
 +        repos_url = self.make_client("a", "dc")
 +        self.build_tree({'dc/foo/bar': None})
 +        self.client_add('dc/foo')
 +        self.client_commit('dc', 'a')
 +        self.client_update('dc')
 +        self.client_copy('dc/foo', 'dc/bla')
 +        self.client_commit('dc', 'b')
 +        repos = Repository.open(repos_url)
 +        ret = list(repos.iter_changes('bla/bar', 2, BzrSvnMappingv3(SingleBranchingScheme('bla/bar'))))
 +        self.assertEquals(1, len(ret))
 +        self.assertEquals("bla/bar", ret[0][0])
 +
 +    def test_set_make_working_trees(self):
 +        repos_url = self.make_repository("a")
 +        repos = Repository.open(repos_url)
 +        repos.set_make_working_trees(True)
 +        self.assertFalse(repos.make_working_trees())
 +
 +    def test_get_fileid_map(self):
 +        repos_url = self.make_repository("a")
 +        repos = Repository.open(repos_url)
 +        mapping = repos.get_mapping()
 +        self.assertEqual({u"": (mapping.generate_file_id(repos.uuid, 0, "", u""), mapping.generate_revision_id(repos.uuid, 0, ""))}, repos.get_fileid_map(0, "", mapping))
 +
 +    def test_generate_revision_id_forced_revid(self):
 +        repos_url = self.make_client("a", "dc")
 +        self.client_set_prop("dc", SVN_PROP_BZR_REVISION_ID+"none", 
 +                             "2 someid\n")
 +        self.client_commit("dc", "set id")
 +        repos = Repository.open(repos_url)
 +        revid = repos.generate_revision_id(1, "", repos.get_mapping())
 +        self.assertEquals("someid", revid)
 +
 +    def test_generate_revision_id_forced_revid_invalid(self):
 +        repos_url = self.make_client("a", "dc")
 +        self.client_set_prop("dc", SVN_PROP_BZR_REVISION_ID+"none", 
 +                             "corrupt-id\n")
 +        self.client_commit("dc", "set id")
 +        repos = Repository.open(repos_url)
 +        revid = repos.generate_revision_id(1, "", repos.get_mapping())
 +        self.assertEquals(
 +                repos.get_mapping().generate_revision_id(repos.uuid, 1, ""),
 +                revid)
 +
 +    def test_add_revision(self):
 +        repos_url = self.make_repository("a")
 +        repos = Repository.open(repos_url)
 +        self.assertRaises(NotImplementedError, repos.add_revision, "revid", 
 +                None)
 +
 +    def test_has_signature_for_revision_id_no(self):
 +        repos_url = self.make_repository("a")
 +        repos = Repository.open(repos_url)
 +        self.assertFalse(repos.has_signature_for_revision_id("foo"))
 +
 +    def test_set_signature(self):
 +        repos_url = self.make_client("a", "dc")
 +        repos = Repository.open(repos_url)
 +        self.build_tree({"dc/foo": "bar"})
 +        self.client_add("dc/foo")
 +        self.client_commit("dc", "msg")
 +        revid = repos.get_mapping().generate_revision_id(repos.uuid, 1, "")
 +        repos.add_signature_text(revid, "TEXT")
 +        self.assertTrue(repos.has_signature_for_revision_id(revid))
 +        self.assertEquals(repos.get_signature_text(revid), "TEXT")
 +
 +    def test_repr(self):
 +        repos_url = self.make_client("a", "dc")
 +        self.build_tree({'dc/foo': "data"})
 +        self.client_add("dc/foo")
 +        self.client_commit("dc", "My Message")
 +
 +        repos = Repository.open(repos_url)
 +
 +        self.assertEqual("SvnRepository('%s/')" % urlutils.local_path_to_url(urlutils.join(self.test_dir, "a")), repos.__repr__())
 +
 +    def test_get_branch_invalid_revision(self):
 +        repos_url = self.make_repository("a")
 +        repos = Repository.open(repos_url)
 +        self.assertRaises(NoSuchRevision, list, 
 +               repos.iter_reverse_branch_changes("/", 20, NoBranchingScheme()))
 +
 +    def test_follow_branch_switched_parents(self):
 +        repos_url = self.make_client('a', 'dc')
 +        self.build_tree({'dc/pykleur/trunk/pykleur': None})
 +        self.client_add("dc/pykleur")
 +        self.client_commit("dc", "initial")
 +        self.build_tree({'dc/pykleur/trunk/pykleur/afile': 'contents'})
 +        self.client_add("dc/pykleur/trunk/pykleur/afile")
 +        self.client_commit("dc", "add file")
 +        self.client_copy("dc/pykleur", "dc/pygments", 1)
 +        self.client_delete('dc/pykleur')
 +        self.client_update("dc")
 +        self.client_commit("dc", "commit")
 +        repos = Repository.open(repos_url)
-         self.assertEquals([
++        results = [(l.branch_path, l.paths, l.revnum) for l in repos.iter_reverse_branch_changes("pygments/trunk", 3, TrunkBranchingScheme(1))]
++
++        # Results differ per Subversion version
++        # For <= 1.4:
++        if ra.version()[1] <= 4:
++            self.assertEquals([
 +            ('pygments/trunk', {'pygments': (u'A', 'pykleur', 1),
 +                                'pygments/trunk': (u'R', 'pykleur/trunk', 2),
 +                                'pykleur': (u'D', None, -1)}, 3),
 +            ('pykleur/trunk', {'pykleur/trunk/pykleur/afile': (u'A', None, -1)}, 2),
 +            ('pykleur/trunk',
 +                    {'pykleur': (u'A', None, -1),
 +                     'pykleur/trunk': (u'A', None, -1),
 +                     'pykleur/trunk/pykleur': (u'A', None, -1)},
-              1)],
-             [(l.branch_path, l.paths, l.revnum) for l in repos.iter_reverse_branch_changes("pygments/trunk", 3, TrunkBranchingScheme(1))])
++             1)], results
++            )
++        else:
++            self.assertEquals(
++               [('pygments/trunk', {'pygments': (u'A', 'pykleur', 1), 'pykleur': (u'D', None, -1)}, 3),
++                ('pykleur/trunk', {'pykleur': (u'A', None, -1), 
++                                   'pykleur/trunk': (u'A', None, -1), 
++                                   'pykleur/trunk/pykleur': (u'A', None, -1)}, 
++                1)], results
++            )
 +
 +    def test_follow_branch_move_single(self):
 +        repos_url = self.make_client('a', 'dc')
 +        self.build_tree({'dc/pykleur/bla': None})
 +        self.client_add("dc/pykleur")
 +        self.client_commit("dc", "initial")
 +        self.client_copy("dc/pykleur", "dc/pygments", 1)
 +        self.client_update("dc")
 +        self.client_commit("dc", "commit")
 +        repos = Repository.open(repos_url)
 +        changes = repos.iter_reverse_branch_changes("pygments", 2, SingleBranchingScheme("pygments"))
 +        self.assertEquals([('pygments',
 +              {'pygments/bla': ('A', None, -1), 'pygments': ('A', None, -1)},
 +                2)],
 +                [(l.branch_path, l.paths, l.revnum) for l in changes])
 +
 +    def test_history_all(self):
 +        repos_url = self.make_client("a", "dc")
 +        self.build_tree({'dc/trunk/file': "data", "dc/foo/file":"data"})
 +        self.client_add("dc/trunk")
 +        self.client_add("dc/foo")
 +        self.client_commit("dc", "My Message")
 +
 +        repos = Repository.open(repos_url)
 +
 +        self.assertEqual(2, 
 +                   len(list(repos.all_revision_ids(repos.get_layout()))))
 +
 +    def test_all_revs_empty(self):
 +        repos_url = self.make_repository("a")
 +        repos = Repository.open(repos_url)
 +        set_branching_scheme(repos, TrunkBranchingScheme())
 +        self.assertEqual([], list(repos.all_revision_ids()))
 +
 +    def test_all_revs(self):
 +        repos_url = self.make_client("a", "dc")
 +        self.build_tree({'dc/trunk/file': "data", "dc/foo/file":"data"})
 +        self.client_add("dc/trunk")
 +        self.client_commit("dc", "add trunk")
 +        self.build_tree({'dc/branches/somebranch/somefile': 'data'})
 +        self.client_add("dc/branches")
 +        self.client_commit("dc", "add a branch")
 +        self.client_delete("dc/branches/somebranch")
 +        self.client_commit("dc", "remove branch")
 +
 +        repos = Repository.open(repos_url)
 +        set_branching_scheme(repos, TrunkBranchingScheme())
 +        mapping = repos.get_mapping()
 +        self.assertEqual([
 +            repos.generate_revision_id(1, "trunk", mapping), 
 +            repos.generate_revision_id(2, "branches/somebranch", mapping)],
 +            list(repos.all_revision_ids()))
 +
 +    def test_follow_history_empty(self):
 +        repos_url = self.make_repository("a")
 +        repos = Repository.open(repos_url)
 +        self.assertEqual([repos.generate_revision_id(0, '', repos.get_mapping())], 
 +              list(repos.all_revision_ids(repos.get_layout())))
 +
 +    def test_follow_history_empty_branch(self):
 +        repos_url = self.make_client("a", "dc")
 +        self.build_tree({'dc/trunk/afile': "data", "dc/branches": None})
 +        self.client_add("dc/trunk")
 +        self.client_add("dc/branches")
 +        self.client_commit("dc", "My Message")
 +        repos = Repository.open(repos_url)
 +        set_branching_scheme(repos, TrunkBranchingScheme())
 +        self.assertEqual([repos.generate_revision_id(1, 'trunk', repos.get_mapping())], 
 +                list(repos.all_revision_ids(repos.get_layout())))
 +
 +    def test_follow_history_follow(self):
 +        repos_url = self.make_client("a", "dc")
 +        self.build_tree({'dc/trunk/afile': "data", "dc/branches": None})
 +        self.client_add("dc/trunk")
 +        self.client_add("dc/branches")
 +        self.client_commit("dc", "My Message")
 +
 +        self.client_copy("dc/trunk", "dc/branches/abranch")
 +        self.client_commit("dc", "Create branch")
 +
 +        repos = Repository.open(repos_url)
 +        set_branching_scheme(repos, TrunkBranchingScheme())
 +
 +        items = list(repos.all_revision_ids(repos.get_layout()))
 +        self.assertEqual([repos.generate_revision_id(1, 'trunk', repos.get_mapping()),
 +                          repos.generate_revision_id(2, 'branches/abranch', repos.get_mapping())
 +                          ], items)
 +
 +    def test_branch_log_specific(self):
 +        repos_url = self.make_client("a", "dc")
 +        self.build_tree({
 +            'dc/branches': None,
 +            'dc/branches/brancha': None,
 +            'dc/branches/branchab': None,
 +            'dc/branches/brancha/data': "data", 
 +            "dc/branches/branchab/data":"data"})
 +        self.client_add("dc/branches")
 +        self.client_commit("dc", "My Message")
 +
 +        repos = Repository.open(repos_url)
 +        set_branching_scheme(repos, TrunkBranchingScheme())
 +
 +        self.assertEqual(1, len(list(repos.iter_reverse_branch_changes("branches/brancha",
 +            1, TrunkBranchingScheme()))))
 +
 +    def test_branch_log_specific_ignore(self):
 +        repos_url = self.make_client("a", "dc")
 +        self.build_tree({'dc/branches': None})
 +        self.client_add("dc/branches")
 +        self.build_tree({
 +            'dc/branches/brancha': None,
 +            'dc/branches/branchab': None,
 +            'dc/branches/brancha/data': "data", 
 +            "dc/branches/branchab/data":"data"})
 +        self.client_add("dc/branches/brancha")
 +        self.client_commit("dc", "My Message")
 +
 +        self.client_add("dc/branches/branchab")
 +        self.client_commit("dc", "My Message2")
 +
 +        repos = Repository.open(repos_url)
 +        set_branching_scheme(repos, TrunkBranchingScheme())
 +
 +        self.assertEqual(1, len(list(repos.iter_reverse_branch_changes("branches/brancha",
 +            2, TrunkBranchingScheme()))))
 +
 +    def test_find_branches(self):
 +        repos_url = self.make_client("a", "dc")
 +        self.build_tree({
 +            'dc/branches/brancha': None,
 +            'dc/branches/branchab': None,
 +            'dc/branches/brancha/data': "data", 
 +            "dc/branches/branchab/data":"data"})
 +        self.client_add("dc/branches")
 +        self.client_commit("dc", "My Message")
 +        repos = Repository.open(repos_url)
 +        set_branching_scheme(repos, TrunkBranchingScheme())
 +        branches = repos.find_branches()
 +        self.assertEquals(2, len(branches))
 +        self.assertEquals(urlutils.join(repos.base, "branches/brancha"), 
 +                          branches[1].base)
 +        self.assertEquals(urlutils.join(repos.base, "branches/branchab"), 
 +                          branches[0].base)
 +
 +    def test_find_branchpaths_moved(self):
 +        repos_url = self.make_client("a", "dc")
 +        self.build_tree({
 +            'dc/tmp/branches/brancha': None,
 +            'dc/tmp/branches/branchab': None,
 +            'dc/tmp/branches/brancha/data': "data", 
 +            "dc/tmp/branches/branchab/data":"data"})
 +        self.client_add("dc/tmp")
 +        self.client_commit("dc", "My Message")
 +        self.client_copy("dc/tmp/branches", "dc/tags")
 +        self.client_commit("dc", "My Message 2")
 +
 +        repos = Repository.open(repos_url)
 +        set_branching_scheme(repos, TrunkBranchingScheme())
 +
 +        self.assertEqual([("tags/branchab", 2, True), 
 +                          ("tags/brancha", 2, True)], 
 +                list(repos.find_branchpaths(TrunkBranchingScheme(), to_revnum=2)))
 +
 +    def test_find_branchpaths_start_revno(self):
 +        repos_url = self.make_client("a", "dc")
 +        self.build_tree({'dc/branches/brancha': None})
 +        self.client_add("dc/branches")
 +        self.client_commit("dc", "My Message")
 +        self.build_tree({'dc/branches/branchb': None})
 +        self.client_add("dc/branches/branchb")
 +        self.client_commit("dc", "My Message 2")
 +
 +        repos = Repository.open(repos_url)
 +        set_branching_scheme(repos, TrunkBranchingScheme())
 +
 +        self.assertEqual([("branches/branchb", 2, True)],
 +                list(repos.find_branchpaths(TrunkBranchingScheme(), from_revnum=2, 
 +                    to_revnum=2)))
 +
 +    def test_find_branchpaths_file_moved_from_nobranch(self):
 +        repos_url = self.make_client("a", "dc")
 +        self.build_tree({
 +            'dc/tmp/trunk': None,
 +            'dc/bla/somefile': "contents"})
 +        self.client_add("dc/tmp")
 +        self.client_add("dc/bla")
 +        self.client_commit("dc", "My Message")
 +        self.client_copy("dc/bla", "dc/tmp/branches")
 +        self.client_delete("dc/tmp/branches/somefile")
 +        self.client_commit("dc", "My Message 2")
 +
 +        Repository.open(repos_url).find_branchpaths(TrunkBranchingScheme(2))
 +
 +    def test_find_branchpaths_deleted_from_nobranch(self):
 +        repos_url = self.make_client("a", "dc")
 +        self.build_tree({
 +            'dc/tmp/trunk': None,
 +            'dc/bla/somefile': "contents"})
 +        self.client_add("dc/tmp")
 +        self.client_add("dc/bla")
 +        self.client_commit("dc", "My Message")
 +        self.client_copy("dc/bla", "dc/tmp/branches")
 +        self.client_delete("dc/tmp/branches/somefile")
 +        self.client_commit("dc", "My Message 2")
 +
 +        Repository.open(repos_url).find_branchpaths(TrunkBranchingScheme(1))
 +
 +    def test_find_branchpaths_moved_nobranch(self):
 +        repos_url = self.make_client("a", "dc")
 +        self.build_tree({
 +            'dc/tmp/nested/foobar': None,
 +            'dc/tmp/nested/branches/brancha': None,
 +            'dc/tmp/nested/branches/branchab': None,
 +            'dc/tmp/nested/branches/brancha/data': "data", 
 +            "dc/tmp/nested/branches/branchab/data":"data"})
 +        self.client_add("dc/tmp")
 +        self.client_commit("dc", "My Message")
 +        self.client_copy("dc/tmp/nested", "dc/t2")
 +        self.client_commit("dc", "My Message 2")
 +
 +        repos = Repository.open(repos_url)
 +        set_branching_scheme(repos, TrunkBranchingScheme(1))
 +
 +        self.assertEqual([("t2/branches/brancha", 2, True), 
 +                          ("t2/branches/branchab", 2, True)], 
 +                list(repos.find_branchpaths(TrunkBranchingScheme(1), to_revnum=2)))
 +
 +    def test_find_branchpaths_no(self):
 +        repos_url = self.make_repository("a")
 +
 +        repos = Repository.open(repos_url)
 +        set_branching_scheme(repos, NoBranchingScheme())
 +
 +        self.assertEqual([("", 0, True)], 
 +                list(repos.find_branchpaths(NoBranchingScheme(), to_revnum=0)))
 +
 +    def test_find_branchpaths_no_later(self):
 +        repos_url = self.make_repository("a")
 +
 +        repos = Repository.open(repos_url)
 +        set_branching_scheme(repos, NoBranchingScheme())
 +
 +        self.assertEqual([("", 0, True)], 
 +                list(repos.find_branchpaths(NoBranchingScheme(), to_revnum=0)))
 +
 +    def test_find_branchpaths_trunk_empty(self):
 +        repos_url = self.make_repository("a")
 +
 +        repos = Repository.open(repos_url)
 +        set_branching_scheme(repos, TrunkBranchingScheme())
 +
 +        self.assertEqual([], 
 +                list(repos.find_branchpaths(TrunkBranchingScheme(), to_revnum=0)))
 +
 +    def test_find_branchpaths_trunk_one(self):
 +        repos_url = self.make_client("a", "dc")
 +
 +        repos = Repository.open(repos_url)
 +        set_branching_scheme(repos, TrunkBranchingScheme())
 +
 +        self.build_tree({'dc/trunk/foo': "data"})
 +        self.client_add("dc/trunk")
 +        self.client_commit("dc", "My Message")
 +
 +        self.assertEqual([("trunk", 1, True)], 
 +                list(repos.find_branchpaths(TrunkBranchingScheme(), to_revnum=1)))
 +
 +    def test_find_branchpaths_removed(self):
 +        repos_url = self.make_client("a", "dc")
 +
 +        repos = Repository.open(repos_url)
 +        set_branching_scheme(repos, TrunkBranchingScheme())
 +
 +        self.build_tree({'dc/trunk/foo': "data"})
 +        self.client_add("dc/trunk")
 +        self.client_commit("dc", "My Message")
 +
 +        self.client_delete("dc/trunk")
 +        self.client_commit("dc", "remove")
 +
 +        self.assertEqual([("trunk", 1, True)], 
 +                list(repos.find_branchpaths(TrunkBranchingScheme(), to_revnum=1)))
 +        self.assertEqual([("trunk", 1, False)], 
 +                list(repos.find_branchpaths(TrunkBranchingScheme(), to_revnum=2)))
 +
 +    def test_url(self):
 +        """ Test repository URL is kept """
 +        bzrdir = self.make_local_bzrdir('b', 'bc')
 +        self.assertTrue(isinstance(bzrdir, BzrDir))
 +
 +    def test_uuid(self):
 +        """ Test UUID is retrieved correctly """
 +        bzrdir = self.make_local_bzrdir('c', 'cc')
 +        self.assertTrue(isinstance(bzrdir, BzrDir))
 +        repository = bzrdir._find_repository()
 +        fs = self.open_fs('c')
 +        self.assertEqual(fs.get_uuid(), repository.uuid)
 +
 +    def test_get_inventory_weave(self):
 +        bzrdir = self.make_client_and_bzrdir('d', 'dc')
 +        repository = bzrdir.find_repository()
 +        self.assertRaises(NotImplementedError, repository.get_inventory_weave)
 +
 +    def test_has_revision(self):
 +        bzrdir = self.make_client_and_bzrdir('d', 'dc')
 +        repository = bzrdir.find_repository()
 +        self.build_tree({'dc/foo': "data"})
 +        self.client_add("dc/foo")
 +        self.client_commit("dc", "My Message")
 +        self.assertTrue(repository.has_revision(
 +            repository.generate_revision_id(1, "", repository.get_mapping())))
 +        self.assertFalse(repository.has_revision("some-other-revision"))
 +
 +    def test_has_revision_none(self):
 +        bzrdir = self.make_client_and_bzrdir('d', 'dc')
 +        repository = bzrdir.find_repository()
 +        self.assertTrue(repository.has_revision(None))
 +
 +    def test_has_revision_future(self):
 +        bzrdir = self.make_client_and_bzrdir('d', 'dc')
 +        repository = bzrdir.find_repository()
 +        self.assertFalse(repository.has_revision(
 +            repository.get_mapping().generate_revision_id(repository.uuid, 5, "")))
 +
 +    def test_get_parent_map(self):
 +        repos_url = self.make_client('d', 'dc')
 +        self.build_tree({'dc/foo': "data"})
 +        self.client_add("dc/foo")
 +        self.client_commit("dc", "My Message")
 +        self.build_tree({'dc/foo': "data2"})
 +        self.client_commit("dc", "Second Message")
 +        repository = Repository.open("svn+%s" % repos_url)
 +        mapping = repository.get_mapping()
 +        revid = repository.generate_revision_id(0, "", mapping)
 +        self.assertEqual({revid: (NULL_REVISION,)}, repository.get_parent_map([revid]))
 +        revid = repository.generate_revision_id(1, "", mapping)
 +        self.assertEqual({revid: (repository.generate_revision_id(0, "", mapping),)}, repository.get_parent_map([revid]))
 +        revid = repository.generate_revision_id(2, "", mapping)
 +        self.assertEqual({revid: (repository.generate_revision_id(1, "", mapping),)},
 +            repository.get_parent_map([revid]))
 +        self.assertEqual({}, repository.get_parent_map(["notexisting"]))
 +
 +    def test_revision_fileidmap(self):
 +        repos_url = self.make_client('d', 'dc')
 +        self.build_tree({'dc/foo': "data"})
 +        self.client_add("dc/foo")
 +        self.client_set_prop("dc", "bzr:revision-info", "")
 +        self.client_set_prop("dc", "bzr:file-ids", "foo\tsomeid\n")
 +        self.client_commit("dc", "My Message")
 +        repository = Repository.open("svn+%s" % repos_url)
 +        tree = repository.revision_tree(Branch.open(repos_url).last_revision())
 +        self.assertEqual("someid", tree.inventory.path2id("foo"))
 +        self.assertFalse("1@%s::foo" % repository.uuid in tree.inventory)
 +
 +    def test_revision_ghost_parents(self):
 +        repos_url = self.make_client('d', 'dc')
 +        self.build_tree({'dc/foo': "data"})
 +        self.client_add("dc/foo")
 +        self.client_commit("dc", "My Message")
 +        self.client_update("dc")
 +        self.build_tree({'dc/foo': "data2"})
 +        self.client_set_prop("dc", "bzr:ancestry:v3-none", "ghostparent\n")
 +        self.client_commit("dc", "Second Message")
 +        repository = Repository.open("svn+%s" % repos_url)
 +        mapping = repository.get_mapping()
 +        self.assertEqual((),
 +                repository.get_revision(
 +                    repository.generate_revision_id(0, "", mapping)).parent_ids)
 +        self.assertEqual((repository.generate_revision_id(0, "", mapping),),
 +                repository.get_revision(
 +                    repository.generate_revision_id(1, "", mapping)).parent_ids)
 +        self.assertEqual((repository.generate_revision_id(1, "", mapping),
 +            "ghostparent"), 
 +                repository.get_revision(
 +                    repository.generate_revision_id(2, "", mapping)).parent_ids)
 + 
 +    def test_revision_svk_parent(self):
 +        repos_url = self.make_client('d', 'dc')
 +        self.build_tree({'dc/trunk/foo': "data", 'dc/branches/foo': None})
 +        self.client_add("dc/trunk")
 +        self.client_add("dc/branches")
 +        self.client_commit("dc", "My Message")
 +        self.client_update("dc")
 +        self.build_tree({'dc/trunk/foo': "data2"})
 +        repository = Repository.open("svn+%s" % repos_url)
 +        set_branching_scheme(repository, TrunkBranchingScheme())
 +        self.client_set_prop("dc/trunk", "svk:merge", 
 +            "%s:/branches/foo:1\n" % repository.uuid)
 +        self.client_commit("dc", "Second Message")
 +        mapping = repository.get_mapping()
 +        self.assertEqual((repository.generate_revision_id(1, "trunk", mapping),
 +            repository.generate_revision_id(1, "branches/foo", mapping)), 
 +                repository.get_revision(
 +                    repository.generate_revision_id(2, "trunk", mapping)).parent_ids)
 +    
 +    def test_get_revision(self):
 +        repos_url = self.make_client('d', 'dc')
 +        repository = Repository.open("svn+%s" % repos_url)
 +        self.assertRaises(NoSuchRevision, repository.get_revision, 
 +                "nonexisting")
 +        self.build_tree({'dc/foo': "data"})
 +        self.client_add("dc/foo")
 +        self.client_commit("dc", "My Message")
 +        self.client_update("dc")
 +        self.build_tree({'dc/foo': "data2"})
 +        (num, date, author) = self.client_commit("dc", "Second Message")
 +        repository = Repository.open("svn+%s" % repos_url)
 +        mapping = repository.get_mapping()
 +        rev = repository.get_revision(
 +            repository.generate_revision_id(2, "", mapping))
 +        self.assertEqual((repository.generate_revision_id(1, "", mapping),),
 +                rev.parent_ids)
 +        self.assertEqual(rev.revision_id, 
 +                repository.generate_revision_id(2, "", mapping))
 +        self.assertEqual(author, rev.committer)
 +        self.assertIsInstance(rev.properties, dict)
 +
 +    def test_get_revision_id_overriden(self):
 +        repos_url = self.make_client('d', 'dc')
 +        repository = Repository.open("svn+%s" % repos_url)
 +        self.assertRaises(NoSuchRevision, repository.get_revision, "nonexisting")
 +        self.build_tree({'dc/foo': "data"})
 +        self.client_add("dc/foo")
 +        self.client_commit("dc", "My Message")
 +        self.build_tree({'dc/foo': "data2"})
 +        self.client_set_prop("dc", "bzr:revision-id:v3-none", 
 +                            "3 myrevid\n")
 +        self.client_update("dc")
 +        (num, date, author) = self.client_commit("dc", "Second Message")
 +        repository = Repository.open("svn+%s" % repos_url)
 +        mapping = repository.get_mapping()
 +        revid = mapping.generate_revision_id(repository.uuid, 2, "")
 +        rev = repository.get_revision("myrevid")
 +        self.assertEqual((repository.generate_revision_id(1, "", mapping),),
 +                rev.parent_ids)
 +        self.assertEqual(rev.revision_id, 
 +                         repository.generate_revision_id(2, "", mapping))
 +        self.assertEqual(author, rev.committer)
 +        self.assertIsInstance(rev.properties, dict)
 +
 +    def test_get_revision_zero(self):
 +        repos_url = self.make_client('d', 'dc')
 +        repository = Repository.open("svn+%s" % repos_url)
 +        mapping = repository.get_mapping()
 +        rev = repository.get_revision(
 +            repository.generate_revision_id(0, "", mapping))
 +        self.assertEqual(repository.generate_revision_id(0, "", mapping), 
 +                         rev.revision_id)
 +        self.assertEqual("", rev.committer)
 +        self.assertEqual({}, rev.properties)
 +        self.assertEqual(None, rev.timezone)
 +
 +    def test_store_branching_scheme(self):
 +        repos_url = self.make_client('d', 'dc')
 +        repository = Repository.open(repos_url)
 +        set_branching_scheme(repository, TrunkBranchingScheme(42))
 +        repository = Repository.open(repos_url)
 +        self.assertEquals("trunk42", str(repository.get_mapping().scheme))
 +
 +    def test_get_ancestry(self):
 +        repos_url = self.make_client('d', 'dc')
 +        repository = Repository.open("svn+%s" % repos_url)
 +        self.assertRaises(NoSuchRevision, repository.get_revision, "nonexisting")
 +        self.build_tree({'dc/foo': "data"})
 +        self.client_add("dc/foo")
 +        self.client_commit("dc", "My Message")
 +        self.client_update("dc")
 +        self.build_tree({'dc/foo': "data2"})
 +        self.client_commit("dc", "Second Message")
 +        self.client_update("dc")
 +        self.build_tree({'dc/foo': "data3"})
 +        self.client_commit("dc", "Third Message")
 +        self.client_update("dc")
 +        repository = Repository.open("svn+%s" % repos_url)
 +        mapping = repository.get_mapping()
 +        self.assertEqual([None, 
 +            repository.generate_revision_id(0, "", mapping),
 +            repository.generate_revision_id(1, "", mapping),
 +            repository.generate_revision_id(2, "", mapping),
 +            repository.generate_revision_id(3, "", mapping)],
 +                repository.get_ancestry(
 +                    repository.generate_revision_id(3, "", mapping)))
 +        self.assertEqual([None, 
 +            repository.generate_revision_id(0, "", mapping),
 +            repository.generate_revision_id(1, "", mapping),
 +            repository.generate_revision_id(2, "", mapping)],
 +                repository.get_ancestry(
 +                    repository.generate_revision_id(2, "", mapping)))
 +        self.assertEqual([None,
 +                    repository.generate_revision_id(0, "", mapping),
 +                    repository.generate_revision_id(1, "", mapping)],
 +                repository.get_ancestry(
 +                    repository.generate_revision_id(1, "", mapping)))
 +        self.assertEqual([None, repository.generate_revision_id(0, "", mapping)],
 +                repository.get_ancestry(
 +                    repository.generate_revision_id(0, "", mapping)))
 +        self.assertEqual([None], repository.get_ancestry(NULL_REVISION))
 +
 +    def test_get_ancestry2(self):
 +        repos_url = self.make_client('d', 'dc')
 +        self.build_tree({'dc/foo': "data"})
 +        self.client_add("dc/foo")
 +        self.client_commit("dc", "My Message")
 +        self.build_tree({'dc/foo': "data2"})
 +        self.client_commit("dc", "Second Message")
 +        repository = Repository.open("svn+%s" % repos_url)
 +        mapping = repository.get_mapping()
 +        self.assertEqual([None, repository.generate_revision_id(0, "", mapping)],
 +                repository.get_ancestry(
 +                    repository.generate_revision_id(0, "", mapping)))
 +        self.assertEqual([None, repository.generate_revision_id(0, "", mapping),
 +            repository.generate_revision_id(1, "", mapping)],
 +                repository.get_ancestry(
 +                    repository.generate_revision_id(1, "", mapping)))
 +        self.assertEqual([None, 
 +            repository.generate_revision_id(0, "", mapping),
 +            repository.generate_revision_id(1, "", mapping),
 +            repository.generate_revision_id(2, "", mapping)], 
 +                repository.get_ancestry(
 +                    repository.generate_revision_id(2, "", mapping)))
 +
 +    def test_get_ancestry_merged(self):
 +        repos_url = self.make_client('d', 'dc')
 +        self.build_tree({'dc/foo': "data"})
 +        self.client_add("dc/foo")
 +        self.client_commit("dc", "My Message")
 +        self.client_update("dc")
 +        self.client_set_prop("dc", "bzr:ancestry:v3-none", "a-parent\n")
 +        self.build_tree({'dc/foo': "data2"})
 +        self.client_commit("dc", "Second Message")
 +        repository = Repository.open("svn+%s" % repos_url)
 +        mapping = repository.get_mapping()
 +        self.assertEqual([None, repository.generate_revision_id(0, "", mapping)],
 +                repository.get_ancestry(
 +                    repository.generate_revision_id(0, "", mapping)))
 +        self.assertEqual([None, repository.generate_revision_id(0, "", mapping),
 +            repository.generate_revision_id(1, "", mapping)],
 +                repository.get_ancestry(
 +                    repository.generate_revision_id(1, "", mapping)))
 +        self.assertEqual([None, 
 +            repository.generate_revision_id(0, "", mapping), "a-parent", 
 +            repository.generate_revision_id(1, "", mapping), 
 +                  repository.generate_revision_id(2, "", mapping)], 
 +                repository.get_ancestry(
 +                    repository.generate_revision_id(2, "", mapping)))
 +
 +    def test_get_inventory(self):
 +        repos_url = self.make_client('d', 'dc')
 +        repository = Repository.open("svn+%s" % repos_url)
 +        self.assertRaises(NoSuchRevision, repository.get_inventory, 
 +                "nonexisting")
 +        self.build_tree({'dc/foo': "data", 'dc/blah': "other data"})
 +        self.client_add("dc/foo")
 +        self.client_add("dc/blah")
 +        self.client_commit("dc", "My Message") #1
 +        self.client_update("dc")
 +        self.build_tree({'dc/foo': "data2", "dc/bar/foo": "data3"})
 +        self.client_add("dc/bar")
 +        self.client_commit("dc", "Second Message") #2
 +        self.client_update("dc")
 +        self.build_tree({'dc/foo': "data3"})
 +        self.client_commit("dc", "Third Message") #3
 +        self.client_update("dc")
 +        repository = Repository.open("svn+%s" % repos_url)
 +        mapping = repository.get_mapping()
 +        inv = repository.get_inventory(
 +                repository.generate_revision_id(1, "", mapping))
 +        self.assertIsInstance(inv, Inventory)
 +        self.assertIsInstance(inv.path2id("foo"), basestring)
 +        inv = repository.get_inventory(
 +            repository.generate_revision_id(2, "", mapping))
 +        self.assertEqual(repository.generate_revision_id(2, "", mapping), 
 +                         inv[inv.path2id("foo")].revision)
 +        self.assertEqual(repository.generate_revision_id(1, "", mapping), 
 +                         inv[inv.path2id("blah")].revision)
 +        self.assertIsInstance(inv, Inventory)
 +        self.assertIsInstance(inv.path2id("foo"), basestring)
 +        self.assertIsInstance(inv.path2id("bar"), basestring)
 +        self.assertIsInstance(inv.path2id("bar/foo"), basestring)
 +
 +    def test_generate_revision_id(self):
 +        repos_url = self.make_client('d', 'dc')
 +        self.build_tree({'dc/bla/bloe': None})
 +        self.client_add("dc/bla")
 +        self.client_commit("dc", "bla")
 +        repository = Repository.open("svn+%s" % repos_url)
 +        mapping = repository.get_mapping()
 +        self.assertEqual(
 +               mapping.generate_revision_id(repository.uuid, 1, "bla/bloe"), 
 +            repository.generate_revision_id(1, "bla/bloe", mapping))
 +
 +    def test_generate_revision_id_zero(self):
 +        repos_url = self.make_client('d', 'dc')
 +        repository = Repository.open("svn+%s" % repos_url)
 +        mapping = repository.get_mapping()
 +        self.assertEqual(mapping.generate_revision_id(repository.uuid, 0, ""), 
 +                repository.generate_revision_id(0, "", mapping))
 +
 +    def test_lookup_revision_id(self):
 +        repos_url = self.make_client('d', 'dc')
 +        self.build_tree({'dc/bloe': None})
 +        self.client_add("dc/bloe")
 +        self.client_commit("dc", "foobar")
 +        repository = Repository.open("svn+%s" % repos_url)
 +        self.assertRaises(NoSuchRevision, repository.lookup_revision_id, 
 +            "nonexisting")
 +        mapping = repository.get_mapping()
 +        self.assertEqual(("bloe", 1), 
 +            repository.lookup_revision_id(
 +                repository.generate_revision_id(1, "bloe", mapping))[:2])
 +
 +    def test_lookup_revision_id_overridden(self):
 +        repos_url = self.make_client('d', 'dc')
 +        self.build_tree({'dc/bloe': None})
 +        self.client_add("dc/bloe")
 +        self.client_set_prop("dc", SVN_PROP_BZR_REVISION_ID+"none", "2 myid\n")
 +        self.client_commit("dc", "foobar")
 +        repository = Repository.open("svn+%s" % repos_url)
 +        mapping = repository.get_mapping()
 +        self.assertEqual(("", 1), repository.lookup_revision_id( 
 +            mapping.generate_revision_id(repository.uuid, 1, ""))[:2])
 +        self.assertEqual(("", 1), 
 +                repository.lookup_revision_id("myid")[:2])
 +
 +    def test_lookup_revision_id_overridden_invalid(self):
 +        repos_url = self.make_client('d', 'dc')
 +        self.build_tree({'dc/bloe': None})
 +        self.client_add("dc/bloe")
 +        self.client_set_prop("dc", SVN_PROP_BZR_REVISION_ID+"none", 
 +                             "corrupt-entry\n")
 +        self.client_commit("dc", "foobar")
 +        repository = Repository.open("svn+%s" % repos_url)
 +        mapping = repository.get_mapping()
 +        self.assertEqual(("", 1), repository.lookup_revision_id( 
 +            mapping.generate_revision_id(repository.uuid, 1, ""))[:2])
 +        self.assertRaises(NoSuchRevision, repository.lookup_revision_id, 
 +            "corrupt-entry")
 +
 +    def test_lookup_revision_id_overridden_invalid_dup(self):
 +        repos_url = self.make_client('d', 'dc')
 +        self.build_tree({'dc/bloe': None})
 +        self.client_add("dc/bloe")
 +        self.client_set_prop("dc", SVN_PROP_BZR_REVISION_ID+"none", 
 +                             "corrupt-entry\n")
 +        self.client_commit("dc", "foobar")
 +        self.build_tree({'dc/bla': None})
 +        self.client_add("dc/bla")
 +        self.client_set_prop("dc", SVN_PROP_BZR_REVISION_ID+"none", 
 +                "corrupt-entry\n2 corrupt-entry\n")
 +        self.client_commit("dc", "foobar")
 +        repository = Repository.open("svn+%s" % repos_url)
 +        mapping = repository.get_mapping()
 +        self.assertEqual(("", 2), repository.lookup_revision_id( 
 +            mapping.generate_revision_id(repository.uuid, 2, ""))[:2])
 +        self.assertEqual(("", 1), repository.lookup_revision_id( 
 +            mapping.generate_revision_id(repository.uuid, 1, ""))[:2])
 +        self.assertEqual(("", 2), repository.lookup_revision_id( 
 +            "corrupt-entry")[:2])
 +
 +    def test_lookup_revision_id_overridden_not_found(self):
 +        """Make sure a revision id that is looked up but doesn't exist 
 +        doesn't accidently end up in the revid cache."""
 +        repos_url = self.make_client('d', 'dc')
 +        self.build_tree({'dc/bloe': None})
 +        self.client_add("dc/bloe")
 +        self.client_set_prop("dc", SVN_PROP_BZR_REVISION_ID+"none", "2 myid\n")
 +        self.client_commit("dc", "foobar")
 +        repository = Repository.open("svn+%s" % repos_url)
 +        self.assertRaises(NoSuchRevision, 
 +                repository.lookup_revision_id, "foobar")
 +
 +    def test_set_branching_scheme_property(self):
 +        repos_url = self.make_client('d', 'dc')
 +        self.client_set_prop("dc", SVN_PROP_BZR_BRANCHING_SCHEME, 
 +            "trunk\nbranches/*\nbranches/tmp/*")
 +        self.client_commit("dc", "set scheme")
 +        repository = Repository.open("svn+%s" % repos_url)
 +        self.assertEquals(ListBranchingScheme(["trunk", "branches/*", "branches/tmp/*"]).branch_list,
 +                          repository.get_mapping().scheme.branch_list)
 +
 +    def test_set_property_scheme(self):
 +        repos_url = self.make_client('d', 'dc')
 +        repos = Repository.open(repos_url)
 +        set_property_scheme(repos, ListBranchingScheme(["bla/*"]))
 +        self.client_update("dc")
 +        self.assertEquals("bla/*\n", 
 +                   self.client_get_prop("dc", SVN_PROP_BZR_BRANCHING_SCHEME))
 +        self.assertEquals("Updating branching scheme for Bazaar.", 
 +                self.client_log("dc", 1, 1)[1][3])
 +
 +    def test_lookup_revision_id_invalid_uuid(self):
 +        repos_url = self.make_client('d', 'dc')
 +        repository = Repository.open("svn+%s" % repos_url)
 +        mapping = repository.get_mapping()
 +        self.assertRaises(NoSuchRevision, 
 +            repository.lookup_revision_id, 
 +                mapping.generate_revision_id("invaliduuid", 0, ""))
 +        
 +    def test_check(self):
 +        repos_url = self.make_client('d', 'dc')
 +        self.build_tree({'dc/foo': "data"})
 +        self.client_add("dc/foo")
 +        self.client_commit("dc", "My Message")
 +        repository = Repository.open("svn+%s" % repos_url)
 +        mapping = repository.get_mapping()
 +        repository.check([
 +            repository.generate_revision_id(0, "", mapping), 
 +            repository.generate_revision_id(1, "", mapping)])
 +
 +    def test_copy_contents_into(self):
 +        repos_url = self.make_client('d', 'dc')
 +        self.build_tree({'dc/foo/bla': "data"})
 +        self.client_add("dc/foo")
 +        self.client_commit("dc", "My Message")
 +        self.build_tree({'dc/foo/blo': "data2", "dc/bar/foo": "data3", 'dc/foo/bla': "data"})
 +        self.client_add("dc/foo/blo")
 +        self.client_add("dc/bar")
 +        self.client_commit("dc", "Second Message")
 +        repository = Repository.open("svn+%s" % repos_url)
 +        mapping = repository.get_mapping()
 +
 +        to_bzrdir = BzrDir.create("e", format.get_rich_root_format())
 +        to_repos = to_bzrdir.create_repository()
 +
 +        repository.copy_content_into(to_repos, 
 +                repository.generate_revision_id(2, "", mapping))
 +
 +        self.assertTrue(repository.has_revision(
 +            repository.generate_revision_id(2, "", mapping)))
 +        self.assertTrue(repository.has_revision(
 +            repository.generate_revision_id(1, "", mapping)))
 +
 +    def test_is_shared(self):
 +        repos_url = self.make_client('d', 'dc')
 +        self.build_tree({'dc/foo/bla': "data"})
 +        self.client_add("dc/foo")
 +        self.client_commit("dc", "My Message")
 +        repository = Repository.open("svn+%s" % repos_url)
 +        self.assertTrue(repository.is_shared())
 +
 +    def test_fetch_property_change_only_trunk(self):
 +        repos_url = self.make_client('d', 'dc')
 +        self.build_tree({'dc/trunk/bla': "data"})
 +        self.client_add("dc/trunk")
 +        self.client_commit("dc", "My Message")
 +        self.client_set_prop("dc/trunk", "some:property", "some data\n")
 +        self.client_commit("dc", "My 3")
 +        self.client_set_prop("dc/trunk", "some2:property", "some data\n")
 +        self.client_commit("dc", "My 2")
 +        self.client_set_prop("dc/trunk", "some:property", "some other data\n")
 +        self.client_commit("dc", "My 4")
 +        oldrepos = Repository.open("svn+"+repos_url)
 +        self.assertEquals([('trunk', {'trunk': (u'M', None, -1)}, 3), 
 +                           ('trunk', {'trunk': (u'M', None, -1)}, 2), 
 +                           ('trunk', {'trunk/bla': (u'A', None, -1), 'trunk': (u'A', None, -1)}, 1)], 
 +                   [(l.branch_path, l.paths, l.revnum) for l in oldrepos.iter_reverse_branch_changes("trunk", 3, TrunkBranchingScheme())])
 +
 +    def test_control_code_msg(self):
 +        repos_url = self.make_client('d', 'dc')
 +
 +        self.build_tree({'dc/trunk': None})
 +        self.client_add("dc/trunk")
 +        self.client_commit("dc", "\x24")
 +
 +        self.build_tree({'dc/trunk/hosts': 'hej2'})
 +        self.client_add("dc/trunk/hosts")
 +        self.client_commit("dc", "bla\xfcbla") #2
 +
 +        self.build_tree({'dc/trunk/hosts': 'hej3'})
 +        self.client_commit("dc", "a\x0cb") #3
 +
 +        self.build_tree({'dc/branches/foobranch/file': 'foohosts'})
 +        self.client_add("dc/branches")
 +        self.client_commit("dc", "foohosts") #4
 +
 +        oldrepos = Repository.open("svn+"+repos_url)
 +        set_branching_scheme(oldrepos, TrunkBranchingScheme())
 +        dir = BzrDir.create("f",format=format.get_rich_root_format())
 +        newrepos = dir.create_repository()
 +        oldrepos.copy_content_into(newrepos)
 +
 +        mapping = oldrepos.get_mapping()
 +
 +        self.assertTrue(newrepos.has_revision(
 +            oldrepos.generate_revision_id(1, "trunk", mapping)))
 +        self.assertTrue(newrepos.has_revision(
 +            oldrepos.generate_revision_id(2, "trunk", mapping)))
 +        self.assertTrue(newrepos.has_revision(
 +            oldrepos.generate_revision_id(3, "trunk", mapping)))
 +        self.assertTrue(newrepos.has_revision(
 +            oldrepos.generate_revision_id(4, "branches/foobranch", mapping)))
 +        self.assertFalse(newrepos.has_revision(
 +            oldrepos.generate_revision_id(4, "trunk", mapping)))
 +        self.assertFalse(newrepos.has_revision(
 +            oldrepos.generate_revision_id(2, "", mapping)))
 +
 +        rev = newrepos.get_revision(oldrepos.generate_revision_id(1, "trunk", mapping))
 +        self.assertEqual("$", rev.message)
 +
 +        rev = newrepos.get_revision(
 +            oldrepos.generate_revision_id(2, "trunk", mapping))
 +        self.assertEqual('bla\xc3\xbcbla', rev.message.encode("utf-8"))
 +
 +        rev = newrepos.get_revision(oldrepos.generate_revision_id(3, "trunk", mapping))
 +        self.assertEqual(u"a\\x0cb", rev.message)
 +
 +    def test_set_branching_scheme(self):
 +        repos_url = self.make_client('d', 'dc')
 +        repos = Repository.open(repos_url)
 +        set_branching_scheme(repos, NoBranchingScheme())
 +
 +    def testlhs_revision_parent_none(self):
 +        repos_url = self.make_client('d', 'dc')
 +        repos = Repository.open(repos_url)
 +        set_branching_scheme(repos, NoBranchingScheme())
 +        self.assertEquals(NULL_REVISION, repos.lhs_revision_parent("", 0, NoBranchingScheme()))
 +
 +    def testlhs_revision_parent_first(self):
 +        repos_url = self.make_client('d', 'dc')
 +        repos = Repository.open(repos_url)
 +        set_branching_scheme(repos, NoBranchingScheme())
 +        self.build_tree({'dc/adir/afile': "data"})
 +        self.client_add("dc/adir")
 +        self.client_commit("dc", "Initial commit")
 +        mapping = repos.get_mapping()
 +        self.assertEquals(repos.generate_revision_id(0, "", mapping), \
 +                repos.lhs_revision_parent("", 1, mapping))
 +
 +    def testlhs_revision_parent_simple(self):
 +        repos_url = self.make_client('d', 'dc')
 +        self.build_tree({'dc/trunk/adir/afile': "data", 
 +                         'dc/trunk/adir/stationary': None,
 +                         'dc/branches/abranch': None})
 +        self.client_add("dc/trunk")
 +        self.client_add("dc/branches")
 +        self.client_commit("dc", "Initial commit")
 +        self.build_tree({'dc/trunk/adir/afile': "bla"})
 +        self.client_commit("dc", "Incremental commit")
 +        repos = Repository.open(repos_url)
 +        set_branching_scheme(repos, TrunkBranchingScheme())
 +        mapping = repos.get_mapping()
 +        self.assertEquals(repos.generate_revision_id(1, "trunk", mapping), \
 +                repos.lhs_revision_parent("trunk", 2, mapping))
 +
 +    def testlhs_revision_parent_copied(self):
 +        repos_url = self.make_client('d', 'dc')
 +        self.build_tree({'dc/py/trunk/adir/afile': "data", 
 +                         'dc/py/trunk/adir/stationary': None})
 +        self.client_add("dc/py")
 +        self.client_commit("dc", "Initial commit")
 +        self.client_copy("dc/py", "dc/de")
 +        self.client_commit("dc", "Incremental commit")
 +        self.build_tree({'dc/de/trunk/adir/afile': "bla"})
 +        self.client_commit("dc", "Change de")
 +        repos = Repository.open(repos_url)
 +        set_branching_scheme(repos, TrunkBranchingScheme(1))
 +        mapping = repos.get_mapping()
 +        self.assertEquals(repos.generate_revision_id(1, "py/trunk", mapping), \
 +                repos.lhs_revision_parent("de/trunk", 3, mapping))
 +
 +    def test_mainline_revision_copied(self):
 +        repos_url = self.make_client('d', 'dc')
 +        self.build_tree({'dc/py/trunk/adir/afile': "data", 
 +                         'dc/py/trunk/adir/stationary': None})
 +        self.client_add("dc/py")
 +        self.client_commit("dc", "Initial commit")
 +        self.build_tree({'dc/de':None})
 +        self.client_add("dc/de")
 +        self.client_copy("dc/py/trunk", "dc/de/trunk")
 +        self.client_commit("dc", "Copy trunk")
 +        repos = Repository.open(repos_url)
 +        set_branching_scheme(repos, TrunkBranchingScheme(1))
 +        mapping = repos.get_mapping()
 +        self.assertEquals(repos.generate_revision_id(1, "py/trunk", mapping), \
 +                repos.lhs_revision_parent("de/trunk", 2, mapping))
 +
 +    def test_mainline_revision_nested_deleted(self):
 +        repos_url = self.make_client('d', 'dc')
 +        self.build_tree({'dc/py/trunk/adir/afile': "data", 
 +                         'dc/py/trunk/adir/stationary': None})
 +        self.client_add("dc/py")
 +        self.client_commit("dc", "Initial commit")
 +        self.client_copy("dc/py", "dc/de")
 +        self.client_commit("dc", "Incremental commit")
 +        self.client_delete("dc/de/trunk/adir")
 +        self.client_commit("dc", "Another incremental commit")
 +        repos = Repository.open(repos_url)
 +        set_branching_scheme(repos, TrunkBranchingScheme(1))
 +        mapping = repos.get_mapping()
 +        self.assertEquals(repos.generate_revision_id(1, "py/trunk", mapping), \
 +                repos.lhs_revision_parent("de/trunk", 3, mapping))
 +
 +    def test_mainline_revision_missing(self):
 +        repos_url = self.make_client('d', 'dc')
 +        repos = Repository.open(repos_url)
 +        self.build_tree({'dc/py/trunk/adir/afile': "data", 
 +                         'dc/py/trunk/adir/stationary': None})
 +        self.client_add("dc/py")
 +        self.client_commit("dc", "Initial commit")
 +        self.assertRaises(NoSuchRevision, 
 +                lambda: repos.lhs_revision_parent("trunk", 2, repos.get_mapping()))
 +
 +
 +class TestSvnRevisionTree(TestCaseWithSubversionRepository):
 +    def setUp(self):
 +        super(TestSvnRevisionTree, self).setUp()
 +        repos_url = self.make_client('d', 'dc')
 +        self.build_tree({'dc/foo/bla': "data"})
 +        self.client_add("dc/foo")
 +        self.client_commit("dc", "My Message")
 +        self.repos = Repository.open(repos_url)
 +        mapping = self.repos.get_mapping()
 +        self.inventory = self.repos.get_inventory(
 +                self.repos.generate_revision_id(1, "", mapping))
 +        self.tree = self.repos.revision_tree(
 +                self.repos.generate_revision_id(1, "", mapping))
 +
 +    def test_inventory(self):
 +        self.assertIsInstance(self.tree.inventory, Inventory)
 +        self.assertEqual(self.inventory, self.tree.inventory)
 +
 +    def test_get_parent_ids(self):
 +        mapping = self.repos.get_mapping()
 +        self.assertEqual((self.repos.generate_revision_id(0, "", mapping),), self.tree.get_parent_ids())
 +
 +    def test_get_parent_ids_zero(self):
 +        mapping = self.repos.get_mapping()
 +        tree = self.repos.revision_tree(
 +                self.repos.generate_revision_id(0, "", mapping))
 +        self.assertEqual((), tree.get_parent_ids())
 +
 +    def test_get_revision_id(self):
 +        mapping = self.repos.get_mapping()
 +        self.assertEqual(self.repos.generate_revision_id(1, "", mapping),
 +                         self.tree.get_revision_id())
 +
 +    def test_get_file_lines(self):
 +        self.assertEqual(["data"], 
 +                self.tree.get_file_lines(self.inventory.path2id("foo/bla")))
 +
 +    def test_executable(self):
 +        self.client_set_prop("dc/foo/bla", "svn:executable", "*")
 +        self.client_commit("dc", "My Message")
 +
 +        mapping = self.repos.get_mapping()
 +        
 +        inventory = self.repos.get_inventory(
 +                self.repos.generate_revision_id(2, "", mapping))
 +
 +        self.assertTrue(inventory[inventory.path2id("foo/bla")].executable)
 +
 +    def test_symlink(self):
 +        if not has_symlinks():
 +            return
 +        os.symlink('foo/bla', 'dc/bar')
 +        self.client_add('dc/bar')
 +        self.client_commit("dc", "My Message")
 +
 +        mapping = self.repos.get_mapping()
 +        
 +        inventory = self.repos.get_inventory(
 +                self.repos.generate_revision_id(2, "", mapping))
 +
 +        self.assertEqual('symlink', inventory[inventory.path2id("bar")].kind)
 +        self.assertEqual('foo/bla', 
 +                inventory[inventory.path2id("bar")].symlink_target)
 +
 +    def test_not_executable(self):
 +        self.assertFalse(self.inventory[
 +            self.inventory.path2id("foo/bla")].executable)
 +
 +
 +class EscapeTest(TestCase):
 +    def test_escape_svn_path_none(self):      
 +        self.assertEqual("", escape_svn_path(""))
 +
 +    def test_escape_svn_path_simple(self):
 +        self.assertEqual("ab", escape_svn_path("ab"))
 +
 +    def test_escape_svn_path_percent(self):
 +        self.assertEqual("a%25b", escape_svn_path("a%b"))
 +
 +    def test_escape_svn_path_whitespace(self):
 +        self.assertEqual("foobar%20", escape_svn_path("foobar "))
 +
 +    def test_escape_svn_path_slash(self):
 +        self.assertEqual("foobar%2F", escape_svn_path("foobar/"))
 +
 +    def test_escape_svn_path_special_char(self):
 +        self.assertEqual("foobar%8A", escape_svn_path("foobar\x8a"))
 +
 +    def test_unescape_svn_path_slash(self):
 +        self.assertEqual("foobar/", unescape_svn_path("foobar%2F"))
 +
 +    def test_unescape_svn_path_none(self):
 +        self.assertEqual("foobar", unescape_svn_path("foobar"))
 +
 +    def test_unescape_svn_path_percent(self):
 +        self.assertEqual("foobar%b", unescape_svn_path("foobar%25b"))
 +
 +    def test_escape_svn_path_nordic(self):
 +        self.assertEqual("foobar%C3%A6", escape_svn_path(u"foobar\xe6".encode("utf-8")))
 +
 +
 +class SvnRepositoryFormatTests(TestCase):
 +    def setUp(self):
 +        self.format = SvnRepositoryFormat()
 +
 +    def test_initialize(self):
 +        self.assertRaises(UninitializableFormat, self.format.initialize, None)
 +
 +    def test_get_format_description(self):
 +        self.assertEqual("Subversion Repository", 
 +                         self.format.get_format_description())
 +
 +    def test_conversion_target_self(self):
 +        self.assertTrue(self.format.check_conversion_target(self.format))
 +
 +    def test_conversion_target_incompatible(self):
 +        self.assertFalse(self.format.check_conversion_target(
 +              format_registry.make_bzrdir('weave').repository_format))
 +
 +    def test_conversion_target_compatible(self):
 +        self.assertTrue(self.format.check_conversion_target(
 +          format_registry.make_bzrdir('rich-root').repository_format))
 +
 +
 +
diff --cc tree.py
index e74d8589,558434db..fe2d6005
--- a/tree.py
+++ b/tree.py
@@@ -29,9 -27,10 +27,10 @@@ import md
  from cStringIO import StringIO
  import urllib
  
 -from bzrlib.plugins.svn import errors, properties, core, wc
 +from bzrlib.plugins.svn import core, errors, properties, wc
  from bzrlib.plugins.svn.delta import apply_txdelta_handler
  
+ 
  def parse_externals_description(base_url, val):
      """Parse an svn:externals property value.