Merge "Remove unicode prefix from code"
diff --git a/manila_tempest_tests/tests/api/test_access_rules_metadata_negative.py b/manila_tempest_tests/tests/api/test_access_rules_metadata_negative.py
index c212095..a351422 100644
--- a/manila_tempest_tests/tests/api/test_access_rules_metadata_negative.py
+++ b/manila_tempest_tests/tests/api/test_access_rules_metadata_negative.py
@@ -67,7 +67,7 @@
cls.share = cls.create_share(share_type_id=cls.share_type_id)
cls.access = cls.shares_v2_client.create_access_rule(
cls.share["id"], cls.access_type, cls.access_to,
- 'rw', metadata={u"key1": u"value1"})['access']
+ 'rw', metadata={"key1": "value1"})['access']
waiters.wait_for_resource_status(
cls.shares_v2_client, cls.share["id"], "active",
resource_name='access_rule', rule_id=cls.access["id"])
diff --git a/manila_tempest_tests/tests/api/test_metadata.py b/manila_tempest_tests/tests/api/test_metadata.py
index be5b1cf..3574d64 100644
--- a/manila_tempest_tests/tests/api/test_metadata.py
+++ b/manila_tempest_tests/tests/api/test_metadata.py
@@ -46,7 +46,7 @@
@tc.attr(base.TAG_POSITIVE, base.TAG_API_WITH_BACKEND)
def test_set_metadata_in_share_creation(self):
- md = {u"key1": u"value1", u"key2": u"value2", }
+ md = {"key1": "value1", "key2": "value2", }
# create share with metadata
share = self.create_share(share_type_id=self.share_type_id,
@@ -60,7 +60,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", u"key.5.1": u"value.5"}
+ md = {"key3": "value3", "key4": "value4", "key.5.1": "value.5"}
# create share
share = self.create_share(share_type_id=self.share_type_id,
@@ -85,8 +85,8 @@
@decorators.idempotent_id('4e5f8159-62b6-4d5c-f729-d8b1f029d7de')
@tc.attr(base.TAG_POSITIVE, base.TAG_API_WITH_BACKEND)
def test_set_metadata_not_delete_pre_metadata(self):
- md1 = {u"key9": u"value9", u"key10": u"value10", }
- md2 = {u"key11": u"value11", u"key12": u"value12", }
+ md1 = {"key9": "value9", "key10": "value10", }
+ md2 = {"key11": "value11", "key12": "value12", }
# create share
share = self.create_share(share_type_id=self.share_type_id,
@@ -121,8 +121,8 @@
@decorators.idempotent_id('2ec70ba5-050b-3b17-c862-c149e53543c0')
@tc.attr(base.TAG_POSITIVE, base.TAG_API_WITH_BACKEND)
def test_set_metadata_key_already_exist(self):
- md1 = {u"key9": u"value9", u"key10": u"value10", }
- md2 = {u"key9": u"value13", u"key11": u"value11", }
+ md1 = {"key9": "value9", "key10": "value10", }
+ md2 = {"key9": "value13", "key11": "value11", }
# create share
share = self.create_share(share_type_id=self.share_type_id,
@@ -138,8 +138,8 @@
self.shares_v2_client.set_metadata(share["id"], md2)
# verify metadata
- md = {u"key9": u"value13", u"key10": u"value10",
- u"key11": u"value11"}
+ md = {"key9": "value13", "key10": "value10",
+ "key11": "value11"}
self._verify_share_metadata(share, md)
# delete metadata
@@ -156,8 +156,8 @@
@tc.attr(base.TAG_POSITIVE, base.TAG_API_WITH_BACKEND)
def test_set_and_update_metadata_by_key(self):
- md1 = {u"key5": u"value5", u"key6": u"value6", }
- md2 = {u"key7": u"value7", u"key8": u"value8", }
+ md1 = {"key5": "value5", "key6": "value6", }
+ md2 = {"key7": "value7", "key8": "value8", }
# create share
share = self.create_share(share_type_id=self.share_type_id,
diff --git a/manila_tempest_tests/tests/api/test_metadata_negative.py b/manila_tempest_tests/tests/api/test_metadata_negative.py
index 5caca5e..38ae739 100644
--- a/manila_tempest_tests/tests/api/test_metadata_negative.py
+++ b/manila_tempest_tests/tests/api/test_metadata_negative.py
@@ -28,7 +28,7 @@
@tc.attr(base.TAG_NEGATIVE, base.TAG_API)
@ddt.data(True, False)
def test_try_set_metadata_to_unexisting_share(self, is_v2_client):
- md = {u"key1": u"value1", u"key2": u"value2", }
+ md = {"key1": "value1", "key2": "value2", }
client = self.shares_v2_client if is_v2_client else self.shares_client
self.assertRaises(lib_exc.NotFound,
client.set_metadata,
@@ -38,7 +38,7 @@
@tc.attr(base.TAG_NEGATIVE, base.TAG_API)
@ddt.data(True, False)
def test_try_update_all_metadata_for_unexisting_share(self, is_v2_client):
- md = {u"key1": u"value1", u"key2": u"value2", }
+ md = {"key1": "value1", "key2": "value2", }
client = self.shares_v2_client if is_v2_client else self.shares_client
self.assertRaises(lib_exc.NotFound,
client.update_all_metadata,
diff --git a/manila_tempest_tests/tests/api/test_rules_negative.py b/manila_tempest_tests/tests/api/test_rules_negative.py
index 0757c8e..2b62ad5 100644
--- a/manila_tempest_tests/tests/api/test_rules_negative.py
+++ b/manila_tempest_tests/tests/api/test_rules_negative.py
@@ -492,7 +492,7 @@
@decorators.idempotent_id('7b33c073-353e-4952-97dc-c3948a3cd037')
@tc.attr(base.TAG_NEGATIVE, base.TAG_API_WITH_BACKEND)
- @ddt.data('jane.doe', u"bj\u00F6rn")
+ @ddt.data('jane.doe', "bj\u00F6rn")
def test_create_access_rule_cephx_with_invalid_cephx_id(self, access_to):
self.assertRaises(lib_exc.BadRequest,
self.shares_v2_client.create_access_rule,
diff --git a/manila_tempest_tests/tests/api/test_share_network_subnet_metadata.py b/manila_tempest_tests/tests/api/test_share_network_subnet_metadata.py
index 2251b68..0f2c7c7 100644
--- a/manila_tempest_tests/tests/api/test_share_network_subnet_metadata.py
+++ b/manila_tempest_tests/tests/api/test_share_network_subnet_metadata.py
@@ -75,7 +75,7 @@
data['share_network_id'] = share_network['id']
data['availability_zone'] = az_name
- md = {u"key1": u"value1", u"key2": u"value2", }
+ md = {"key1": "value1", "key2": "value2", }
# create network subnet with metadata
subnet = self.create_share_network_subnet(metadata=md,
@@ -91,7 +91,7 @@
@decorators.idempotent_id('ec5c02e9-fcee-4890-87bc-7937a247afe9')
@tc.attr(base.TAG_POSITIVE, base.TAG_API)
def test_set_get_delete_metadata(self):
- md = {u"key3": u"value3", u"key4": u"value4", u"key.5.1": u"value.5"}
+ md = {"key3": "value3", "key4": "value4", "key.5.1": "value.5"}
# create subnet
subnet = self.create_share_network_subnet(**self.data)
@@ -126,8 +126,8 @@
@decorators.idempotent_id('9ff9c3b4-9bd0-4e8a-a317-726cab640a67')
@tc.attr(base.TAG_POSITIVE, base.TAG_API)
def test_set_metadata_not_delete_pre_metadata(self):
- md1 = {u"key9": u"value9", u"key10": u"value10", }
- md2 = {u"key11": u"value11", u"key12": u"value12", }
+ md1 = {"key9": "value9", "key10": "value10", }
+ md2 = {"key11": "value11", "key12": "value12", }
# create subnet
subnet = self.create_share_network_subnet(**self.data)
@@ -174,8 +174,8 @@
@decorators.idempotent_id('1973bcb0-93a5-49a5-84fb-a03f6f6ff43b')
@tc.attr(base.TAG_POSITIVE, base.TAG_API)
def test_set_metadata_key_already_exist(self):
- md1 = {u"key9": u"value9", u"key10": u"value10", }
- md2 = {u"key9": u"value13", u"key11": u"value11", }
+ md1 = {"key9": "value9", "key10": "value10", }
+ md2 = {"key9": "value13", "key11": "value11", }
# create subnet
subnet = self.create_share_network_subnet(**self.data)
@@ -220,8 +220,8 @@
@decorators.idempotent_id('d37ea163-7215-4c35-996a-1bc165e554de')
@tc.attr(base.TAG_POSITIVE, base.TAG_API)
def test_set_and_update_metadata_by_key(self):
- md1 = {u"key5": u"value5", u"key6": u"value6", }
- md2 = {u"key7": u"value7", u"key8": u"value8", }
+ md1 = {"key5": "value5", "key6": "value6", }
+ md2 = {"key7": "value7", "key8": "value8", }
# create subnet
subnet = self.create_share_network_subnet(**self.data)
diff --git a/manila_tempest_tests/tests/api/test_share_network_subnet_metadata_negative.py b/manila_tempest_tests/tests/api/test_share_network_subnet_metadata_negative.py
index 0ad9d46..4e4f2cd 100644
--- a/manila_tempest_tests/tests/api/test_share_network_subnet_metadata_negative.py
+++ b/manila_tempest_tests/tests/api/test_share_network_subnet_metadata_negative.py
@@ -48,7 +48,7 @@
@tc.attr(base.TAG_NEGATIVE, base.TAG_API)
def test_try_set_metadata_to_unexisting_subnet(self):
share_network = self.create_share_network()
- md = {u"key1": u"value1", u"key2": u"value2", }
+ md = {"key1": "value1", "key2": "value2", }
self.assertRaises(lib_exc.NotFound,
self.shares_v2_client.set_metadata,
"wrong_subnet_id", md, resource="subnet",
@@ -59,7 +59,7 @@
@tc.attr(base.TAG_NEGATIVE, base.TAG_API)
def test_try_update_all_metadata_to_unexisting_subnet(self):
share_network = self.create_share_network()
- md = {u"key1": u"value1", u"key2": u"value2", }
+ md = {"key1": "value1", "key2": "value2", }
self.assertRaises(lib_exc.NotFound,
self.shares_v2_client.update_all_metadata,
"wrong_subnet_id", md, resource="subnet",
diff --git a/manila_tempest_tests/tests/api/test_snapshot_metadata.py b/manila_tempest_tests/tests/api/test_snapshot_metadata.py
index 0596213..e5df030 100644
--- a/manila_tempest_tests/tests/api/test_snapshot_metadata.py
+++ b/manila_tempest_tests/tests/api/test_snapshot_metadata.py
@@ -79,7 +79,7 @@
@tc.attr(base.TAG_POSITIVE, base.TAG_API_WITH_BACKEND)
def test_set_metadata_in_snapshot_creation(self):
- md = {u"key1": u"value1", u"key2": u"value2", }
+ md = {"key1": "value1", "key2": "value2", }
# create snapshot with metadata
snapshot = self.create_snapshot_wait_for_active(
@@ -93,7 +93,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", u"key.5.1": u"value.5"}
+ md = {"key3": "value3", "key4": "value4", "key.5.1": "value.5"}
# create snapshot
snapshot = self.create_snapshot_wait_for_active(
@@ -120,8 +120,8 @@
@decorators.idempotent_id('23ec837d-1b50-499c-bbb9-a7bde843c9e8')
@tc.attr(base.TAG_POSITIVE, base.TAG_API_WITH_BACKEND)
def test_set_metadata_not_delete_pre_metadata(self):
- md1 = {u"key9": u"value9", u"key10": u"value10", }
- md2 = {u"key11": u"value11", u"key12": u"value12", }
+ md1 = {"key9": "value9", "key10": "value10", }
+ md2 = {"key11": "value11", "key12": "value12", }
# create snapshot
snapshot = self.create_snapshot_wait_for_active(
@@ -159,8 +159,8 @@
@decorators.idempotent_id('b7a00be5-3dd1-4d25-8723-c662581c923f')
@tc.attr(base.TAG_POSITIVE, base.TAG_API_WITH_BACKEND)
def test_set_metadata_key_already_exist(self):
- md1 = {u"key9": u"value9", u"key10": u"value10", }
- md2 = {u"key9": u"value13", u"key11": u"value11", }
+ md1 = {"key9": "value9", "key10": "value10", }
+ md2 = {"key9": "value13", "key11": "value11", }
# create snapshot
snapshot = self.create_snapshot_wait_for_active(
@@ -196,8 +196,8 @@
@tc.attr(base.TAG_POSITIVE, base.TAG_API_WITH_BACKEND)
def test_set_and_update_metadata_by_key(self):
- md1 = {u"key5": u"value5", u"key6": u"value6", }
- md2 = {u"key7": u"value7", u"key8": u"value8", }
+ md1 = {"key5": "value5", "key6": "value6", }
+ md2 = {"key7": "value7", "key8": "value8", }
# create snapshot
snapshot = self.create_snapshot_wait_for_active(
diff --git a/manila_tempest_tests/tests/api/test_snapshot_metadata_negative.py b/manila_tempest_tests/tests/api/test_snapshot_metadata_negative.py
index 7a80e4a..53a05c6 100644
--- a/manila_tempest_tests/tests/api/test_snapshot_metadata_negative.py
+++ b/manila_tempest_tests/tests/api/test_snapshot_metadata_negative.py
@@ -71,7 +71,7 @@
@decorators.idempotent_id('8be4773b-6af9-413f-97e2-8acdb6149e7a')
@tc.attr(base.TAG_NEGATIVE, base.TAG_API)
def test_try_set_metadata_to_unexisting_snapshot(self):
- md = {u"key1": u"value1", u"key2": u"value2", }
+ md = {"key1": "value1", "key2": "value2", }
self.assertRaises(lib_exc.NotFound,
self.shares_v2_client.set_metadata,
"wrong_snapshot_id", md, resource="snapshot")
@@ -79,7 +79,7 @@
@decorators.idempotent_id('03a7f6e9-de8b-4669-87e1-b179308b477d')
@tc.attr(base.TAG_NEGATIVE, base.TAG_API)
def test_try_update_all_metadata_to_unexisting_snapshot(self):
- md = {u"key1": u"value1", u"key2": u"value2", }
+ md = {"key1": "value1", "key2": "value2", }
self.assertRaises(lib_exc.NotFound,
self.shares_v2_client.update_all_metadata,
"wrong_snapshot_id", md, resource="snapshot")