From 866a17037979e0e23ecc3e416c873a5be93044b6 Mon Sep 17 00:00:00 2001 From: Akihiro Motoki Date: Fri, 4 Jun 2021 16:18:56 +0900 Subject: [PATCH] integration tests: Relax router interface status check Previously the router interface test in the integration tests checked the status of an added router interface, but the initial status varies depending on a network backend used in neutron. In case of neutron L3 agent, a new router interface has "Down" status, but in case of OVN a new router interface has "Active" status. This caused the integration test failure after DevStack switched the default network backend from ML2/OVS to ML2/OVN. The initial status of a new router interface depends on a network backend and the neutron API does not define the initial status, so it is too strict for the horizon integration tests to check the status. This commit drops the status check. It is sufficient to check the existence of a new router interface. Change-Id: Idfd4a9206e85558ba17e476a47b38ee9c1f566fa Closes-Bug: #1930405 --- openstack_dashboard/test/integration_tests/tests/test_router.py | 2 -- 1 file changed, 2 deletions(-) diff --git a/openstack_dashboard/test/integration_tests/tests/test_router.py b/openstack_dashboard/test/integration_tests/tests/test_router.py index e6e43ae7f3..feba833e09 100644 --- a/openstack_dashboard/test/integration_tests/tests/test_router.py +++ b/openstack_dashboard/test/integration_tests/tests/test_router.py @@ -61,8 +61,6 @@ class TestRouters(helpers.TestCase): interfaces_page.find_message_and_dismiss(messages.ERROR)) interface_name = interfaces_page.interface_name self.assertTrue(interfaces_page.is_interface_present(interface_name)) - self.assertTrue( - interfaces_page.is_interface_status(interface_name, 'Down')) def _delete_interface(self, interfaces_page, interface_name): interfaces_page.delete_interface(interface_name)