openstack
/
nova
Code Issues Proposed changes

Remove unused "id" and "rules" from secgroup body 

In test_delete_security_group_by_admin, security_group_template is
used for getting a test request body. And current code specifies
id and rules. However, "create a security group" API just doesn't
use them at all.
So this patch removes them.

Change-Id: I4ae54e1082b8cd73d2a09d321d71ada1d896eee6
This commit is contained in:
Ken'ichi Ohmichi 2015-06-16 00:41:17 +00:00
parent c0625a4e93
commit de86db32c5
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
nova/tests/unit/api/openstack/compute/contrib/test_security_groups.py
View File

@ -570,7 +570,7 @@ class TestSecurityGroupsV21(test.TestCase):
self.assertTrue(self.called)
def test_delete_security_group_by_admin(self):
sg = security_group_template(id=2, rules=[])
sg = security_group_template()
req = fakes.HTTPRequest.blank('/v2/fake/os-security-groups')
self.controller.create(req, {'security_group': sg})