diff options
author | Riccardo Pittau <elfosardo@gmail.com> | 2020-07-16 11:53:55 +0200 |
---|---|---|
committer | Riccardo Pittau <elfosardo@gmail.com> | 2020-07-17 07:08:07 +0000 |
commit | a7ee205019e5e307b5d5c598dd3518ea6e42348e (patch) | |
tree | 54dcfb9efe58546e8a7637bde3b2eb9df7c68ce0 | |
parent | 5abb3aedce2ac19e1464d5f24c1846fccb1a54c0 (diff) | |
download | ironic-a7ee205019e5e307b5d5c598dd3518ea6e42348e.tar.gz |
Enforce autospec in test_notification_utils
And remove corresponding filter from tox.ini
Change-Id: I4724a5efd522c313d2f6dbb14dcdf19700f8cda7
-rw-r--r-- | ironic/tests/unit/conductor/test_notification_utils.py | 19 | ||||
-rw-r--r-- | tox.ini | 1 |
2 files changed, 12 insertions, 8 deletions
diff --git a/ironic/tests/unit/conductor/test_notification_utils.py b/ironic/tests/unit/conductor/test_notification_utils.py index cc8fe5320..dc3e1c43d 100644 --- a/ironic/tests/unit/conductor/test_notification_utils.py +++ b/ironic/tests/unit/conductor/test_notification_utils.py @@ -40,7 +40,8 @@ class TestNotificationUtils(db_base.DbTestCase): 'upgrade_lock', 'shared']) self.task.node = self.node - @mock.patch.object(notif_utils, '_emit_conductor_node_notification') + @mock.patch.object(notif_utils, '_emit_conductor_node_notification', + autospec=True) def test_emit_power_state_corrected_notification(self, mock_cond_emit): notif_utils.emit_power_state_corrected_notification( self.task, states.POWER_ON) @@ -54,7 +55,8 @@ class TestNotificationUtils(db_base.DbTestCase): from_power=states.POWER_ON ) - @mock.patch.object(notif_utils, '_emit_conductor_node_notification') + @mock.patch.object(notif_utils, '_emit_conductor_node_notification', + autospec=True) def test_emit_power_set_notification(self, mock_cond_emit): notif_utils.emit_power_set_notification( self.task, @@ -71,7 +73,8 @@ class TestNotificationUtils(db_base.DbTestCase): to_power=states.POWER_ON ) - @mock.patch.object(notif_utils, '_emit_conductor_node_notification') + @mock.patch.object(notif_utils, '_emit_conductor_node_notification', + autospec=True) def test_emit_console_notification(self, mock_cond_emit): notif_utils.emit_console_notification( self.task, 'console_set', fields.NotificationStatus.END) @@ -84,7 +87,8 @@ class TestNotificationUtils(db_base.DbTestCase): fields.NotificationStatus.END, ) - @mock.patch.object(notif_utils, '_emit_conductor_node_notification') + @mock.patch.object(notif_utils, '_emit_conductor_node_notification', + autospec=True) def test_emit_console_notification_error_status(self, mock_cond_emit): notif_utils.emit_console_notification( self.task, 'console_set', fields.NotificationStatus.ERROR) @@ -97,7 +101,7 @@ class TestNotificationUtils(db_base.DbTestCase): fields.NotificationStatus.ERROR, ) - @mock.patch.object(notification, 'mask_secrets') + @mock.patch.object(notification, 'mask_secrets', autospec=True) def test__emit_conductor_node_notification(self, mock_secrets): mock_notify_method = mock.Mock() # Required for exception handling @@ -152,7 +156,7 @@ class TestNotificationUtils(db_base.DbTestCase): self.assertFalse(mock_notify_method.called) - @mock.patch.object(notification, 'mask_secrets') + @mock.patch.object(notification, 'mask_secrets', autospec=True) def test__emit_conductor_node_notification_known_notify_exc(self, mock_secrets): """Test exception caught for a known notification exception.""" @@ -179,7 +183,8 @@ class TestNotificationUtils(db_base.DbTestCase): class ProvisionNotifyTestCase(tests_base.TestCase): - @mock.patch('ironic.objects.node.NodeSetProvisionStateNotification') + @mock.patch('ironic.objects.node.NodeSetProvisionStateNotification', + autospec=True) def test_emit_notification(self, provision_mock): provision_mock.__name__ = 'NodeSetProvisionStateNotification' self.config(host='fake-host') @@ -131,7 +131,6 @@ per-file-ignores = ironic/cmd/__init__.py:E402 ironic/tests/base.py:E402 ironic/tests/unit/api/*:H210 - ironic/tests/unit/conductor/test_notification_utils.py:H210 ironic/tests/unit/common/*:H210 ironic/tests/unit/drivers/modules/test_console_utils.py:H210 ironic/tests/unit/drivers/modules/test_iscsi_deploy.py:H210 |