Merge "Add test coverage for shell snapshot-create w/ metadata"

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 58caddc..ee71620 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')