##// END OF EJS Templates
ip-restritions: fix issue with loading empty IP rules for non-logged not-anonymous users....
ip-restritions: fix issue with loading empty IP rules for non-logged not-anonymous users. This case won't have any IP rules set. And we accidentally loaded them becuase of no empty user filter was present. This caused some odd display warnings for users.

File last commit:

r2487:fcee5614 default
r2838:d1dcf373 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 -*-
release: update copyright year to 2018
r2487 # Copyright (C) 2010-2018 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()