misc: fix function-redefined pylint error (#62099)
This commit is contained in:
parent
3dbee7bc63
commit
928086b47e
|
@ -21,9 +21,9 @@ def add_logging_to_cmislib():
|
|||
'''Monkeypatch cmislib request module to log requests and responses.'''
|
||||
|
||||
from cmislib.atompub import binding
|
||||
from cmislib.net import RESTService
|
||||
from cmislib.net import RESTService as CMISRESTService
|
||||
|
||||
class RESTService(RESTService):
|
||||
class RESTService(CMISRESTService):
|
||||
def get(self, url, *args, **kwargs):
|
||||
logger = kwargs.pop('passerelle_logger')
|
||||
logger.debug('cmislib GET request to %s', url)
|
||||
|
|
|
@ -76,7 +76,7 @@ def test_check_status(mocked_get, connector):
|
|||
|
||||
|
||||
@mock.patch('passerelle.utils.Request.get')
|
||||
def test_check_status(mocked_get, connector):
|
||||
def test_check_status_error(mocked_get, connector):
|
||||
hello_response = json.loads(FAKE_HELLO_RESPONSE)
|
||||
del hello_response['Data']
|
||||
mocked_get.return_value = tests.utils.FakedResponse(content=json.dumps(hello_response), status_code=200)
|
||||
|
@ -208,12 +208,12 @@ def test_get_user_forms_failure_no_json(mocked_post, mocked_get, app, connector)
|
|||
resp = app.get(endpoint)
|
||||
result = resp.json
|
||||
assert result['err'] == 1
|
||||
assert result['err_desc'] == 'No JSON content returned: content'
|
||||
assert result['err_desc'] == "No JSON content returned: 'content'"
|
||||
|
||||
|
||||
@mock.patch('passerelle.utils.Request.get')
|
||||
@mock.patch('passerelle.utils.Request.post')
|
||||
def test_get_user_forms_failure_no_json(mocked_post, mocked_get, app, connector):
|
||||
def test_get_user_forms_failure_arpege_error(mocked_post, mocked_get, app, connector):
|
||||
endpoint = reverse(
|
||||
'generic-endpoint',
|
||||
kwargs={
|
||||
|
|
|
@ -128,13 +128,13 @@ def test_jsonresponse_error_header():
|
|||
assert data == {'test': 'un test', 'err': 0}
|
||||
|
||||
@to_json()
|
||||
def test_func(req):
|
||||
def test_func2(req):
|
||||
class CustomException(Exception):
|
||||
http_status = 200
|
||||
|
||||
raise CustomException
|
||||
|
||||
result = test_func(req)
|
||||
result = test_func2(req)
|
||||
data = json_loads(result.content)
|
||||
assert 'err_class' in data
|
||||
assert 'err' in data
|
||||
|
|
|
@ -167,7 +167,7 @@ def test_diffusion(app, connector):
|
|||
|
||||
# sender name not allowed
|
||||
@httmock.urlmatch(netloc=NETLOC)
|
||||
def mocked_response(url, request):
|
||||
def mocked_response2(url, request):
|
||||
request_body = json.loads(force_text(request.body))
|
||||
assert request_body['smsParam']['senderName'] == 'john'
|
||||
error_response = [
|
||||
|
@ -182,7 +182,7 @@ def test_diffusion(app, connector):
|
|||
orange.provide_sender = True
|
||||
orange.save()
|
||||
with pytest.raises(OrangeError, match='The given sender name is not allowed.'):
|
||||
with httmock.HTTMock(mocked_response):
|
||||
with httmock.HTTMock(mocked_response2):
|
||||
orange.diffusion('my_token', 'gid2', PAYLOAD['to'], PAYLOAD['message'], PAYLOAD['from'])
|
||||
|
||||
|
||||
|
|
Loading…
Reference in New Issue