a6ad8da299
Use before_cursor_execute and after_cursor_execute instead of before_execute and after_execute. 1) Migration with custom types will work now (even if tracing is enabled) 2) We don't need to render 2 times SQL expressesion => less overhead Change-Id: I7c6b6909ce0f15a69ce5caad544e1351a647b472
76 lines
2.7 KiB
Python
76 lines
2.7 KiB
Python
# Copyright 2014 Mirantis Inc.
|
|
# All Rights Reserved.
|
|
#
|
|
# Licensed under the Apache License, Version 2.0 (the "License"); you may
|
|
# not use this file except in compliance with the License. You may obtain
|
|
# a copy of the License at
|
|
#
|
|
# http://www.apache.org/licenses/LICENSE-2.0
|
|
#
|
|
# Unless required by applicable law or agreed to in writing, software
|
|
# distributed under the License is distributed on an "AS IS" BASIS, WITHOUT
|
|
# WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the
|
|
# License for the specific language governing permissions and limitations
|
|
# under the License.
|
|
|
|
import mock
|
|
|
|
from osprofiler import sqlalchemy
|
|
|
|
from tests import test
|
|
|
|
|
|
class SqlalchemyTracingTestCase(test.TestCase):
|
|
|
|
@mock.patch("osprofiler.sqlalchemy.profiler")
|
|
def test_before_execute(self, mock_profiler):
|
|
handler = sqlalchemy._before_cursor_execute("sql")
|
|
|
|
handler(mock.MagicMock(), 1, 2, 3, 4, 5)
|
|
expected_info = {
|
|
"db.statement": 2,
|
|
"db.params": 3
|
|
}
|
|
mock_profiler.start.assert_called_once_with("sql", info=expected_info)
|
|
|
|
@mock.patch("osprofiler.sqlalchemy.profiler")
|
|
def test_after_execute(self, mock_profiler):
|
|
handler = sqlalchemy._after_cursor_execute()
|
|
handler(mock.MagicMock(), 1, 2, 3, 4, 5)
|
|
mock_profiler.stop.assert_called_once_with()
|
|
|
|
@mock.patch("osprofiler.sqlalchemy._before_cursor_execute")
|
|
@mock.patch("osprofiler.sqlalchemy._after_cursor_execute")
|
|
def test_add_tracing(self, mock_after_exc, mock_before_exc):
|
|
sa = mock.MagicMock()
|
|
engine = mock.MagicMock()
|
|
|
|
mock_before_exc.return_value = "before"
|
|
mock_after_exc.return_value = "after"
|
|
|
|
sqlalchemy.add_tracing(sa, engine, "sql")
|
|
|
|
mock_before_exc.assert_called_once_with("sql")
|
|
mock_after_exc.assert_called_once_with()
|
|
expected_calls = [
|
|
mock.call(engine, "before_cursor_execute", "before"),
|
|
mock.call(engine, "after_cursor_execute", "after")
|
|
]
|
|
self.assertEqual(sa.event.listen.call_args_list, expected_calls)
|
|
|
|
@mock.patch("osprofiler.sqlalchemy._before_cursor_execute")
|
|
@mock.patch("osprofiler.sqlalchemy._after_cursor_execute")
|
|
def test_disable_and_enable(self, mock_after_exc, mock_before_exc):
|
|
sqlalchemy.disable()
|
|
|
|
sa = mock.MagicMock()
|
|
engine = mock.MagicMock()
|
|
sqlalchemy.add_tracing(sa, engine, "sql")
|
|
self.assertFalse(mock_after_exc.called)
|
|
self.assertFalse(mock_before_exc.called)
|
|
|
|
sqlalchemy.enable()
|
|
sqlalchemy.add_tracing(sa, engine, "sql")
|
|
self.assertTrue(mock_after_exc.called)
|
|
self.assertTrue(mock_before_exc.called)
|