python-openstackclient 6.6.1__py3-none-any.whl → 7.0.0__py3-none-any.whl

openstackclient/tests/unit/volume/v3/test_volume_group.py

@@ -12,7 +12,6 @@
 
 from unittest import mock
 
-from cinderclient import api_versions
 from osc_lib import exceptions
 
 from openstackclient.tests.unit import utils as tests_utils
@@ -96,7 +95,7 @@ class TestVolumeGroupCreate(TestVolumeGroup):
         self.cmd = volume_group.CreateVolumeGroup(self.app, None)
 
     def test_volume_group_create(self):
-        self.volume_client.api_version = api_versions.APIVersion('3.13')
+        self.set_volume_api_version('3.13')
 
         arglist = [
             '--volume-group-type',
@@ -132,7 +131,7 @@ class TestVolumeGroupCreate(TestVolumeGroup):
         self.assertCountEqual(self.data, data)
 
     def test_volume_group_create__legacy(self):
-        self.volume_client.api_version = api_versions.APIVersion('3.13')
+        self.set_volume_api_version('3.13')
 
         arglist = [
             self.fake_volume_group_type.id,
@@ -172,7 +171,7 @@ class TestVolumeGroupCreate(TestVolumeGroup):
         )
 
     def test_volume_group_create_no_volume_type(self):
-        self.volume_client.api_version = api_versions.APIVersion('3.13')
+        self.set_volume_api_version('3.13')
 
         arglist = [
             '--volume-group-type',
@@ -194,7 +193,7 @@ class TestVolumeGroupCreate(TestVolumeGroup):
         )
 
     def test_volume_group_create_with_options(self):
-        self.volume_client.api_version = api_versions.APIVersion('3.13')
+        self.set_volume_api_version('3.13')
 
         arglist = [
             '--volume-group-type',
@@ -236,7 +235,7 @@ class TestVolumeGroupCreate(TestVolumeGroup):
         self.assertCountEqual(self.data, data)
 
     def test_volume_group_create_pre_v313(self):
-        self.volume_client.api_version = api_versions.APIVersion('3.12')
+        self.set_volume_api_version('3.12')
 
         arglist = [
             '--volume-group-type',
@@ -261,7 +260,7 @@ class TestVolumeGroupCreate(TestVolumeGroup):
         )
 
     def test_volume_group_create_from_source_group(self):
-        self.volume_client.api_version = api_versions.APIVersion('3.14')
+        self.set_volume_api_version('3.14')
 
         arglist = [
             '--source-group',
@@ -290,7 +289,7 @@ class TestVolumeGroupCreate(TestVolumeGroup):
         self.assertCountEqual(self.data, data)
 
     def test_volume_group_create_from_group_snapshot(self):
-        self.volume_client.api_version = api_versions.APIVersion('3.14')
+        self.set_volume_api_version('3.14')
 
         arglist = [
             '--group-snapshot',
@@ -319,7 +318,7 @@ class TestVolumeGroupCreate(TestVolumeGroup):
         self.assertCountEqual(self.data, data)
 
     def test_volume_group_create_from_src_pre_v314(self):
-        self.volume_client.api_version = api_versions.APIVersion('3.13')
+        self.set_volume_api_version('3.13')
 
         arglist = [
             '--source-group',
@@ -338,7 +337,7 @@ class TestVolumeGroupCreate(TestVolumeGroup):
         )
 
     def test_volume_group_create_from_src_source_group_group_snapshot(self):
-        self.volume_client.api_version = api_versions.APIVersion('3.14')
+        self.set_volume_api_version('3.14')
 
         arglist = [
             '--source-group',
@@ -376,7 +375,7 @@ class TestVolumeGroupDelete(TestVolumeGroup):
         self.cmd = volume_group.DeleteVolumeGroup(self.app, None)
 
     def test_volume_group_delete(self):
-        self.volume_client.api_version = api_versions.APIVersion('3.13')
+        self.set_volume_api_version('3.13')
 
         arglist = [
             self.fake_volume_group.id,
@@ -397,7 +396,7 @@ class TestVolumeGroupDelete(TestVolumeGroup):
         self.assertIsNone(result)
 
     def test_volume_group_delete_pre_v313(self):
-        self.volume_client.api_version = api_versions.APIVersion('3.12')
+        self.set_volume_api_version('3.12')
 
         arglist = [
             self.fake_volume_group.id,
@@ -455,7 +454,7 @@ class TestVolumeGroupSet(TestVolumeGroup):
         self.cmd = volume_group.SetVolumeGroup(self.app, None)
 
     def test_volume_group_set(self):
-        self.volume_client.api_version = api_versions.APIVersion('3.13')
+        self.set_volume_api_version('3.13')
 
         arglist = [
             self.fake_volume_group.id,
@@ -482,7 +481,7 @@ class TestVolumeGroupSet(TestVolumeGroup):
         self.assertCountEqual(self.data, data)
 
     def test_volume_group_with_enable_replication_option(self):
-        self.volume_client.api_version = api_versions.APIVersion('3.38')
+        self.set_volume_api_version('3.38')
 
         arglist = [
             self.fake_volume_group.id,
@@ -503,7 +502,7 @@ class TestVolumeGroupSet(TestVolumeGroup):
         self.assertCountEqual(self.data, data)
 
     def test_volume_group_set_pre_v313(self):
-        self.volume_client.api_version = api_versions.APIVersion('3.12')
+        self.set_volume_api_version('3.12')
 
         arglist = [
             self.fake_volume_group.id,
@@ -527,7 +526,7 @@ class TestVolumeGroupSet(TestVolumeGroup):
         )
 
     def test_volume_group_with_enable_replication_option_pre_v338(self):
-        self.volume_client.api_version = api_versions.APIVersion('3.37')
+        self.set_volume_api_version('3.37')
 
         arglist = [
             self.fake_volume_group.id,
@@ -572,7 +571,7 @@ class TestVolumeGroupList(TestVolumeGroup):
         self.cmd = volume_group.ListVolumeGroup(self.app, None)
 
     def test_volume_group_list(self):
-        self.volume_client.api_version = api_versions.APIVersion('3.13')
+        self.set_volume_api_version('3.13')
 
         arglist = [
             '--all-projects',
@@ -593,7 +592,7 @@ class TestVolumeGroupList(TestVolumeGroup):
         self.assertCountEqual(tuple(self.data), data)
 
     def test_volume_group_list_pre_v313(self):
-        self.volume_client.api_version = api_versions.APIVersion('3.12')
+        self.set_volume_api_version('3.12')
 
         arglist = [
             '--all-projects',
@@ -623,7 +622,7 @@ class TestVolumeGroupFailover(TestVolumeGroup):
         self.cmd = volume_group.FailoverVolumeGroup(self.app, None)
 
     def test_volume_group_failover(self):
-        self.volume_client.api_version = api_versions.APIVersion('3.38')
+        self.set_volume_api_version('3.38')
 
         arglist = [
             self.fake_volume_group.id,
@@ -648,7 +647,7 @@ class TestVolumeGroupFailover(TestVolumeGroup):
         self.assertIsNone(result)
 
     def test_volume_group_failover_pre_v338(self):
-        self.volume_client.api_version = api_versions.APIVersion('3.37')
+        self.set_volume_api_version('3.37')
 
         arglist = [
             self.fake_volume_group.id,

openstackclient/tests/unit/volume/v3/test_volume_group_snapshot.py

@@ -10,28 +10,15 @@
 # License for the specific language governing permissions and limitations
 # under the License.
 
-from unittest import mock
-
-from keystoneauth1 import discover
 from openstack.block_storage.v3 import group as _group
 from openstack.block_storage.v3 import group_snapshot as _group_snapshot
 from openstack.test import fakes as sdk_fakes
-from openstack import utils as sdk_utils
 from osc_lib import exceptions
 
 from openstackclient.tests.unit.volume.v3 import fakes as volume_fakes
 from openstackclient.volume.v3 import volume_group_snapshot
 
 
-def fake_supports_microversion(mocked_version):
-    def supports_microversion(adapter, microversion, raise_exception=False):
-        required = discover.normalize_version_number(microversion)
-        candidate = discover.normalize_version_number(mocked_version)
-        return discover.version_match(required, candidate)
-
-    return supports_microversion
-
-
 class TestVolumeGroupSnapshotCreate(volume_fakes.TestVolume):
     fake_volume_group = sdk_fakes.generate_fake_resource(_group.Group)
     fake_volume_group_snapshot = sdk_fakes.generate_fake_resource(
@@ -70,9 +57,8 @@ class TestVolumeGroupSnapshotCreate(volume_fakes.TestVolume):
             self.app, None
         )
 
-    @mock.patch.object(sdk_utils, 'supports_microversion')
-    def test_volume_group_snapshot_create(self, mock_mv):
-        mock_mv.side_effect = fake_supports_microversion('3.14')
+    def test_volume_group_snapshot_create(self):
+        self.set_volume_api_version('3.14')
 
         arglist = [
             self.fake_volume_group.id,
@@ -99,9 +85,8 @@ class TestVolumeGroupSnapshotCreate(volume_fakes.TestVolume):
         self.assertEqual(self.columns, columns)
         self.assertCountEqual(self.data, data)
 
-    @mock.patch.object(sdk_utils, 'supports_microversion')
-    def test_volume_group_snapshot_create_with_options(self, mock_mv):
-        mock_mv.side_effect = fake_supports_microversion('3.14')
+    def test_volume_group_snapshot_create_with_options(self):
+        self.set_volume_api_version('3.14')
 
         arglist = [
             self.fake_volume_group.id,
@@ -132,9 +117,8 @@ class TestVolumeGroupSnapshotCreate(volume_fakes.TestVolume):
         self.assertEqual(self.columns, columns)
         self.assertCountEqual(self.data, data)
 
-    @mock.patch.object(sdk_utils, 'supports_microversion')
-    def test_volume_group_snapshot_create_pre_v314(self, mock_mv):
-        mock_mv.side_effect = fake_supports_microversion('3.13')
+    def test_volume_group_snapshot_create_pre_v314(self):
+        self.set_volume_api_version('3.13')
 
         arglist = [
             self.fake_volume_group.id,
@@ -174,9 +158,8 @@ class TestVolumeGroupSnapshotDelete(volume_fakes.TestVolume):
             self.app, None
         )
 
-    @mock.patch.object(sdk_utils, 'supports_microversion')
-    def test_volume_group_snapshot_delete(self, mock_mv):
-        mock_mv.side_effect = fake_supports_microversion('3.14')
+    def test_volume_group_snapshot_delete(self):
+        self.set_volume_api_version('3.14')
 
         arglist = [
             self.fake_volume_group_snapshot.id,
@@ -193,9 +176,8 @@ class TestVolumeGroupSnapshotDelete(volume_fakes.TestVolume):
         )
         self.assertIsNone(result)
 
-    @mock.patch.object(sdk_utils, 'supports_microversion')
-    def test_volume_group_snapshot_delete_pre_v314(self, mock_mv):
-        mock_mv.side_effect = fake_supports_microversion('3.13')
+    def test_volume_group_snapshot_delete_pre_v314(self):
+        self.set_volume_api_version('3.13')
 
         arglist = [
             self.fake_volume_group_snapshot.id,
@@ -249,9 +231,8 @@ class TestVolumeGroupSnapshotList(volume_fakes.TestVolume):
             self.app, None
         )
 
-    @mock.patch.object(sdk_utils, 'supports_microversion')
-    def test_volume_group_snapshot_list(self, mock_mv):
-        mock_mv.side_effect = fake_supports_microversion('3.14')
+    def test_volume_group_snapshot_list(self):
+        self.set_volume_api_version('3.14')
 
         arglist = [
             '--all-projects',
@@ -269,9 +250,8 @@ class TestVolumeGroupSnapshotList(volume_fakes.TestVolume):
         self.assertEqual(self.columns, columns)
         self.assertCountEqual(tuple(self.data), data)
 
-    @mock.patch.object(sdk_utils, 'supports_microversion')
-    def test_volume_group_snapshot_list_pre_v314(self, mock_mv):
-        mock_mv.side_effect = fake_supports_microversion('3.13')
+    def test_volume_group_snapshot_list_pre_v314(self):
+        self.set_volume_api_version('3.13')
 
         arglist = []
         verifylist = [

openstackclient/tests/unit/volume/v3/test_volume_group_type.py

@@ -12,7 +12,6 @@
 
 from unittest import mock
 
-from cinderclient import api_versions
 from osc_lib.cli import format_columns
 from osc_lib import exceptions
 
@@ -29,8 +28,6 @@ class TestVolumeGroupType(volume_fakes.TestVolume):
 
 
 class TestVolumeGroupTypeCreate(TestVolumeGroupType):
-    maxDiff = 2000
-
     fake_volume_group_type = volume_fakes.create_one_volume_group_type()
 
     columns = (
@@ -58,7 +55,7 @@ class TestVolumeGroupTypeCreate(TestVolumeGroupType):
         self.cmd = volume_group_type.CreateVolumeGroupType(self.app, None)
 
     def test_volume_group_type_create(self):
-        self.volume_client.api_version = api_versions.APIVersion('3.11')
+        self.set_volume_api_version('3.11')
 
         arglist = [
             self.fake_volume_group_type.name,
@@ -79,7 +76,7 @@ class TestVolumeGroupTypeCreate(TestVolumeGroupType):
         self.assertCountEqual(self.data, data)
 
     def test_volume_group_type_create_with_options(self):
-        self.volume_client.api_version = api_versions.APIVersion('3.11')
+        self.set_volume_api_version('3.11')
 
         arglist = [
             self.fake_volume_group_type.name,
@@ -103,7 +100,7 @@ class TestVolumeGroupTypeCreate(TestVolumeGroupType):
         self.assertCountEqual(self.data, data)
 
     def test_volume_group_type_create_pre_v311(self):
-        self.volume_client.api_version = api_versions.APIVersion('3.10')
+        self.set_volume_api_version('3.10')
 
         arglist = [
             self.fake_volume_group_type.name,
@@ -137,7 +134,7 @@ class TestVolumeGroupTypeDelete(TestVolumeGroupType):
         self.cmd = volume_group_type.DeleteVolumeGroupType(self.app, None)
 
     def test_volume_group_type_delete(self):
-        self.volume_client.api_version = api_versions.APIVersion('3.11')
+        self.set_volume_api_version('3.11')
 
         arglist = [
             self.fake_volume_group_type.id,
@@ -155,7 +152,7 @@ class TestVolumeGroupTypeDelete(TestVolumeGroupType):
         self.assertIsNone(result)
 
     def test_volume_group_type_delete_pre_v311(self):
-        self.volume_client.api_version = api_versions.APIVersion('3.10')
+        self.set_volume_api_version('3.10')
 
         arglist = [
             self.fake_volume_group_type.id,
@@ -210,7 +207,7 @@ class TestVolumeGroupTypeSet(TestVolumeGroupType):
         self.cmd = volume_group_type.SetVolumeGroupType(self.app, None)
 
     def test_volume_group_type_set(self):
-        self.volume_client.api_version = api_versions.APIVersion('3.11')
+        self.set_volume_api_version('3.11')
 
         self.fake_volume_group_type.set_keys.return_value = None
 
@@ -249,7 +246,7 @@ class TestVolumeGroupTypeSet(TestVolumeGroupType):
         self.assertCountEqual(self.data, data)
 
     def test_volume_group_type_with_no_property_option(self):
-        self.volume_client.api_version = api_versions.APIVersion('3.11')
+        self.set_volume_api_version('3.11')
 
         arglist = [
             self.fake_volume_group_type.id,
@@ -280,7 +277,7 @@ class TestVolumeGroupTypeSet(TestVolumeGroupType):
         self.assertCountEqual(self.data, data)
 
     def test_volume_group_type_set_pre_v311(self):
-        self.volume_client.api_version = api_versions.APIVersion('3.10')
+        self.set_volume_api_version('3.10')
 
         arglist = [
             self.fake_volume_group_type.id,
@@ -337,7 +334,7 @@ class TestVolumeGroupTypeUnset(TestVolumeGroupType):
         self.cmd = volume_group_type.UnsetVolumeGroupType(self.app, None)
 
     def test_volume_group_type_unset(self):
-        self.volume_client.api_version = api_versions.APIVersion('3.11')
+        self.set_volume_api_version('3.11')
 
         arglist = [
             self.fake_volume_group_type.id,
@@ -365,7 +362,7 @@ class TestVolumeGroupTypeUnset(TestVolumeGroupType):
         self.assertCountEqual(self.data, data)
 
     def test_volume_group_type_unset_pre_v311(self):
-        self.volume_client.api_version = api_versions.APIVersion('3.10')
+        self.set_volume_api_version('3.10')
 
         arglist = [
             self.fake_volume_group_type.id,
@@ -418,7 +415,7 @@ class TestVolumeGroupTypeList(TestVolumeGroupType):
         self.cmd = volume_group_type.ListVolumeGroupType(self.app, None)
 
     def test_volume_group_type_list(self):
-        self.volume_client.api_version = api_versions.APIVersion('3.11')
+        self.set_volume_api_version('3.11')
 
         arglist = []
         verifylist = [
@@ -433,7 +430,7 @@ class TestVolumeGroupTypeList(TestVolumeGroupType):
         self.assertCountEqual(tuple(self.data), data)
 
     def test_volume_group_type_list_with_default_option(self):
-        self.volume_client.api_version = api_versions.APIVersion('3.11')
+        self.set_volume_api_version('3.11')
 
         arglist = [
             '--default',
@@ -450,7 +447,7 @@ class TestVolumeGroupTypeList(TestVolumeGroupType):
         self.assertCountEqual(tuple([self.data[0]]), data)
 
     def test_volume_group_type_list_pre_v311(self):
-        self.volume_client.api_version = api_versions.APIVersion('3.10')
+        self.set_volume_api_version('3.10')
 
         arglist = []
         verifylist = []

openstackclient/tests/unit/volume/v3/test_volume_message.py

@@ -12,7 +12,6 @@
 
 from unittest.mock import call
 
-from cinderclient import api_versions
 from osc_lib import exceptions
 
 from openstackclient.tests.unit.identity.v3 import fakes as identity_fakes
@@ -24,7 +23,7 @@ class TestVolumeMessage(volume_fakes.TestVolume):
     def setUp(self):
         super().setUp()
 
-        self.projects_mock = self.app.client_manager.identity.projects
+        self.projects_mock = self.identity_client.projects
         self.projects_mock.reset_mock()
 
         self.volume_messages_mock = self.volume_client.messages
@@ -46,7 +45,7 @@ class TestVolumeMessageDelete(TestVolumeMessage):
         self.cmd = volume_message.DeleteMessage(self.app, None)
 
     def test_message_delete(self):
-        self.volume_client.api_version = api_versions.APIVersion('3.3')
+        self.set_volume_api_version('3.3')
 
         arglist = [
             self.fake_messages[0].id,
@@ -64,7 +63,7 @@ class TestVolumeMessageDelete(TestVolumeMessage):
         self.assertIsNone(result)
 
     def test_message_delete_multiple_messages(self):
-        self.volume_client.api_version = api_versions.APIVersion('3.3')
+        self.set_volume_api_version('3.3')
 
         arglist = [
             self.fake_messages[0].id,
@@ -84,7 +83,7 @@ class TestVolumeMessageDelete(TestVolumeMessage):
         self.assertIsNone(result)
 
     def test_message_delete_multiple_messages_with_exception(self):
-        self.volume_client.api_version = api_versions.APIVersion('3.3')
+        self.set_volume_api_version('3.3')
 
         arglist = [
             self.fake_messages[0].id,
@@ -114,7 +113,7 @@ class TestVolumeMessageDelete(TestVolumeMessage):
         self.assertEqual(2, self.volume_messages_mock.delete.call_count)
 
     def test_message_delete_pre_v33(self):
-        self.volume_client.api_version = api_versions.APIVersion('3.2')
+        self.set_volume_api_version('3.2')
 
         arglist = [
             self.fake_messages[0].id,
@@ -172,7 +171,7 @@ class TestVolumeMessageList(TestVolumeMessage):
         self.cmd = volume_message.ListMessages(self.app, None)
 
     def test_message_list(self):
-        self.volume_client.api_version = api_versions.APIVersion('3.3')
+        self.set_volume_api_version('3.3')
 
         arglist = []
         verifylist = [
@@ -196,7 +195,7 @@ class TestVolumeMessageList(TestVolumeMessage):
         self.assertCountEqual(self.data, list(data))
 
     def test_message_list_with_options(self):
-        self.volume_client.api_version = api_versions.APIVersion('3.3')
+        self.set_volume_api_version('3.3')
 
         arglist = [
             '--project',
@@ -227,7 +226,7 @@ class TestVolumeMessageList(TestVolumeMessage):
         self.assertCountEqual(self.data, list(data))
 
     def test_message_list_pre_v33(self):
-        self.volume_client.api_version = api_versions.APIVersion('3.2')
+        self.set_volume_api_version('3.2')
 
         arglist = []
         verifylist = [
@@ -280,7 +279,7 @@ class TestVolumeMessageShow(TestVolumeMessage):
         self.cmd = volume_message.ShowMessage(self.app, None)
 
     def test_message_show(self):
-        self.volume_client.api_version = api_versions.APIVersion('3.3')
+        self.set_volume_api_version('3.3')
 
         arglist = [self.fake_message.id]
         verifylist = [('message_id', self.fake_message.id)]
@@ -293,7 +292,7 @@ class TestVolumeMessageShow(TestVolumeMessage):
         self.assertEqual(self.data, data)
 
     def test_message_show_pre_v33(self):
-        self.volume_client.api_version = api_versions.APIVersion('3.2')
+        self.set_volume_api_version('3.2')
 
         arglist = [self.fake_message.id]
         verifylist = [('message_id', self.fake_message.id)]

openstackclient/tests/unit/volume/v3/test_volume_snapshot.py

@@ -0,0 +1,161 @@
+#
+#   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.
+
+from unittest import mock
+
+from osc_lib import exceptions
+from osc_lib import utils
+
+from openstackclient.tests.unit.volume.v2 import fakes as volume_fakes
+from openstackclient.tests.unit.volume.v3 import fakes as volume_fakes_v3
+from openstackclient.volume.v3 import volume_snapshot
+
+
+class TestVolumeSnapshot(volume_fakes_v3.TestVolume):
+    def setUp(self):
+        super().setUp()
+
+        self.snapshots_mock = self.volume_client.volume_snapshots
+        self.snapshots_mock.reset_mock()
+
+        self.volume_sdk_client.unmanage_snapshot.return_value = None
+
+
+class TestVolumeSnapshotDelete(TestVolumeSnapshot):
+    snapshots = volume_fakes.create_snapshots(count=2)
+
+    def setUp(self):
+        super().setUp()
+
+        self.snapshots_mock.get = volume_fakes.get_snapshots(self.snapshots)
+        self.snapshots_mock.delete.return_value = None
+
+        # Get the command object to mock
+        self.cmd = volume_snapshot.DeleteVolumeSnapshot(self.app, None)
+
+    def test_snapshot_delete(self):
+        arglist = [self.snapshots[0].id]
+        verifylist = [("snapshots", [self.snapshots[0].id])]
+        parsed_args = self.check_parser(self.cmd, arglist, verifylist)
+
+        result = self.cmd.take_action(parsed_args)
+
+        self.snapshots_mock.delete.assert_called_with(
+            self.snapshots[0].id, False
+        )
+        self.assertIsNone(result)
+
+    def test_snapshot_delete_with_force(self):
+        arglist = ['--force', self.snapshots[0].id]
+        verifylist = [('force', True), ("snapshots", [self.snapshots[0].id])]
+        parsed_args = self.check_parser(self.cmd, arglist, verifylist)
+
+        result = self.cmd.take_action(parsed_args)
+
+        self.snapshots_mock.delete.assert_called_with(
+            self.snapshots[0].id, True
+        )
+        self.assertIsNone(result)
+
+    def test_delete_multiple_snapshots(self):
+        arglist = []
+        for s in self.snapshots:
+            arglist.append(s.id)
+        verifylist = [
+            ('snapshots', arglist),
+        ]
+
+        parsed_args = self.check_parser(self.cmd, arglist, verifylist)
+        result = self.cmd.take_action(parsed_args)
+
+        calls = []
+        for s in self.snapshots:
+            calls.append(mock.call(s.id, False))
+        self.snapshots_mock.delete.assert_has_calls(calls)
+        self.assertIsNone(result)
+
+    def test_delete_multiple_snapshots_with_exception(self):
+        arglist = [
+            self.snapshots[0].id,
+            'unexist_snapshot',
+        ]
+        verifylist = [
+            ('snapshots', arglist),
+        ]
+
+        parsed_args = self.check_parser(self.cmd, arglist, verifylist)
+
+        find_mock_result = [self.snapshots[0], exceptions.CommandError]
+        with mock.patch.object(
+            utils, 'find_resource', side_effect=find_mock_result
+        ) as find_mock:
+            try:
+                self.cmd.take_action(parsed_args)
+                self.fail('CommandError should be raised.')
+            except exceptions.CommandError as e:
+                self.assertEqual('1 of 2 snapshots failed to delete.', str(e))
+
+            find_mock.assert_any_call(
+                self.snapshots_mock, self.snapshots[0].id
+            )
+            find_mock.assert_any_call(self.snapshots_mock, 'unexist_snapshot')
+
+            self.assertEqual(2, find_mock.call_count)
+            self.snapshots_mock.delete.assert_called_once_with(
+                self.snapshots[0].id, False
+            )
+
+    def test_snapshot_delete_remote(self):
+        arglist = ['--remote', self.snapshots[0].id]
+        verifylist = [('remote', True), ("snapshots", [self.snapshots[0].id])]
+        parsed_args = self.check_parser(self.cmd, arglist, verifylist)
+
+        result = self.cmd.take_action(parsed_args)
+
+        self.volume_sdk_client.unmanage_snapshot.assert_called_with(
+            self.snapshots[0].id
+        )
+        self.assertIsNone(result)
+
+    def test_snapshot_delete_with_remote_force(self):
+        arglist = ['--remote', '--force', self.snapshots[0].id]
+        verifylist = [
+            ('remote', True),
+            ('force', True),
+            ("snapshots", [self.snapshots[0].id]),
+        ]
+
+        parsed_args = self.check_parser(self.cmd, arglist, verifylist)
+        exc = self.assertRaises(
+            exceptions.CommandError, self.cmd.take_action, parsed_args
+        )
+        self.assertIn(
+            "The --force option is not supported with the --remote "
+            "parameter.",
+            str(exc),
+        )
+
+    def test_delete_multiple_snapshots_remote(self):
+        arglist = ['--remote']
+        for s in self.snapshots:
+            arglist.append(s.id)
+        verifylist = [('remote', True), ('snapshots', arglist[1:])]
+
+        parsed_args = self.check_parser(self.cmd, arglist, verifylist)
+        result = self.cmd.take_action(parsed_args)
+
+        calls = []
+        for s in self.snapshots:
+            calls.append(mock.call(s.id))
+        self.volume_sdk_client.unmanage_snapshot.assert_has_calls(calls)
+        self.assertIsNone(result)