python-openstackclient 9.0.0__py3-none-any.whl → 10.1.0__py3-none-any.whl

openstackclient/tests/unit/identity/v3/test_role_assignment.py

@@ -25,7 +25,7 @@ from openstackclient.identity.v3 import role_assignment
 from openstackclient.tests.unit.identity.v3 import fakes as identity_fakes
 
 
-class TestRoleAssignmentList(identity_fakes.TestIdentityv3):
+class TestRoleAssignmentList(identity_fakes.TestIdentity):
     columns = (
         'Role',
         'User',

openstackclient/tests/unit/identity/v3/test_service.py

@@ -22,7 +22,7 @@ from openstackclient.identity.v3 import service
 from openstackclient.tests.unit.identity.v3 import fakes as identity_fakes
 
 
-class TestServiceCreate(identity_fakes.TestIdentityv3):
+class TestServiceCreate(identity_fakes.TestIdentity):
     columns = (
         'id',
         'name',
@@ -163,7 +163,7 @@ class TestServiceCreate(identity_fakes.TestIdentityv3):
         self.assertEqual(self.datalist, data)
 
 
-class TestServiceDelete(identity_fakes.TestIdentityv3):
+class TestServiceDelete(identity_fakes.TestIdentity):
     service = sdk_fakes.generate_fake_resource(_service.Service)
 
     def setUp(self):
@@ -195,7 +195,7 @@ class TestServiceDelete(identity_fakes.TestIdentityv3):
         self.assertIsNone(result)
 
 
-class TestServiceList(identity_fakes.TestIdentityv3):
+class TestServiceList(identity_fakes.TestIdentity):
     service = sdk_fakes.generate_fake_resource(_service.Service)
 
     def setUp(self):
@@ -259,7 +259,7 @@ class TestServiceList(identity_fakes.TestIdentityv3):
         self.assertEqual(datalist, tuple(data))
 
 
-class TestServiceSet(identity_fakes.TestIdentityv3):
+class TestServiceSet(identity_fakes.TestIdentity):
     service = sdk_fakes.generate_fake_resource(_service.Service)
 
     def setUp(self):
@@ -420,7 +420,7 @@ class TestServiceSet(identity_fakes.TestIdentityv3):
         self.assertIsNone(result)
 
 
-class TestServiceShow(identity_fakes.TestIdentityv3):
+class TestServiceShow(identity_fakes.TestIdentity):
     service = sdk_fakes.generate_fake_resource(_service.Service)
 
     def setUp(self):

openstackclient/tests/unit/identity/v3/test_token.py

@@ -16,7 +16,7 @@ from openstackclient.identity.v3 import token
 from openstackclient.tests.unit.identity.v3 import fakes as identity_fakes
 
 
-class TestTokenIssue(identity_fakes.TestIdentityv3):
+class TestTokenIssue(identity_fakes.TestIdentity):
     def setUp(self):
         super().setUp()
 
@@ -99,7 +99,7 @@ class TestTokenIssue(identity_fakes.TestIdentityv3):
         self.assertEqual(datalist, data)
 
 
-class TestTokenRevoke(identity_fakes.TestIdentityv3):
+class TestTokenRevoke(identity_fakes.TestIdentity):
     TOKEN = 'fob'
 
     def setUp(self):

openstackclient/tests/unit/identity/v3/test_trust.py

@@ -26,7 +26,7 @@ from openstackclient.identity.v3 import trust
 from openstackclient.tests.unit.identity.v3 import fakes as identity_fakes
 
 
-class TestTrustCreate(identity_fakes.TestIdentityv3):
+class TestTrustCreate(identity_fakes.TestIdentity):
     def setUp(self):
         super().setUp()
 
@@ -109,7 +109,7 @@ class TestTrustCreate(identity_fakes.TestIdentityv3):
         self.assertEqual(datalist, data)
 
 
-class TestTrustDelete(identity_fakes.TestIdentityv3):
+class TestTrustDelete(identity_fakes.TestIdentity):
     def setUp(self):
         super().setUp()
 
@@ -166,7 +166,7 @@ class TestTrustDelete(identity_fakes.TestIdentityv3):
         )
 
 
-class TestTrustList(identity_fakes.TestIdentityv3):
+class TestTrustList(identity_fakes.TestIdentity):
     def setUp(self):
         super().setUp()
 
@@ -341,7 +341,7 @@ class TestTrustList(identity_fakes.TestIdentityv3):
         self.assertEqual(datalist, tuple(data))
 
 
-class TestTrustShow(identity_fakes.TestIdentityv3):
+class TestTrustShow(identity_fakes.TestIdentity):
     def setUp(self):
         super().setUp()
 

openstackclient/tests/unit/identity/v3/test_user.py

@@ -31,7 +31,7 @@ from openstackclient.identity.v3 import user
 from openstackclient.tests.unit.identity.v3 import fakes as identity_fakes
 
 
-class TestUserCreate(identity_fakes.TestIdentityv3):
+class TestUserCreate(identity_fakes.TestIdentity):
     domain = sdk_fakes.generate_fake_resource(_domain.Domain)
     project = sdk_fakes.generate_fake_resource(_project.Project)
 
@@ -777,7 +777,7 @@ class TestUserCreate(identity_fakes.TestIdentityv3):
         self.assertEqual(self.datalist, data)
 
 
-class TestUserDelete(identity_fakes.TestIdentityv3):
+class TestUserDelete(identity_fakes.TestIdentity):
     user = sdk_fakes.generate_fake_resource(_user.User)
 
     def setUp(self):
@@ -840,7 +840,7 @@ class TestUserDelete(identity_fakes.TestIdentityv3):
         )
 
 
