Protect vs former interface format

Compass still set publicURL [1]

[1] https://build.opnfv.org/ci/view/functest/job/functest-compass-baremetal-daily-fraser/lastFailedBuild/consoleFull

Change-Id: I88c6ca336a7017f97f4c07719156e567d1afb9df
Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>

diff --git a/functest/opnfv_tests/sdn/odl/odl.py b/functest/opnfv_tests/sdn/odl/odl.py
index 180fa0e..21c879e 100644 (file)
--- a/functest/opnfv_tests/sdn/odl/odl.py
+++ b/functest/opnfv_tests/sdn/odl/odl.py
@@ -159,8 +159,10 @@ class ODLTests(robotframework.RobotFramework):
             cloud = os_client_config.make_shade()
             neutron_id = cloud.search_services('neutron')[0].id
             endpoint = cloud.search_endpoints(
-                filters={'interface': os.environ.get('OS_INTERFACE', 'public'),
-                         'service_id': neutron_id})[0].url
+                filters={
+                    'interface': os.environ.get(
+                        'OS_INTERFACE', 'public').replace('URL', ''),
+                    'service_id': neutron_id})[0].url
             kwargs = {'neutronurl': endpoint}
             kwargs['odlip'] = env.get('SDN_CONTROLLER_IP')
             kwargs['odlwebport'] = '8080'

diff --git a/functest/tests/unit/odl/test_odl.py b/functest/tests/unit/odl/test_odl.py
index bf509cb..58a699f 100644 (file)
--- a/functest/tests/unit/odl/test_odl.py
+++ b/functest/tests/unit/odl/test_odl.py
@@ -388,8 +388,10 @@ class ODLRunTesting(ODLTesting):
         args[0].assert_called_once_with()
         args[0].return_value.search_services.assert_called_once_with('neutron')
         args[0].return_value.search_endpoints.assert_called_once_with(
-            filters={'interface': os.environ.get("OS_INTERFACE", "public"),
-                     'service_id': self._neutron_id})
+            filters={
+                'interface': os.environ.get(
+                    "OS_INTERFACE", "public").replace('URL', ''),
+                'service_id': self._neutron_id})
 
     @mock.patch('os_client_config.make_shade')
     def _test_multiple_suites(self, suites,
@@ -474,6 +476,13 @@ class ODLRunTesting(ODLTesting):
                        odlip=self._sdn_controller_ip,
                        odlwebport=self._odl_webport)
 
+    def test_os_interface_publicurl(self):
+        os.environ["OS_INTERFACE"] = "publicURL"
+        os.environ["SDN_CONTROLLER_IP"] = self._sdn_controller_ip
+        self._test_run(testcase.TestCase.EX_OK, None,
+                       odlip=self._sdn_controller_ip,
+                       odlwebport=self._odl_webport)
+
     def test_os_interface_internal(self):
         os.environ["OS_INTERFACE"] = "internal"
         os.environ["SDN_CONTROLLER_IP"] = self._sdn_controller_ip