##// END OF EJS Templates
user-groups: fix potential problem with group sync of external plugins....
user-groups: fix potential problem with group sync of external plugins. - when using external plugin we used to check for a parameter that set the sync mode. The problem is we only checked if the flag was there. So toggling sync on and off set the value and then left the key still set but with None. This confused the sync and thought the group should be synced !

File last commit:

r1410:a4133cfc default
r2143:4314e88b default
Show More
test_client_http.py
133 lines | 4.0 KiB | text/x-python | PythonLexer
/ rhodecode / tests / vcs / test_client_http.py
project: added all source files and assets
r1 # -*- coding: utf-8 -*-
license: updated copyright year to 2017
r1271 # Copyright (C) 2010-2017 RhodeCode GmbH
project: added all source files and assets
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 logging
import mock
import msgpack
import pytest
from rhodecode.lib import vcs
http-backend: catch errors from HTTP calls that are not raising exceptions, but...
r1410 from rhodecode.lib.vcs import client_http, exceptions
project: added all source files and assets
r1
def is_new_connection(logger, level, message):
return (
logger == 'requests.packages.urllib3.connectionpool' and
message.startswith('Starting new HTTP'))
@pytest.fixture
def stub_session():
"""
Stub of `requests.Session()`.
"""
session = mock.Mock()
http-backend: catch errors from HTTP calls that are not raising exceptions, but...
r1410 post = session.post()
post.content = msgpack.packb({})
post.status_code = 200
session.reset_mock()
return session
@pytest.fixture
def stub_fail_session():
"""
Stub of `requests.Session()`.
"""
session = mock.Mock()
post = session.post()
post.content = msgpack.packb({'error': '500'})
post.status_code = 500
project: added all source files and assets
r1 session.reset_mock()
return session
Martin Bornhold
tests: Add session factory fixture and use it in tests.
r289 @pytest.fixture
def stub_session_factory(stub_session):
"""
Stub of `rhodecode.lib.vcs.client_http.ThreadlocalSessionFactory`.
"""
session_factory = mock.Mock()
session_factory.return_value = stub_session
return session_factory
http-backend: catch errors from HTTP calls that are not raising exceptions, but...
r1410 @pytest.fixture
def stub_session_failing_factory(stub_fail_session):
"""
Stub of `rhodecode.lib.vcs.client_http.ThreadlocalSessionFactory`.
"""
session_factory = mock.Mock()
session_factory.return_value = stub_fail_session
return session_factory
def test_uses_persistent_http_connections(caplog, vcsbackend_hg):
repo = vcsbackend_hg.repo
remote_call = repo._remote.branches
with caplog.at_level(logging.INFO):
for x in range(5):
remote_call(normal=True, closed=False)
new_connections = [
r for r in caplog.record_tuples if is_new_connection(*r)]
assert len(new_connections) <= 1
Martin Bornhold
tests: Add session factory fixture and use it in tests.
r289 def test_repo_maker_uses_session_for_classmethods(stub_session_factory):
project: added all source files and assets
r1 repo_maker = client_http.RepoMaker(
vcs-http: explicitly pass in vcs-type into vcsserver.
r1126 'server_and_port', 'endpoint', 'test_dummy_scm', stub_session_factory)
project: added all source files and assets
r1 repo_maker.example_call()
Martin Bornhold
tests: Add session factory fixture and use it in tests.
r289 stub_session_factory().post.assert_called_with(
project: added all source files and assets
r1 'http://server_and_port/endpoint', data=mock.ANY)
def test_repo_maker_uses_session_for_instance_methods(
Martin Bornhold
tests: Add session factory fixture and use it in tests.
r289 stub_session_factory, config):
project: added all source files and assets
r1 repo_maker = client_http.RepoMaker(
vcs-http: explicitly pass in vcs-type into vcsserver.
r1126 'server_and_port', 'endpoint', 'test_dummy_scm', stub_session_factory)
project: added all source files and assets
r1 repo = repo_maker('stub_path', config)
repo.example_call()
Martin Bornhold
tests: Add session factory fixture and use it in tests.
r289 stub_session_factory().post.assert_called_with(
project: added all source files and assets
r1 'http://server_and_port/endpoint', data=mock.ANY)
Martin Bornhold
tests: Adapt vcs test to check that the session factory returns the same session object.
r288 @mock.patch('rhodecode.lib.vcs.client_http.ThreadlocalSessionFactory')
project: added all source files and assets
r1 @mock.patch('rhodecode.lib.vcs.connection')
Martin Bornhold
tests: Add session factory fixture and use it in tests.
r289 def test_connect_passes_in_the_same_session(
connection, session_factory_class, stub_session):
Martin Bornhold
tests: Adapt vcs test to check that the session factory returns the same session object.
r288 session_factory = session_factory_class.return_value
session_factory.return_value = stub_session
vcs.connect_http('server_and_port')
http-backend: catch errors from HTTP calls that are not raising exceptions, but...
r1410
def test_repo_maker_uses_session_that_throws_error(
stub_session_failing_factory, config):
repo_maker = client_http.RepoMaker(
'server_and_port', 'endpoint', 'test_dummy_scm', stub_session_failing_factory)
repo = repo_maker('stub_path', config)
with pytest.raises(exceptions.HttpVCSCommunicationError):
repo.example_call()