Remove unneeded randomness when calling ddt.data()
The usage of sets introduces unneeded randomness, and the
utils.deduplicate function can be used instead.
This is a follow-up to similar past changes, see:
I9cbd26016238c25487ac8104c1188cd2cf4f467e
I15bfbf6e1ca604f2bf2fcf12dcd4e67bc728fe5e
Change-Id: I8fce553d927da6d94471ecdf4040bbbb4377d9cf
diff --git a/manila_tempest_tests/tests/api/test_rules.py b/manila_tempest_tests/tests/api/test_rules.py
index 30b1fc5..0128534 100644
--- a/manila_tempest_tests/tests/api/test_rules.py
+++ b/manila_tempest_tests/tests/api/test_rules.py
@@ -109,8 +109,11 @@
@decorators.idempotent_id('3390df2d-f6f8-4634-a562-87c1be994f6a')
@tc.attr(base.TAG_POSITIVE, base.TAG_BACKEND)
@ddt.data(*itertools.chain(
- itertools.product({'1.0', '2.9', '2.37', LATEST_MICROVERSION}, {4}),
- itertools.product({'2.38', LATEST_MICROVERSION}, {6})
+ itertools.product(
+ utils.deduplicate(['1.0', '2.9', '2.37', LATEST_MICROVERSION]),
+ [4]),
+ itertools.product(
+ utils.deduplicate(['2.38', LATEST_MICROVERSION]), [6])
))
@ddt.unpack
def test_create_delete_access_rules_with_one_ip(self, version,
@@ -166,8 +169,11 @@
@decorators.idempotent_id('5d25168a-d646-443e-8cf1-3151eb7887f5')
@tc.attr(base.TAG_POSITIVE, base.TAG_BACKEND)
@ddt.data(*itertools.chain(
- itertools.product({'1.0', '2.9', '2.37', LATEST_MICROVERSION}, {4}),
- itertools.product({'2.38', LATEST_MICROVERSION}, {6})
+ itertools.product(
+ utils.deduplicate(['1.0', '2.9', '2.37', LATEST_MICROVERSION]),
+ [4]),
+ itertools.product(
+ utils.deduplicate(['2.38', LATEST_MICROVERSION]), [6])
))
@ddt.unpack
def test_create_delete_access_rule_with_cidr(self, version, ip_version):