Merge "Added test for Limit formatting routes when adding resources"
diff --git a/manila_tempest_tests/tests/api/test_metadata.py b/manila_tempest_tests/tests/api/test_metadata.py
index a2a92cf..ff9953b 100644
--- a/manila_tempest_tests/tests/api/test_metadata.py
+++ b/manila_tempest_tests/tests/api/test_metadata.py
@@ -42,7 +42,7 @@
@tc.attr(base.TAG_POSITIVE, base.TAG_API_WITH_BACKEND)
def test_set_get_delete_metadata(self):
- md = {u"key3": u"value3", u"key4": u"value4", }
+ md = {u"key3": u"value3", u"key4": u"value4", u"key.5.1": u"value.5"}
# create share
share = self.create_share(cleanup_in_class=False)
diff --git a/manila_tempest_tests/tests/api/test_metadata_negative.py b/manila_tempest_tests/tests/api/test_metadata_negative.py
index a4ad260..4e37424 100644
--- a/manila_tempest_tests/tests/api/test_metadata_negative.py
+++ b/manila_tempest_tests/tests/api/test_metadata_negative.py
@@ -42,6 +42,7 @@
"wrong_share_id", md)
+@ddt.ddt
class SharesMetadataNegativeTest(base.BaseSharesTest):
@classmethod
@@ -98,3 +99,19 @@
self.assertRaises(lib_exc.NotFound,
self.shares_client.delete_metadata,
self.share["id"], "wrong_key")
+
+ @tc.attr(base.TAG_NEGATIVE, base.TAG_API_WITH_BACKEND)
+ @ddt.data(("foo.xml", False), ("foo.json", False),
+ ("foo.xml", True), ("foo.json", True))
+ @ddt.unpack
+ def test_try_delete_metadata_with_unsupport_format_key(
+ self, key, is_v2_client):
+ md = {key: u"value.test"}
+
+ client = self.shares_v2_client if is_v2_client else self.shares_client
+ # set metadata
+ client.set_metadata(self.share["id"], md)
+
+ self.assertRaises(lib_exc.NotFound,
+ client.delete_metadata,
+ self.share["id"], key)