Test exception handling paths of long_running_operation result

Signed-off-by: mulhern <amulhern@redhat.com>

Author: mulkieran

src/stratis_cli/_actions/_utils.py

@@ -297,12 +297,11 @@ def wrapper(namespace: Namespace):
         try:
             func(namespace)
         except DPClientInvocationError as err:
-            # sim engine completes all operations rapidly
             if not any(
                 isinstance(e, DBusException)
                 and e.get_dbus_name() == "org.freedesktop.DBus.Error.NoReply"
                 for e in get_errors(err)
-            ):  # pragma: no cover
+            ):
                 raise err
             print("Operation initiated", file=sys.stderr)
 

tests/whitebox/unit/test_running.py

@@ -0,0 +1,61 @@
+# Copyright 2025 Red Hat, Inc.
+#
+# 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.
+"""
+Test 'long_running_operation'.
+"""
+
+# isort: STDLIB
+import unittest
+
+# isort: THIRDPARTY
+import dbus
+
+# isort: FIRSTPARTY
+from dbus_python_client_gen import DPClientInvocationError
+
+# isort: LOCAL
+from stratis_cli._actions._utils import long_running_operation
+
+
+class LongRunningOperationTestCase(unittest.TestCase):
+    """
+    Test long_running_operation error paths that don't show up in the sim
+    engine.
+    """
+
+    def test_raise_dbus_exception(self):
+        """
+        Should succeed because it catches the distinguishing NoReply D-Bus
+        error.
+        """
+
+        def raises_error(_):
+            raise DPClientInvocationError(
+                "fake", "intf", None
+            ) from dbus.exceptions.DBusException(
+                name="org.freedesktop.DBus.Error.NoReply"
+            )
+
+        self.assertIsNone(long_running_operation(raises_error)(None))
+
+    def test_no_dbus_exception(self):
+        """
+        Should raise an exception that was previously raised.
+        """
+
+        def raises_error(_):
+            raise DPClientInvocationError("fake", "intf", None)
+
+        with self.assertRaises(DPClientInvocationError):
+            long_running_operation(raises_error)(None)