openstack
/
neutron-tempest-plugin
Code Issues Proposed changes

Clean routers in DvrRoutersNegativeTestExtended 

It completes the initial patch "Add tempest tests for DVR router state
management" which doesn't clean 2 routers [1].

[1] https://review.opendev.org/#/c/668760/

Change-Id: Iec14343578ca587143a55bbd3d602790730d31b6
Signed-off-by: Cédric Ollivier <ollivier.cedric@gmail.com>
This commit is contained in:
Cédric Ollivier 2019-08-21 09:26:37 +02:00
parent d11f4ec31a
commit 0a3705f937
1 changed files with 4 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
neutron_tempest_plugin/api/test_routers_negative.py
View File

@ -90,9 +90,9 @@ class DvrRoutersNegativeTestExtended(RoutersNegativeTestBase):
# create a centralized router
router_args = {'tenant_id': self.client.tenant_id,
'distributed': False}
router = self.admin_client.create_router(
router = self._create_admin_router(
data_utils.rand_name('router'), admin_state_up=True,
**router_args)['router']
**router_args)
self.assertTrue(router['admin_state_up'])
self.assertFalse(router['distributed'])
# attempt to set the router to distributed, catch BadRequest exception
@ -107,9 +107,9 @@ class DvrRoutersNegativeTestExtended(RoutersNegativeTestBase):
# create a centralized router
router_args = {'tenant_id': self.client.tenant_id,
'distributed': False}
router = self.admin_client.create_router(
router = self._create_admin_router(
data_utils.rand_name('router'), admin_state_up=True,
**router_args)['router']
**router_args)
self.assertTrue(router['admin_state_up'])
self.assertFalse(router['distributed'])
# take the router down to modify distributed->True