openstack/oslo.messaging
Code Issues Proposed changes

Merge "Fix notification tests not unmocking logging"

Browse Source
This commit is contained in:
Jenkins 2017-05-03 20:29:40 +00:00 committed by Gerrit Code Review
commit e569c92cd9
1 changed files with 16 additions and 13 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

29
oslo_messaging/tests/notify/test_notifier.py
View File

@ -321,6 +321,8 @@ class TestLogNotifier(test_utils.BaseTestCase):
mock_utcnow.return_value = datetime.datetime.utcnow()
logger = mock.Mock()
message = {
'message_id': str(message_id),
'publisher_id': 'test.localhost',
@ -330,15 +332,15 @@ class TestLogNotifier(test_utils.BaseTestCase):
'timestamp': str(timeutils.utcnow()),
}
logger = mock.Mock()
logging.getLogger = mock.Mock()
logging.getLogger.return_value = logger
with mock.patch.object(logging, 'getLogger') as gl:
gl.return_value = logger
notifier.info({}, 'test.notify', 'bar')
notifier.info({}, 'test.notify', 'bar')
uuid.uuid4.assert_called_once_with()
logging.getLogger.assert_called_once_with(
'oslo.messaging.notification.test.notify')
uuid.uuid4.assert_called_once_with()
logging.getLogger.assert_called_once_with('oslo.messaging.'
'notification.test.notify')
logger.info.assert_called_once_with(JsonMessageMatcher(message))
self.assertTrue(notifier.is_enabled())
@ -351,14 +353,15 @@ class TestLogNotifier(test_utils.BaseTestCase):
'notification.foo'))
logger.sample = None
logging.getLogger = mock.Mock()
logging.getLogger.return_value = logger
msg = {'event_type': 'foo'}
driver.notify(None, msg, "sample", None)
logging.getLogger.assert_called_once_with('oslo.messaging.'
'notification.foo')
with mock.patch.object(logging, 'getLogger') as gl:
gl.return_value = logger
driver.notify(None, msg, "sample", None)
logging.getLogger.assert_called_once_with('oslo.messaging.'
'notification.foo')
def test_mask_passwords(self):
# Ensure that passwords are masked with notifications