Merge "Follow-up to "Add retry logic to _exec_ipmitool""

1 files changed, 5 insertions, 5 deletions

diff --git a/ironic/tests/drivers/test_ipmitool.py b/ironic/tests/drivers/test_ipmitool.py
index 0828f2c60..2e0097fba 100644
--- a/ironic/tests/drivers/test_ipmitool.py
+++ b/ironic/tests/drivers/test_ipmitool.py
@@ -891,21 +891,21 @@ class IPMIToolPrivateMethodTestCase(db_base.DbTestCase):
 
     @mock.patch.object(ipmi, '_is_option_supported', autospec=True)
     @mock.patch.object(utils, 'execute', autospec=True)
-    def test__exec_ipmitool_exception_retry_failure_unhandable(self,
+    def test__exec_ipmitool_exception_non_retryable_failure(self,
             mock_exec, mock_support, mock_sleep):
 
         ipmi.LAST_CMD_TIME = {}
         mock_support.return_value = False
 
-        # Return a retryable error, then a error that cannot
-        # be retryable thus resulting in a single retry
-        # attempt by _exec_ipmitool that is successful.
+        # Return a retryable error, then an error that cannot
+        # be retried thus resulting in a single retry
+        # attempt by _exec_ipmitool.
         mock_exec.side_effect = iter([
             processutils.ProcessExecutionError(
                 stderr="insufficient resources for session"
             ),
             processutils.ProcessExecutionError(
-                "Unknown"
+                stderr="Unknown"
             ),
             ])