diff --git a/.packit.yaml b/.packit.yaml index 76f4e60ad..917ba5d56 100644 --- a/.packit.yaml +++ b/.packit.yaml @@ -27,7 +27,6 @@ jobs: - job: copr_build trigger: pull_request identifier: copr_pull - manual_trigger: true targets: - fedora-all diff --git a/src/stratis_cli/_error_reporting.py b/src/stratis_cli/_error_reporting.py index f637f47b9..f0158f2a7 100644 --- a/src/stratis_cli/_error_reporting.py +++ b/src/stratis_cli/_error_reporting.py @@ -288,7 +288,7 @@ def _interpret_errors_2( # pylint: disable=too-many-return-statements if dbus_name == "org.freedesktop.zbus.Error" and isinstance( context, DPClientSetPropertyContext - ): # pragma: no cover + ): return ( f"stratisd failed to perform the operation that you " f"requested, because it could not set the D-Bus " @@ -335,15 +335,6 @@ def _interpret_errors_2( # pylint: disable=too-many-return-statements f"service sent the following message: {dbus_message}." ) - if isinstance(context, DPClientSetPropertyContext): # pragma: no cover - return ( - f"stratisd failed to perform the operation that you " - f"requested, because it could not set the D-Bus " - f'property "{context.property_name}" belonging to ' - f'interface "{error.interface_name}" to "{context.value}". ' - f"It returned the following error: {dbus_message}." - ) - if isinstance(context, DPClientGetPropertyContext): # pragma: no cover return ( f"stratisd failed to perform the operation that you " diff --git a/tests/integration/logical/test_create.py b/tests/integration/logical/test_create.py index 831d933c7..f6cfb2521 100644 --- a/tests/integration/logical/test_create.py +++ b/tests/integration/logical/test_create.py @@ -44,10 +44,10 @@ def setUp(self): def test_creation(self): """ - Creation of two volumes at once should fail. + Creation of two volumes at once should succeed. """ command_line = self._MENU + [self._POOLNAME] + self._VOLNAMES - self.check_error(StratisCliEngineError, command_line, _ERROR) + TEST_RUNNER(command_line) class Create4TestCase(SimTestCase): @@ -106,8 +106,6 @@ def setUp(self): command_line = ["pool", "create", self._POOLNAME] + _DEVICE_STRATEGY() RUNNER(command_line) - # Creation of two volumes is split up into two calls to RUNNER, - # since only one filesystem per request is currently allowed. command_line = self._MENU + [self._POOLNAME] + self._VOLNAMES[0:1] RUNNER(command_line)