aboutsummaryrefslogtreecommitdiffstats
path: root/functest/tests/unit/openstack/rally/test_rally.py
diff options
context:
space:
mode:
authorCedric Ollivier <cedric.ollivier@orange.com>2017-09-15 08:39:02 +0000
committerGerrit Code Review <gerrit@opnfv.org>2017-09-15 08:39:02 +0000
commit59320641c1ee0639d4e112ac6a6fa4011f4067df (patch)
treeeef5f710bf92be0841c69d0d11251e1222098835 /functest/tests/unit/openstack/rally/test_rally.py
parent3945146d7448ba57b79dd4c350ce7f8881661a8e (diff)
parentd11b1b3a50af1c8ff86f1f6b269fadffe32c34ab (diff)
Merge "Refactor resource creation and cleanup in rally"
Diffstat (limited to 'functest/tests/unit/openstack/rally/test_rally.py')
-rw-r--r--functest/tests/unit/openstack/rally/test_rally.py95
1 files changed, 57 insertions, 38 deletions
diff --git a/functest/tests/unit/openstack/rally/test_rally.py b/functest/tests/unit/openstack/rally/test_rally.py
index 95222ca8..63c0192b 100644
--- a/functest/tests/unit/openstack/rally/test_rally.py
+++ b/functest/tests/unit/openstack/rally/test_rally.py
@@ -16,35 +16,31 @@ from functest.core import testcase
from functest.opnfv_tests.openstack.rally import rally
from functest.utils.constants import CONST
+from snaps.openstack.os_credentials import OSCreds
+
class OSRallyTesting(unittest.TestCase):
- @mock.patch('functest.opnfv_tests.openstack.rally.rally.os_utils.'
- 'get_nova_client', return_value=mock.Mock())
- @mock.patch('functest.opnfv_tests.openstack.rally.rally.os_utils.'
- 'get_neutron_client', return_value=mock.Mock())
- def setUp(self, mock_func1, mock_func2):
- self.rally_base = rally.RallyBase()
- self.rally_base.network_dict['net_id'] = 'test_net_id'
- self.polling_iter = 2
- mock_func1.assert_called()
- mock_func2.assert_called()
-
- @mock.patch('functest.opnfv_tests.openstack.rally.rally.os_utils.'
- 'get_external_net', return_value=None)
- def test_build_task_args_missing_floating_network(self, mock_func):
+ def setUp(self):
+ os_creds = OSCreds(
+ username='user', password='pass',
+ auth_url='http://foo.com:5000/v3', project_name='bar')
+ with mock.patch('snaps.openstack.tests.openstack_tests.'
+ 'get_credentials', return_value=os_creds) as m:
+ self.rally_base = rally.RallyBase()
+ self.polling_iter = 2
+ self.assertTrue(m.called)
+
+ def test_build_task_args_missing_floating_network(self):
CONST.__setattr__('OS_AUTH_URL', None)
+ self.rally_base.ext_net_name = ''
task_args = self.rally_base._build_task_args('test_file_name')
self.assertEqual(task_args['floating_network'], '')
- mock_func.assert_called()
- @mock.patch('functest.opnfv_tests.openstack.rally.rally.os_utils.'
- 'get_external_net', return_value='test_floating_network')
- def test_build_task_args_missing_net_id(self, mock_func):
+ def test_build_task_args_missing_net_id(self):
CONST.__setattr__('OS_AUTH_URL', None)
- self.rally_base.network_dict['net_id'] = ''
+ self.rally_base.priv_net_id = ''
task_args = self.rally_base._build_task_args('test_file_name')
self.assertEqual(task_args['netid'], '')
- mock_func.assert_called()
@staticmethod
def check_scenario_file(value):
@@ -284,27 +280,53 @@ class OSRallyTesting(unittest.TestCase):
with self.assertRaises(Exception):
self.rally_base._prepare_env()
- @mock.patch('functest.opnfv_tests.openstack.rally.rally.os_utils.'
- 'get_or_create_image', return_value=(True, None))
- def test_prepare_env_image_missing(self, mock_get_img):
+ @mock.patch('functest.opnfv_tests.openstack.snaps.snaps_utils.'
+ 'get_ext_net_name', return_value='test_net_name')
+ @mock.patch('snaps.openstack.utils.deploy_utils.create_image',
+ return_value=None)
+ def test_prepare_env_image_missing(self, mock_get_img, mock_get_net):
self.rally_base.TESTS = ['test1', 'test2']
self.rally_base.test_name = 'test1'
with self.assertRaises(Exception):
self.rally_base._prepare_env()
mock_get_img.assert_called()
+ mock_get_net.assert_called()
- @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=None)
- def test_prepare_env_image_shared_network_creation_failed(
- self, mock_create_net, mock_get_img):
+ @mock.patch('functest.opnfv_tests.openstack.snaps.snaps_utils.'
+ 'get_ext_net_name', return_value='test_net_name')
+ @mock.patch('snaps.openstack.utils.deploy_utils.create_image',
+ return_value=mock.Mock())
+ @mock.patch('snaps.openstack.utils.deploy_utils.create_network',
+ return_value=None)
+ def test_prepare_env_network_creation_failed(
+ self, mock_create_net, mock_get_img, mock_get_net):
+ self.rally_base.TESTS = ['test1', 'test2']
+ self.rally_base.test_name = 'test1'
+ with self.assertRaises(Exception):
+ self.rally_base._prepare_env()
+ mock_create_net.assert_called()
+ mock_get_img.assert_called()
+ mock_get_net.assert_called()
+
+ @mock.patch('functest.opnfv_tests.openstack.snaps.snaps_utils.'
+ 'get_ext_net_name', return_value='test_net_name')
+ @mock.patch('snaps.openstack.utils.deploy_utils.create_image',
+ return_value=mock.Mock())
+ @mock.patch('snaps.openstack.utils.deploy_utils.create_network',
+ return_value=mock.Mock())
+ @mock.patch('snaps.openstack.utils.deploy_utils.create_router',
+ return_value=None)
+ def test_prepare_env_router_creation_failed(
+ self, mock_create_router, mock_create_net, mock_get_img,
+ mock_get_net):
self.rally_base.TESTS = ['test1', 'test2']
self.rally_base.test_name = 'test1'
with self.assertRaises(Exception):
self.rally_base._prepare_env()
mock_create_net.assert_called()
mock_get_img.assert_called()
+ mock_get_net.assert_called()
+ mock_create_router.assert_called()
@mock.patch('functest.opnfv_tests.openstack.rally.rally.RallyBase.'
'_run_task', return_value=mock.Mock())
@@ -323,16 +345,13 @@ class OSRallyTesting(unittest.TestCase):
self.rally_base._run_tests()
mock_run_task.assert_any_call('test1')
- @mock.patch('functest.opnfv_tests.openstack.rally.rally.os_utils.'
- 'delete_glance_image')
- def test_clean_up_default(self, mock_glance_method):
- 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()
+ def test_clean_up_default(self):
+ creator1 = mock.Mock()
+ creator2 = mock.Mock()
+ self.rally_base.creators = [creator1, creator2]
self.rally_base._clean_up()
- mock_glance_method.assert_any_call(self.rally_base.nova_client,
- 1)
+ self.assertTrue(creator1.clean.called)
+ self.assertTrue(creator2.clean.called)
@mock.patch('functest.opnfv_tests.openstack.rally.rally.RallyBase.'
'_prepare_env')