aboutsummaryrefslogtreecommitdiffstats
path: root/functest/tests
diff options
context:
space:
mode:
Diffstat (limited to 'functest/tests')
-rw-r--r--functest/tests/unit/odl/test_odl.py26
-rw-r--r--functest/tests/unit/opnfv_tests/openstack/tempest/__init__.py0
-rw-r--r--functest/tests/unit/opnfv_tests/openstack/tempest/test_conf_utils.py92
-rw-r--r--functest/tests/unit/opnfv_tests/openstack/tempest/test_tempest.py279
-rw-r--r--functest/tests/unit/utils/test_openstack_clean.py57
5 files changed, 444 insertions, 10 deletions
diff --git a/functest/tests/unit/odl/test_odl.py b/functest/tests/unit/odl/test_odl.py
index 8f2a5d7e..5961940f 100644
--- a/functest/tests/unit/odl/test_odl.py
+++ b/functest/tests/unit/odl/test_odl.py
@@ -30,6 +30,7 @@ class ODLTesting(unittest.TestCase):
_keystone_ip = "127.0.0.1"
_neutron_ip = "127.0.0.2"
_sdn_controller_ip = "127.0.0.3"
+ _os_auth_url = "http://{}:5000/v2.0".format(_keystone_ip)
_os_tenantname = "admin"
_os_username = "admin"
_os_password = "admin"
@@ -42,14 +43,15 @@ class ODLTesting(unittest.TestCase):
for var in ("INSTALLER_TYPE", "SDN_CONTROLLER", "SDN_CONTROLLER_IP"):
if var in os.environ:
del os.environ[var]
+ os.environ["OS_AUTH_URL"] = self._os_auth_url
os.environ["OS_USERNAME"] = self._os_username
os.environ["OS_PASSWORD"] = self._os_password
os.environ["OS_TENANT_NAME"] = self._os_tenantname
self.test = odl.ODLTests()
self.defaultargs = {'odlusername': self._odl_username,
'odlpassword': self._odl_password,
- 'keystoneip': self._keystone_ip,
'neutronip': self._keystone_ip,
+ 'osauthurl': self._os_auth_url,
'osusername': self._os_username,
'ostenantname': self._os_tenantname,
'ospassword': self._os_password,
@@ -157,8 +159,8 @@ class ODLTesting(unittest.TestCase):
def _get_main_kwargs(self, key=None):
kwargs = {'odlusername': self._odl_username,
'odlpassword': self._odl_password,
- 'keystoneip': self._keystone_ip,
'neutronip': self._neutron_ip,
+ 'osauthurl': self._os_auth_url,
'osusername': self._os_username,
'ostenantname': self._os_tenantname,
'ospassword': self._os_password,
@@ -178,6 +180,7 @@ class ODLTesting(unittest.TestCase):
if len(args) > 1:
variable = ['KEYSTONE:{}'.format(self._keystone_ip),
'NEUTRON:{}'.format(self._neutron_ip),
+ 'OS_AUTH_URL:"{}"'.format(self._os_auth_url),
'OSUSERNAME:"{}"'.format(self._os_username),
'OSTENANTNAME:"{}"'.format(self._os_tenantname),
'OSPASSWORD:"{}"'.format(self._os_password),
@@ -207,12 +210,12 @@ class ODLTesting(unittest.TestCase):
def test_main_missing_odlpassword(self):
self._test_main_missing_keyword('odlpassword')
- def test_main_missing_keystoneip(self):
- self._test_main_missing_keyword('keystoneip')
-
def test_main_missing_neutronip(self):
self._test_main_missing_keyword('neutronip')
+ def test_main_missing_osauthurl(self):
+ self._test_main_missing_keyword('osauthurl')
+
def test_main_missing_osusername(self):
self._test_main_missing_keyword('osusername')
@@ -347,10 +350,11 @@ class ODLTesting(unittest.TestCase):
self.assertEqual(self.test.run(), status)
self.test.main.assert_called_once_with(
odl.ODLTests.default_suites,
- keystoneip=self._keystone_ip, neutronip=self._neutron_ip,
+ neutronip=self._neutron_ip,
odlip=odlip, odlpassword=self._odl_password,
odlrestconfport=odlrestconfport,
odlusername=self._odl_username, odlwebport=odlwebport,
+ osauthurl=self._os_auth_url,
ospassword=self._os_password, ostenantname=self._os_tenantname,
osusername=self._os_username)
@@ -368,10 +372,11 @@ class ODLTesting(unittest.TestCase):
self.assertEqual(self.test.run(suites=suites), status)
self.test.main.assert_called_once_with(
suites,
- keystoneip=self._keystone_ip, neutronip=self._neutron_ip,
+ neutronip=self._neutron_ip,
odlip=odlip, odlpassword=self._odl_password,
odlrestconfport=odlrestconfport,
odlusername=self._odl_username, odlwebport=odlwebport,
+ osauthurl=self._os_auth_url,
ospassword=self._os_password, ostenantname=self._os_tenantname,
osusername=self._os_username)
@@ -381,6 +386,9 @@ class ODLTesting(unittest.TestCase):
self.assertEqual(self.test.run(),
testcase_base.TestcaseBase.EX_RUN_ERROR)
+ def test_run_missing_os_auth_url(self):
+ self._test_run_missing_env_var("OS_AUTH_URL")
+
def test_run_missing_os_username(self):
self._test_run_missing_env_var("OS_USERNAME")
@@ -507,8 +515,8 @@ class ODLTesting(unittest.TestCase):
def test_argparser_odlpassword(self):
self._test_argparser('odlpassword', 'foo')
- def test_argparser_keystoneip(self):
- self._test_argparser('keystoneip', '127.0.0.4')
+ def test_argparser_osauthurl(self):
+ self._test_argparser('osauthurl', 'http://127.0.0.4:5000/v2')
def test_argparser_neutronip(self):
self._test_argparser('neutronip', '127.0.0.4')
diff --git a/functest/tests/unit/opnfv_tests/openstack/tempest/__init__.py b/functest/tests/unit/opnfv_tests/openstack/tempest/__init__.py
new file mode 100644
index 00000000..e69de29b
--- /dev/null
+++ b/functest/tests/unit/opnfv_tests/openstack/tempest/__init__.py
diff --git a/functest/tests/unit/opnfv_tests/openstack/tempest/test_conf_utils.py b/functest/tests/unit/opnfv_tests/openstack/tempest/test_conf_utils.py
new file mode 100644
index 00000000..6121f4eb
--- /dev/null
+++ b/functest/tests/unit/opnfv_tests/openstack/tempest/test_conf_utils.py
@@ -0,0 +1,92 @@
+#!/usr/bin/env python
+
+# All rights reserved. This program and the accompanying materials
+# are made available under the terms of the Apache License, Version 2.0
+# which accompanies this distribution, and is available at
+# http://www.apache.org/licenses/LICENSE-2.0
+
+import logging
+import unittest
+
+import mock
+
+from functest.opnfv_tests.openstack.tempest import conf_utils
+from functest.utils.constants import CONST
+
+
+class OSTempestConfUtilsTesting(unittest.TestCase):
+
+ logging.disable(logging.CRITICAL)
+
+ def test_get_verifier_id_missing_verifier(self):
+ CONST.tempest_deployment_name = 'test_deploy_name'
+ with mock.patch('functest.opnfv_tests.openstack.tempest.'
+ 'conf_utils.subprocess.Popen') as mock_popen, \
+ self.assertRaises(Exception):
+ mock_stdout = mock.Mock()
+ attrs = {'stdout.readline.return_value': ''}
+ mock_stdout.configure_mock(**attrs)
+ mock_popen.return_value = mock_stdout
+ conf_utils.get_verifier_id(),
+
+ def test_get_verifier_id_default(self):
+ CONST.tempest_deployment_name = 'test_deploy_name'
+ with mock.patch('functest.opnfv_tests.openstack.tempest.'
+ 'conf_utils.subprocess.Popen') as mock_popen:
+ mock_stdout = mock.Mock()
+ attrs = {'stdout.readline.return_value': 'test_deploy_id'}
+ mock_stdout.configure_mock(**attrs)
+ mock_popen.return_value = mock_stdout
+
+ self.assertEqual(conf_utils.get_verifier_id(),
+ 'test_deploy_id')
+
+ def test_get_verifier_deployment_id_missing_rally(self):
+ CONST.rally_deployment_name = 'test_rally_deploy_name'
+ with mock.patch('functest.opnfv_tests.openstack.tempest.'
+ 'conf_utils.subprocess.Popen') as mock_popen, \
+ self.assertRaises(Exception):
+ mock_stdout = mock.Mock()
+ attrs = {'stdout.readline.return_value': ''}
+ mock_stdout.configure_mock(**attrs)
+ mock_popen.return_value = mock_stdout
+ conf_utils.get_verifier_deployment_id(),
+
+ def test_get_verifier_deployment_id_default(self):
+ CONST.rally_deployment_name = 'test_rally_deploy_name'
+ with mock.patch('functest.opnfv_tests.openstack.tempest.'
+ 'conf_utils.subprocess.Popen') as mock_popen:
+ mock_stdout = mock.Mock()
+ attrs = {'stdout.readline.return_value': 'test_deploy_id'}
+ mock_stdout.configure_mock(**attrs)
+ mock_popen.return_value = mock_stdout
+
+ self.assertEqual(conf_utils.get_verifier_deployment_id(),
+ 'test_deploy_id')
+
+ def test_get_verifier_repo_dir_default(self):
+ with mock.patch('functest.opnfv_tests.openstack.tempest.'
+ 'conf_utils.os.path.join',
+ return_value='test_verifier_repo_dir'), \
+ mock.patch('functest.opnfv_tests.openstack.tempest.'
+ 'conf_utils.get_verifier_id') as m:
+ self.assertEqual(conf_utils.get_verifier_repo_dir(''),
+ 'test_verifier_repo_dir')
+ self.assertTrue(m.called)
+
+ def test_get_verifier_deployment_dir_default(self):
+ with mock.patch('functest.opnfv_tests.openstack.tempest.'
+ 'conf_utils.os.path.join',
+ return_value='test_verifier_repo_dir'), \
+ mock.patch('functest.opnfv_tests.openstack.tempest.'
+ 'conf_utils.get_verifier_id') as m1, \
+ mock.patch('functest.opnfv_tests.openstack.tempest.'
+ 'conf_utils.get_verifier_deployment_id') as m2:
+ self.assertEqual(conf_utils.get_verifier_deployment_dir('', ''),
+ 'test_verifier_repo_dir')
+ self.assertTrue(m1.called)
+ self.assertTrue(m2.called)
+
+
+if __name__ == "__main__":
+ unittest.main(verbosity=2)
diff --git a/functest/tests/unit/opnfv_tests/openstack/tempest/test_tempest.py b/functest/tests/unit/opnfv_tests/openstack/tempest/test_tempest.py
new file mode 100644
index 00000000..8ae7b4db
--- /dev/null
+++ b/functest/tests/unit/opnfv_tests/openstack/tempest/test_tempest.py
@@ -0,0 +1,279 @@
+#!/usr/bin/env python
+
+# All rights reserved. This program and the accompanying materials
+# are made available under the terms of the Apache License, Version 2.0
+# which accompanies this distribution, and is available at
+# http://www.apache.org/licenses/LICENSE-2.0
+
+import logging
+import unittest
+
+import mock
+
+from functest.core import testcase_base
+from functest.opnfv_tests.openstack.tempest import tempest
+from functest.opnfv_tests.openstack.tempest import conf_utils
+from functest.utils.constants import CONST
+
+
+class OSTempestTesting(unittest.TestCase):
+
+ logging.disable(logging.CRITICAL)
+
+ def setUp(self):
+ with mock.patch('functest.opnfv_tests.openstack.tempest.tempest.'
+ 'conf_utils.get_verifier_id',
+ return_value='test_deploy_id'), \
+ mock.patch('functest.opnfv_tests.openstack.tempest.tempest.'
+ 'conf_utils.get_verifier_deployment_id',
+ return_value='test_deploy_id'), \
+ mock.patch('functest.opnfv_tests.openstack.tempest.tempest.'
+ 'conf_utils.get_verifier_repo_dir',
+ return_value='test_verifier_repo_dir'), \
+ mock.patch('functest.opnfv_tests.openstack.tempest.tempest.'
+ 'conf_utils.get_verifier_deployment_dir',
+ return_value='test_verifier_deploy_dir'):
+ self.tempestcommon = tempest.TempestCommon()
+
+ def test_create_tempest_resources_missing_network_dic(self):
+ with mock.patch('functest.opnfv_tests.openstack.tempest.tempest.'
+ 'os_utils.get_keystone_client',
+ return_value=mock.Mock()), \
+ mock.patch('functest.opnfv_tests.openstack.tempest.tempest.'
+ 'os_utils.create_tenant',
+ return_value='test_tenant_id'), \
+ mock.patch('functest.opnfv_tests.openstack.tempest.tempest.'
+ 'os_utils.create_user',
+ return_value='test_user_id'), \
+ mock.patch('functest.opnfv_tests.openstack.tempest.tempest.'
+ 'os_utils.create_shared_network_full',
+ return_value=None), \
+ self.assertRaises(Exception) as context:
+ self.tempestcommon.create_tempest_resources()
+ msg = 'Failed to create private network'
+ self.assertTrue(msg in context)
+
+ def test_create_tempest_resources_missing_image(self):
+ CONST.tempest_use_custom_images = 'test_image'
+ with mock.patch('functest.opnfv_tests.openstack.tempest.tempest.'
+ 'os_utils.get_keystone_client',
+ return_value=mock.Mock()), \
+ mock.patch('functest.opnfv_tests.openstack.tempest.tempest.'
+ 'os_utils.create_tenant',
+ return_value='test_tenant_id'), \
+ mock.patch('functest.opnfv_tests.openstack.tempest.tempest.'
+ 'os_utils.create_user',
+ return_value='test_user_id'), \
+ mock.patch('functest.opnfv_tests.openstack.tempest.tempest.'
+ 'os_utils.create_shared_network_full',
+ return_value=mock.Mock()), \
+ mock.patch('functest.opnfv_tests.openstack.tempest.tempest.'
+ 'os_utils.get_or_create_image',
+ return_value=(mock.Mock(), None)), \
+ self.assertRaises(Exception) as context:
+ self.tempestcommon.create_tempest_resources()
+ msg = 'Failed to create image'
+ self.assertTrue(msg in context)
+
+ def test_create_tempest_resources_missing_flavor(self):
+ CONST.tempest_use_custom_images = 'test_image'
+ CONST.tempest_use_custom_flavors = 'test_flavour'
+ with mock.patch('functest.opnfv_tests.openstack.tempest.tempest.'
+ 'os_utils.get_keystone_client',
+ return_value=mock.Mock()), \
+ mock.patch('functest.opnfv_tests.openstack.tempest.tempest.'
+ 'os_utils.create_tenant',
+ return_value='test_tenant_id'), \
+ mock.patch('functest.opnfv_tests.openstack.tempest.tempest.'
+ 'os_utils.create_user',
+ return_value='test_user_id'), \
+ mock.patch('functest.opnfv_tests.openstack.tempest.tempest.'
+ 'os_utils.create_shared_network_full',
+ return_value=mock.Mock()), \
+ mock.patch('functest.opnfv_tests.openstack.tempest.tempest.'
+ 'os_utils.get_or_create_image',
+ return_value=(mock.Mock(), 'image_id')), \
+ mock.patch('functest.opnfv_tests.openstack.tempest.tempest.'
+ 'os_utils.get_or_create_flavor',
+ return_value=(mock.Mock(), None)), \
+ self.assertRaises(Exception) as context:
+ self.tempestcommon.create_tempest_resources()
+ msg = 'Failed to create flavor'
+ self.assertTrue(msg in context)
+
+ @mock.patch('functest.opnfv_tests.openstack.tempest.tempest.logger.debug')
+ def test_generate_test_list_defcore_mode(self, mock_logger_debug):
+ self.tempestcommon.MODE = 'defcore'
+ with mock.patch('functest.opnfv_tests.openstack.tempest.tempest.'
+ 'shutil.copyfile') as m:
+ self.tempestcommon.generate_test_list('test_verifier_repo_dir')
+ self.assertTrue(m.called)
+
+ @mock.patch('functest.opnfv_tests.openstack.tempest.tempest.logger.error')
+ @mock.patch('functest.opnfv_tests.openstack.tempest.tempest.logger.debug')
+ def test_generate_test_list_custom_mode_missing_file(self,
+ mock_logger_debug,
+ mock_logger_error):
+ self.tempestcommon.MODE = 'custom'
+ with mock.patch('functest.opnfv_tests.openstack.tempest.tempest.'
+ 'os.path.isfile', return_value=False), \
+ self.assertRaises(Exception) as context:
+ msg = "Tempest test list file %s NOT found."
+ self.tempestcommon.generate_test_list('test_verifier_repo_dir')
+ self.assertTrue((msg % conf_utils.TEMPEST_CUSTOM) in context)
+
+ def test_generate_test_list_custom_mode_default(self):
+ self.tempestcommon.MODE = 'custom'
+ with mock.patch('functest.opnfv_tests.openstack.tempest.tempest.'
+ 'shutil.copyfile') as m, \
+ mock.patch('functest.opnfv_tests.openstack.tempest.tempest.'
+ 'os.path.isfile', return_value=True):
+ self.tempestcommon.generate_test_list('test_verifier_repo_dir')
+ self.assertTrue(m.called)
+
+ def _test_generate_test_list_mode_default(self, mode):
+ self.tempestcommon.MODE = mode
+ if self.tempestcommon.MODE == 'smoke':
+ testr_mode = "smoke"
+ elif self.tempestcommon.MODE == 'feature_multisite':
+ testr_mode = "'[Kk]ingbird'"
+ elif self.tempestcommon.MODE == 'full':
+ testr_mode = ""
+ else:
+ testr_mode = 'tempest.api.' + self.tempestcommon.MODE
+ conf_utils.TEMPEST_RAW_LIST = 'raw_list'
+ verifier_repo_dir = 'test_verifier_repo_dir'
+ with mock.patch('functest.opnfv_tests.openstack.tempest.tempest.'
+ 'ft_utils.execute_command') as m:
+ cmd = ("cd {0};"
+ "testr list-tests {1} > {2};"
+ "cd -;".format(verifier_repo_dir,
+ testr_mode,
+ conf_utils.TEMPEST_RAW_LIST))
+ self.tempestcommon.generate_test_list('test_verifier_repo_dir')
+ m.assert_any_call(cmd)
+
+ def test_generate_test_list_smoke_mode(self):
+ self._test_generate_test_list_mode_default('smoke')
+
+ def test_generate_test_list_feature_multisite_mode(self):
+ self._test_generate_test_list_mode_default('feature_multisite')
+
+ def test_generate_test_list_full_mode(self):
+ self._test_generate_test_list_mode_default('full')
+
+ def test_parse_verifier_result_missing_verification_uuid(self):
+ self.tempestcommon.VERIFICATION_ID = ''
+ with self.assertRaises(Exception):
+ self.tempestcommon.parse_verifier_result()
+
+ @mock.patch('functest.opnfv_tests.openstack.tempest.tempest.logger.info')
+ def test_parse_verifier_result_default(self, mock_logger_info):
+ self.tempestcommon.VERIFICATION_ID = 'test_uuid'
+ self.tempestcommon.case_name = 'test_case_name'
+ stdout = ['Testscount||2', 'Success||2', 'Skipped||0', 'Failures||0']
+ with mock.patch('functest.opnfv_tests.openstack.tempest.tempest.'
+ 'subprocess.Popen') as mock_popen, \
+ mock.patch('functest.opnfv_tests.openstack.tempest.tempest.'
+ 'ft_utils.check_success_rate') as mock_method, \
+ mock.patch('__builtin__.open', mock.mock_open()):
+ mock_stdout = mock.Mock()
+ attrs = {'stdout': stdout}
+ mock_stdout.configure_mock(**attrs)
+ mock_popen.return_value = mock_stdout
+
+ self.tempestcommon.parse_verifier_result()
+ mock_method.assert_any_call('test_case_name', 100)
+
+ def test_run_missing_create_tempest_dir(self):
+ ret = testcase_base.TestcaseBase.EX_RUN_ERROR
+ with mock.patch('functest.opnfv_tests.openstack.tempest.tempest.'
+ 'os.path.exists', return_value=False), \
+ mock.patch('functest.opnfv_tests.openstack.tempest.tempest.'
+ 'os.makedirs') as mock_os_makedirs, \
+ mock.patch.object(self.tempestcommon, 'create_tempest_resources',
+ return_value=ret):
+ self.assertEqual(self.tempestcommon.run(),
+ ret)
+ self.assertTrue(mock_os_makedirs.called)
+
+ def test_run_missing_configure_tempest(self):
+ ret = testcase_base.TestcaseBase.EX_RUN_ERROR
+ ret_ok = testcase_base.TestcaseBase.EX_OK
+ with mock.patch('functest.opnfv_tests.openstack.tempest.tempest.'
+ 'os.path.exists', return_value=False), \
+ mock.patch('functest.opnfv_tests.openstack.tempest.tempest.'
+ 'os.makedirs') as mock_os_makedirs, \
+ mock.patch.object(self.tempestcommon,
+ 'create_tempest_resources',
+ return_value=ret_ok), \
+ mock.patch('functest.opnfv_tests.openstack.tempest.tempest.'
+ 'conf_utils.configure_tempest',
+ return_value=ret):
+ self.assertEqual(self.tempestcommon.run(),
+ ret)
+ self.assertTrue(mock_os_makedirs.called)
+
+ def test_run_missing_generate_test_list(self):
+ ret = testcase_base.TestcaseBase.EX_RUN_ERROR
+ ret_ok = testcase_base.TestcaseBase.EX_OK
+ with mock.patch('functest.opnfv_tests.openstack.tempest.tempest.'
+ 'os.path.exists', return_value=False), \
+ mock.patch('functest.opnfv_tests.openstack.tempest.tempest.'
+ 'os.makedirs') as mock_os_makedirs, \
+ mock.patch.object(self.tempestcommon, 'create_tempest_resources',
+ return_value=ret_ok), \
+ mock.patch('functest.opnfv_tests.openstack.tempest.tempest.'
+ 'conf_utils.configure_tempest',
+ return_value=ret_ok), \
+ mock.patch.object(self.tempestcommon, 'generate_test_list',
+ return_value=ret):
+ self.assertEqual(self.tempestcommon.run(),
+ ret)
+ self.assertTrue(mock_os_makedirs.called)
+
+ def test_run_missing_apply_tempest_blacklist(self):
+ ret = testcase_base.TestcaseBase.EX_RUN_ERROR
+ ret_ok = testcase_base.TestcaseBase.EX_OK
+ with mock.patch('functest.opnfv_tests.openstack.tempest.tempest.'
+ 'os.path.exists', return_value=False), \
+ mock.patch('functest.opnfv_tests.openstack.tempest.tempest.'
+ 'os.makedirs') as mock_os_makedirs, \
+ mock.patch.object(self.tempestcommon, 'create_tempest_resources',
+ return_value=ret_ok), \
+ mock.patch('functest.opnfv_tests.openstack.tempest.tempest.'
+ 'conf_utils.configure_tempest',
+ return_value=ret_ok), \
+ mock.patch.object(self.tempestcommon, 'generate_test_list',
+ return_value=ret_ok), \
+ mock.patch.object(self.tempestcommon, 'apply_tempest_blacklist',
+ return_value=ret):
+ self.assertEqual(self.tempestcommon.run(),
+ ret)
+ self.assertTrue(mock_os_makedirs.called)
+
+ def test_run_missing_default_criteria_pass(self):
+ ret_ok = testcase_base.TestcaseBase.EX_OK
+ self.tempestcommon.criteria = "PASS"
+ with mock.patch('functest.opnfv_tests.openstack.tempest.tempest.'
+ 'os.path.exists', return_value=False), \
+ mock.patch('functest.opnfv_tests.openstack.tempest.tempest.'
+ 'os.makedirs') as mock_os_makedirs, \
+ mock.patch.object(self.tempestcommon, 'create_tempest_resources',
+ return_value=ret_ok), \
+ mock.patch('functest.opnfv_tests.openstack.tempest.tempest.'
+ 'conf_utils.configure_tempest',
+ return_value=ret_ok), \
+ mock.patch.object(self.tempestcommon, 'generate_test_list',
+ return_value=ret_ok), \
+ mock.patch.object(self.tempestcommon, 'apply_tempest_blacklist',
+ return_value=ret_ok), \
+ mock.patch.object(self.tempestcommon, 'run_verifier_tests'), \
+ mock.patch.object(self.tempestcommon, 'parse_verifier_result'):
+ self.assertEqual(self.tempestcommon.run(),
+ ret_ok)
+ self.assertTrue(mock_os_makedirs.called)
+
+
+if __name__ == "__main__":
+ unittest.main(verbosity=2)
diff --git a/functest/tests/unit/utils/test_openstack_clean.py b/functest/tests/unit/utils/test_openstack_clean.py
index 28eab4f8..15669538 100644
--- a/functest/tests/unit/utils/test_openstack_clean.py
+++ b/functest/tests/unit/utils/test_openstack_clean.py
@@ -20,13 +20,41 @@ class OSCleanTesting(unittest.TestCase):
def _get_instance(self, key):
mock_obj = mock.Mock()
attrs = {'id': 'id' + str(key), 'name': 'name' + str(key),
- 'ip': 'ip' + str(key)}
+ 'ip': 'ip' + str(key), 'status': 'ACTIVE',
+ 'OS-EXT-STS:task_state': '-'}
+ mock_obj.configure_mock(**attrs)
+ return mock_obj
+
+ def _get_instance_deleted(self, key):
+ mock_obj = mock.Mock()
+ attrs = {'id': 'id' + str(key), 'name': 'name' + str(key),
+ 'ip': 'ip' + str(key), 'status': 'DELETED',
+ 'OS-EXT-STS:task_state': '-'}
+ mock_obj.configure_mock(**attrs)
+ return mock_obj
+
+ def _get_instance_deleting(self, key):
+ mock_obj = mock.Mock()
+ attrs = {'id': 'id' + str(key), 'name': 'name' + str(key),
+ 'ip': 'ip' + str(key), 'status': 'BUILD',
+ 'OS-EXT-STS:task_state': 'deleting'}
+ mock_obj.configure_mock(**attrs)
+ return mock_obj
+
+ def _get_instance_other(self, key):
+ mock_obj = mock.Mock()
+ attrs = {'id': 'id' + str(key), 'name': 'name' + str(key),
+ 'ip': 'ip' + str(key), 'status': 'BUILD',
+ 'OS-EXT-STS:task_state': 'networking'}
mock_obj.configure_mock(**attrs)
return mock_obj
def setUp(self):
self.client = mock.Mock()
self.test_list = [self._get_instance(1), self._get_instance(2)]
+ self.deleted_list = [self._get_instance_deleted(5),
+ self._get_instance_deleting(6)]
+ self.other_list = [self._get_instance_other(7)]
self.update_list = {'id1': 'name1', 'id2': 'name2'}
self.remove_list = {'id3': 'name3', 'id4': 'name4'}
self.test_dict_list = [{'id': 'id1', 'name': 'name1', 'ip': 'ip1',
@@ -77,6 +105,33 @@ class OSCleanTesting(unittest.TestCase):
" '\s*\S+'"
" ..."))
+ @mock.patch('functest.utils.openstack_clean.logger.debug')
+ def test_remove_instances_pending_delete_success(self, mock_logger_debug):
+ with mock.patch('functest.utils.openstack_clean.os_utils'
+ '.get_instances', return_value=self.deleted_list), \
+ mock.patch('functest.utils.openstack_clean.os_utils'
+ '.delete_instance', return_value=True):
+ openstack_clean.remove_instances(self.client, self.remove_list)
+ mock_logger_debug.assert_any_call("Removing Nova instances...")
+ mock_logger_debug.test_utils.RegexMatch("Removing"
+ " instance"
+ " '\s*\S+'"
+ " ...").assert_not_called()
+
+ @mock.patch('functest.utils.openstack_clean.logger.debug')
+ def test_remove_instances_other_delete_success(self, mock_logger_debug):
+ with mock.patch('functest.utils.openstack_clean.os_utils'
+ '.get_instances', return_value=self.other_list), \
+ mock.patch('functest.utils.openstack_clean.os_utils'
+ '.delete_instance', return_value=True):
+ openstack_clean.remove_instances(self.client, self.remove_list)
+ mock_logger_debug.assert_any_call("Removing Nova instances...")
+ mock_logger_debug.assert_any_call(" > Request sent.")
+ mock_logger_debug.assert_any_call(test_utils.RegexMatch("Removing"
+ " instance"
+ " '\s*\S+'"
+ " ..."))
+
@mock.patch('functest.utils.openstack_clean.logger.error')
@mock.patch('functest.utils.openstack_clean.logger.debug')
def test_remove_instances_delete_failed(self, mock_logger_debug,