2015-12-10 15:22:10 +00:00
|
|
|
import contextlib
|
2015-11-18 21:15:37 +00:00
|
|
|
import logging
|
2015-12-10 15:22:10 +00:00
|
|
|
import os
|
2015-11-19 22:29:07 +00:00
|
|
|
import json
|
2015-12-10 15:22:10 +00:00
|
|
|
import unittest
|
2015-11-18 21:15:37 +00:00
|
|
|
|
2015-11-19 13:02:47 +00:00
|
|
|
from tornado import httputil, testing, web
|
2015-12-10 15:22:10 +00:00
|
|
|
import mock
|
2015-11-18 21:15:37 +00:00
|
|
|
|
2015-12-10 15:22:10 +00:00
|
|
|
import sprockets.http.mixins
|
2015-11-18 21:15:37 +00:00
|
|
|
import examples
|
|
|
|
|
|
|
|
|
|
|
|
class RecordingHandler(logging.Handler):
|
|
|
|
|
|
|
|
def __init__(self):
|
|
|
|
super(RecordingHandler, self).__init__()
|
|
|
|
self.emitted = []
|
|
|
|
|
|
|
|
def emit(self, record):
|
|
|
|
self.emitted.append((record, self.format(record)))
|
|
|
|
|
|
|
|
|
2015-12-10 15:22:10 +00:00
|
|
|
class RaisingHandler(sprockets.http.mixins.ErrorLogger,
|
|
|
|
sprockets.http.mixins.ErrorWriter,
|
2015-11-19 22:29:07 +00:00
|
|
|
web.RequestHandler):
|
2015-11-19 13:02:47 +00:00
|
|
|
|
|
|
|
def get(self, status_code):
|
|
|
|
raise web.HTTPError(int(status_code),
|
|
|
|
reason=self.get_query_argument('reason', None))
|
|
|
|
|
|
|
|
|
2015-12-10 15:22:10 +00:00
|
|
|
class MockHelper(unittest.TestCase):
|
|
|
|
|
|
|
|
def setUp(self):
|
|
|
|
super(MockHelper, self).setUp()
|
|
|
|
self._mocks = []
|
|
|
|
|
|
|
|
def tearDown(self):
|
|
|
|
super(MockHelper, self).tearDown()
|
|
|
|
for mocker in self._mocks:
|
|
|
|
mocker.stop()
|
|
|
|
del self._mocks[:]
|
|
|
|
|
|
|
|
def start_mock(self, target):
|
|
|
|
mocked = mock.patch(target)
|
|
|
|
self._mocks.append(mocked)
|
|
|
|
return mocked.start()
|
|
|
|
|
|
|
|
|
|
|
|
@contextlib.contextmanager
|
|
|
|
def override_environment_variable(name, value):
|
|
|
|
stash = os.environ.pop(name, None)
|
|
|
|
if value is not None:
|
|
|
|
os.environ[name] = value
|
|
|
|
try:
|
|
|
|
yield
|
|
|
|
finally:
|
|
|
|
os.environ.pop(name, None)
|
|
|
|
if stash is not None:
|
|
|
|
os.environ[name] = stash
|
|
|
|
|
|
|
|
|
2015-11-19 16:54:23 +00:00
|
|
|
class ErrorLoggerTests(testing.AsyncHTTPTestCase):
|
2015-11-18 21:15:37 +00:00
|
|
|
|
|
|
|
def setUp(self):
|
2015-11-19 16:54:23 +00:00
|
|
|
super(ErrorLoggerTests, self).setUp()
|
2015-11-18 21:15:37 +00:00
|
|
|
self.recorder = RecordingHandler()
|
|
|
|
root_logger = logging.getLogger()
|
|
|
|
root_logger.addHandler(self.recorder)
|
|
|
|
|
|
|
|
def tearDown(self):
|
2015-11-19 16:54:23 +00:00
|
|
|
super(ErrorLoggerTests, self).tearDown()
|
2015-11-18 21:15:37 +00:00
|
|
|
logging.getLogger().removeHandler(self.recorder)
|
|
|
|
|
|
|
|
def get_app(self):
|
2015-11-19 13:02:47 +00:00
|
|
|
return web.Application([
|
|
|
|
web.url(r'/status/(?P<status_code>\d+)', examples.StatusHandler),
|
|
|
|
web.url(r'/fail/(?P<status_code>\d+)', RaisingHandler),
|
|
|
|
])
|
2015-11-18 21:15:37 +00:00
|
|
|
|
|
|
|
def assert_message_logged(self, level, msg_fmt, *msg_args):
|
|
|
|
suffix = msg_fmt.format(*msg_args)
|
|
|
|
for record, message in self.recorder.emitted:
|
2015-11-19 13:02:47 +00:00
|
|
|
if record.levelno == level and message.endswith(suffix):
|
|
|
|
return
|
|
|
|
self.fail('Expected message ending in "%s" to be logged in %r'
|
|
|
|
% (suffix, self.recorder.emitted))
|
2015-11-18 21:15:37 +00:00
|
|
|
|
|
|
|
def test_that_client_error_logged_as_warning(self):
|
|
|
|
self.fetch('/status/400')
|
|
|
|
self.assert_message_logged(
|
|
|
|
logging.WARNING, 'failed with 400: {}', httputil.responses[400])
|
|
|
|
|
|
|
|
def test_that_server_error_logged_as_error(self):
|
|
|
|
self.fetch('/status/500')
|
|
|
|
self.assert_message_logged(
|
|
|
|
logging.ERROR, 'failed with 500: {}', httputil.responses[500])
|
|
|
|
|
|
|
|
def test_that_custom_status_codes_logged_as_unknown(self):
|
|
|
|
self.fetch('/status/623')
|
|
|
|
self.assert_message_logged(logging.ERROR, 'failed with 623: Unknown')
|
|
|
|
|
|
|
|
def test_that_custom_reasons_are_supported(self):
|
|
|
|
self.fetch('/status/456?reason=oops')
|
|
|
|
self.assert_message_logged(logging.WARNING, 'failed with 456: oops')
|
2015-11-19 13:02:47 +00:00
|
|
|
|
|
|
|
def test_that_status_code_extracted_from_http_errors(self):
|
|
|
|
self.fetch('/fail/400')
|
|
|
|
self.assert_message_logged(
|
|
|
|
logging.WARNING, 'failed with 400: {}', httputil.responses[400])
|
|
|
|
|
|
|
|
def test_that_reason_extracted_from_http_errors(self):
|
|
|
|
self.fetch('/fail/400?reason=oopsie')
|
|
|
|
self.assert_message_logged(logging.WARNING, 'failed with 400: oopsie')
|
2015-11-19 16:51:34 +00:00
|
|
|
|
|
|
|
def test_that_log_message_is_honored(self):
|
|
|
|
self.fetch('/status/400?log_message=injected%20message')
|
|
|
|
self.assert_message_logged(logging.WARNING,
|
|
|
|
'failed with 400: injected message')
|
2015-11-19 22:29:07 +00:00
|
|
|
|
|
|
|
|
|
|
|
class ErrorWriterTests(testing.AsyncHTTPTestCase):
|
|
|
|
|
2015-11-20 12:25:05 +00:00
|
|
|
def setUp(self):
|
|
|
|
self._application = None
|
|
|
|
super(ErrorWriterTests, self).setUp()
|
|
|
|
|
|
|
|
@property
|
|
|
|
def application(self):
|
|
|
|
if self._application is None:
|
|
|
|
self._application = web.Application([
|
|
|
|
web.url(r'/status/(?P<status_code>\d+)',
|
|
|
|
examples.StatusHandler),
|
|
|
|
web.url(r'/fail/(?P<status_code>\d+)', RaisingHandler),
|
|
|
|
])
|
|
|
|
return self._application
|
|
|
|
|
2015-11-19 22:29:07 +00:00
|
|
|
def get_app(self):
|
2015-11-20 12:25:05 +00:00
|
|
|
return self.application
|
2015-11-19 22:29:07 +00:00
|
|
|
|
|
|
|
def _decode_response(self, response):
|
|
|
|
content_type = response.headers['Content-Type']
|
|
|
|
self.assertTrue(content_type.startswith('application/json'),
|
|
|
|
'Incorrect content type received')
|
|
|
|
return json.loads(response.body.decode('utf-8'))
|
|
|
|
|
|
|
|
def test_that_error_json_contains_error_type(self):
|
|
|
|
response = self.fetch('/fail/400')
|
|
|
|
self.assertEqual(response.code, 400)
|
|
|
|
|
|
|
|
exc = web.HTTPError(400)
|
|
|
|
body = self._decode_response(response)
|
|
|
|
self.assertEqual(body['type'], exc.__class__.__name__)
|
|
|
|
|
|
|
|
def test_that_error_json_contains_error_message(self):
|
|
|
|
response = self.fetch('/fail/400')
|
|
|
|
self.assertEqual(response.code, 400)
|
|
|
|
|
|
|
|
exc = web.HTTPError(400)
|
|
|
|
body = self._decode_response(response)
|
|
|
|
self.assertEqual(body['message'], str(exc))
|
|
|
|
|
|
|
|
def test_that_error_json_ignores_the_log_message(self):
|
|
|
|
response = self.fetch('/status/500?log_message=something%20good')
|
|
|
|
self.assertEqual(response.code, 500)
|
|
|
|
|
|
|
|
body = self._decode_response(response)
|
|
|
|
self.assertEqual(body['message'], httputil.responses[500])
|
|
|
|
|
|
|
|
def test_that_error_json_contains_type_none_for_non_exceptions(self):
|
|
|
|
response = self.fetch('/status/500')
|
|
|
|
self.assertEqual(response.code, 500)
|
|
|
|
|
|
|
|
body = self._decode_response(response)
|
|
|
|
self.assertIsNone(body['type'])
|
|
|
|
|
|
|
|
def test_that_error_json_contains_reason_for_non_exceptions(self):
|
|
|
|
response = self.fetch('/status/500')
|
|
|
|
self.assertEqual(response.code, 500)
|
|
|
|
|
|
|
|
body = self._decode_response(response)
|
|
|
|
self.assertEqual(body['message'], httputil.responses[500])
|
|
|
|
|
|
|
|
def test_that_error_json_reason_contains_unknown_in_some_cases(self):
|
|
|
|
response = self.fetch('/status/567')
|
|
|
|
self.assertEqual(response.code, 567)
|
|
|
|
|
|
|
|
body = self._decode_response(response)
|
|
|
|
self.assertEqual(body['message'], 'Unknown')
|
2015-11-20 12:25:05 +00:00
|
|
|
|
|
|
|
def test_that_error_json_honors_serve_traceback(self):
|
|
|
|
self.application.settings['serve_traceback'] = True
|
|
|
|
|
|
|
|
response = self.fetch('/fail/400')
|
|
|
|
self.assertEqual(response.code, 400)
|
|
|
|
|
|
|
|
body = self._decode_response(response)
|
|
|
|
self.assertGreater(len(body['traceback']), 0)
|
2015-11-20 19:14:39 +00:00
|
|
|
|
|
|
|
def test_that_mediatype_mixin_is_honored(self):
|
|
|
|
send_response = mock.Mock()
|
|
|
|
setattr(examples.StatusHandler, 'send_response', send_response)
|
|
|
|
response = self.fetch('/status/500')
|
|
|
|
self.assertEqual(response.code, 500)
|
|
|
|
send_response.assert_called_once_with({
|
|
|
|
'type': None,
|
|
|
|
'message': 'Internal Server Error',
|
|
|
|
'traceback': None
|
|
|
|
})
|
|
|
|
delattr(examples.StatusHandler, 'send_response')
|
2015-12-10 15:22:10 +00:00
|
|
|
|
|
|
|
|
|
|
|
class RunTests(MockHelper, unittest.TestCase):
|
|
|
|
|
|
|
|
def setUp(self):
|
|
|
|
super(RunTests, self).setUp()
|
|
|
|
self.runner_cls = self.start_mock('sprockets.http.runner.Runner')
|
2015-12-10 15:41:59 +00:00
|
|
|
self.get_logging_config = self.start_mock(
|
2015-12-10 15:44:52 +00:00
|
|
|
'sprockets.http._get_logging_config')
|
2015-12-10 15:41:59 +00:00
|
|
|
self.get_logging_config.return_value = {'version': 1}
|
|
|
|
self.logging_dict_config = self.start_mock(
|
|
|
|
'sprockets.http.logging.config').dictConfig
|
2015-12-10 15:22:10 +00:00
|
|
|
|
|
|
|
@property
|
|
|
|
def runner_instance(self):
|
|
|
|
return self.runner_cls.return_value
|
|
|
|
|
|
|
|
def test_that_runner_run_called_with_created_application(self):
|
|
|
|
create_app = mock.Mock()
|
|
|
|
sprockets.http.run(create_app)
|
|
|
|
self.assertEqual(create_app.call_count, 1)
|
|
|
|
self.runner_cls.assert_called_once_with(create_app.return_value)
|
|
|
|
|
|
|
|
def test_that_debug_envvar_enables_debug_flag(self):
|
|
|
|
create_app = mock.Mock()
|
|
|
|
with override_environment_variable('DEBUG', '1'):
|
|
|
|
sprockets.http.run(create_app)
|
|
|
|
create_app.assert_called_once_with(debug=True)
|
2015-12-10 15:41:59 +00:00
|
|
|
self.get_logging_config.assert_called_once_with(True)
|
2015-12-10 15:22:10 +00:00
|
|
|
|
|
|
|
def test_that_false_debug_envvar_disables_debug_flag(self):
|
|
|
|
create_app = mock.Mock()
|
|
|
|
with override_environment_variable('DEBUG', '0'):
|
|
|
|
sprockets.http.run(create_app)
|
|
|
|
create_app.assert_called_once_with(debug=False)
|
2015-12-10 15:41:59 +00:00
|
|
|
self.get_logging_config.assert_called_once_with(False)
|
2015-12-10 15:22:10 +00:00
|
|
|
|
|
|
|
def test_that_unset_debug_envvar_disables_debug_flag(self):
|
|
|
|
create_app = mock.Mock()
|
|
|
|
with override_environment_variable('DEBUG', None):
|
|
|
|
sprockets.http.run(create_app)
|
|
|
|
create_app.assert_called_once_with(debug=False)
|
2015-12-10 15:41:59 +00:00
|
|
|
self.get_logging_config.assert_called_once_with(False)
|
2015-12-10 15:22:10 +00:00
|
|
|
|
|
|
|
def test_that_port_defaults_to_8000(self):
|
|
|
|
sprockets.http.run(mock.Mock())
|
|
|
|
self.runner_instance.run.called_once_with(8000, mock.ANY)
|
|
|
|
|
|
|
|
def test_that_port_envvar_sets_port_number(self):
|
|
|
|
with override_environment_variable('PORT', '8888'):
|
|
|
|
sprockets.http.run(mock.Mock())
|
|
|
|
self.runner_instance.run.called_once_with(8888, mock.ANY)
|
|
|
|
|
|
|
|
def test_that_port_kwarg_sets_port_number(self):
|
|
|
|
sprockets.http.run(mock.Mock(), settings={'port': 8888})
|
2015-12-10 15:41:59 +00:00
|
|
|
self.runner_instance.run.assert_called_once_with(8888, mock.ANY)
|
2015-12-10 15:22:10 +00:00
|
|
|
|
|
|
|
def test_that_number_of_procs_defaults_to_zero(self):
|
|
|
|
sprockets.http.run(mock.Mock())
|
2015-12-10 15:41:59 +00:00
|
|
|
self.runner_instance.run.assert_called_once_with(mock.ANY, 0)
|
2015-12-10 15:22:10 +00:00
|
|
|
|
|
|
|
def test_that_number_of_process_kwarg_sets_number_of_procs(self):
|
|
|
|
sprockets.http.run(mock.Mock(), settings={'number_of_procs': 1})
|
2015-12-10 15:41:59 +00:00
|
|
|
self.runner_instance.run.assert_called_once_with(mock.ANY, 1)
|
|
|
|
|
|
|
|
def test_that_logging_dict_config_is_called_appropriately(self):
|
|
|
|
sprockets.http.run(mock.Mock())
|
|
|
|
self.logging_dict_config.assert_called_once_with(
|
|
|
|
self.get_logging_config.return_value)
|
2015-12-10 15:48:25 +00:00
|
|
|
|
|
|
|
def test_that_logconfig_override_is_used(self):
|
|
|
|
sprockets.http.run(mock.Mock(), log_config=mock.sentinel.config)
|
|
|
|
self.logging_dict_config.assert_called_once_with(
|
|
|
|
mock.sentinel.config)
|