Browse Source

Add duplicate uuid checking for tempest tests

This patch adds a pep8 test for duplicate uuids in the tempest tests.
It also corrects some existing duplicates.

Change-Id: I2fc03b3cc604528903b8f70ce7f1e4e641aeafd2
changes/18/751618/1
Michael Johnson 1 year ago
parent
commit
46658f2ff8
  1. 2
      octavia_tempest_plugin/tests/api/v2/test_load_balancer.py
  2. 2
      octavia_tempest_plugin/tests/scenario/v2/test_member.py
  3. 7
      tox.ini

2
octavia_tempest_plugin/tests/api/v2/test_load_balancer.py

@ -197,7 +197,7 @@ class LoadBalancerAPITest(test_base.LoadBalancerBaseTest):
CONF.load_balancer.lb_build_interval,
CONF.load_balancer.lb_build_timeout)
@decorators.idempotent_id('643ef031-c800-45f2-b229-3c8f8b37c829')
@decorators.idempotent_id('abd784e3-485f-442a-85da-d91365c6b5dd')
def test_load_balancer_delete_cascade(self):
"""Tests load balancer create and cascade delete APIs.

2
octavia_tempest_plugin/tests/scenario/v2/test_member.py

@ -671,7 +671,7 @@ class MemberScenarioTest(test_base.LoadBalancerBaseTest):
const.TCP, const.TCP, const.LB_ALGORITHM_ROUND_ROBIN)
self._test_mixed_member_create(pool_id)
@decorators.idempotent_id('0623aa1f-753d-44e7-afa1-017d274eace7')
@decorators.idempotent_id('e74b28cf-ab1a-423b-a1c5-d940e3c0a5ab')
@testtools.skipUnless(CONF.load_balancer.test_with_ipv6,
'IPv6 testing is disabled')
# Skipping test for amphora driver until "UDP load balancers cannot mix

7
tox.ini

@ -18,7 +18,12 @@ commands =
stestr slowest
[testenv:pep8]
commands = flake8
commands =
flake8
check-uuid --package octavia_tempest_plugin
[testenv:uuidgen]
commands = check-uuid --fix --package octavia_tempest_plugin
[testenv:venv]
commands = {posargs}

Loading…
Cancel
Save