You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@allura.apache.org by br...@apache.org on 2013/02/13 00:17:06 UTC
[2/2] git commit: [#5810] standardize slashes in tree_paths
Updated Branches:
refs/heads/db/5810 [created] 5388f06ca
[#5810] standardize slashes in tree_paths
Project: http://git-wip-us.apache.org/repos/asf/incubator-allura/repo
Commit: http://git-wip-us.apache.org/repos/asf/incubator-allura/commit/5388f06c
Tree: http://git-wip-us.apache.org/repos/asf/incubator-allura/tree/5388f06c
Diff: http://git-wip-us.apache.org/repos/asf/incubator-allura/diff/5388f06c
Branch: refs/heads/db/5810
Commit: 5388f06ca6dcda701bcf0cb020ec440e54347e81
Parents: 708786f
Author: Dave Brondsema <db...@geek.net>
Authored: Tue Feb 12 23:16:59 2013 +0000
Committer: Dave Brondsema <db...@geek.net>
Committed: Tue Feb 12 23:16:59 2013 +0000
----------------------------------------------------------------------
ForgeSVN/forgesvn/model/svn.py | 2 +-
.../forgesvn/tests/model/test_svnimplementation.py | 39 +++++++++++++++
2 files changed, 40 insertions(+), 1 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/incubator-allura/blob/5388f06c/ForgeSVN/forgesvn/model/svn.py
----------------------------------------------------------------------
diff --git a/ForgeSVN/forgesvn/model/svn.py b/ForgeSVN/forgesvn/model/svn.py
index 64a36ca..11e79c1 100644
--- a/ForgeSVN/forgesvn/model/svn.py
+++ b/ForgeSVN/forgesvn/model/svn.py
@@ -396,7 +396,7 @@ class SVNImplementation(M.RepositoryImplementation):
def compute_tree_new(self, commit, tree_path='/'):
from allura.model import repo as RM
- tree_path = tree_path[:-1]
+ tree_path = '/' + tree_path.strip('/') # always leading slash, never trailing
tree_id = self._tree_oid(commit._id, tree_path)
tree = RM.Tree.query.get(_id=tree_id)
if tree:
http://git-wip-us.apache.org/repos/asf/incubator-allura/blob/5388f06c/ForgeSVN/forgesvn/tests/model/test_svnimplementation.py
----------------------------------------------------------------------
diff --git a/ForgeSVN/forgesvn/tests/model/test_svnimplementation.py b/ForgeSVN/forgesvn/tests/model/test_svnimplementation.py
new file mode 100644
index 0000000..417e9cd
--- /dev/null
+++ b/ForgeSVN/forgesvn/tests/model/test_svnimplementation.py
@@ -0,0 +1,39 @@
+from mock import Mock, MagicMock, patch
+import pysvn
+from nose.tools import assert_equal
+
+from allura.model.repo import Commit
+from forgesvn.model.svn import Repository, SVNImplementation
+
+
+class TestSVNImplementation(object):
+
+ def test_compute_tree_new(self):
+ self._test_compute_tree_new('/trunk/foo/')
+ self._test_compute_tree_new('/trunk/foo')
+ self._test_compute_tree_new('trunk/foo/')
+ self._test_compute_tree_new('trunk/foo')
+
+ @patch('allura.model.repo.LastCommitDoc.m.update_partial')
+ @patch('allura.model.repo.TreesDoc.m.update_partial')
+ @patch('allura.model.repo.Tree.upsert')
+ @patch('allura.model.repo.Tree.query.get')
+ def _test_compute_tree_new(self, path, tree_get, tree_upsert, treesdoc_partial, lcd_partial):
+ repo = Mock(fs_path='/tmp/')
+ repo.name = 'code'
+ impl = SVNImplementation(repo)
+ impl._svn.info2 = Mock()
+ impl._svn.info2.return_value = [('foo', Mock())]
+ tree_get.return_value = None # no existing tree
+ commit = Commit()
+ commit._id = '5057636b9c1040636b81e4b1:6'
+ tree_upsert.return_value = (Mock(), True)
+
+ tree_id = impl.compute_tree_new(commit, path)
+
+ assert_equal(impl._svn.info2.call_args[0][0], 'file:///tmp/code/trunk/foo')
+ treesdoc_partial.assert_called()
+ lcd_partial.assert_called()
+
+ def test_last_commit_ids(self):
+ pass
\ No newline at end of file