-class TestUserList(identity_fakes.TestIdentityv3):
+class TestUserList(identity_fakes.TestIdentity):
     domain = sdk_fakes.generate_fake_resource(_domain.Domain)
     project = sdk_fakes.generate_fake_resource(_project.Project)
     user = sdk_fakes.generate_fake_resource(
@@ -1034,8 +1034,75 @@ class TestUserList(identity_fakes.TestIdentityv3):
         self.assertEqual(self.columns, columns)
         self.assertEqual(self.datalist, tuple(data))
 
+    def test_user_list_with_pagination(self):
+        arglist = [
+            '--limit',
+            '2',
+            '--marker',
+            'some-marker',
+        ]
+        verifylist = [
+            ('limit', 2),
+            ('marker', 'some-marker'),
+        ]
+        parsed_args = self.check_parser(self.cmd, arglist, verifylist)
+
+        columns, data = self.cmd.take_action(parsed_args)
+
+        kwargs = {
+            'domain_id': None,
+            'limit': 2,
+            'marker': 'some-marker',
+        }
+        self.identity_sdk_client.users.assert_called_with(**kwargs)
+
+        self.assertEqual(self.columns, columns)
+        self.assertEqual(self.datalist, tuple(data))
+
+    def test_user_list_with_pagination_and_group(self):
+        arglist = [
+            '--group',
+            self.group.name,
+            '--limit',
+            '5',
+        ]
+        verifylist = [
+            ('group', self.group.name),
+            ('limit', 5),
+        ]
+        parsed_args = self.check_parser(self.cmd, arglist, verifylist)
+
+        columns, data = self.cmd.take_action(parsed_args)
+
+        kwargs = {
+            'domain_id': None,
+            'group': self.group.id,
+            'limit': 5,
+        }
+        self.identity_sdk_client.group_users.assert_called_with(**kwargs)
 
-class TestUserSet(identity_fakes.TestIdentityv3):
+        self.assertEqual(self.columns, columns)
+        self.assertEqual(self.datalist, tuple(data))
+
+    def test_user_list_pagination_with_project_fails(self):
+        arglist = [
+            '--project',
+            self.project.name,
+            '--limit',
+            '2',
+        ]
+        verifylist = [
+            ('project', self.project.name),
+            ('limit', 2),
+        ]
+        parsed_args = self.check_parser(self.cmd, arglist, verifylist)
+
+        self.assertRaises(
+            exceptions.CommandError, self.cmd.take_action, parsed_args
+        )
+
+
+class TestUserSet(identity_fakes.TestIdentity):
     project = sdk_fakes.generate_fake_resource(_project.Project)
     domain = sdk_fakes.generate_fake_resource(_domain.Domain)
     user = sdk_fakes.generate_fake_resource(
@@ -1684,7 +1751,7 @@ class TestUserSet(identity_fakes.TestIdentityv3):
         self.assertIsNone(result)
 
 
-class TestUserSetPassword(identity_fakes.TestIdentityv3):
+class TestUserSetPassword(identity_fakes.TestIdentity):
     def setUp(self):
         super().setUp()
         self.cmd = user.SetPasswordUser(self.app, None)
@@ -1716,7 +1783,7 @@ class TestUserSetPassword(identity_fakes.TestIdentityv3):
         conn = self.app.client_manager.sdk_connection
         user_id = conn.config.get_auth().get_user_id(conn.identity)
 
-        self.identity_sdk_client.update_user.assert_called_with(
+        self.identity_sdk_client.update_password.assert_called_with(
             user=user_id, current_password=current_pass, password=new_pass
         )
 
@@ -1733,7 +1800,7 @@ class TestUserSetPassword(identity_fakes.TestIdentityv3):
         conn = self.app.client_manager.sdk_connection
         user_id = conn.config.get_auth().get_user_id(conn.identity)
 
-        self.identity_sdk_client.update_user.assert_called_with(
+        self.identity_sdk_client.update_password.assert_called_with(
             user=user_id, current_password=current_pass, password=new_pass
         )
 
@@ -1758,12 +1825,12 @@ class TestUserSetPassword(identity_fakes.TestIdentityv3):
         conn = self.app.client_manager.sdk_connection
         user_id = conn.config.get_auth().get_user_id(conn.identity)
 
-        self.identity_sdk_client.update_user.assert_called_with(
+        self.identity_sdk_client.update_password.assert_called_with(
             user=user_id, current_password=current_pass, password=new_pass
         )
 
 
-class TestUserShow(identity_fakes.TestIdentityv3):
+class TestUserShow(identity_fakes.TestIdentity):
     user = sdk_fakes.generate_fake_resource(_user.User)
 
     def setUp(self):
@@ -1773,7 +1840,7 @@ class TestUserShow(identity_fakes.TestIdentityv3):
 
         # Get the command object to test
         self.cmd = user.ShowUser(self.app, None)
-        self.identity_client.auth.client.get_user_id.return_value = (  # noqa: E501
+        self.identity_client.auth.client.get_user_id.return_value = (
             self.user.id
         )
         self.identity_client.tokens.get_token_data.return_value = {

openstackclient/tests/unit/image/v2/test_image.py

@@ -305,7 +305,7 @@ class TestImageCreate(TestImage):
         ]
         parsed_args = self.check_parser(self.cmd, arglist, verifylist)
 
-        columns, data = self.cmd.take_action(parsed_args)
+        self.cmd.take_action(parsed_args)
 
         self.image_client.create_image.assert_called_with(
             name=self.new_image.name,
@@ -336,7 +336,7 @@ class TestImageCreate(TestImage):
         ]
         parsed_args = self.check_parser(self.cmd, arglist, verifylist)
 
-        columns, data = self.cmd.take_action(parsed_args)
+        self.cmd.take_action(parsed_args)
 
         self.volume_sdk_client.upload_volume_to_image.assert_called_once_with(
             volume.id,
@@ -397,7 +397,7 @@ class TestImageCreate(TestImage):
         ]
         parsed_args = self.check_parser(self.cmd, arglist, verifylist)
 
-        columns, data = self.cmd.take_action(parsed_args)
+        self.cmd.take_action(parsed_args)
 
         self.volume_sdk_client.upload_volume_to_image.assert_called_once_with(
             volume.id,
@@ -946,7 +946,7 @@ class TestImageList(TestImage):
         ]
         parsed_args = self.check_parser(self.cmd, arglist, verifylist)
 
-        columns, data = self.cmd.take_action(parsed_args)
+        self.cmd.take_action(parsed_args)
         self.image_client.images.assert_called_with(
             marker=self._image.id,
         )
@@ -966,7 +966,7 @@ class TestImageList(TestImage):
         ]
         parsed_args = self.check_parser(self.cmd, arglist, verifylist)
 
-        columns, data = self.cmd.take_action(parsed_args)
+        self.cmd.take_action(parsed_args)
         self.image_client.images.assert_called_with(
             name='abc',
             # marker=self._image.id
@@ -982,7 +982,7 @@ class TestImageList(TestImage):
         ]
         parsed_args = self.check_parser(self.cmd, arglist, verifylist)
 
-        columns, data = self.cmd.take_action(parsed_args)
+        self.cmd.take_action(parsed_args)
         self.image_client.images.assert_called_with(status='active')
 
     def test_image_list_hidden_option(self):
@@ -994,7 +994,7 @@ class TestImageList(TestImage):
         ]
         parsed_args = self.check_parser(self.cmd, arglist, verifylist)
 
-        columns, data = self.cmd.take_action(parsed_args)
+        self.cmd.take_action(parsed_args)
         self.image_client.images.assert_called_with(is_hidden=True)
 
     def test_image_list_tag_option(self):
@@ -1004,7 +1004,7 @@ class TestImageList(TestImage):
         ]
         parsed_args = self.check_parser(self.cmd, arglist, verifylist)
 
-        columns, data = self.cmd.take_action(parsed_args)
+        self.cmd.take_action(parsed_args)
         self.image_client.images.assert_called_with(tag=['abc', 'cba'])
 
 
@@ -1329,7 +1329,7 @@ class TestImageSet(TestImage):
         self.image_client.update_image.assert_called()
         call_args = self.image_client.update_image.call_args
         if call_args:
-            args, kwargs = call_args
+            _args, kwargs = call_args
             self.assertNotIn('owner_id', kwargs)
 
     def test_image_set_membership_reject_with_project_no_owner_change(self):
@@ -1366,7 +1366,7 @@ class TestImageSet(TestImage):
         self.image_client.update_image.assert_called()
         call_args = self.image_client.update_image.call_args
         if call_args:
-            args, kwargs = call_args
+            _args, kwargs = call_args
             self.assertNotIn('owner_id', kwargs)
 
     def test_image_set_membership_pending_with_project_no_owner_change(self):
@@ -1403,7 +1403,7 @@ class TestImageSet(TestImage):
         self.image_client.update_image.assert_called()
         call_args = self.image_client.update_image.call_args
         if call_args:
-            args, kwargs = call_args
+            _args, kwargs = call_args
             self.assertNotIn('owner_id', kwargs)
 
     def test_image_set_options(self):

openstackclient/tests/unit/image/v2/test_metadef_namespaces.py

@@ -12,6 +12,8 @@
 #   License for the specific language governing permissions and limitations
 #   under the License.
 
+from unittest import mock
+
 from openstackclient.image.v2 import metadef_namespaces
 from openstackclient.tests.unit.image.v2 import fakes as image_fakes
 
@@ -24,6 +26,7 @@ class TestMetadefNamespaceCreate(image_fakes.TestImagev2):
         'display_name',
         'namespace',
         'owner',
+        'tags',
         'visibility',
     )
     expected_data = (
@@ -31,6 +34,7 @@ class TestMetadefNamespaceCreate(image_fakes.TestImagev2):
         _metadef_namespace.display_name,
         _metadef_namespace.namespace,
         _metadef_namespace.owner,
+        _metadef_namespace.tags,
         _metadef_namespace.visibility,
     )
 
@@ -114,21 +118,25 @@ class TestMetadefNamespaceList(image_fakes.TestImagev2):
 
 
 class TestMetadefNamespaceSet(image_fakes.TestImagev2):
-    _metadef_namespace = image_fakes.create_one_metadef_namespace()
-
     def setUp(self):
         super().setUp()
 
+        self.metadef_namespace = image_fakes.create_one_metadef_namespace()
+
+        self.image_client.get_metadef_namespace.return_value = (
+            self.metadef_namespace
+        )
         self.image_client.update_metadef_namespace.return_value = (
-            self._metadef_namespace
+            self.metadef_namespace
         )
+        self.image_client.add_tag_to_metadef_namespace.return_value = None
+
         self.cmd = metadef_namespaces.SetMetadefNamespace(self.app, None)
-        self.datalist = self._metadef_namespace
 
     def test_namespace_set_no_options(self):
-        arglist = [self._metadef_namespace.namespace]
+        arglist = [self.metadef_namespace.namespace]
         verifylist = [
-            ('namespace', self._metadef_namespace.namespace),
+            ('namespace', self.metadef_namespace.namespace),
         ]
 
         parsed_args = self.check_parser(self.cmd, arglist, verifylist)
@@ -137,6 +145,95 @@ class TestMetadefNamespaceSet(image_fakes.TestImagev2):
 
         self.assertIsNone(result)
 
+    def test_namespace_set_tag(self):
+        arglist = [
+            self.metadef_namespace.namespace,
+            '--tag',
+            't1',
+            '--tag',
+            't2',
+        ]
+        verifylist = [
+            ('namespace', self.metadef_namespace.namespace),
+            ('tags', ['t1', 't2']),
+        ]
+
+        parsed_args = self.check_parser(self.cmd, arglist, verifylist)
+
+        self.cmd.take_action(parsed_args)
+
+        self.image_client.add_tag_to_metadef_namespace.assert_has_calls(
+            [
+                mock.call(self.metadef_namespace.namespace, 't1'),
+                mock.call(self.metadef_namespace.namespace, 't2'),
+            ]
+        )
+
+
+class TestMetadefNamespaceUnset(image_fakes.TestImagev2):
+    def setUp(self):
+        super().setUp()
+
+        self.metadef_namespace = image_fakes.create_one_metadef_namespace(
+            attrs={'tags': [{'name': 't1'}]}
+        )
+
+        self.image_client.get_metadef_namespace.return_value = (
+            self.metadef_namespace
+        )
+        self.image_client.update_metadef_namespace.return_value = (
+            self.metadef_namespace
+        )
+        self.image_client.remove_tag_from_metadef_namespace.return_value = None
+        self.image_client.remove_tags_from_metadef_namespace.return_value = (
+            None
+        )
+
+        self.cmd = metadef_namespaces.UnsetMetadefNamespace(self.app, None)
+
+    def test_namespace_unset_tag(self):
+        arglist = [
+            self.metadef_namespace.namespace,
+            '--tag',
+            't1',
+            '--tag',
+            't2',
+        ]
+        verifylist = [
+            ('namespace', self.metadef_namespace.namespace),
+            ('tags', ['t1', 't2']),
+        ]
+
+        parsed_args = self.check_parser(self.cmd, arglist, verifylist)
+
+        self.cmd.take_action(parsed_args)
+
+        self.image_client.remove_tag_from_metadef_namespace.assert_has_calls(
+            [
+                mock.call(self.metadef_namespace, 't1'),
+                mock.call(self.metadef_namespace, 't2'),
+            ]
+        )
+        self.image_client.remove_tags_from_metadef_namespace.assert_not_called()
+
+    def test_namespace_unset_all_tag(self):
+        arglist = [
+            self.metadef_namespace.namespace,
+            '--all-tags',
+        ]
+        verifylist = [
+            ('namespace', self.metadef_namespace.namespace),
+        ]
+
+        parsed_args = self.check_parser(self.cmd, arglist, verifylist)
+
+        self.cmd.take_action(parsed_args)
+
+        self.image_client.remove_tags_from_metadef_namespace.assert_called_once_with(
+            self.metadef_namespace
+        )
+        self.image_client.remove_tag_from_metadef_namespace.assert_not_called()
+
 
 class TestMetadefNamespaceShow(image_fakes.TestImagev2):
     _metadef_namespace = image_fakes.create_one_metadef_namespace()
@@ -146,6 +243,7 @@ class TestMetadefNamespaceShow(image_fakes.TestImagev2):
         'display_name',
         'namespace',
         'owner',
+        'tags',
         'visibility',
     )
     expected_data = (
@@ -153,6 +251,7 @@ class TestMetadefNamespaceShow(image_fakes.TestImagev2):
         _metadef_namespace.display_name,
         _metadef_namespace.namespace,
         _metadef_namespace.owner,
+        _metadef_namespace.tags,
         _metadef_namespace.visibility,
     )
 

openstackclient/tests/unit/network/test_common.py

@@ -13,91 +13,11 @@
 
 from unittest import mock
 
-import openstack
-from osc_lib import exceptions
 
 from openstackclient.network import common
 from openstackclient.tests.unit import utils
 
 
-def _add_common_argument(parser):
-    parser.add_argument(
-        'common',
-        metavar='<common>',
-        help='Common argument',
-    )
-    return parser
-
-
-def _add_network_argument(parser):
-    parser.add_argument(
-        'network',
-        metavar='<network>',
-        help='Network argument',
-    )
-    return parser
-
-
-def _add_compute_argument(parser):
-    parser.add_argument(
-        'compute',
-        metavar='<compute>',
-        help='Compute argument',
-    )
-    return parser
-
-
-class FakeNetworkAndComputeCommand(common.NetworkAndComputeCommand):
-    def update_parser_common(self, parser):
-        return _add_common_argument(parser)
-
-    def update_parser_network(self, parser):
-        return _add_network_argument(parser)
-
-    def update_parser_compute(self, parser):
-        return _add_compute_argument(parser)
-
-    def take_action_network(self, client, parsed_args):
-        return client.network_action(parsed_args)
-
-    def take_action_compute(self, client, parsed_args):
-        return client.compute_action(parsed_args)
-
-
-class FakeNetworkAndComputeLister(common.NetworkAndComputeLister):
-    def update_parser_common(self, parser):
-        return _add_common_argument(parser)
-
-    def update_parser_network(self, parser):
-        return _add_network_argument(parser)
-
-    def update_parser_compute(self, parser):
-        return _add_compute_argument(parser)
-
-    def take_action_network(self, client, parsed_args):
-        return client.network_action(parsed_args)
-
-    def take_action_compute(self, client, parsed_args):
-        return client.compute_action(parsed_args)
-
-
-class FakeNetworkAndComputeShowOne(common.NetworkAndComputeShowOne):
-    def update_parser_common(self, parser):
-        return _add_common_argument(parser)
-
-    def update_parser_network(self, parser):
-        return _add_network_argument(parser)
-
-    def update_parser_compute(self, parser):
-        return _add_compute_argument(parser)
-
-    def take_action_network(self, client, parsed_args):
-        return client.network_action(parsed_args)
-
-    def take_action_compute(self, client, parsed_args):
-        return client.compute_action(parsed_args)
-
-
 class FakeCreateNeutronCommandWithExtraArgs(
     common.NeutronCommandWithExtraArgs
 ):
@@ -119,81 +39,6 @@ class FakeCreateNeutronCommandWithExtraArgs(
         client.test_create_action(**attrs)
 
 
-class TestNetworkAndCompute(utils.TestCommand):
-    def setUp(self):
-        super().setUp()
-
-        # Create client mocks. Note that we intentionally do not use specced
-        # mocks since we want to test fake methods.
-
-        self.app.client_manager.network = mock.Mock()  # noqa: O401
-        self.network_client = self.app.client_manager.network  # noqa: O401
-        self.network_client.network_action.return_value = 'take_action_network'
-
-        self.app.client_manager.compute = mock.Mock()  # noqa: O401
-        self.compute_client = self.app.client_manager.compute  # noqa: O401
-        self.compute_client.compute_action.return_value = 'take_action_compute'
-
-        self.cmd = FakeNetworkAndComputeCommand(self.app, None)
-
-    def test_take_action_network(self):
-        arglist = ['common', 'network']
-        verifylist = [('common', 'common'), ('network', 'network')]
-
-        parsed_args = self.check_parser(self.cmd, arglist, verifylist)
-        result = self.cmd.take_action(parsed_args)
-        self.network_client.network_action.assert_called_with(parsed_args)
-        self.assertEqual('take_action_network', result)
-
-    def test_take_action_compute(self):
-        arglist = ['common', 'compute']
-        verifylist = [('common', 'common'), ('compute', 'compute')]
-
-        self.app.client_manager.network_endpoint_enabled = False
-        parsed_args = self.check_parser(self.cmd, arglist, verifylist)
-        result = self.cmd.take_action(parsed_args)
-        self.compute_client.compute_action.assert_called_with(parsed_args)
-        self.assertEqual('take_action_compute', result)
-
-
-class TestNetworkAndComputeCommand(TestNetworkAndCompute):
-    def setUp(self):
-        super().setUp()
-        self.cmd = FakeNetworkAndComputeCommand(self.app, None)
-
-
-class TestNetworkAndComputeLister(TestNetworkAndCompute):
-    def setUp(self):
-        super().setUp()
-        self.cmd = FakeNetworkAndComputeLister(self.app, None)
-
-
-class TestNetworkAndComputeShowOne(TestNetworkAndCompute):
-    def setUp(self):
-        super().setUp()
-        self.cmd = FakeNetworkAndComputeShowOne(self.app, None)
-
-    def test_take_action_with_http_exception(self):
-        with mock.patch.object(self.cmd, 'take_action_network') as m_action:
-            m_action.side_effect = openstack.exceptions.HttpException("bar")
-            self.assertRaisesRegex(
-                exceptions.CommandError,
-                "bar",
-                self.cmd.take_action,
-                mock.Mock(),
-            )
-
-        self.app.client_manager.network_endpoint_enabled = False
-        with mock.patch.object(self.cmd, 'take_action_compute') as m_action:
-            m_action.side_effect = openstack.exceptions.HttpException("bar")
-            self.assertRaisesRegex(
-                exceptions.CommandError,
-                "bar",
-                self.cmd.take_action,
-                mock.Mock(),
-            )
-
-
 class TestNeutronCommandWithExtraArgs(utils.TestCommand):
     def setUp(self):
         super().setUp()