summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
-rw-r--r--ironic/tests/unit/drivers/modules/test_agent_base.py14
-rw-r--r--tox.ini1
2 files changed, 7 insertions, 8 deletions
diff --git a/ironic/tests/unit/drivers/modules/test_agent_base.py b/ironic/tests/unit/drivers/modules/test_agent_base.py
index d684fda9b..21dd38881 100644
--- a/ironic/tests/unit/drivers/modules/test_agent_base.py
+++ b/ironic/tests/unit/drivers/modules/test_agent_base.py
@@ -533,7 +533,7 @@ class HeartbeatMixinTest(AgentDeployMixinBaseTest):
mock_notify.assert_called_once_with(task, 'clean')
mock_set_steps.assert_called_once_with(task)
- @mock.patch.object(manager_utils, 'cleaning_error_handler')
+ @mock.patch.object(manager_utils, 'cleaning_error_handler', autospec=True)
@mock.patch.object(objects.node.Node, 'touch_provisioning', autospec=True)
@mock.patch.object(agent_base.HeartbeatMixin,
'refresh_steps', autospec=True)
@@ -611,7 +611,7 @@ class HeartbeatMixinTest(AgentDeployMixinBaseTest):
mock_touch.assert_called_once_with(mock.ANY)
self.assertFalse(mock_continue.called)
- @mock.patch.object(manager_utils, 'cleaning_error_handler')
+ @mock.patch.object(manager_utils, 'cleaning_error_handler', autospec=True)
@mock.patch.object(agent_base.HeartbeatMixin,
'continue_cleaning', autospec=True)
def test_heartbeat_continue_cleaning_fails(self, mock_continue,
@@ -634,7 +634,7 @@ class HeartbeatMixinTest(AgentDeployMixinBaseTest):
mock_continue.assert_called_once_with(mock.ANY, task)
mock_handler.assert_called_once_with(task, mock.ANY)
- @mock.patch.object(manager_utils, 'rescuing_error_handler')
+ @mock.patch.object(manager_utils, 'rescuing_error_handler', autospec=True)
@mock.patch.object(agent_base.HeartbeatMixin, '_finalize_rescue',
autospec=True)
def test_heartbeat_rescue(self, mock_finalize_rescue,
@@ -648,7 +648,7 @@ class HeartbeatMixinTest(AgentDeployMixinBaseTest):
mock_finalize_rescue.assert_called_once_with(mock.ANY, task)
self.assertFalse(mock_rescue_err_handler.called)
- @mock.patch.object(manager_utils, 'rescuing_error_handler')
+ @mock.patch.object(manager_utils, 'rescuing_error_handler', autospec=True)
@mock.patch.object(agent_base.HeartbeatMixin, '_finalize_rescue',
autospec=True)
def test_heartbeat_rescue_fails(self, mock_finalize,
@@ -848,7 +848,7 @@ class AgentRescueTests(AgentDeployMixinBaseTest):
class AgentDeployMixinTest(AgentDeployMixinBaseTest):
- @mock.patch.object(manager_utils, 'power_on_node_if_needed')
+ @mock.patch.object(manager_utils, 'power_on_node_if_needed', autospec=True)
@mock.patch.object(manager_utils, 'notify_conductor_resume_deploy',
autospec=True)
@mock.patch.object(driver_utils, 'collect_ramdisk_logs', autospec=True)
@@ -997,7 +997,7 @@ class AgentDeployMixinTest(AgentDeployMixinBaseTest):
self.assertEqual(states.ACTIVE, task.node.target_provision_state)
self.assertFalse(mock_collect.called)
- @mock.patch.object(manager_utils, 'power_on_node_if_needed')
+ @mock.patch.object(manager_utils, 'power_on_node_if_needed', autospec=True)
@mock.patch.object(manager_utils, 'notify_conductor_resume_deploy',
autospec=True)
@mock.patch.object(driver_utils, 'collect_ramdisk_logs', autospec=True)
@@ -2109,7 +2109,7 @@ class AgentDeployMixinTest(AgentDeployMixinBaseTest):
@mock.patch.object(manager_utils, 'restore_power_state_if_needed',
autospec=True)
- @mock.patch.object(manager_utils, 'power_on_node_if_needed')
+ @mock.patch.object(manager_utils, 'power_on_node_if_needed', autospec=True)
@mock.patch.object(manager_utils, 'notify_conductor_resume_deploy',
autospec=True)
@mock.patch.object(driver_utils, 'collect_ramdisk_logs', autospec=True)
diff --git a/tox.ini b/tox.ini
index ce4dbe76c..144c8023e 100644
--- a/tox.ini
+++ b/tox.ini
@@ -133,7 +133,6 @@ per-file-ignores =
ironic/tests/unit/api/*:H210
ironic/tests/unit/conductor/*:H210
ironic/tests/unit/common/*:H210
- ironic/tests/unit/drivers/modules/test_agent_base.py:H210
ironic/tests/unit/drivers/modules/test_console_utils.py:H210
ironic/tests/unit/drivers/modules/test_deploy_utils.py:H210
ironic/tests/unit/drivers/modules/test_ipxe.py:H210