Add Python3 support in functest/ci

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

diff --git a/functest/ci/check_deployment.py b/functest/ci/check_deployment.py
index ae016a8..8cc522f 100644 (file)
--- a/functest/ci/check_deployment.py
+++ b/functest/ci/check_deployment.py
@@ -19,16 +19,16 @@ import logging
 import logging.config
 import os
 import pkg_resources
+from six.moves.urllib.parse import urlparse
 import socket
-from urlparse import urlparse
 
 from functest.opnfv_tests.openstack.snaps import snaps_utils
 
+from snaps.openstack.tests import openstack_tests
 from snaps.openstack.utils import glance_utils
 from snaps.openstack.utils import keystone_utils
 from snaps.openstack.utils import neutron_utils
 from snaps.openstack.utils import nova_utils
-from snaps.openstack.tests import openstack_tests
 
 __author__ = "Jose Lausuch <jose.lausuch@ericsson.com>"
 

diff --git a/functest/ci/run_tests.py b/functest/ci/run_tests.py
index d476bde..67a6f14 100644 (file)
--- a/functest/ci/run_tests.py
+++ b/functest/ci/run_tests.py
@@ -19,6 +19,7 @@ import sys
 import textwrap
 
 import prettytable
+import six
 import yaml
 
 import functest.ci.tier_builder as tb
@@ -118,7 +119,7 @@ class Runner(object):
             raise Exception("RC file %s does not exist..." % rc_file)
         logger.debug("Sourcing the OpenStack RC file...")
         os_utils.source_credentials(rc_file)
-        for key, value in os.environ.iteritems():
+        for key, value in six.iteritems(os.environ):
             if re.search("OS_", key):
                 if key == 'OS_AUTH_URL':
                     CONST.__setattr__('OS_AUTH_URL', value)

diff --git a/functest/tests/unit/ci/test_check_deployment.py b/functest/tests/unit/ci/test_check_deployment.py
index 24e3ce5..66d1b7a 100644 (file)
--- a/functest/tests/unit/ci/test_check_deployment.py
+++ b/functest/tests/unit/ci/test_check_deployment.py
@@ -41,7 +41,7 @@ class CheckDeploymentTesting(unittest.TestCase):
     def test_check_rc(self):
         with mock.patch('functest.ci.check_deployment.os.path.isfile',
                         returns=True) as m, \
-                mock.patch('__builtin__.open',
+                mock.patch('six.moves.builtins.open',
                            mock.mock_open(read_data='OS_AUTH_URL')):
             self.deployment.check_rc()
             self.assertTrue(m.called)
@@ -55,7 +55,7 @@ class CheckDeploymentTesting(unittest.TestCase):
             self.assertTrue(msg in context)
 
     def test_check_rc_missing_os_auth(self):
-        with mock.patch('__builtin__.open',
+        with mock.patch('six.moves.builtins.open',
                         mock.mock_open(read_data='test')), \
                 self.assertRaises(Exception) as context:
             msg = 'OS_AUTH_URL not defined in {}.'.format(self.rc_file)

diff --git a/functest/tests/unit/ci/test_tier_builder.py b/functest/tests/unit/ci/test_tier_builder.py
index 700c6e9..1dec9ae 100644 (file)
--- a/functest/tests/unit/ci/test_tier_builder.py
+++ b/functest/tests/unit/ci/test_tier_builder.py
@@ -39,7 +39,7 @@ class TierBuilderTesting(unittest.TestCase):
 
         with mock.patch('functest.ci.tier_builder.yaml.safe_load',
                         return_value=self.mock_yaml), \
-                mock.patch('__builtin__.open', mock.mock_open()):
+                mock.patch('six.moves.builtins.open', mock.mock_open()):
             self.tierbuilder = tier_builder.TierBuilder('test_installer',
                                                         'test_scenario',
                                                         'testcases_file')

diff --git a/tox.ini b/tox.ini
index 8eeb1d1..1e5487b 100644 (file)
--- a/tox.ini
+++ b/tox.ini
@@ -54,6 +54,7 @@ commands =
 
 [testenv:py35]
 dirs =
+  functest/tests/unit/ci
   functest/tests/unit/core
   functest/tests/unit/odl
   functest/tests/unit/utils/test_decorators.py