##// END OF EJS Templates
core: no longer rely on webob exception inside get_or_404 function....
core: no longer rely on webob exception inside get_or_404 function. - relly on pyramid exc only - helps reduce number of appenlight problems with 404 handling via webob

File last commit:

r1785:1cce4ff2 default
r1956:a7a935de default
Show More
test_base.py
255 lines | 9.1 KiB | text/x-python | PythonLexer
# -*- coding: utf-8 -*-
# Copyright (C) 2010-2017 RhodeCode GmbH
#
# 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 pytest
from mock import Mock, patch
from rhodecode.lib import base
from rhodecode.lib.vcs.exceptions import RepositoryRequirementError
from rhodecode.lib import helpers as h
from rhodecode.model import db
@pytest.mark.parametrize('result_key, expected_value', [
('username', 'stub_username'),
('action', 'stub_action'),
('repository', 'stub_repo_name'),
('scm', 'stub_scm'),
('hooks', ['stub_hook']),
('config', 'stub_ini_filename'),
('ip', '1.2.3.4'),
('server_url', 'https://example.com'),
('user_agent', 'client-text-v1.1'),
# TODO: johbo: Commpare locking parameters with `_get_rc_scm_extras`
# in hooks_utils.
('make_lock', None),
('locked_by', [None, None, None]),
])
def test_vcs_operation_context_parameters(result_key, expected_value):
result = call_vcs_operation_context()
assert result[result_key] == expected_value
@patch('rhodecode.model.db.User.get_by_username', Mock())
@patch('rhodecode.model.db.Repository.get_by_repo_name')
def test_vcs_operation_context_checks_locking(mock_get_by_repo_name):
mock_get_locking_state = mock_get_by_repo_name().get_locking_state
mock_get_locking_state.return_value = (None, None, [None, None, None])
call_vcs_operation_context(check_locking=True)
assert mock_get_locking_state.called
@patch('rhodecode.model.db.Repository.get_locking_state')
def test_vcs_operation_context_skips_locking_checks_if_anonymouse(
mock_get_locking_state):
call_vcs_operation_context(
username=db.User.DEFAULT_USER, check_locking=True)
assert not mock_get_locking_state.called
@patch('rhodecode.model.db.Repository.get_locking_state')
def test_vcs_operation_context_can_skip_locking_check(mock_get_locking_state):
call_vcs_operation_context(check_locking=False)
assert not mock_get_locking_state.called
@patch.object(
base, 'get_enabled_hook_classes', Mock(return_value=['stub_hook']))
@patch('rhodecode.lib.utils2.get_server_url',
Mock(return_value='https://example.com'))
def call_vcs_operation_context(**kwargs_override):
kwargs = {
'repo_name': 'stub_repo_name',
'username': 'stub_username',
'action': 'stub_action',
'scm': 'stub_scm',
'check_locking': False,
}
kwargs.update(kwargs_override)
config_file_patch = patch.dict(
'rhodecode.CONFIG', {'__file__': 'stub_ini_filename'})
settings_patch = patch.object(base, 'VcsSettingsModel')
with config_file_patch, settings_patch as settings_mock:
result = base.vcs_operation_context(
environ={'HTTP_USER_AGENT': 'client-text-v1.1',
'REMOTE_ADDR': '1.2.3.4'}, **kwargs)
settings_mock.assert_called_once_with(repo='stub_repo_name')
return result
class TestBaseRepoController(object):
def test_context_is_updated_when_update_global_counters_is_called(self):
followers = 1
forks = 2
pull_requests = 3
is_following = True
scm_model = Mock(name="scm_model")
db_repo = Mock(name="db_repo")
scm_model.get_followers.return_value = followers
scm_model.get_forks.return_value = forks
scm_model.get_pull_requests.return_value = pull_requests
scm_model.is_following_repo.return_value = is_following
controller = base.BaseRepoController()
with patch.object(base, 'c') as context_mock:
controller._update_global_counters(scm_model, db_repo)
scm_model.get_pull_requests.assert_called_once_with(db_repo)
assert context_mock.repository_pull_requests == pull_requests
class TestBaseRepoControllerHandleMissingRequirements(object):
def test_logs_error_and_sets_repo_to_none(self, app):
controller = base.BaseRepoController()
error_message = 'Some message'
error = RepositoryRequirementError(error_message)
context_patcher = patch.object(base, 'c')
log_patcher = patch.object(base, 'log')
request_patcher = patch.object(base, 'request')
redirect_patcher = patch.object(base, 'redirect')
controller.rhodecode_repo = 'something'
with context_patcher as context_mock, log_patcher as log_mock, \
request_patcher, redirect_patcher:
context_mock.repo_name = 'abcde'
controller._handle_missing_requirements(error)
expected_log_message = (
'Requirements are missing for repository %s: %s', 'abcde',
error_message)
log_mock.error.assert_called_once_with(*expected_log_message)
assert controller.rhodecode_repo is None
@pytest.mark.parametrize('path, should_redirect', [
('/abcde', False),
('/abcde/settings', False),
('/abcde/settings/vcs', False),
('/_admin/repos/abcde', False), # Settings update
('/abcde/changelog', True),
('/abcde/files/tip', True),
('/abcde/settings/statistics', True),
])
def test_redirects_if_not_summary_or_settings_page(
self, app, path, should_redirect):
repo_name = 'abcde'
controller = base.BaseRepoController()
error = RepositoryRequirementError('Some message')
context_patcher = patch.object(base, 'c')
controller.rhodecode_repo = repo_name
request_patcher = patch.object(base, 'request')
redirect_patcher = patch.object(base, 'redirect')
with context_patcher as context_mock, \
request_patcher as request_mock, \
redirect_patcher as redirect_mock:
request_mock.path = path
context_mock.repo_name = repo_name
controller._handle_missing_requirements(error)
expected_url = h.route_path('repo_summary', repo_name=repo_name)
if should_redirect:
redirect_mock.assert_called_once_with(expected_url)
else:
redirect_mock.call_count == 0
class TestBaseRepoControllerBefore(object):
def test_flag_is_true_when_requirements_are_missing(self, before_mocks):
controller = self._get_controller()
handle_patcher = patch.object(
controller, '_handle_missing_requirements')
error = RepositoryRequirementError()
before_mocks.repository.scm_instance.side_effect = error
with handle_patcher as handle_mock:
controller.__before__()
handle_mock.assert_called_once_with(error)
assert before_mocks['context'].repository_requirements_missing is True
def test_flag_is_false_when_no_requirements_are_missing(
self, before_mocks):
controller = self._get_controller()
handle_patcher = patch.object(
controller, '_handle_missing_requirements')
with handle_patcher as handle_mock:
controller.__before__()
handle_mock.call_count == 0
assert before_mocks['context'].repository_requirements_missing is False
def test_update_global_counters_is_called(self, before_mocks):
controller = self._get_controller()
update_counters_patcher = patch.object(
controller, '_update_global_counters')
with update_counters_patcher as update_counters_mock:
controller.__before__()
update_counters_mock.assert_called_once_with(
controller.scm_model, before_mocks.repository)
def _get_controller(self):
controller = base.BaseRepoController()
controller.scm_model = Mock()
controller.rhodecode_repo = Mock()
return controller
@pytest.fixture
def before_mocks(request):
patcher = BeforePatcher()
patcher.start()
request.addfinalizer(patcher.stop)
return patcher
class BeforePatcher(object):
patchers = {}
mocks = {}
repository = None
def __init__(self):
self.repository = Mock()
def start(self):
self.patchers = {
'request': patch.object(base, 'request'),
'before': patch.object(base.BaseController, '__before__'),
'context': patch.object(base, 'c'),
'repo': patch.object(
base.Repository, 'get_by_repo_name',
return_value=self.repository)
}
self.mocks = {
p: self.patchers[p].start() for p in self.patchers
}
def stop(self):
for patcher in self.patchers.values():
patcher.stop()
def __getitem__(self, key):
return self.mocks[key]