test_vcs_operations_locking.py
183 lines
| 7.9 KiB
| text/x-python
|
PythonLexer
r5088 | # Copyright (C) 2010-2023 RhodeCode GmbH | |||
r2456 | # | |||
# 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/ | ||||
""" | ||||
Test suite for making push/pull operations, on specially modified INI files | ||||
""" | ||||
import pytest | ||||
from rhodecode.model.db import User, Repository | ||||
from rhodecode.model.meta import Session | ||||
from rhodecode.model.repo import RepoModel | ||||
r5607 | from rhodecode.tests import GIT_REPO, HG_REPO, TEST_USER_ADMIN_LOGIN, TEST_USER_REGULAR_LOGIN, TEST_USER_REGULAR_PASS | |||
from rhodecode.tests.vcs_operations import Command, _check_proper_clone, _check_proper_git_push, _add_files_and_push | ||||
r2456 | ||||
r5607 | custom_code = [ | |||
{'app:main': {'auth_ret_code': '423'}}, | ||||
] | ||||
r2456 | ||||
r5607 | @pytest.mark.parametrize('rcstack', custom_code, indirect=True) | |||
@pytest.mark.usefixtures( | ||||
"init_pyramid_app", | ||||
"repo_group_repos", | ||||
"disable_anonymous_user", | ||||
"disable_locking", | ||||
) | ||||
class TestVCSOperationsOnLockingRepos(object): | ||||
def test_clone_and_create_lock_hg(self, rcstack, tmpdir): | ||||
r2456 | # enable locking | |||
r = Repository.get_by_repo_name(HG_REPO) | ||||
r.enable_locking = True | ||||
Session().add(r) | ||||
Session().commit() | ||||
# clone | ||||
r5607 | clone_url = rcstack.repo_clone_url(HG_REPO) | |||
stdout, stderr = Command(tmpdir.strpath).execute("hg clone", clone_url, tmpdir.strpath) | ||||
r2456 | ||||
# check if lock was made | ||||
r = Repository.get_by_repo_name(HG_REPO) | ||||
r5607 | assert r.locked[0] == User.get_by_username(TEST_USER_ADMIN_LOGIN).user_id | |||
r2456 | ||||
r5607 | def test_clone_and_create_lock_git(self, rcstack, tmpdir): | |||
r2456 | # enable locking | |||
r = Repository.get_by_repo_name(GIT_REPO) | ||||
r.enable_locking = True | ||||
Session().add(r) | ||||
Session().commit() | ||||
# clone | ||||
r5607 | clone_url = rcstack.repo_clone_url(GIT_REPO) | |||
stdout, stderr = Command(tmpdir.strpath).execute("git clone", clone_url, tmpdir.strpath) | ||||
r2456 | ||||
# check if lock was made | ||||
r = Repository.get_by_repo_name(GIT_REPO) | ||||
r5607 | assert r.locked[0] == User.get_by_username(TEST_USER_ADMIN_LOGIN).user_id | |||
r2456 | ||||
r5607 | def test_clone_after_repo_was_locked_hg(self, rcstack, tmpdir): | |||
r2456 | # lock repo | |||
r = Repository.get_by_repo_name(HG_REPO) | ||||
Repository.lock(r, User.get_by_username(TEST_USER_ADMIN_LOGIN).user_id) | ||||
# pull fails since repo is locked | ||||
r5607 | clone_url = rcstack.repo_clone_url(HG_REPO) | |||
stdout, stderr = Command(tmpdir.strpath).execute("hg clone", clone_url, tmpdir.strpath) | ||||
msg = f"abort: HTTP Error 423: Repository `{HG_REPO}` locked by user `{TEST_USER_ADMIN_LOGIN}`" | ||||
r2456 | assert msg in stderr | |||
r5607 | def test_clone_after_repo_was_locked_git(self, rcstack, tmpdir): | |||
r2456 | # lock repo | |||
r = Repository.get_by_repo_name(GIT_REPO) | ||||
Repository.lock(r, User.get_by_username(TEST_USER_ADMIN_LOGIN).user_id) | ||||
# pull fails since repo is locked | ||||
r5607 | clone_url = rcstack.repo_clone_url(GIT_REPO) | |||
stdout, stderr = Command(tmpdir.strpath).execute("git clone", clone_url, tmpdir.strpath) | ||||
r2456 | ||||
r5607 | lock_msg = "remote: ERROR: Repository `vcs_test_git` locked by user `test_admin`. Reason:`lock_auto`" | |||
r2456 | assert lock_msg in stderr | |||
r5607 | assert "fatal: remote did not send all necessary objects" in stderr | |||
assert "remote: Pre pull hook failed: aborting" in stderr | ||||
r2456 | ||||
r5607 | def test_push_on_locked_repo_by_other_user_hg(self, rcstack, tmpdir): | |||
clone_url = rcstack.repo_clone_url(HG_REPO) | ||||
stdout, stderr = Command(tmpdir.strpath).execute("hg clone", clone_url, tmpdir.strpath) | ||||
r2456 | ||||
# lock repo | ||||
r = Repository.get_by_repo_name(HG_REPO) | ||||
# let this user actually push ! | ||||
r5607 | RepoModel().grant_user_permission(repo=r, user=TEST_USER_REGULAR_LOGIN, perm="repository.write") | |||
r2456 | Session().commit() | |||
Repository.lock(r, User.get_by_username(TEST_USER_ADMIN_LOGIN).user_id) | ||||
# push fails repo is locked by other user ! | ||||
r5607 | push_url = rcstack.repo_clone_url(HG_REPO, user=TEST_USER_REGULAR_LOGIN, passwd=TEST_USER_REGULAR_PASS) | |||
stdout, stderr = _add_files_and_push("hg", tmpdir.strpath, clone_url=push_url) | ||||
msg = f"abort: HTTP Error 423: Repository `{HG_REPO}` locked by user `{TEST_USER_ADMIN_LOGIN}`" | ||||
r2456 | assert msg in stderr | |||
r5607 | def test_push_on_locked_repo_by_other_user_git(self, rcstack, tmpdir): | |||
clone_url = rcstack.repo_clone_url(GIT_REPO) | ||||
stdout, stderr = Command(tmpdir.strpath).execute("git clone", clone_url, tmpdir.strpath) | ||||
r2456 | ||||
# lock repo | ||||
r = Repository.get_by_repo_name(GIT_REPO) | ||||
# let this user actually push ! | ||||
r5607 | RepoModel().grant_user_permission(repo=r, user=TEST_USER_REGULAR_LOGIN, perm="repository.write") | |||
r2456 | Session().commit() | |||
Repository.lock(r, User.get_by_username(TEST_USER_ADMIN_LOGIN).user_id) | ||||
# push fails repo is locked by other user! | ||||
r5607 | push_url = rcstack.repo_clone_url(GIT_REPO, user=TEST_USER_REGULAR_LOGIN, passwd=TEST_USER_REGULAR_PASS) | |||
stdout, stderr = _add_files_and_push("git", tmpdir.strpath, clone_url=push_url) | ||||
r2456 | ||||
r5607 | err = f"Repository `{GIT_REPO}` locked by user `{TEST_USER_ADMIN_LOGIN}`" | |||
r2456 | # err = 'RPC failed; result=22, HTTP code = 423' | |||
assert err in stderr | ||||
r5607 | def test_push_unlocks_repository_hg(self, rcstack, tmpdir): | |||
r2456 | # enable locking | |||
r = Repository.get_by_repo_name(HG_REPO) | ||||
r.enable_locking = True | ||||
Session().add(r) | ||||
Session().commit() | ||||
r5607 | clone_url = rcstack.repo_clone_url(HG_REPO) | |||
stdout, stderr = Command(tmpdir.strpath).execute("hg clone", clone_url, tmpdir.strpath) | ||||
_check_proper_clone(stdout, stderr, "hg") | ||||
r2456 | ||||
# check for lock repo after clone | ||||
r = Repository.get_by_repo_name(HG_REPO) | ||||
uid = User.get_by_username(TEST_USER_ADMIN_LOGIN).user_id | ||||
assert r.locked[0] == uid | ||||
# push is ok and repo is now unlocked | ||||
r5607 | stdout, stderr = _add_files_and_push("hg", tmpdir.strpath, clone_url=clone_url) | |||
assert f"remote: Released lock on repo `{HG_REPO}`" in stdout | ||||
r2456 | # we need to cleanup the Session Here ! | |||
Session.remove() | ||||
r = Repository.get_by_repo_name(HG_REPO) | ||||
assert r.locked == [None, None, None] | ||||
r5607 | def test_push_unlocks_repository_git(self, rcstack, tmpdir): | |||
r2456 | # Note: Did a first debugging session. Seems that | |||
# Repository.get_locking_state is called twice. The second call | ||||
# has the action "pull" and does not reset the lock. | ||||
# enable locking | ||||
r = Repository.get_by_repo_name(GIT_REPO) | ||||
r.enable_locking = True | ||||
Session().add(r) | ||||
Session().commit() | ||||
r5607 | clone_url = rcstack.repo_clone_url(GIT_REPO) | |||
stdout, stderr = Command(tmpdir.strpath).execute("git clone", clone_url, tmpdir.strpath) | ||||
_check_proper_clone(stdout, stderr, "git") | ||||
r2456 | ||||
# check for lock repo after clone | ||||
r = Repository.get_by_repo_name(GIT_REPO) | ||||
r5607 | assert r.locked[0] == User.get_by_username(TEST_USER_ADMIN_LOGIN).user_id | |||
r2456 | ||||
# push is ok and repo is now unlocked | ||||
r5607 | stdout, stderr = _add_files_and_push("git", tmpdir.strpath, clone_url=clone_url) | |||
r2456 | _check_proper_git_push(stdout, stderr) | |||
# assert ('remote: Released lock on repo `%s`' % GIT_REPO) in stdout | ||||
# we need to cleanup the Session Here ! | ||||
Session.remove() | ||||
r = Repository.get_by_repo_name(GIT_REPO) | ||||
assert r.locked == [None, None, None] | ||||