Rev 1892: Add test for seen_bzr_revprops. in file:///data/jelmer/bzr-svn/trunk/

[Jelmer Vernooij]

At file:///data/jelmer/bzr-svn/trunk/

------------------------------------------------------------
revno: 1892
revision-id: jelmer at samba.org-20080909162126-x0y1g097i0yrn8wf
parent: jelmer at samba.org-20080909160532-tyc75az5lgaheo2n
committer: Jelmer Vernooij <jelmer at samba.org>
branch nick: trunk
timestamp: Tue 2008-09-09 18:21:26 +0200
message:
  Add test for seen_bzr_revprops.
modified:
  repository.py                  repository.py-20060306123302-1f8c5069b3fe0265
  revmeta.py                     revmeta.py-20080901215045-n8a6arqybs9ez5hl-1
  tests/test_repository.py       test_repos.py-20060508151940-ddc49a59257ca712
=== modified file 'repository.py'
--- a/repository.py	2008-09-07 22:15:06 +0000
+++ b/repository.py	2008-09-09 16:21:26 +0000
@@ -507,9 +507,9 @@
         repository.
 
         """
-        if self.check_revprops == False:
+        if self.transport.has_capability("commit-revprops") == False:
             return False
-        for revmeta in self._revmeta_provider.iter_all_changes(self.get_layout(), self.get_latest_revnum()):
+        for revmeta in self._revmeta_provider.iter_all_changes(self.get_layout(), None, self.get_latest_revnum()):
             if revmeta.is_bzr_revision_revprops():
                 return True
         return False

=== modified file 'revmeta.py'
--- a/revmeta.py	2008-09-09 13:36:52 +0000
+++ b/revmeta.py	2008-09-09 16:21:26 +0000
@@ -221,7 +221,7 @@
         if self.consider_bzr_fileprops():
             order.append(self.is_bzr_revision_fileprops)
         # Only look for revprops if they could've been committed
-        if (not self._log.quick_revprops and self.consider_bzr_revprops()):
+        if ((not self._log.quick_revprops) and self.consider_bzr_revprops()):
             order.append(self.is_bzr_revision_revprops)
         for fn in order:
             ret = fn()

=== modified file 'tests/test_repository.py'
--- a/tests/test_repository.py	2008-09-04 11:47:09 +0000
+++ b/tests/test_repository.py	2008-09-09 16:21:26 +0000
@@ -84,10 +84,11 @@
         self.assertEqual(fs.get_uuid(), repository.uuid)
 
     def test_is_shared(self):
-        self.make_checkout(self.repos_url, 'dc')
-        self.build_tree({'dc/foo/bla': "data"})
-        self.client_add("dc/foo")
-        self.client_commit("dc", "My Message")
+        dc = self.get_commit_editor(self.repos_url)
+        foo = dc.add_dir("foo")
+        bla = foo.add_file("foo/bla").modify("data")
+        dc.close()
+
         repository = Repository.open(self.repos_url)
         self.assertTrue(repository.is_shared())
 
@@ -109,6 +110,14 @@
         repos = Repository.open(self.repos_url)
         self.assertFalse(repos.get_physical_lock_status())
 
+    def test_seen_bzr_revprops(self):
+        repos = Repository.open(self.repos_url)
+        dc = self.get_commit_editor(self.repos_url)
+        dc.add_dir("foo")
+        dc.close()
+
+        self.assertFalse(repos.seen_bzr_revprops())
+
 
 class SvnRepositoryFormatTests(TestCase):
     def setUp(self):