From a078ddd7c39f5bc3de154d3f9611b4713522b673 Mon Sep 17 00:00:00 2001 From: Soniya Vyas Date: Fri, 4 Dec 2020 18:34:44 +0530 Subject: [PATCH] Need to add kwargs for the api Added kwargs parameter for create_security_group() api Implements: blueprint tempest-scenario-manager-stable Signed-off by: Soniya Vyas Change-Id: Ia01153c84ccb6a085231753aba7caaa22a8f2535 --- tempest/scenario/manager.py | 14 ++++++++------ 1 file changed, 8 insertions(+), 6 deletions(-) diff --git a/tempest/scenario/manager.py b/tempest/scenario/manager.py index ff860d5b90..274ecfb6da 100644 --- a/tempest/scenario/manager.py +++ b/tempest/scenario/manager.py @@ -527,14 +527,16 @@ class ScenarioTest(tempest.test.BaseTestCase): rules.append(sg_rule) return rules - def _create_security_group(self): + def _create_security_group(self, **kwargs): """Create security group and add rules to security group""" - sg_name = data_utils.rand_name(self.__class__.__name__) - sg_desc = sg_name + " description" + if not kwargs.get('name'): + kwargs['name'] = data_utils.rand_name(self.__class__.__name__) + if not kwargs.get('description'): + kwargs['description'] = kwargs['name'] + " description" secgroup = self.compute_security_groups_client.create_security_group( - name=sg_name, description=sg_desc)['security_group'] - self.assertEqual(secgroup['name'], sg_name) - self.assertEqual(secgroup['description'], sg_desc) + **kwargs)['security_group'] + self.assertEqual(secgroup['name'], kwargs['name']) + self.assertEqual(secgroup['description'], kwargs['description']) self.addCleanup( test_utils.call_and_ignore_notfound_exc, self.compute_security_groups_client.delete_security_group,