Remove further randomness when calling ddt.data()
Avoid the usage of set to dedeplicate the lists and
instead use the specific utils.deduplicate function,
which preserves the original order.
This is a follow-up of: I9cbd26016238c25487ac8104c1188cd2cf4f467e
Change-Id: I15bfbf6e1ca604f2bf2fcf12dcd4e67bc728fe5e
diff --git a/manila_tempest_tests/tests/api/admin/test_share_group_types.py b/manila_tempest_tests/tests/api/admin/test_share_group_types.py
index eb62838..89d43fa 100644
--- a/manila_tempest_tests/tests/api/admin/test_share_group_types.py
+++ b/manila_tempest_tests/tests/api/admin/test_share_group_types.py
@@ -60,7 +60,7 @@
@decorators.idempotent_id('e2ba1754-cecc-4178-ad39-eefbb59e4d6d')
@tc.attr(base.TAG_POSITIVE, base.TAG_API)
@ddt.data(
- *itertools.product(('id', 'name'), set(
+ *itertools.product(('id', 'name'), utils.deduplicate(
[LATEST_MICROVERSION, constants.MIN_SHARE_GROUP_MICROVERSION,
constants.SHARE_GROUPS_GRADUATION_VERSION])))
@ddt.unpack
diff --git a/manila_tempest_tests/tests/api/test_quotas.py b/manila_tempest_tests/tests/api/test_quotas.py
index 958cb63..b25db80 100644
--- a/manila_tempest_tests/tests/api/test_quotas.py
+++ b/manila_tempest_tests/tests/api/test_quotas.py
@@ -88,7 +88,7 @@
self.assertGreater(int(quotas["replica_gigabytes"]), -2)
@ddt.data(
- *itertools.product(set(
+ *itertools.product(utils.deduplicate(
["2.25", "2.53", CONF.share.max_api_microversion]), (True, False))
)
@ddt.unpack
diff --git a/manila_tempest_tests/tests/api/test_rules.py b/manila_tempest_tests/tests/api/test_rules.py
index 2a2420a..5c3bdef 100644
--- a/manila_tempest_tests/tests/api/test_rules.py
+++ b/manila_tempest_tests/tests/api/test_rules.py
@@ -494,7 +494,7 @@
@decorators.idempotent_id('4e636fd2-26ef-4b63-96eb-77860a8b6cdf')
@tc.attr(base.TAG_POSITIVE, base.TAG_BACKEND)
@ddt.data(*itertools.product(
- set(['2.13', '2.27', '2.28', LATEST_MICROVERSION]),
+ utils.deduplicate(['2.13', '2.27', '2.28', LATEST_MICROVERSION]),
("alice", "alice_bob", "alice bob"),
('rw', 'ro')))
@ddt.unpack