test_git.py
1251 lines
| 52.5 KiB
| text/x-python
|
PythonLexer
r5608 | # Copyright (C) 2010-2024 RhodeCode GmbH | |||
r1 | # | |||
# This program is free software: you can redistribute it and/or modify | ||||
# it under the terms of the GNU Affero General Public License, version 3 | ||||
# (only), as published by the Free Software Foundation. | ||||
# | ||||
# 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 Affero General Public License | ||||
# along with this program. If not, see <http://www.gnu.org/licenses/>. | ||||
# | ||||
# This program is dual-licensed. If you wish to learn more about the | ||||
# RhodeCode Enterprise Edition, including its added features, Support services, | ||||
# and proprietary license terms, please see https://rhodecode.com/licenses/ | ||||
import datetime | ||||
import mock | ||||
import os | ||||
import sys | ||||
r1577 | import shutil | |||
r1 | ||||
import pytest | ||||
r1577 | from rhodecode.lib.utils import make_db_config | |||
r1 | from rhodecode.lib.vcs.backends.base import Reference | |||
r5607 | from rhodecode.lib.vcs.backends.git import GitRepository, GitCommit, discover_git_version | |||
from rhodecode.lib.vcs.exceptions import RepositoryError, VCSError, NodeDoesNotExistError | ||||
r5647 | from rhodecode.lib.vcs.nodes import NodeKind, FileNode, DirNode, NodeState, SubModuleNode, RootNode | |||
r1 | from rhodecode.tests import TEST_GIT_REPO, TEST_GIT_REPO_CLONE, get_new_dir | |||
r2453 | from rhodecode.tests.vcs.conftest import BackendTestMixin | |||
r1 | ||||
pytestmark = pytest.mark.backends("git") | ||||
r5607 | DIFF_FROM_REMOTE = rb"""diff --git a/foobar b/foobar | |||
r5087 | new file mode 100644 | |||
index 0000000..f6ea049 | ||||
--- /dev/null | ||||
+++ b/foobar | ||||
@@ -0,0 +1 @@ | ||||
+foobar | ||||
\ No newline at end of file | ||||
diff --git a/foobar2 b/foobar2 | ||||
new file mode 100644 | ||||
index 0000000..e8c9d6b | ||||
--- /dev/null | ||||
+++ b/foobar2 | ||||
@@ -0,0 +1 @@ | ||||
+foobar2 | ||||
\ No newline at end of file | ||||
""" | ||||
def callable_get_diff(*args, **kwargs): | ||||
return DIFF_FROM_REMOTE | ||||
r3778 | class TestGitRepository(object): | |||
r1 | @pytest.fixture(autouse=True) | |||
r2351 | def prepare(self, request, baseapp): | |||
r1 | self.repo = GitRepository(TEST_GIT_REPO, bare=True) | |||
r3842 | self.repo.count() | |||
r1 | ||||
r3951 | def get_clone_repo(self, tmpdir): | |||
r1 | """ | |||
Return a non bare clone of the base repo. | ||||
""" | ||||
r5607 | clone_path = str(tmpdir.join("clone-repo")) | |||
repo_clone = GitRepository(clone_path, create=True, src_url=self.repo.path, bare=False) | ||||
r1 | ||||
return repo_clone | ||||
r3951 | def get_empty_repo(self, tmpdir, bare=False): | |||
r1 | """ | |||
Return a non bare empty repo. | ||||
""" | ||||
r5607 | clone_path = str(tmpdir.join("empty-repo")) | |||
r3778 | return GitRepository(clone_path, create=True, bare=bare) | |||
r1 | ||||
def test_wrong_repo_path(self): | ||||
r5607 | wrong_repo_path = "/tmp/errorrepo_git" | |||
r1 | with pytest.raises(RepositoryError): | |||
GitRepository(wrong_repo_path) | ||||
r3778 | def test_repo_clone(self, tmp_path_factory): | |||
r1 | repo = GitRepository(TEST_GIT_REPO) | |||
r5607 | clone_path = f"{tmp_path_factory.mktemp('_')}_{TEST_GIT_REPO_CLONE}" | |||
repo_clone = GitRepository(clone_path, src_url=TEST_GIT_REPO, create=True, do_workspace_checkout=True) | ||||
r3778 | ||||
r1 | assert len(repo.commit_ids) == len(repo_clone.commit_ids) | |||
# Checking hashes of commits should be enough | ||||
for commit in repo.get_commits(): | ||||
raw_id = commit.raw_id | ||||
assert raw_id == repo_clone.get_commit(raw_id).raw_id | ||||
def test_repo_clone_without_create(self): | ||||
with pytest.raises(RepositoryError): | ||||
r5607 | GitRepository(TEST_GIT_REPO_CLONE + "_wo_create", src_url=TEST_GIT_REPO) | |||
r1 | ||||
r3778 | def test_repo_clone_with_update(self, tmp_path_factory): | |||
r1 | repo = GitRepository(TEST_GIT_REPO) | |||
r5607 | clone_path = "{}_{}_update".format(tmp_path_factory.mktemp("_"), TEST_GIT_REPO_CLONE) | |||
r3778 | ||||
r5607 | repo_clone = GitRepository(clone_path, create=True, src_url=TEST_GIT_REPO, do_workspace_checkout=True) | |||
r1 | assert len(repo.commit_ids) == len(repo_clone.commit_ids) | |||
# check if current workdir was updated | ||||
r5607 | fpath = os.path.join(clone_path, "MANIFEST.in") | |||
r1 | assert os.path.isfile(fpath) | |||
r3778 | def test_repo_clone_without_update(self, tmp_path_factory): | |||
r1 | repo = GitRepository(TEST_GIT_REPO) | |||
r5607 | clone_path = "{}_{}_without_update".format(tmp_path_factory.mktemp("_"), TEST_GIT_REPO_CLONE) | |||
repo_clone = GitRepository(clone_path, create=True, src_url=TEST_GIT_REPO, do_workspace_checkout=False) | ||||
r1 | assert len(repo.commit_ids) == len(repo_clone.commit_ids) | |||
# check if current workdir was *NOT* updated | ||||
r5607 | fpath = os.path.join(clone_path, "MANIFEST.in") | |||
r1 | # Make sure it's not bare repo | |||
assert not repo_clone.bare | ||||
assert not os.path.isfile(fpath) | ||||
r3778 | def test_repo_clone_into_bare_repo(self, tmp_path_factory): | |||
r1 | repo = GitRepository(TEST_GIT_REPO) | |||
r5607 | clone_path = "{}_{}_bare.git".format(tmp_path_factory.mktemp("_"), TEST_GIT_REPO_CLONE) | |||
repo_clone = GitRepository(clone_path, create=True, src_url=repo.path, bare=True) | ||||
r1 | assert repo_clone.bare | |||
def test_create_repo_is_not_bare_by_default(self): | ||||
r5607 | repo = GitRepository(get_new_dir("not-bare-by-default"), create=True) | |||
r1 | assert not repo.bare | |||
def test_create_bare_repo(self): | ||||
r5607 | repo = GitRepository(get_new_dir("bare-repo"), create=True, bare=True) | |||
r1 | assert repo.bare | |||
def test_update_server_info(self): | ||||
self.repo._update_server_info() | ||||
def test_fetch(self, vcsbackend_git): | ||||
# Note: This is a git specific part of the API, it's only implemented | ||||
# by the git backend. | ||||
source_repo = vcsbackend_git.repo | ||||
r3078 | target_repo = vcsbackend_git.create_repo(bare=True) | |||
r1 | target_repo.fetch(source_repo.path) | |||
# Note: Get a fresh instance, avoids caching trouble | ||||
target_repo = vcsbackend_git.backend(target_repo.path) | ||||
assert len(source_repo.commit_ids) == len(target_repo.commit_ids) | ||||
def test_commit_ids(self): | ||||
# there are 112 commits (by now) | ||||
# so we can assume they would be available from now on | ||||
r5607 | subset = { | |||
"c1214f7e79e02fc37156ff215cd71275450cffc3", | ||||
"38b5fe81f109cb111f549bfe9bb6b267e10bc557", | ||||
"fa6600f6848800641328adbf7811fd2372c02ab2", | ||||
"102607b09cdd60e2793929c4f90478be29f85a17", | ||||
"49d3fd156b6f7db46313fac355dca1a0b94a0017", | ||||
"2d1028c054665b962fa3d307adfc923ddd528038", | ||||
"d7e0d30fbcae12c90680eb095a4f5f02505ce501", | ||||
"ff7ca51e58c505fec0dd2491de52c622bb7a806b", | ||||
"dd80b0f6cf5052f17cc738c2951c4f2070200d7f", | ||||
"8430a588b43b5d6da365400117c89400326e7992", | ||||
"d955cd312c17b02143c04fa1099a352b04368118", | ||||
"f67b87e5c629c2ee0ba58f85197e423ff28d735b", | ||||
"add63e382e4aabc9e1afdc4bdc24506c269b7618", | ||||
"f298fe1189f1b69779a4423f40b48edf92a703fc", | ||||
"bd9b619eb41994cac43d67cf4ccc8399c1125808", | ||||
"6e125e7c890379446e98980d8ed60fba87d0f6d1", | ||||
"d4a54db9f745dfeba6933bf5b1e79e15d0af20bd", | ||||
"0b05e4ed56c802098dfc813cbe779b2f49e92500", | ||||
"191caa5b2c81ed17c0794bf7bb9958f4dcb0b87e", | ||||
"45223f8f114c64bf4d6f853e3c35a369a6305520", | ||||
"ca1eb7957a54bce53b12d1a51b13452f95bc7c7e", | ||||
"f5ea29fc42ef67a2a5a7aecff10e1566699acd68", | ||||
"27d48942240f5b91dfda77accd2caac94708cc7d", | ||||
"622f0eb0bafd619d2560c26f80f09e3b0b0d78af", | ||||
"e686b958768ee96af8029fe19c6050b1a8dd3b2b", | ||||
} | ||||
r1 | assert subset.issubset(set(self.repo.commit_ids)) | |||
def test_slicing(self): | ||||
# 4 1 5 10 95 | ||||
r5607 | for sfrom, sto, size in [(0, 4, 4), (1, 2, 1), (10, 15, 5), (10, 20, 10), (5, 100, 95)]: | |||
r1 | commit_ids = list(self.repo[sfrom:sto]) | |||
assert len(commit_ids) == size | ||||
assert commit_ids[0] == self.repo.get_commit(commit_idx=sfrom) | ||||
assert commit_ids[-1] == self.repo.get_commit(commit_idx=sto - 1) | ||||
def test_branches(self): | ||||
# TODO: Need more tests here | ||||
# Removed (those are 'remotes' branches for cloned repo) | ||||
# assert 'master' in self.repo.branches | ||||
# assert 'gittree' in self.repo.branches | ||||
# assert 'web-branch' in self.repo.branches | ||||
for __, commit_id in self.repo.branches.items(): | ||||
assert isinstance(self.repo.get_commit(commit_id), GitCommit) | ||||
def test_tags(self): | ||||
# TODO: Need more tests here | ||||
r5607 | assert "v0.1.1" in self.repo.tags | |||
assert "v0.1.2" in self.repo.tags | ||||
r1 | for __, commit_id in self.repo.tags.items(): | |||
assert isinstance(self.repo.get_commit(commit_id), GitCommit) | ||||
def _test_single_commit_cache(self, commit_id): | ||||
commit = self.repo.get_commit(commit_id) | ||||
assert commit_id in self.repo.commits | ||||
assert commit is self.repo.commits[commit_id] | ||||
def test_initial_commit(self): | ||||
commit_id = self.repo.commit_ids[0] | ||||
init_commit = self.repo.get_commit(commit_id) | ||||
init_author = init_commit.author | ||||
r5607 | assert init_commit.message == "initial import\n" | |||
assert init_author == "Marcin Kuzminski <marcin@python-blog.com>" | ||||
r1 | assert init_author == init_commit.committer | |||
r5647 | assert sorted(init_commit.added_paths) == sorted( | |||
[ | ||||
b"vcs/__init__.py", | ||||
b"vcs/backends/BaseRepository.py", | ||||
b"vcs/backends/__init__.py", | ||||
] | ||||
) | ||||
assert sorted(init_commit.affected_files) == sorted( | ||||
[ | ||||
b"vcs/__init__.py", | ||||
b"vcs/backends/BaseRepository.py", | ||||
b"vcs/backends/__init__.py", | ||||
] | ||||
) | ||||
for path in (b"vcs/__init__.py", b"vcs/backends/BaseRepository.py", b"vcs/backends/__init__.py"): | ||||
r1 | assert isinstance(init_commit.get_node(path), FileNode) | |||
r5647 | for path in (b"", b"vcs", b"vcs/backends"): | |||
r1 | assert isinstance(init_commit.get_node(path), DirNode) | |||
with pytest.raises(NodeDoesNotExistError): | ||||
r5647 | init_commit.get_node(path=b"foobar") | |||
r1 | ||||
r5647 | node = init_commit.get_node(b"vcs/") | |||
r5607 | assert hasattr(node, "kind") | |||
r1 | assert node.kind == NodeKind.DIR | |||
r5647 | node = init_commit.get_node(b"vcs") | |||
r5607 | assert hasattr(node, "kind") | |||
r1 | assert node.kind == NodeKind.DIR | |||
r5647 | node = init_commit.get_node(b"vcs/__init__.py") | |||
r5607 | assert hasattr(node, "kind") | |||
r1 | assert node.kind == NodeKind.FILE | |||
def test_not_existing_commit(self): | ||||
with pytest.raises(RepositoryError): | ||||
r5607 | self.repo.get_commit("f" * 40) | |||
r1 | ||||
def test_commit10(self): | ||||
commit10 = self.repo.get_commit(self.repo.commit_ids[9]) | ||||
README = """=== | ||||
VCS | ||||
=== | ||||
Various Version Control System management abstraction layer for Python. | ||||
Introduction | ||||
------------ | ||||
TODO: To be written... | ||||
""" | ||||
r5647 | node = commit10.get_node(b"README.rst") | |||
r1 | assert node.kind == NodeKind.FILE | |||
r5087 | assert node.str_content == README | |||
r1 | ||||
def test_head(self): | ||||
assert self.repo.head == self.repo.get_commit().raw_id | ||||
r3951 | def test_checkout_with_create(self, tmpdir): | |||
repo_clone = self.get_clone_repo(tmpdir) | ||||
r1 | ||||
r5607 | new_branch = "new_branch" | |||
assert repo_clone._current_branch() == "master" | ||||
assert set(repo_clone.branches) == {"master"} | ||||
r1 | repo_clone._checkout(new_branch, create=True) | |||
# Branches is a lazy property so we need to recrete the Repo object. | ||||
repo_clone = GitRepository(repo_clone.path) | ||||
r5607 | assert set(repo_clone.branches) == {"master", new_branch} | |||
r1 | assert repo_clone._current_branch() == new_branch | |||
r3951 | def test_checkout(self, tmpdir): | |||
repo_clone = self.get_clone_repo(tmpdir) | ||||
r1 | ||||
r5607 | repo_clone._checkout("new_branch", create=True) | |||
repo_clone._checkout("master") | ||||
r1 | ||||
r5607 | assert repo_clone._current_branch() == "master" | |||
r1 | ||||
r3951 | def test_checkout_same_branch(self, tmpdir): | |||
repo_clone = self.get_clone_repo(tmpdir) | ||||
r1 | ||||
r5607 | repo_clone._checkout("master") | |||
assert repo_clone._current_branch() == "master" | ||||
r1 | ||||
r3951 | def test_checkout_branch_already_exists(self, tmpdir): | |||
repo_clone = self.get_clone_repo(tmpdir) | ||||
r1 | ||||
with pytest.raises(RepositoryError): | ||||
r5607 | repo_clone._checkout("master", create=True) | |||
r1 | ||||
def test_checkout_bare_repo(self): | ||||
with pytest.raises(RepositoryError): | ||||
r5607 | self.repo._checkout("master") | |||
r1 | ||||
def test_current_branch_bare_repo(self): | ||||
with pytest.raises(RepositoryError): | ||||
self.repo._current_branch() | ||||
r3951 | def test_current_branch_empty_repo(self, tmpdir): | |||
repo = self.get_empty_repo(tmpdir) | ||||
r1 | assert repo._current_branch() is None | |||
r3778 | def test_local_clone(self, tmp_path_factory): | |||
r5607 | clone_path = str(tmp_path_factory.mktemp("test-local-clone")) | |||
self.repo._local_clone(clone_path, "master") | ||||
r1 | repo_clone = GitRepository(clone_path) | |||
assert self.repo.commit_ids == repo_clone.commit_ids | ||||
r3951 | def test_local_clone_with_specific_branch(self, tmpdir): | |||
source_repo = self.get_clone_repo(tmpdir) | ||||
r1 | ||||
# Create a new branch in source repo | ||||
new_branch_commit = source_repo.commit_ids[-3] | ||||
source_repo._checkout(new_branch_commit) | ||||
r5607 | source_repo._checkout("new_branch", create=True) | |||
r1 | ||||
r5607 | clone_path = str(tmpdir.join("git-clone-path-1")) | |||
source_repo._local_clone(clone_path, "new_branch") | ||||
r1 | repo_clone = GitRepository(clone_path) | |||
r5607 | assert source_repo.commit_ids[: -3 + 1] == repo_clone.commit_ids | |||
r1 | ||||
r5607 | clone_path = str(tmpdir.join("git-clone-path-2")) | |||
source_repo._local_clone(clone_path, "master") | ||||
r1 | repo_clone = GitRepository(clone_path) | |||
assert source_repo.commit_ids == repo_clone.commit_ids | ||||
def test_local_clone_fails_if_target_exists(self): | ||||
with pytest.raises(RepositoryError): | ||||
r5607 | self.repo._local_clone(self.repo.path, "master") | |||
r1 | ||||
r3951 | def test_local_fetch(self, tmpdir): | |||
target_repo = self.get_empty_repo(tmpdir) | ||||
source_repo = self.get_clone_repo(tmpdir) | ||||
r1 | ||||
# Create a new branch in source repo | ||||
master_commit = source_repo.commit_ids[-1] | ||||
new_branch_commit = source_repo.commit_ids[-3] | ||||
source_repo._checkout(new_branch_commit) | ||||
r5607 | source_repo._checkout("new_branch", create=True) | |||
r1 | ||||
r5607 | target_repo._local_fetch(source_repo.path, "new_branch") | |||
r1 | assert target_repo._last_fetch_heads() == [new_branch_commit] | |||
r5607 | target_repo._local_fetch(source_repo.path, "master") | |||
r1 | assert target_repo._last_fetch_heads() == [master_commit] | |||
r3951 | def test_local_fetch_from_bare_repo(self, tmpdir): | |||
target_repo = self.get_empty_repo(tmpdir) | ||||
r5607 | target_repo._local_fetch(self.repo.path, "master") | |||
r1 | ||||
master_commit = self.repo.commit_ids[-1] | ||||
assert target_repo._last_fetch_heads() == [master_commit] | ||||
def test_local_fetch_from_same_repo(self): | ||||
with pytest.raises(ValueError): | ||||
r5607 | self.repo._local_fetch(self.repo.path, "master") | |||
r1 | ||||
r3951 | def test_local_fetch_branch_does_not_exist(self, tmpdir): | |||
target_repo = self.get_empty_repo(tmpdir) | ||||
r1 | ||||
with pytest.raises(RepositoryError): | ||||
r5607 | target_repo._local_fetch(self.repo.path, "new_branch") | |||
r1 | ||||
r3951 | def test_local_pull(self, tmpdir): | |||
target_repo = self.get_empty_repo(tmpdir) | ||||
source_repo = self.get_clone_repo(tmpdir) | ||||
r1 | ||||
# Create a new branch in source repo | ||||
master_commit = source_repo.commit_ids[-1] | ||||
new_branch_commit = source_repo.commit_ids[-3] | ||||
source_repo._checkout(new_branch_commit) | ||||
r5607 | source_repo._checkout("new_branch", create=True) | |||
r1 | ||||
r5607 | target_repo._local_pull(source_repo.path, "new_branch") | |||
r1 | target_repo = GitRepository(target_repo.path) | |||
assert target_repo.head == new_branch_commit | ||||
r5607 | target_repo._local_pull(source_repo.path, "master") | |||
r1 | target_repo = GitRepository(target_repo.path) | |||
assert target_repo.head == master_commit | ||||
def test_local_pull_in_bare_repo(self): | ||||
with pytest.raises(RepositoryError): | ||||
r5607 | self.repo._local_pull(self.repo.path, "master") | |||
r1 | ||||
r3951 | def test_local_merge(self, tmpdir): | |||
target_repo = self.get_empty_repo(tmpdir) | ||||
source_repo = self.get_clone_repo(tmpdir) | ||||
r1 | ||||
# Create a new branch in source repo | ||||
master_commit = source_repo.commit_ids[-1] | ||||
new_branch_commit = source_repo.commit_ids[-3] | ||||
source_repo._checkout(new_branch_commit) | ||||
r5607 | source_repo._checkout("new_branch", create=True) | |||
r1 | ||||
# This is required as one cannot do a -ff-only merge in an empty repo. | ||||
r5607 | target_repo._local_pull(source_repo.path, "new_branch") | |||
r1 | ||||
r5607 | target_repo._local_fetch(source_repo.path, "master") | |||
merge_message = "Merge message\n\nDescription:..." | ||||
user_name = "Albert Einstein" | ||||
user_email = "albert@einstein.com" | ||||
target_repo._local_merge(merge_message, user_name, user_email, target_repo._last_fetch_heads()) | ||||
r1 | ||||
target_repo = GitRepository(target_repo.path) | ||||
assert target_repo.commit_ids[-2] == master_commit | ||||
last_commit = target_repo.get_commit(target_repo.head) | ||||
assert last_commit.message.strip() == merge_message | ||||
r5607 | assert last_commit.author == "%s <%s>" % (user_name, user_email) | |||
r1 | ||||
r5607 | assert not os.path.exists(os.path.join(target_repo.path, ".git", "MERGE_HEAD")) | |||
r1 | ||||
def test_local_merge_raises_exception_on_conflict(self, vcsbackend_git): | ||||
target_repo = vcsbackend_git.create_repo(number_of_commits=1) | ||||
r5607 | vcsbackend_git.ensure_file(b"README", b"I will conflict with you!!!") | |||
r1 | ||||
r5607 | target_repo._local_fetch(self.repo.path, "master") | |||
r1 | with pytest.raises(RepositoryError): | |||
r5607 | target_repo._local_merge("merge_message", "user name", "user@name.com", target_repo._last_fetch_heads()) | |||
r1 | ||||
# Check we are not left in an intermediate merge state | ||||
r5607 | assert not os.path.exists(os.path.join(target_repo.path, ".git", "MERGE_HEAD")) | |||
r1 | ||||
r3951 | def test_local_merge_into_empty_repo(self, tmpdir): | |||
target_repo = self.get_empty_repo(tmpdir) | ||||
r1 | ||||
# This is required as one cannot do a -ff-only merge in an empty repo. | ||||
r5607 | target_repo._local_fetch(self.repo.path, "master") | |||
r1 | with pytest.raises(RepositoryError): | |||
r5607 | target_repo._local_merge("merge_message", "user name", "user@name.com", target_repo._last_fetch_heads()) | |||
r1 | ||||
def test_local_merge_in_bare_repo(self): | ||||
with pytest.raises(RepositoryError): | ||||
r5607 | self.repo._local_merge("merge_message", "user name", "user@name.com", None) | |||
r1 | ||||
r3951 | def test_local_push_non_bare(self, tmpdir): | |||
target_repo = self.get_empty_repo(tmpdir) | ||||
r1 | ||||
r5607 | pushed_branch = "pushed_branch" | |||
self.repo._local_push("master", target_repo.path, pushed_branch) | ||||
r1 | # Fix the HEAD of the target repo, or otherwise GitRepository won't | |||
# report any branches. | ||||
r5607 | with open(os.path.join(target_repo.path, ".git", "HEAD"), "w") as f: | |||
f.write("ref: refs/heads/%s" % pushed_branch) | ||||
r1 | ||||
target_repo = GitRepository(target_repo.path) | ||||
r5607 | assert target_repo.branches[pushed_branch] == self.repo.branches["master"] | |||
r1 | ||||
r3951 | def test_local_push_bare(self, tmpdir): | |||
target_repo = self.get_empty_repo(tmpdir, bare=True) | ||||
r1 | ||||
r5607 | pushed_branch = "pushed_branch" | |||
self.repo._local_push("master", target_repo.path, pushed_branch) | ||||
r1 | # Fix the HEAD of the target repo, or otherwise GitRepository won't | |||
# report any branches. | ||||
r5607 | with open(os.path.join(target_repo.path, "HEAD"), "w") as f: | |||
f.write("ref: refs/heads/%s" % pushed_branch) | ||||
r1 | ||||
target_repo = GitRepository(target_repo.path) | ||||
r5607 | assert target_repo.branches[pushed_branch] == self.repo.branches["master"] | |||
r1 | ||||
r3951 | def test_local_push_non_bare_target_branch_is_checked_out(self, tmpdir): | |||
target_repo = self.get_clone_repo(tmpdir) | ||||
r1 | ||||
r5607 | pushed_branch = "pushed_branch" | |||
r1 | # Create a new branch in source repo | |||
new_branch_commit = target_repo.commit_ids[-3] | ||||
target_repo._checkout(new_branch_commit) | ||||
target_repo._checkout(pushed_branch, create=True) | ||||
r5607 | self.repo._local_push("master", target_repo.path, pushed_branch) | |||
r1 | ||||
target_repo = GitRepository(target_repo.path) | ||||
r5607 | assert target_repo.branches[pushed_branch] == self.repo.branches["master"] | |||
r1 | ||||
def test_local_push_raises_exception_on_conflict(self, vcsbackend_git): | ||||
target_repo = vcsbackend_git.create_repo(number_of_commits=1) | ||||
with pytest.raises(RepositoryError): | ||||
r5607 | self.repo._local_push("master", target_repo.path, "master") | |||
r1 | ||||
r3951 | def test_hooks_can_be_enabled_via_env_variable_for_local_push(self, tmpdir): | |||
target_repo = self.get_empty_repo(tmpdir, bare=True) | ||||
r1 | ||||
r5607 | with mock.patch.object(self.repo, "run_git_command") as run_mock: | |||
self.repo._local_push("master", target_repo.path, "master", enable_hooks=True) | ||||
env = run_mock.call_args[1]["extra_env"] | ||||
assert "RC_SKIP_HOOKS" not in env | ||||
r1 | ||||
def _add_failing_hook(self, repo_path, hook_name, bare=False): | ||||
r5607 | path_components = ["hooks", hook_name] if bare else [".git", "hooks", hook_name] | |||
r1 | hook_path = os.path.join(repo_path, *path_components) | |||
r5607 | with open(hook_path, "w") as f: | |||
r1 | script_lines = [ | |||
r5607 | "#!%s" % sys.executable, | |||
"import os", | ||||
"import sys", | ||||
r1 | 'if os.environ.get("RC_SKIP_HOOKS"):', | |||
r5607 | " sys.exit(0)", | |||
"sys.exit(1)", | ||||
r1 | ] | |||
r5607 | f.write("\n".join(script_lines)) | |||
r3268 | os.chmod(hook_path, 0o755) | |||
r1 | ||||
r3951 | def test_local_push_does_not_execute_hook(self, tmpdir): | |||
target_repo = self.get_empty_repo(tmpdir) | ||||
r1 | ||||
r5607 | pushed_branch = "pushed_branch" | |||
self._add_failing_hook(target_repo.path, "pre-receive") | ||||
self.repo._local_push("master", target_repo.path, pushed_branch) | ||||
r1 | # Fix the HEAD of the target repo, or otherwise GitRepository won't | |||
# report any branches. | ||||
r5607 | with open(os.path.join(target_repo.path, ".git", "HEAD"), "w") as f: | |||
f.write("ref: refs/heads/%s" % pushed_branch) | ||||
r1 | ||||
target_repo = GitRepository(target_repo.path) | ||||
r5607 | assert target_repo.branches[pushed_branch] == self.repo.branches["master"] | |||
r1 | ||||
r3951 | def test_local_push_executes_hook(self, tmpdir): | |||
target_repo = self.get_empty_repo(tmpdir, bare=True) | ||||
r5607 | self._add_failing_hook(target_repo.path, "pre-receive", bare=True) | |||
r1 | with pytest.raises(RepositoryError): | |||
r5607 | self.repo._local_push("master", target_repo.path, "master", enable_hooks=True) | |||
r1 | ||||
def test_maybe_prepare_merge_workspace(self): | ||||
workspace = self.repo._maybe_prepare_merge_workspace( | ||||
r5607 | 2, "pr2", Reference("branch", "master", "unused"), Reference("branch", "master", "unused") | |||
) | ||||
r1 | ||||
assert os.path.isdir(workspace) | ||||
workspace_repo = GitRepository(workspace) | ||||
assert workspace_repo.branches == self.repo.branches | ||||
# Calling it a second time should also succeed | ||||
workspace = self.repo._maybe_prepare_merge_workspace( | ||||
r5607 | 2, "pr2", Reference("branch", "master", "unused"), Reference("branch", "master", "unused") | |||
) | ||||
r2473 | assert os.path.isdir(workspace) | |||
def test_maybe_prepare_merge_workspace_different_refs(self): | ||||
workspace = self.repo._maybe_prepare_merge_workspace( | ||||
r5607 | 2, "pr2", Reference("branch", "master", "unused"), Reference("branch", "develop", "unused") | |||
) | ||||
r2473 | ||||
assert os.path.isdir(workspace) | ||||
workspace_repo = GitRepository(workspace) | ||||
assert workspace_repo.branches == self.repo.branches | ||||
# Calling it a second time should also succeed | ||||
workspace = self.repo._maybe_prepare_merge_workspace( | ||||
r5607 | 2, "pr2", Reference("branch", "master", "unused"), Reference("branch", "develop", "unused") | |||
) | ||||
r1 | assert os.path.isdir(workspace) | |||
def test_cleanup_merge_workspace(self): | ||||
workspace = self.repo._maybe_prepare_merge_workspace( | ||||
r5607 | 2, "pr3", Reference("branch", "master", "unused"), Reference("branch", "master", "unused") | |||
) | ||||
self.repo.cleanup_merge_workspace(2, "pr3") | ||||
r1 | ||||
assert not os.path.exists(workspace) | ||||
def test_cleanup_merge_workspace_invalid_workspace_id(self): | ||||
# No assert: because in case of an inexistent workspace this function | ||||
# should still succeed. | ||||
r5607 | self.repo.cleanup_merge_workspace(1, "pr4") | |||
r1 | ||||
def test_set_refs(self): | ||||
r5607 | test_ref = "refs/test-refs/abcde" | |||
test_commit_id = "ecb86e1f424f2608262b130db174a7dfd25a6623" | ||||
r1 | ||||
self.repo.set_refs(test_ref, test_commit_id) | ||||
r5607 | stdout, _ = self.repo.run_git_command(["show-ref"]) | |||
r1 | assert test_ref in stdout | |||
assert test_commit_id in stdout | ||||
def test_remove_ref(self): | ||||
r5607 | test_ref = "refs/test-refs/abcde" | |||
test_commit_id = "ecb86e1f424f2608262b130db174a7dfd25a6623" | ||||
r1 | self.repo.set_refs(test_ref, test_commit_id) | |||
r5607 | stdout, _ = self.repo.run_git_command(["show-ref"]) | |||
r1 | assert test_ref in stdout | |||
assert test_commit_id in stdout | ||||
self.repo.remove_ref(test_ref) | ||||
r5607 | stdout, _ = self.repo.run_git_command(["show-ref"]) | |||
r1 | assert test_ref not in stdout | |||
assert test_commit_id not in stdout | ||||
class TestGitCommit(object): | ||||
@pytest.fixture(autouse=True) | ||||
def prepare(self): | ||||
self.repo = GitRepository(TEST_GIT_REPO) | ||||
def test_default_commit(self): | ||||
tip = self.repo.get_commit() | ||||
assert tip == self.repo.get_commit(None) | ||||
r5607 | assert tip == self.repo.get_commit("tip") | |||
r1 | ||||
def test_root_node(self): | ||||
tip = self.repo.get_commit() | ||||
r5647 | assert tip.root is tip.get_node(b"") | |||
r1 | ||||
def test_lazy_fetch(self): | ||||
""" | ||||
Test if commit's nodes expands and are cached as we walk through | ||||
the commit. This test is somewhat hard to write as order of tests | ||||
is a key here. Written by running command after command in a shell. | ||||
""" | ||||
r5607 | commit_id = "2a13f185e4525f9d4b59882791a2d397b90d5ddc" | |||
r1 | assert commit_id in self.repo.commit_ids | |||
commit = self.repo.get_commit(commit_id) | ||||
assert len(commit.nodes) == 0 | ||||
root = commit.root | ||||
assert len(commit.nodes) == 1 | ||||
assert len(root.nodes) == 8 | ||||
# accessing root.nodes updates commit.nodes | ||||
assert len(commit.nodes) == 9 | ||||
r5647 | docs = commit.get_node(b"docs") | |||
r1 | # we haven't yet accessed anything new as docs dir was already cached | |||
assert len(commit.nodes) == 9 | ||||
assert len(docs.nodes) == 8 | ||||
# accessing docs.nodes updates commit.nodes | ||||
assert len(commit.nodes) == 17 | ||||
r5647 | assert docs is commit.get_node(b"docs") | |||
r1 | assert docs is root.nodes[0] | |||
assert docs is root.dirs[0] | ||||
r5647 | assert docs is commit.get_node(b"docs") | |||
r1 | ||||
def test_nodes_with_commit(self): | ||||
r5607 | commit_id = "2a13f185e4525f9d4b59882791a2d397b90d5ddc" | |||
r1 | commit = self.repo.get_commit(commit_id) | |||
root = commit.root | ||||
r5647 | assert isinstance(root, RootNode) | |||
docs = commit.get_node(b"docs") | ||||
assert docs is commit.get_node(b"docs") | ||||
api = commit.get_node(b"docs/api") | ||||
assert api is commit.get_node(b"docs/api") | ||||
index = commit.get_node(b"docs/api/index.rst") | ||||
assert index is commit.get_node(b"docs/api/index.rst") | ||||
r1 | ||||
def test_branch_and_tags(self): | ||||
""" | ||||
rev0 = self.repo.commit_ids[0] | ||||
commit0 = self.repo.get_commit(rev0) | ||||
assert commit0.branch == 'master' | ||||
assert commit0.tags == [] | ||||
rev10 = self.repo.commit_ids[10] | ||||
commit10 = self.repo.get_commit(rev10) | ||||
assert commit10.branch == 'master' | ||||
assert commit10.tags == [] | ||||
rev44 = self.repo.commit_ids[44] | ||||
commit44 = self.repo.get_commit(rev44) | ||||
assert commit44.branch == 'web-branch' | ||||
tip = self.repo.get_commit('tip') | ||||
assert 'tip' in tip.tags | ||||
""" | ||||
# Those tests would fail - branches are now going | ||||
# to be changed at main API in order to support git backend | ||||
pass | ||||
def test_file_size(self): | ||||
to_check = ( | ||||
r5647 | ("c1214f7e79e02fc37156ff215cd71275450cffc3", b"vcs/backends/BaseRepository.py", 502), | |||
("d7e0d30fbcae12c90680eb095a4f5f02505ce501", b"vcs/backends/hg.py", 854), | ||||
("6e125e7c890379446e98980d8ed60fba87d0f6d1", b"setup.py", 1068), | ||||
("d955cd312c17b02143c04fa1099a352b04368118", b"vcs/backends/base.py", 2921), | ||||
("ca1eb7957a54bce53b12d1a51b13452f95bc7c7e", b"vcs/backends/base.py", 3936), | ||||
("f50f42baeed5af6518ef4b0cb2f1423f3851a941", b"vcs/backends/base.py", 6189), | ||||
r1 | ) | |||
for commit_id, path, size in to_check: | ||||
node = self.repo.get_commit(commit_id).get_node(path) | ||||
assert node.is_file() | ||||
assert node.size == size | ||||
def test_file_history_from_commits(self): | ||||
r5647 | node = self.repo[10].get_node(b"setup.py") | |||
r1 | commit_ids = [commit.raw_id for commit in node.history] | |||
r5607 | assert ["ff7ca51e58c505fec0dd2491de52c622bb7a806b"] == commit_ids | |||
r1 | ||||
r5647 | node = self.repo[20].get_node(b"setup.py") | |||
r1 | node_ids = [commit.raw_id for commit in node.history] | |||
r5607 | assert ["191caa5b2c81ed17c0794bf7bb9958f4dcb0b87e", "ff7ca51e58c505fec0dd2491de52c622bb7a806b"] == node_ids | |||
r1 | ||||
# special case we check history from commit that has this particular | ||||
# file changed this means we check if it's included as well | ||||
r5647 | node = self.repo.get_commit("191caa5b2c81ed17c0794bf7bb9958f4dcb0b87e").get_node(b"setup.py") | |||
r1 | node_ids = [commit.raw_id for commit in node.history] | |||
r5607 | assert ["191caa5b2c81ed17c0794bf7bb9958f4dcb0b87e", "ff7ca51e58c505fec0dd2491de52c622bb7a806b"] == node_ids | |||
r1 | ||||
def test_file_history(self): | ||||
# we can only check if those commits are present in the history | ||||
# as we cannot update this test every time file is changed | ||||
files = { | ||||
r5647 | b"setup.py": [ | |||
r5607 | "54386793436c938cff89326944d4c2702340037d", | |||
"51d254f0ecf5df2ce50c0b115741f4cf13985dab", | ||||
"998ed409c795fec2012b1c0ca054d99888b22090", | ||||
"5e0eb4c47f56564395f76333f319d26c79e2fb09", | ||||
"0115510b70c7229dbc5dc49036b32e7d91d23acd", | ||||
"7cb3fd1b6d8c20ba89e2264f1c8baebc8a52d36e", | ||||
"2a13f185e4525f9d4b59882791a2d397b90d5ddc", | ||||
"191caa5b2c81ed17c0794bf7bb9958f4dcb0b87e", | ||||
"ff7ca51e58c505fec0dd2491de52c622bb7a806b", | ||||
r1 | ], | |||
r5647 | b"vcs/nodes.py": [ | |||
r5607 | "33fa3223355104431402a888fa77a4e9956feb3e", | |||
"fa014c12c26d10ba682fadb78f2a11c24c8118e1", | ||||
"e686b958768ee96af8029fe19c6050b1a8dd3b2b", | ||||
"ab5721ca0a081f26bf43d9051e615af2cc99952f", | ||||
"c877b68d18e792a66b7f4c529ea02c8f80801542", | ||||
"4313566d2e417cb382948f8d9d7c765330356054", | ||||
"6c2303a793671e807d1cfc70134c9ca0767d98c2", | ||||
"54386793436c938cff89326944d4c2702340037d", | ||||
"54000345d2e78b03a99d561399e8e548de3f3203", | ||||
"1c6b3677b37ea064cb4b51714d8f7498f93f4b2b", | ||||
"2d03ca750a44440fb5ea8b751176d1f36f8e8f46", | ||||
"2a08b128c206db48c2f0b8f70df060e6db0ae4f8", | ||||
"30c26513ff1eb8e5ce0e1c6b477ee5dc50e2f34b", | ||||
"ac71e9503c2ca95542839af0ce7b64011b72ea7c", | ||||
"12669288fd13adba2a9b7dd5b870cc23ffab92d2", | ||||
"5a0c84f3e6fe3473e4c8427199d5a6fc71a9b382", | ||||
"12f2f5e2b38e6ff3fbdb5d722efed9aa72ecb0d5", | ||||
"5eab1222a7cd4bfcbabc218ca6d04276d4e27378", | ||||
"f50f42baeed5af6518ef4b0cb2f1423f3851a941", | ||||
"d7e390a45f6aa96f04f5e7f583ad4f867431aa25", | ||||
"f15c21f97864b4f071cddfbf2750ec2e23859414", | ||||
"e906ef056cf539a4e4e5fc8003eaf7cf14dd8ade", | ||||
"ea2b108b48aa8f8c9c4a941f66c1a03315ca1c3b", | ||||
"84dec09632a4458f79f50ddbbd155506c460b4f9", | ||||
"0115510b70c7229dbc5dc49036b32e7d91d23acd", | ||||
"2a13f185e4525f9d4b59882791a2d397b90d5ddc", | ||||
"3bf1c5868e570e39569d094f922d33ced2fa3b2b", | ||||
"b8d04012574729d2c29886e53b1a43ef16dd00a1", | ||||
"6970b057cffe4aab0a792aa634c89f4bebf01441", | ||||
"dd80b0f6cf5052f17cc738c2951c4f2070200d7f", | ||||
"ff7ca51e58c505fec0dd2491de52c622bb7a806b", | ||||
r1 | ], | |||
r5647 | b"vcs/backends/git.py": [ | |||
r5607 | "4cf116ad5a457530381135e2f4c453e68a1b0105", | |||
"9a751d84d8e9408e736329767387f41b36935153", | ||||
"cb681fb539c3faaedbcdf5ca71ca413425c18f01", | ||||
"428f81bb652bcba8d631bce926e8834ff49bdcc6", | ||||
"180ab15aebf26f98f714d8c68715e0f05fa6e1c7", | ||||
"2b8e07312a2e89e92b90426ab97f349f4bce2a3a", | ||||
"50e08c506174d8645a4bb517dd122ac946a0f3bf", | ||||
"54000345d2e78b03a99d561399e8e548de3f3203", | ||||
r1 | ], | |||
} | ||||
for path, commit_ids in files.items(): | ||||
node = self.repo.get_commit(commit_ids[0]).get_node(path) | ||||
node_ids = [commit.raw_id for commit in node.history] | ||||
assert set(commit_ids).issubset(set(node_ids)), ( | ||||
"We assumed that %s is subset of commit_ids for which file %s " | ||||
r5607 | "has been changed, and history of that node returned: %s" % (commit_ids, path, node_ids) | |||
) | ||||
r1 | ||||
def test_file_annotate(self): | ||||
files = { | ||||
r5647 | b"vcs/backends/__init__.py": { | |||
r5607 | "c1214f7e79e02fc37156ff215cd71275450cffc3": { | |||
"lines_no": 1, | ||||
"commits": [ | ||||
"c1214f7e79e02fc37156ff215cd71275450cffc3", | ||||
r1 | ], | |||
}, | ||||
r5607 | "16fba1ae9334d79b66d7afed2c2dfbfa2ae53647": { | |||
"lines_no": 21, | ||||
"commits": [ | ||||
"49d3fd156b6f7db46313fac355dca1a0b94a0017", | ||||
"16fba1ae9334d79b66d7afed2c2dfbfa2ae53647", | ||||
"16fba1ae9334d79b66d7afed2c2dfbfa2ae53647", | ||||
"16fba1ae9334d79b66d7afed2c2dfbfa2ae53647", | ||||
"16fba1ae9334d79b66d7afed2c2dfbfa2ae53647", | ||||
"16fba1ae9334d79b66d7afed2c2dfbfa2ae53647", | ||||
"16fba1ae9334d79b66d7afed2c2dfbfa2ae53647", | ||||
"16fba1ae9334d79b66d7afed2c2dfbfa2ae53647", | ||||
"16fba1ae9334d79b66d7afed2c2dfbfa2ae53647", | ||||
"16fba1ae9334d79b66d7afed2c2dfbfa2ae53647", | ||||
"16fba1ae9334d79b66d7afed2c2dfbfa2ae53647", | ||||
"16fba1ae9334d79b66d7afed2c2dfbfa2ae53647", | ||||
"16fba1ae9334d79b66d7afed2c2dfbfa2ae53647", | ||||
"16fba1ae9334d79b66d7afed2c2dfbfa2ae53647", | ||||
"16fba1ae9334d79b66d7afed2c2dfbfa2ae53647", | ||||
"16fba1ae9334d79b66d7afed2c2dfbfa2ae53647", | ||||
"16fba1ae9334d79b66d7afed2c2dfbfa2ae53647", | ||||
"16fba1ae9334d79b66d7afed2c2dfbfa2ae53647", | ||||
"16fba1ae9334d79b66d7afed2c2dfbfa2ae53647", | ||||
"16fba1ae9334d79b66d7afed2c2dfbfa2ae53647", | ||||
"16fba1ae9334d79b66d7afed2c2dfbfa2ae53647", | ||||
r1 | ], | |||
}, | ||||
r5607 | "e29b67bd158580fc90fc5e9111240b90e6e86064": { | |||
"lines_no": 32, | ||||
"commits": [ | ||||
"16fba1ae9334d79b66d7afed2c2dfbfa2ae53647", | ||||
"16fba1ae9334d79b66d7afed2c2dfbfa2ae53647", | ||||
"5eab1222a7cd4bfcbabc218ca6d04276d4e27378", | ||||
"16fba1ae9334d79b66d7afed2c2dfbfa2ae53647", | ||||
"16fba1ae9334d79b66d7afed2c2dfbfa2ae53647", | ||||
"992f38217b979d0b0987d0bae3cc26dac85d9b19", | ||||
"16fba1ae9334d79b66d7afed2c2dfbfa2ae53647", | ||||
"54000345d2e78b03a99d561399e8e548de3f3203", | ||||
"16fba1ae9334d79b66d7afed2c2dfbfa2ae53647", | ||||
"16fba1ae9334d79b66d7afed2c2dfbfa2ae53647", | ||||
"78c3f0c23b7ee935ec276acb8b8212444c33c396", | ||||
"992f38217b979d0b0987d0bae3cc26dac85d9b19", | ||||
"992f38217b979d0b0987d0bae3cc26dac85d9b19", | ||||
"992f38217b979d0b0987d0bae3cc26dac85d9b19", | ||||
"992f38217b979d0b0987d0bae3cc26dac85d9b19", | ||||
"2a13f185e4525f9d4b59882791a2d397b90d5ddc", | ||||
"992f38217b979d0b0987d0bae3cc26dac85d9b19", | ||||
"78c3f0c23b7ee935ec276acb8b8212444c33c396", | ||||
"992f38217b979d0b0987d0bae3cc26dac85d9b19", | ||||
"992f38217b979d0b0987d0bae3cc26dac85d9b19", | ||||
"992f38217b979d0b0987d0bae3cc26dac85d9b19", | ||||
"16fba1ae9334d79b66d7afed2c2dfbfa2ae53647", | ||||
"16fba1ae9334d79b66d7afed2c2dfbfa2ae53647", | ||||
"16fba1ae9334d79b66d7afed2c2dfbfa2ae53647", | ||||
"16fba1ae9334d79b66d7afed2c2dfbfa2ae53647", | ||||
"992f38217b979d0b0987d0bae3cc26dac85d9b19", | ||||
"16fba1ae9334d79b66d7afed2c2dfbfa2ae53647", | ||||
"992f38217b979d0b0987d0bae3cc26dac85d9b19", | ||||
"992f38217b979d0b0987d0bae3cc26dac85d9b19", | ||||
"16fba1ae9334d79b66d7afed2c2dfbfa2ae53647", | ||||
"16fba1ae9334d79b66d7afed2c2dfbfa2ae53647", | ||||
"16fba1ae9334d79b66d7afed2c2dfbfa2ae53647", | ||||
r1 | ], | |||
}, | ||||
}, | ||||
} | ||||
for fname, commit_dict in files.items(): | ||||
for commit_id, __ in commit_dict.items(): | ||||
commit = self.repo.get_commit(commit_id) | ||||
l1_1 = [x[1] for x in commit.get_file_annotate(fname)] | ||||
l1_2 = [x[2]().raw_id for x in commit.get_file_annotate(fname)] | ||||
assert l1_1 == l1_2 | ||||
l1 = l1_1 | ||||
r5607 | l2 = files[fname][commit_id]["commits"] | |||
r1 | assert l1 == l2, ( | |||
"The lists of commit_ids for %s@commit_id %s" | ||||
"from annotation list should match each other, " | ||||
r5607 | "got \n%s \nvs \n%s " % (fname, commit_id, l1, l2) | |||
) | ||||
r1 | ||||
def test_files_state(self): | ||||
""" | ||||
Tests state of FileNodes. | ||||
""" | ||||
r5647 | commit = self.repo.get_commit("e6ea6d16e2f26250124a1f4b4fe37a912f9d86a0") | |||
node = commit.get_node(b"vcs/utils/diffs.py") | ||||
assert node.bytes_path in commit.added_paths | ||||
r1 | ||||
r5647 | commit = self.repo.get_commit("33fa3223355104431402a888fa77a4e9956feb3e") | |||
node = commit.get_node(b".hgignore") | ||||
assert node.bytes_path in commit.changed_paths | ||||
r1 | ||||
r5647 | commit = self.repo.get_commit("e29b67bd158580fc90fc5e9111240b90e6e86064") | |||
node = commit.get_node(b"setup.py") | ||||
assert node.bytes_path not in commit.affected_files | ||||
r1 | ||||
# If node has REMOVED state then trying to fetch it would raise | ||||
# CommitError exception | ||||
r5607 | commit = self.repo.get_commit("fa6600f6848800641328adbf7811fd2372c02ab2") | |||
r5647 | path = b"vcs/backends/BaseRepository.py" | |||
r1 | with pytest.raises(NodeDoesNotExistError): | |||
commit.get_node(path) | ||||
r5647 | ||||
r1 | # but it would be one of ``removed`` (commit's attribute) | |||
r5647 | assert path in [rf for rf in commit.removed_paths] | |||
r1 | ||||
r5607 | commit = self.repo.get_commit("54386793436c938cff89326944d4c2702340037d") | |||
r5647 | changed = [b"setup.py", b"tests/test_nodes.py", b"vcs/backends/hg.py", b"vcs/nodes.py"] | |||
assert set(changed) == set([f for f in commit.changed_paths]) | ||||
r1 | ||||
r784 | def test_unicode_branch_refs(self): | |||
r1 | unicode_branches = { | |||
r5607 | "refs/heads/unicode": "6c0ce52b229aa978889e91b38777f800e85f330b", | |||
"refs/heads/uniçö∂e": "ürl", | ||||
r1 | } | |||
r5607 | with mock.patch(("rhodecode.lib.vcs.backends.git.repository" ".GitRepository._refs"), unicode_branches): | |||
r1 | branches = self.repo.branches | |||
r5607 | assert "unicode" in branches | |||
assert "uniçö∂e" in branches | ||||
r1 | ||||
r784 | def test_unicode_tag_refs(self): | |||
unicode_tags = { | ||||
r5607 | "refs/tags/unicode": "6c0ce52b229aa978889e91b38777f800e85f330b", | |||
"refs/tags/uniçö∂e": "6c0ce52b229aa978889e91b38777f800e85f330b", | ||||
r784 | } | |||
r5607 | with mock.patch(("rhodecode.lib.vcs.backends.git.repository" ".GitRepository._refs"), unicode_tags): | |||
r784 | tags = self.repo.tags | |||
r5607 | assert "unicode" in tags | |||
assert "uniçö∂e" in tags | ||||
r784 | ||||
r1 | def test_commit_message_is_unicode(self): | |||
for commit in self.repo: | ||||
r4952 | assert type(commit.message) == str | |||
r1 | ||||
def test_commit_author_is_unicode(self): | ||||
for commit in self.repo: | ||||
r4952 | assert type(commit.author) == str | |||
r1 | ||||
r5087 | def test_repo_files_content_types(self): | |||
r1 | commit = self.repo.get_commit() | |||
r5647 | for node in commit.get_node(b"/"): | |||
r1 | if node.is_file(): | |||
r5087 | assert type(node.content) == bytes | |||
assert type(node.str_content) == str | ||||
r1 | ||||
def test_wrong_path(self): | ||||
# There is 'setup.py' in the root dir but not there: | ||||
r5647 | path = b"foo/bar/setup.py" | |||
r1 | tip = self.repo.get_commit() | |||
with pytest.raises(VCSError): | ||||
tip.get_node(path) | ||||
r5607 | @pytest.mark.parametrize( | |||
"author_email, commit_id", | ||||
[ | ||||
("marcin@python-blog.com", "c1214f7e79e02fc37156ff215cd71275450cffc3"), | ||||
("lukasz.balcerzak@python-center.pl", "ff7ca51e58c505fec0dd2491de52c622bb7a806b"), | ||||
("none@none", "8430a588b43b5d6da365400117c89400326e7992"), | ||||
], | ||||
) | ||||
r1 | def test_author_email(self, author_email, commit_id): | |||
commit = self.repo.get_commit(commit_id) | ||||
assert author_email == commit.author_email | ||||
r5607 | @pytest.mark.parametrize( | |||
"author, commit_id", | ||||
[ | ||||
("Marcin Kuzminski", "c1214f7e79e02fc37156ff215cd71275450cffc3"), | ||||
("Lukasz Balcerzak", "ff7ca51e58c505fec0dd2491de52c622bb7a806b"), | ||||
("marcink", "8430a588b43b5d6da365400117c89400326e7992"), | ||||
], | ||||
) | ||||
r1 | def test_author_username(self, author, commit_id): | |||
commit = self.repo.get_commit(commit_id) | ||||
assert author == commit.author_name | ||||
r1577 | class TestLargeFileRepo(object): | |||
def test_large_file(self, backend_git): | ||||
conf = make_db_config() | ||||
r5607 | git_largefiles_store = conf.get("vcs_git_lfs", "store_location") | |||
repo = backend_git.create_test_repo("largefiles", conf) | ||||
r1577 | ||||
tip = repo.scm_instance().get_commit() | ||||
r5647 | node = tip.get_node(b"1MB.zip") | |||
r1577 | ||||
# extract stored LF node into the origin cache | ||||
r5607 | repo_lfs_store: str = os.path.join(repo.repo_path, repo.repo_name, "lfs_store") | |||
r1577 | ||||
r5607 | oid: str = "7b331c02e313c7599d5a90212e17e6d3cb729bd2e1c9b873c302a63c95a2f9bf" | |||
# where the OID actually is INSIDE the repo... | ||||
oid_path = os.path.join(repo_lfs_store, oid) | ||||
r5264 | ||||
r5607 | # Note: oid path depends on LFSOidStore.store_suffix. Once it will be changed update below line accordingly | |||
oid_destination = os.path.join(git_largefiles_store, f"objects/{oid[:2]}/{oid[2:4]}/{oid}") | ||||
r5609 | spec_path = os.path.dirname(oid_destination) | |||
os.makedirs(spec_path, exist_ok=True) | ||||
r1577 | shutil.copy(oid_path, oid_destination) | |||
lf_node = node.get_largefile_node() | ||||
assert lf_node.is_largefile() is True | ||||
assert lf_node.size == 1024000 | ||||
r5647 | assert lf_node.name == b"1MB.zip" | |||
r1577 | ||||
r2453 | @pytest.mark.usefixtures("vcs_repository_support") | |||
r1 | class TestGitSpecificWithRepo(BackendTestMixin): | |||
@classmethod | ||||
def _get_commits(cls): | ||||
return [ | ||||
{ | ||||
r5607 | "message": "Initial", | |||
"author": "Joe Doe <joe.doe@example.com>", | ||||
"date": datetime.datetime(2010, 1, 1, 20), | ||||
"added": [ | ||||
FileNode(b"foobar/static/js/admin/base.js", content=b"base"), | ||||
FileNode(b"foobar/static/admin", content=b"admin", mode=0o120000), # this is a link | ||||
FileNode(b"foo", content=b"foo"), | ||||
r1 | ], | |||
}, | ||||
{ | ||||
r5607 | "message": "Second", | |||
"author": "Joe Doe <joe.doe@example.com>", | ||||
"date": datetime.datetime(2010, 1, 1, 22), | ||||
"added": [ | ||||
FileNode(b"foo2", content=b"foo2"), | ||||
r1 | ], | |||
}, | ||||
] | ||||
def test_paths_slow_traversing(self): | ||||
commit = self.repo.get_commit() | ||||
r5647 | assert commit.get_node(b"foobar/static/js/admin/base.js").content == b"base" | |||
r1 | ||||
def test_paths_fast_traversing(self): | ||||
commit = self.repo.get_commit() | ||||
r5647 | assert commit.get_node(b"foobar/static/js/admin/base.js").content == b"base" | |||
r1 | ||||
def test_get_diff_runs_git_command_with_hashes(self): | ||||
r3777 | comm1 = self.repo[0] | |||
comm2 = self.repo[1] | ||||
r3862 | ||||
r5607 | with mock.patch.object(self.repo, "_remote", return_value=mock.Mock()) as remote_mock: | |||
r5087 | remote_mock.diff = mock.MagicMock(side_effect=callable_get_diff) | |||
r3862 | self.repo.get_diff(comm1, comm2) | |||
r3777 | ||||
r3862 | remote_mock.diff.assert_called_once_with( | |||
r5607 | comm1.raw_id, comm2.raw_id, file_filter=None, opt_ignorews=False, context=3 | |||
) | ||||
r3777 | ||||
def test_get_diff_runs_git_command_with_str_hashes(self): | ||||
comm2 = self.repo[1] | ||||
r5087 | ||||
r5607 | with mock.patch.object(self.repo, "_remote", return_value=mock.Mock()) as remote_mock: | |||
r5087 | remote_mock.diff = mock.MagicMock(side_effect=callable_get_diff) | |||
r3862 | self.repo.get_diff(self.repo.EMPTY_COMMIT, comm2) | |||
r5087 | ||||
r3862 | remote_mock.diff.assert_called_once_with( | |||
r5607 | self.repo.EMPTY_COMMIT.raw_id, comm2.raw_id, file_filter=None, opt_ignorews=False, context=3 | |||
) | ||||
r3777 | ||||
def test_get_diff_runs_git_command_with_path_if_its_given(self): | ||||
comm1 = self.repo[0] | ||||
comm2 = self.repo[1] | ||||
r5087 | ||||
r5607 | with mock.patch.object(self.repo, "_remote", return_value=mock.Mock()) as remote_mock: | |||
r5087 | remote_mock.diff = mock.MagicMock(side_effect=callable_get_diff) | |||
r5607 | self.repo.get_diff(comm1, comm2, "foo") | |||
r5087 | ||||
r3862 | remote_mock.diff.assert_called_once_with( | |||
r5607 | self.repo._lookup_commit(0), comm2.raw_id, file_filter="foo", opt_ignorews=False, context=3 | |||
) | ||||
r1 | ||||
r2453 | @pytest.mark.usefixtures("vcs_repository_support") | |||
r1 | class TestGitRegression(BackendTestMixin): | |||
@classmethod | ||||
def _get_commits(cls): | ||||
return [ | ||||
{ | ||||
r5607 | "message": "Initial", | |||
"author": "Joe Doe <joe.doe@example.com>", | ||||
"date": datetime.datetime(2010, 1, 1, 20), | ||||
"added": [ | ||||
FileNode(b"bot/__init__.py", content=b"base"), | ||||
FileNode(b"bot/templates/404.html", content=b"base"), | ||||
FileNode(b"bot/templates/500.html", content=b"base"), | ||||
r1 | ], | |||
}, | ||||
{ | ||||
r5607 | "message": "Second", | |||
"author": "Joe Doe <joe.doe@example.com>", | ||||
"date": datetime.datetime(2010, 1, 1, 22), | ||||
"added": [ | ||||
FileNode(b"bot/build/migrations/1.py", content=b"foo2"), | ||||
FileNode(b"bot/build/migrations/2.py", content=b"foo2"), | ||||
FileNode(b"bot/build/static/templates/f.html", content=b"foo2"), | ||||
FileNode(b"bot/build/static/templates/f1.html", content=b"foo2"), | ||||
FileNode(b"bot/build/templates/err.html", content=b"foo2"), | ||||
FileNode(b"bot/build/templates/err2.html", content=b"foo2"), | ||||
r1 | ], | |||
}, | ||||
] | ||||
r5607 | @pytest.mark.parametrize( | |||
"path, expected_paths", | ||||
[ | ||||
r5647 | (b"bot", ["bot/build", "bot/templates", "bot/__init__.py"]), | |||
(b"bot/build", ["bot/build/migrations", "bot/build/static", "bot/build/templates"]), | ||||
(b"bot/build/static", ["bot/build/static/templates"]), | ||||
( | ||||
b"bot/build/static/templates", | ||||
["bot/build/static/templates/f.html", "bot/build/static/templates/f1.html"], | ||||
), | ||||
(b"bot/build/templates", ["bot/build/templates/err.html", "bot/build/templates/err2.html"]), | ||||
(b"bot/templates/", ["bot/templates/404.html", "bot/templates/500.html"]), | ||||
r5607 | ], | |||
) | ||||
r1 | def test_similar_paths(self, path, expected_paths): | |||
commit = self.repo.get_commit() | ||||
paths = [n.path for n in commit.get_nodes(path)] | ||||
assert paths == expected_paths | ||||
r3078 | class TestDiscoverGitVersion(object): | |||
r2351 | def test_returns_git_version(self, baseapp): | |||
r1 | version = discover_git_version() | |||
assert version | ||||
def test_returns_empty_string_without_vcsserver(self): | ||||
mock_connection = mock.Mock() | ||||
r5607 | mock_connection.discover_git_version = mock.Mock(side_effect=Exception) | |||
with mock.patch("rhodecode.lib.vcs.connection.Git", mock_connection): | ||||
r1 | version = discover_git_version() | |||
r5607 | assert version == "" | |||
r1 | ||||
class TestGetSubmoduleUrl(object): | ||||
def test_submodules_file_found(self): | ||||
r5607 | commit = GitCommit(repository=mock.Mock(), raw_id="abcdef12", idx=1) | |||
r1 | node = mock.Mock() | |||
r5151 | ||||
r5607 | with mock.patch.object(commit, "get_node", return_value=node) as get_node_mock: | |||
r5151 | node.str_content = ( | |||
r5607 | '[submodule "subrepo1"]\n' "\tpath = subrepo1\n" "\turl = https://code.rhodecode.com/dulwich\n" | |||
r1 | ) | |||
r5647 | result = commit._get_submodule_url(b"subrepo1") | |||
get_node_mock.assert_called_once_with(b".gitmodules") | ||||
r5607 | assert result == "https://code.rhodecode.com/dulwich" | |||
r1 | ||||
def test_complex_submodule_path(self): | ||||
r5607 | commit = GitCommit(repository=mock.Mock(), raw_id="abcdef12", idx=1) | |||
r1 | node = mock.Mock() | |||
r5151 | ||||
r5607 | with mock.patch.object(commit, "get_node", return_value=node) as get_node_mock: | |||
r5151 | node.str_content = ( | |||
r1 | '[submodule "complex/subrepo/path"]\n' | |||
r5607 | "\tpath = complex/subrepo/path\n" | |||
"\turl = https://code.rhodecode.com/dulwich\n" | ||||
r1 | ) | |||
r5647 | result = commit._get_submodule_url(b"complex/subrepo/path") | |||
get_node_mock.assert_called_once_with(b".gitmodules") | ||||
r5607 | assert result == "https://code.rhodecode.com/dulwich" | |||
r1 | ||||
def test_submodules_file_not_found(self): | ||||
r5607 | commit = GitCommit(repository=mock.Mock(), raw_id="abcdef12", idx=1) | |||
with mock.patch.object(commit, "get_node", side_effect=NodeDoesNotExistError): | ||||
r5647 | result = commit._get_submodule_url(b"complex/subrepo/path") | |||
r1 | assert result is None | |||
def test_path_not_found(self): | ||||
r5607 | commit = GitCommit(repository=mock.Mock(), raw_id="abcdef12", idx=1) | |||
r1 | node = mock.Mock() | |||
r5151 | ||||
r5607 | with mock.patch.object(commit, "get_node", return_value=node) as get_node_mock: | |||
r5151 | node.str_content = ( | |||
r5607 | '[submodule "subrepo1"]\n' "\tpath = subrepo1\n" "\turl = https://code.rhodecode.com/dulwich\n" | |||
r1 | ) | |||
r5647 | result = commit._get_submodule_url(b"subrepo2") | |||
get_node_mock.assert_called_once_with(b".gitmodules") | ||||
r1 | assert result is None | |||
def test_returns_cached_values(self): | ||||
r5607 | commit = GitCommit(repository=mock.Mock(), raw_id="abcdef12", idx=1) | |||
r1 | node = mock.Mock() | |||
r5151 | ||||
r5607 | with mock.patch.object(commit, "get_node", return_value=node) as get_node_mock: | |||
r5151 | node.str_content = ( | |||
r5607 | '[submodule "subrepo1"]\n' "\tpath = subrepo1\n" "\turl = https://code.rhodecode.com/dulwich\n" | |||
r1 | ) | |||
for _ in range(3): | ||||
r5647 | commit._get_submodule_url(b"subrepo1") | |||
get_node_mock.assert_called_once_with(b".gitmodules") | ||||
r1 | ||||
def test_get_node_returns_a_link(self): | ||||
repository = mock.Mock() | ||||
r5607 | repository.alias = "git" | |||
commit = GitCommit(repository=repository, raw_id="abcdef12", idx=1) | ||||
submodule_url = "https://code.rhodecode.com/dulwich" | ||||
r5647 | get_id_patch = mock.patch.object(commit, "_get_path_tree_id_and_type", return_value=(1, NodeKind.SUBMODULE)) | |||
r5607 | get_submodule_patch = mock.patch.object(commit, "_get_submodule_url", return_value=submodule_url) | |||
r1 | ||||
with get_id_patch, get_submodule_patch as submodule_mock: | ||||
r5647 | node = commit.get_node(b"/abcde") | |||
r1 | ||||
r5647 | submodule_mock.assert_called_once_with(b"/abcde") | |||
r1 | assert type(node) == SubModuleNode | |||
assert node.url == submodule_url | ||||
def test_get_nodes_returns_links(self): | ||||
repository = mock.MagicMock() | ||||
r5607 | repository.alias = "git" | |||
r5651 | # obj_name, stat_, tree_item_id, node_kind, pre_load_data | |||
repository._remote.get_nodes.return_value = [(b"subrepo", "stat", 1, NodeKind.SUBMODULE, [])] | ||||
r5607 | commit = GitCommit(repository=repository, raw_id="abcdef12", idx=1) | |||
submodule_url = "https://code.rhodecode.com/dulwich" | ||||
r5651 | ||||
r5647 | get_id_patch = mock.patch.object(commit, "_get_path_tree_id_and_type", return_value=(1, NodeKind.DIR)) | |||
r5607 | get_submodule_patch = mock.patch.object(commit, "_get_submodule_url", return_value=submodule_url) | |||
r1 | ||||
with get_id_patch, get_submodule_patch as submodule_mock: | ||||
r5647 | nodes = commit.get_nodes(b"/abcde") | |||
r1 | ||||
assert len(nodes) == 1 | ||||
assert type(nodes[0]) == SubModuleNode | ||||
assert nodes[0].url == submodule_url | ||||
r5651 | submodule_mock.assert_called_once_with(b"/abcde/subrepo") | |||
r5607 | ||||
class TestGetShadowInstance(object): | ||||
@pytest.fixture() | ||||
def repo(self, vcsbackend_git): | ||||
_git_repo = vcsbackend_git.repo | ||||
mock.patch.object(_git_repo, "config", mock.Mock()) | ||||
connection_mock = mock.Mock(unsafe=True, name="connection.Hg") | ||||
mock.patch("rhodecode.lib.vcs.connection.Git", connection_mock) | ||||
return _git_repo | ||||
def test_getting_shadow_instance_copies_config(self, repo): | ||||
shadow = repo.get_shadow_instance(repo.path) | ||||
assert shadow.config.serialize() == repo.config.serialize() | ||||