summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorEric Harney <eharney@redhat.com>2022-11-07 14:44:51 -0500
committerEric Harney <eharney@redhat.com>2022-11-17 13:32:11 -0500
commitdec8d4a9ee9208b33a08141d04879369d7210da0 (patch)
treef11e7fffcc998bea623c2007de37f4ec511c628a
parent9df653571d4da06c25222189be27e87a6da75628 (diff)
downloadpython-cinderclient-dec8d4a9ee9208b33a08141d04879369d7210da0.tar.gz
Add test coverage for shell snapshot-create w/ metadata
This path is not currently unit tested. Change-Id: If38c6352f5e1f0a50e4a0d29fbdd5263ccba3b29
-rw-r--r--cinderclient/tests/unit/v3/test_shell.py18
1 files changed, 18 insertions, 0 deletions
diff --git a/cinderclient/tests/unit/v3/test_shell.py b/cinderclient/tests/unit/v3/test_shell.py
index 7c5f110..841cab9 100644
--- a/cinderclient/tests/unit/v3/test_shell.py
+++ b/cinderclient/tests/unit/v3/test_shell.py
@@ -970,6 +970,15 @@ class ShellTest(utils.TestCase):
}
}
+ SNAP_BODY_3_66_W_METADATA = {
+ 'snapshot': {
+ 'volume_id': '123456',
+ 'name': None,
+ 'description': None,
+ 'metadata': {'a': 'b'}
+ }
+ }
+
@ddt.data(True, 'true', 'on', '1')
@mock.patch('cinderclient.utils.find_resource')
def test_snapshot_create_3_66_with_force_true(self, f_val, mock_find_vol):
@@ -1037,6 +1046,15 @@ class ShellTest(utils.TestCase):
self.assert_called_anytime('POST', '/snapshots',
body=pre_3_66_request_body)
+ @mock.patch('cinderclient.utils.find_resource')
+ def test_snapshot_create_w_metadata(self, mock_find_vol):
+ mock_find_vol.return_value = volumes.Volume(
+ self, {'id': '123456'}, loaded=True)
+ self.run_command('--os-volume-api-version 3.66 '
+ 'snapshot-create 123456 --metadata a=b')
+ self.assert_called_anytime('POST', '/snapshots',
+ body=self.SNAP_BODY_3_66_W_METADATA)
+
def test_snapshot_manageable_list(self):
self.run_command('--os-volume-api-version 3.8 '
'snapshot-manageable-list fakehost')