summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
-rw-r--r--docs/configguide/index.rst122
-rwxr-xr-xfunctest/ci/check_os.sh6
-rwxr-xr-xfunctest/ci/exec_test.sh4
-rw-r--r--functest/opnfv_tests/openstack/rally/rally.py3
-rw-r--r--functest/opnfv_tests/openstack/tempest/conf_utils.py2
-rwxr-xr-xfunctest/opnfv_tests/sdn/odl/odl.py2
-rw-r--r--functest/tests/unit/odl/test_odl.py14
-rw-r--r--functest/tests/unit/opnfv_tests/__init__.py0
-rw-r--r--functest/tests/unit/opnfv_tests/openstack/__init__.py0
-rw-r--r--functest/tests/unit/opnfv_tests/openstack/rally/__init__.py0
-rw-r--r--functest/tests/unit/opnfv_tests/openstack/rally/test_rally.py391
-rw-r--r--functest/tests/unit/utils/test_functest_utils.py34
-rw-r--r--functest/utils/functest_utils.py11
-rwxr-xr-xrun_unit_tests.sh1
-rw-r--r--test-requirements.txt1
15 files changed, 504 insertions, 87 deletions
diff --git a/docs/configguide/index.rst b/docs/configguide/index.rst
index 58e8a1343..f12739e31 100644
--- a/docs/configguide/index.rst
+++ b/docs/configguide/index.rst
@@ -37,67 +37,69 @@ High level architecture
The high level architecture of Functest within OPNFV can be described as
follows::
- CIMC/Lights+out management Admin Private Public Storage
+ CIMC/Lights+out management Admin Mgmt/API Public Storage Private
PXE
- + + + + +
- | | | | |
- | +----------------------------+ | | | |
- | | | | | | |
- +-----+ Jumphost | | | | |
- | | +--------+ | | |
- | | | | | | |
- | | +--------------------+ | | | | |
- | | | | | | | | |
- | | | Tools | +----------------+ | |
- | | | - Rally | | | | | |
- | | | - Robot | | | | | |
- | | | - TestON | | | | | |
- | | | | | | | | |
- | | | Testcases | | | | | |
- | | | - VIM | | | | | |
- | | | -- healthcheck | | | | | |
- | | | -- vPing_ssh | | | | | |
- | | | -- vPing_userdata | | | | | |
- | | | -- SNAPS_cases | | | | | |
- | | | -- Tempest_smoke | | | | | |
- | | | -- Rally_sanity | | | | | |
- | | | -- Tempest_full | | | | | |
- | | | -- Rally_full | | | | | |
- | | | | | | | | |
- | | | - SDN Controller | | | | | |
- | | | -- odl | | | | | |
- | | | -- onos | | | | | |
- | | | | | | | | |
- | | | - Features | | | | | |
- | | | | | | | | |
- | | | - VNF | | | | | |
- | | | | | | | | |
- | | +--------------------+ | | | | |
- | | Functest Docker +-------------------------+ |
- | | | | | | |
- | | +---------------------------------+
- | | | | | | |
- | +----------------------------+ | | | |
- | | | | |
- | +----------------+ | | | |
- | | 1 | | | | |
- +----+ +--------------+-+ | | | |
- | | | 2 | | | | |
- | | | +--------------+-+ | | | |
- | | | | 3 | | | | |
- | | | | +--------------+-+ | | | |
- | | | | | 4 | | | | |
- | +-+ | | +--------------+-+ | | | |
- | | | | | 5 +-------------+ | | |
- | +-+ | | nodes for | | | | |
- | | | | deploying +---------------------+ | |
- | +-+ | OPNFV | | | | |
- | | | +------------------------------+ |
- | +-+ SUT | | | | |
- | | +--------------------------------------+
- | +----------------+ | | | |
- | | | | |
- + + + + +
+ + + + + + +
+ | | | | | |
+ | +----------------------------+ | | | | |
+ | | | | | | | |
+ +-----+ Jumphost | | | | | |
+ | | +--------+ | | | |
+ | | | | | | | |
+ | | +--------------------+ | | | | | |
+ | | | | | | | | | |
+ | | | Tools | +----------------+ | | |
+ | | | - Rally | | | | | | |
+ | | | - Robot | | | | | | |
+ | | | - TestON | | | | | | |
+ | | | | |-------------------------+ | |
+ | | | Testcases | | | | | | |
+ | | | - VIM | | | | | | |
+ | | | -- healthcheck | | | | | | |
+ | | | -- vPing_ssh | | | | | | |
+ | | | -- vPing_userdata | | | | | | |
+ | | | -- SNAPS_cases | | | | | | |
+ | | | -- Tempest_smoke | | | | | | |
+ | | | -- Rally_sanity | | | | | | |
+ | | | -- Tempest_full | | | | | | |
+ | | | -- Rally_full | | | | | | |
+ | | | | | | | | | |
+ | | | - SDN Controller | | | | | | |
+ | | | -- odl | | | | | | |
+ | | | -- onos | | | | | | |
+ | | | | | | | | | |
+ | | | - Features | | | | | | |
+ | | | | | | | | | |
+ | | | - VNF | | | | | | |
+ | | | | | | | | | |
+ | | +--------------------+ | | | | | |
+ | | Functest Docker + | | | | |
+ | | | | | | | |
+ | | | | | | | |
+ | | | | | | | |
+ | +----------------------------+ | | | | |
+ | | | | | |
+ | +----------------+ | | | | |
+ | | 1 | | | | | |
+ +----+ +--------------+-+ | | | | |
+ | | | 2 | | | | | |
+ | | | +--------------+-+ | | | | |
+ | | | | 3 | | | | | |
+ | | | | +--------------+-+ | | | | |
+ | | | | | 4 | | | | | |
+ | +-+ | | +--------------+-+ | | | | |
+ | | | | | 5 +-------------+ | | | |
+ | +-+ | | nodes for | | | | | |
+ | | | | deploying +---------------------+ | | |
+ | +-+ | OPNFV | | | | | |
+ | | | +------------------------------+ | |
+ | +-+ SUT | | | | | |
+ | | +--------------------------------------+ |
+ | | | | | | | |
+ | | +----------------------------------------------+
+ | +----------------+ | | | | |
+ | | | | | |
+ + + + + + +
SUT = System Under Test
All the libraries and dependencies needed by all of the Functest tools
diff --git a/functest/ci/check_os.sh b/functest/ci/check_os.sh
index e2471026c..b875a173e 100755
--- a/functest/ci/check_os.sh
+++ b/functest/ci/check_os.sh
@@ -57,11 +57,11 @@ echo " ...OK"
echo "Checking OpenStack basic services:"
-commands=('openstack endpoint list' 'nova list' 'neutron net-list' \
- 'glance image-list' 'cinder list')
+commands=('openstack endpoint list' 'openstack server list' 'openstack network list' \
+ 'openstack image list' 'openstack volume list')
for cmd in "${commands[@]}"
do
- service=$(echo $cmd | awk '{print $1}')
+ service=$(echo $cmd | awk '{print $1, $2}')
echo ">>Checking $service service..."
$cmd &>/dev/null
result=$?
diff --git a/functest/ci/exec_test.sh b/functest/ci/exec_test.sh
index b288fe367..6a2b55a29 100755
--- a/functest/ci/exec_test.sh
+++ b/functest/ci/exec_test.sh
@@ -54,6 +54,10 @@ function odl_tests(){
odl_ip=$SDN_CONTROLLER_IP
odl_port=8081
odl_restport=8081
+ elif [ "$INSTALLER_TYPE" == "netvirt" ]; then
+ odl_ip=$SDN_CONTROLLER_IP
+ odl_port=8081
+ odl_restport=8081
elif [ "$INSTALLER_TYPE" == "joid" ]; then
odl_ip=$SDN_CONTROLLER
elif [ "$INSTALLER_TYPE" == "compass" ]; then
diff --git a/functest/opnfv_tests/openstack/rally/rally.py b/functest/opnfv_tests/openstack/rally/rally.py
index f984c3689..16a872fc2 100644
--- a/functest/opnfv_tests/openstack/rally/rally.py
+++ b/functest/opnfv_tests/openstack/rally/rally.py
@@ -66,6 +66,7 @@ class RallyBase(testcase_base.TestcaseBase):
self.cinder_client = os_utils.get_cinder_client()
self.network_dict = {}
self.volume_type = None
+ self.smoke = None
def _build_task_args(self, test_file_name):
task_args = {'service_list': [test_file_name]}
@@ -287,7 +288,7 @@ class RallyBase(testcase_base.TestcaseBase):
cmd_line = ("rally task validate "
"--task {0} "
"--task-args \"{1}\""
- .format(task_file, self.__build_task_args(test_name)))
+ .format(task_file, self._build_task_args(test_name)))
logger.debug('running command line: {}'.format(cmd_line))
p = subprocess.Popen(cmd_line, stdout=subprocess.PIPE,
stderr=subprocess.STDOUT, shell=True)
diff --git a/functest/opnfv_tests/openstack/tempest/conf_utils.py b/functest/opnfv_tests/openstack/tempest/conf_utils.py
index 0c9064a14..91a5bb4b0 100644
--- a/functest/opnfv_tests/openstack/tempest/conf_utils.py
+++ b/functest/opnfv_tests/openstack/tempest/conf_utils.py
@@ -208,7 +208,7 @@ def configure_tempest_multisite(deployment_dir):
# cmd = ("openstack endpoint show kingbird | grep publicurl |"
# "awk '{print $4}' | awk -F '/' '{print $4}'")
# kingbird_api_version = os.popen(cmd).read()
- kingbird_api_version = os_utils.get_endpoint(service_type='kingbird')
+ kingbird_api_version = os_utils.get_endpoint(service_type='multisite')
if CI_INSTALLER_TYPE == 'fuel':
# For MOS based setup, the service is accessible
diff --git a/functest/opnfv_tests/sdn/odl/odl.py b/functest/opnfv_tests/sdn/odl/odl.py
index 9502be93f..250759572 100755
--- a/functest/opnfv_tests/sdn/odl/odl.py
+++ b/functest/opnfv_tests/sdn/odl/odl.py
@@ -164,7 +164,7 @@ class ODLTests(testcase_base.TestcaseBase):
kwargs['ospassword'] = os.environ['OS_PASSWORD']
if installer_type == 'fuel':
kwargs['odlwebport'] = '8282'
- elif installer_type == 'apex':
+ elif installer_type == 'apex' or installer_type == 'netvirt':
kwargs['odlip'] = os.environ['SDN_CONTROLLER_IP']
kwargs['odlwebport'] = '8081'
kwargs['odlrestconfport'] = '8081'
diff --git a/functest/tests/unit/odl/test_odl.py b/functest/tests/unit/odl/test_odl.py
index 810b35920..8f2a5d7e4 100644
--- a/functest/tests/unit/odl/test_odl.py
+++ b/functest/tests/unit/odl/test_odl.py
@@ -443,6 +443,20 @@ class ODLTesting(unittest.TestCase):
odlip=self._sdn_controller_ip, odlwebport='8081',
odlrestconfport='8081')
+ def test_run_netvirt_missing_sdn_controller_ip(self):
+ with mock.patch('functest.utils.openstack_utils.get_endpoint',
+ side_effect=self._fake_url_for):
+ os.environ["INSTALLER_TYPE"] = "netvirt"
+ self.assertEqual(self.test.run(),
+ testcase_base.TestcaseBase.EX_RUN_ERROR)
+
+ def test_run_netvirt(self):
+ os.environ["SDN_CONTROLLER_IP"] = self._sdn_controller_ip
+ os.environ["INSTALLER_TYPE"] = "netvirt"
+ self._test_run(testcase_base.TestcaseBase.EX_OK,
+ odlip=self._sdn_controller_ip, odlwebport='8081',
+ odlrestconfport='8081')
+
def test_run_joid_missing_sdn_controller(self):
with mock.patch('functest.utils.openstack_utils.get_endpoint',
side_effect=self._fake_url_for):
diff --git a/functest/tests/unit/opnfv_tests/__init__.py b/functest/tests/unit/opnfv_tests/__init__.py
new file mode 100644
index 000000000..e69de29bb
--- /dev/null
+++ b/functest/tests/unit/opnfv_tests/__init__.py
diff --git a/functest/tests/unit/opnfv_tests/openstack/__init__.py b/functest/tests/unit/opnfv_tests/openstack/__init__.py
new file mode 100644
index 000000000..e69de29bb
--- /dev/null
+++ b/functest/tests/unit/opnfv_tests/openstack/__init__.py
diff --git a/functest/tests/unit/opnfv_tests/openstack/rally/__init__.py b/functest/tests/unit/opnfv_tests/openstack/rally/__init__.py
new file mode 100644
index 000000000..e69de29bb
--- /dev/null
+++ b/functest/tests/unit/opnfv_tests/openstack/rally/__init__.py
diff --git a/functest/tests/unit/opnfv_tests/openstack/rally/test_rally.py b/functest/tests/unit/opnfv_tests/openstack/rally/test_rally.py
new file mode 100644
index 000000000..ad39be48c
--- /dev/null
+++ b/functest/tests/unit/opnfv_tests/openstack/rally/test_rally.py
@@ -0,0 +1,391 @@
+#!/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 json
+import logging
+import os
+import unittest
+
+import mock
+
+from functest.core import testcase_base
+from functest.opnfv_tests.openstack.rally import rally
+from functest.utils.constants import CONST
+
+
+class OSRallyTesting(unittest.TestCase):
+
+ logging.disable(logging.CRITICAL)
+
+ def setUp(self):
+ self.nova_client = mock.Mock()
+ self.neutron_client = mock.Mock()
+ self.cinder_client = mock.Mock()
+ with mock.patch('functest.opnfv_tests.openstack.rally.rally.'
+ 'os_utils.get_nova_client',
+ return_value=self.nova_client), \
+ mock.patch('functest.opnfv_tests.openstack.rally.rally.'
+ 'os_utils.get_neutron_client',
+ return_value=self.neutron_client), \
+ mock.patch('functest.opnfv_tests.openstack.rally.rally.'
+ 'os_utils.get_cinder_client',
+ return_value=self.cinder_client):
+ self.rally_base = rally.RallyBase()
+ self.rally_base.network_dict['net_id'] = 'test_net_id'
+ self.polling_iter = 2
+
+ def test_build_task_args_missing_floating_network(self):
+ CONST.OS_AUTH_URL = None
+ with mock.patch('functest.opnfv_tests.openstack.rally.rally.'
+ 'os_utils.get_external_net',
+ return_value=None):
+ task_args = self.rally_base._build_task_args('test_file_name')
+ self.assertEqual(task_args['floating_network'], '')
+
+ def test_build_task_args_missing_net_id(self):
+ CONST.OS_AUTH_URL = None
+ self.rally_base.network_dict['net_id'] = ''
+ with mock.patch('functest.opnfv_tests.openstack.rally.rally.'
+ 'os_utils.get_external_net',
+ return_value='test_floating_network'):
+ task_args = self.rally_base._build_task_args('test_file_name')
+ self.assertEqual(task_args['netid'], '')
+
+ def test_build_task_args_missing_auth_url(self):
+ CONST.OS_AUTH_URL = None
+ with mock.patch('functest.opnfv_tests.openstack.rally.rally.'
+ 'os_utils.get_external_net',
+ return_value='test_floating_network'):
+ task_args = self.rally_base._build_task_args('test_file_name')
+ self.assertEqual(task_args['request_url'], '')
+
+ def check_scenario_file(self, value):
+ yaml_file = 'opnfv-{}.yaml'.format('test_file_name')
+ if yaml_file in value:
+ return False
+ return True
+
+ def test_prepare_test_list_missing_scenario_file(self):
+ with mock.patch('functest.opnfv_tests.openstack.rally.rally.'
+ 'os.path.exists',
+ side_effect=self.check_scenario_file), \
+ self.assertRaises(Exception):
+ self.rally_base._prepare_test_list('test_file_name')
+
+ def check_temp_dir(self, value):
+ yaml_file = 'opnfv-{}.yaml'.format('test_file_name')
+ if yaml_file in value:
+ return True
+ return False
+
+ def test_prepare_test_list_missing_temp_dir(self):
+ with mock.patch('functest.opnfv_tests.openstack.rally.rally.'
+ 'os.path.exists',
+ side_effect=self.check_temp_dir), \
+ mock.patch('functest.opnfv_tests.openstack.rally.rally.'
+ 'os.makedirs') as mock_os_makedir, \
+ mock.patch.object(self.rally_base, 'apply_blacklist',
+ return_value=mock.Mock()) as mock_method:
+ yaml_file = 'opnfv-{}.yaml'.format('test_file_name')
+ ret_val = os.path.join(self.rally_base.TEMP_DIR, yaml_file)
+ self.assertEqual(self.rally_base.
+ _prepare_test_list('test_file_name'),
+ ret_val)
+ self.assertTrue(mock_method.called)
+ self.assertTrue(mock_os_makedir.called)
+
+ def test_get_task_id_default(self):
+ cmd_raw = 'Task 1: started'
+ self.assertEqual(self.rally_base.get_task_id(cmd_raw),
+ '1')
+
+ def test_get_task_id_missing_id(self):
+ cmd_raw = ''
+ self.assertEqual(self.rally_base.get_task_id(cmd_raw),
+ None)
+
+ def test_task_succeed_fail(self):
+ json_raw = json.dumps([None])
+ self.assertEqual(self.rally_base.task_succeed(json_raw),
+ False)
+ json_raw = json.dumps([{'result': [{'error': ['test_error']}]}])
+ self.assertEqual(self.rally_base.task_succeed(json_raw),
+ False)
+
+ def test_task_succeed_success(self):
+ json_raw = json.dumps('')
+ self.assertEqual(self.rally_base.task_succeed(json_raw),
+ True)
+
+ def polling(self):
+ if self.polling_iter == 0:
+ return "something"
+ self.polling_iter -= 1
+ return None
+
+ def test_get_cmd_output(self):
+ proc = mock.Mock()
+ attrs = {'poll.side_effect': self.polling,
+ 'stdout.readline.return_value': 'line'}
+ proc.configure_mock(**attrs)
+ self.assertEqual(self.rally_base.get_cmd_output(proc),
+ 'lineline')
+
+ def test_excl_scenario_default(self):
+ CONST.INSTALLER_TYPE = 'test_installer'
+ CONST.DEPLOY_SCENARIO = 'test_scenario'
+ dic = {'scenario': [{'scenarios': ['test_scenario'],
+ 'installers': ['test_installer'],
+ 'tests': ['test']}]}
+ with mock.patch('__builtin__.open', mock.mock_open()), \
+ mock.patch('functest.opnfv_tests.openstack.rally.rally.'
+ 'yaml.safe_load',
+ return_value=dic):
+ self.assertEqual(self.rally_base.excl_scenario(),
+ ['test'])
+
+ def test_excl_scenario_exception(self):
+ with mock.patch('__builtin__.open', side_effect=Exception):
+ self.assertEqual(self.rally_base.excl_scenario(),
+ [])
+
+ def test_excl_func_default(self):
+ CONST.INSTALLER_TYPE = 'test_installer'
+ CONST.DEPLOY_SCENARIO = 'test_scenario'
+ dic = {'functionality': [{'functions': ['no_live_migration'],
+ 'tests': ['test']}]}
+ with mock.patch('__builtin__.open', mock.mock_open()), \
+ mock.patch('functest.opnfv_tests.openstack.rally.rally.'
+ 'yaml.safe_load',
+ return_value=dic), \
+ mock.patch.object(self.rally_base, 'live_migration_supported',
+ return_value=False):
+ self.assertEqual(self.rally_base.excl_func(),
+ ['test'])
+
+ def test_excl_func_exception(self):
+ with mock.patch('__builtin__.open', side_effect=Exception):
+ self.assertEqual(self.rally_base.excl_func(),
+ [])
+
+ def test_file_is_empty_default(self):
+ mock_obj = mock.Mock()
+ attrs = {'st_size': 10}
+ mock_obj.configure_mock(**attrs)
+ with mock.patch('functest.opnfv_tests.openstack.rally.rally.'
+ 'os.stat',
+ return_value=mock_obj):
+ self.assertEqual(self.rally_base.file_is_empty('test_file_name'),
+ False)
+
+ def test_file_is_empty_exception(self):
+ with mock.patch('functest.opnfv_tests.openstack.rally.rally.'
+ 'os.stat',
+ side_effect=Exception):
+ self.assertEqual(self.rally_base.file_is_empty('test_file_name'),
+ True)
+
+ def test_run_task_missing_task_file(self):
+ with mock.patch('functest.opnfv_tests.openstack.rally.rally.'
+ 'os.path.exists',
+ return_value=False), \
+ self.assertRaises(Exception):
+ self.rally_base._run_task('test_name')
+
+ @mock.patch('functest.opnfv_tests.openstack.rally.rally.logger.info')
+ def test_run_task_no_tests_for_scenario(self, mock_logger_info):
+ with mock.patch('functest.opnfv_tests.openstack.rally.rally.'
+ 'os.path.exists',
+ return_value=True), \
+ mock.patch.object(self.rally_base, '_prepare_test_list',
+ return_value='test_file_name'), \
+ mock.patch.object(self.rally_base, 'file_is_empty',
+ return_value=True):
+ self.rally_base._run_task('test_name')
+ str = 'No tests for scenario "test_name"'
+ mock_logger_info.assert_any_call(str)
+
+ @mock.patch('functest.opnfv_tests.openstack.rally.rally.logger.error')
+ def test_run_task_taskid_missing(self, mock_logger_error):
+ with mock.patch('functest.opnfv_tests.openstack.rally.rally.'
+ 'os.path.exists',
+ return_value=True), \
+ mock.patch.object(self.rally_base, '_prepare_test_list',
+ return_value='test_file_name'), \
+ mock.patch.object(self.rally_base, 'file_is_empty',
+ return_value=False), \
+ mock.patch.object(self.rally_base, '_build_task_args',
+ return_value={}), \
+ mock.patch('functest.opnfv_tests.openstack.rally.rally.'
+ 'subprocess.Popen'), \
+ mock.patch.object(self.rally_base, '_get_output',
+ return_value=mock.Mock()), \
+ mock.patch.object(self.rally_base, 'get_task_id',
+ return_value=None), \
+ mock.patch.object(self.rally_base, 'get_cmd_output',
+ return_value=''):
+ self.rally_base._run_task('test_name')
+ str = 'Failed to retrieve task_id, validating task...'
+ mock_logger_error.assert_any_call(str)
+
+ @mock.patch('functest.opnfv_tests.openstack.rally.rally.logger.info')
+ @mock.patch('functest.opnfv_tests.openstack.rally.rally.logger.error')
+ def test_run_task_default(self, mock_logger_error,
+ mock_logger_info):
+ popen = mock.Mock()
+ attrs = {'read.return_value': 'json_result'}
+ popen.configure_mock(**attrs)
+
+ with mock.patch('functest.opnfv_tests.openstack.rally.rally.'
+ 'os.path.exists',
+ return_value=True), \
+ mock.patch.object(self.rally_base, '_prepare_test_list',
+ return_value='test_file_name'), \
+ mock.patch.object(self.rally_base, 'file_is_empty',
+ return_value=False), \
+ mock.patch.object(self.rally_base, '_build_task_args',
+ return_value={}), \
+ mock.patch('functest.opnfv_tests.openstack.rally.rally.'
+ 'subprocess.Popen'), \
+ mock.patch.object(self.rally_base, '_get_output',
+ return_value=mock.Mock()), \
+ mock.patch.object(self.rally_base, 'get_task_id',
+ return_value='1'), \
+ mock.patch.object(self.rally_base, 'get_cmd_output',
+ return_value=''), \
+ mock.patch('functest.opnfv_tests.openstack.rally.rally.'
+ 'os.makedirs'), \
+ mock.patch('functest.opnfv_tests.openstack.rally.rally.'
+ 'os.popen',
+ return_value=popen), \
+ mock.patch('__builtin__.open', mock.mock_open()), \
+ mock.patch.object(self.rally_base, 'task_succeed',
+ return_value=True):
+ self.rally_base._run_task('test_name')
+ str = 'Test scenario: "test_name" OK.\n'
+ mock_logger_info.assert_any_call(str)
+
+ def test_prepare_env_testname_invalid(self):
+ self.rally_base.TESTS = ['test1', 'test2']
+ self.rally_base.test_name = 'test'
+ with self.assertRaises(Exception):
+ self.rally_base._prepare_env()
+
+ def test_prepare_env_volume_creation_failed(self):
+ self.rally_base.TESTS = ['test1', 'test2']
+ self.rally_base.test_name = 'test1'
+ volume_type = None
+ with mock.patch('functest.opnfv_tests.openstack.rally.rally.'
+ 'os_utils.list_volume_types',
+ return_value=None), \
+ mock.patch('functest.opnfv_tests.openstack.rally.rally.'
+ 'os_utils.create_volume_type',
+ return_value=volume_type), \
+ self.assertRaises(Exception):
+ self.rally_base._prepare_env()
+
+ def test_prepare_env_image_missing(self):
+ self.rally_base.TESTS = ['test1', 'test2']
+ self.rally_base.test_name = 'test1'
+ volume_type = mock.Mock()
+ image_id = None
+ with mock.patch('functest.opnfv_tests.openstack.rally.rally.'
+ 'os_utils.list_volume_types',
+ return_value=None), \
+ mock.patch('functest.opnfv_tests.openstack.rally.rally.'
+ 'os_utils.create_volume_type',
+ return_value=volume_type), \
+ mock.patch('functest.opnfv_tests.openstack.rally.rally.'
+ 'os_utils.get_or_create_image',
+ return_value=(True, image_id)), \
+ self.assertRaises(Exception):
+ self.rally_base._prepare_env()
+
+ def test_prepare_env_image_shared_network_creation_failed(self):
+ self.rally_base.TESTS = ['test1', 'test2']
+ self.rally_base.test_name = 'test1'
+ volume_type = mock.Mock()
+ image_id = 'image_id'
+ network_dict = None
+ with mock.patch('functest.opnfv_tests.openstack.rally.rally.'
+ 'os_utils.list_volume_types',
+ return_value=None), \
+ mock.patch('functest.opnfv_tests.openstack.rally.rally.'
+ 'os_utils.create_volume_type',
+ return_value=volume_type), \
+ mock.patch('functest.opnfv_tests.openstack.rally.rally.'
+ 'os_utils.get_or_create_image',
+ return_value=(True, image_id)), \
+ mock.patch('functest.opnfv_tests.openstack.rally.rally.'
+ 'os_utils.create_shared_network_full',
+ return_value=network_dict), \
+ self.assertRaises(Exception):
+ self.rally_base._prepare_env()
+
+ def test_run_tests_all(self):
+ self.rally_base.TESTS = ['test1', 'test2']
+ self.rally_base.test_name = 'all'
+ with mock.patch.object(self.rally_base, '_run_task',
+ return_value=mock.Mock()):
+ self.rally_base._run_tests()
+ self.rally_base._run_task.assert_any_call('test1')
+ self.rally_base._run_task.assert_any_call('test2')
+
+ def test_run_tests_default(self):
+ self.rally_base.TESTS = ['test1', 'test2']
+ self.rally_base.test_name = 'test1'
+ with mock.patch.object(self.rally_base, '_run_task',
+ return_value=mock.Mock()):
+ self.rally_base._run_tests()
+ self.rally_base._run_task.assert_any_call('test1')
+
+ @mock.patch('functest.opnfv_tests.openstack.rally.rally.logger.info')
+ def test_generate_report(self, mock_logger_info):
+ summary = [{'test_name': 'test_name',
+ 'overall_duration': 5,
+ 'nb_tests': 3,
+ 'success': 5}]
+ self.rally_base.summary = summary
+ with mock.patch('functest.opnfv_tests.openstack.rally.rally.'
+ 'ft_utils.check_success_rate',
+ return_value='criteria'):
+ self.rally_base._generate_report()
+ self.assertTrue(mock_logger_info.called)
+
+ def test_clean_up_default(self):
+ self.rally_base.volume_type = mock.Mock()
+ self.rally_base.cinder_client = mock.Mock()
+ self.rally_base.image_exists = False
+ self.rally_base.image_id = 1
+ self.rally_base.nova_client = mock.Mock()
+ with mock.patch('functest.opnfv_tests.openstack.rally.rally.'
+ 'os_utils.delete_volume_type') as mock_vol_method, \
+ mock.patch('functest.opnfv_tests.openstack.rally.rally.'
+ 'os_utils.delete_glance_image') as mock_glance_method:
+ self.rally_base._clean_up()
+ mock_vol_method.assert_any_call(self.rally_base.cinder_client,
+ self.rally_base.volume_type)
+ mock_glance_method.assert_any_call(self.rally_base.nova_client,
+ 1)
+
+ def test_run_default(self):
+ with mock.patch.object(self.rally_base, '_prepare_env'), \
+ mock.patch.object(self.rally_base, '_run_tests'), \
+ mock.patch.object(self.rally_base, '_generate_report'), \
+ mock.patch.object(self.rally_base, '_clean_up'):
+ self.assertEqual(self.rally_base.run(),
+ testcase_base.TestcaseBase.EX_OK)
+
+ def test_run_exception(self):
+ with mock.patch.object(self.rally_base, '_prepare_env',
+ side_effect=Exception):
+ self.assertEqual(self.rally_base.run(),
+ testcase_base.TestcaseBase.EX_RUN_ERROR)
+
+
+if __name__ == "__main__":
+ unittest.main(verbosity=2)
diff --git a/functest/tests/unit/utils/test_functest_utils.py b/functest/tests/unit/utils/test_functest_utils.py
index ce9086a7a..bb8360119 100644
--- a/functest/tests/unit/utils/test_functest_utils.py
+++ b/functest/tests/unit/utils/test_functest_utils.py
@@ -131,15 +131,17 @@ class FunctestUtilsTesting(unittest.TestCase):
self.assertEqual(functest_utils.get_installer_type(),
self.installer)
- @mock.patch('functest.utils.functest_utils.logger.error')
- def test_get_scenario_failed(self, mock_logger_error):
+ @mock.patch('functest.utils.functest_utils.logger.info')
+ def test_get_scenario_failed(self, mock_logger_info):
with mock.patch.dict(os.environ,
{},
clear=True):
self.assertEqual(functest_utils.get_scenario(),
- "Unknown_scenario")
- mock_logger_error.assert_called_once_with("Impossible to retrieve"
- " the scenario")
+ "os-nosdn-nofeature-noha")
+ mock_logger_info.assert_called_once_with("Impossible to retrieve "
+ "the scenario.Use "
+ "default "
+ "os-nosdn-nofeature-noha")
def test_get_scenario_default(self):
with mock.patch.dict(os.environ,
@@ -158,17 +160,17 @@ class FunctestUtilsTesting(unittest.TestCase):
mock_get_build_tag.return_value = "unknown_build_tag"
self.assertEqual(functest_utils.get_version(), "unknown")
- @mock.patch('functest.utils.functest_utils.logger.error')
- def test_get_pod_name_failed(self, mock_logger_error):
+ @mock.patch('functest.utils.functest_utils.logger.info')
+ def test_get_pod_name_failed(self, mock_logger_info):
with mock.patch.dict(os.environ,
{},
clear=True):
self.assertEqual(functest_utils.get_pod_name(),
"unknown-pod")
- mock_logger_error.assert_called_once_with("Unable to retrieve "
- "the POD name from "
- "environment. Using "
- "pod name 'unknown-pod'")
+ mock_logger_info.assert_called_once_with("Unable to retrieve "
+ "the POD name from "
+ "environment. Using "
+ "pod name 'unknown-pod'")
def test_get_pod_name_default(self):
with mock.patch.dict(os.environ,
@@ -177,15 +179,15 @@ class FunctestUtilsTesting(unittest.TestCase):
self.assertEqual(functest_utils.get_pod_name(),
self.node_name)
- @mock.patch('functest.utils.functest_utils.logger.error')
- def test_get_build_tag_failed(self, mock_logger_error):
+ @mock.patch('functest.utils.functest_utils.logger.info')
+ def test_get_build_tag_failed(self, mock_logger_info):
with mock.patch.dict(os.environ,
{},
clear=True):
self.assertEqual(functest_utils.get_build_tag(),
- "unknown_build_tag")
- mock_logger_error.assert_called_once_with("Impossible to retrieve"
- " the build tag")
+ "none")
+ mock_logger_info.assert_called_once_with("Impossible to retrieve"
+ " the build tag")
def test_get_build_tag_default(self):
with mock.patch.dict(os.environ,
diff --git a/functest/utils/functest_utils.py b/functest/utils/functest_utils.py
index 2bf87a05c..040554642 100644
--- a/functest/utils/functest_utils.py
+++ b/functest/utils/functest_utils.py
@@ -96,8 +96,9 @@ def get_scenario():
try:
scenario = os.environ['DEPLOY_SCENARIO']
except KeyError:
- logger.error("Impossible to retrieve the scenario")
- scenario = "Unknown_scenario"
+ logger.info("Impossible to retrieve the scenario."
+ "Use default os-nosdn-nofeature-noha")
+ scenario = "os-nosdn-nofeature-noha"
return scenario
@@ -128,7 +129,7 @@ def get_pod_name():
try:
return os.environ['NODE_NAME']
except KeyError:
- logger.error(
+ logger.info(
"Unable to retrieve the POD name from environment. " +
"Using pod name 'unknown-pod'")
return "unknown-pod"
@@ -141,8 +142,8 @@ def get_build_tag():
try:
build_tag = os.environ['BUILD_TAG']
except KeyError:
- logger.error("Impossible to retrieve the build tag")
- build_tag = "unknown_build_tag"
+ logger.info("Impossible to retrieve the build tag")
+ build_tag = "none"
return build_tag
diff --git a/run_unit_tests.sh b/run_unit_tests.sh
index 5167d78c6..606aedcd5 100755
--- a/run_unit_tests.sh
+++ b/run_unit_tests.sh
@@ -36,6 +36,7 @@ nosetests --with-xunit \
--cover-package=functest.core.testcase_base \
--cover-package=functest.opnfv_tests.sdn.odl.odl \
--cover-package=functest.utils \
+ --cover-package=functest.opnfv_tests.openstack.rally \
--cover-xml \
--cover-html \
--log-config=$(pwd)/functest/tests/unit/test_logging.ini \
diff --git a/test-requirements.txt b/test-requirements.txt
index 2bf297ba8..96b276780 100644
--- a/test-requirements.txt
+++ b/test-requirements.txt
@@ -9,6 +9,7 @@ click==6.6
coverage==4.1
dnspython==1.15.0
gitpython==1.0.1
+iniparse==0.4
mock==1.3.0
nose==1.3.7
python-ceilometerclient==2.6.2