Fix create_security_group_rule_bulk_native to return all created rules

create_security_group_rule_bulk_native should return all of the created
security-group rules, but returns only one, this patch fix the issue and
adds a unittest to validate it.

Closes-Bug: #1434207
Change-Id: I8611c83fecf90e025b24b09fc3a371cbeebce637

diff --git a/neutron/db/securitygroups_db.py b/neutron/db/securitygroups_db.py
index 92dcc7ac4141189081c6d6ac1064b748f67d4b90..5e6deae43a42ba8e3dd4b1e0dfb13dec87d40215 100644 (file)
--- a/neutron/db/securitygroups_db.py
+++ b/neutron/db/securitygroups_db.py
@@ -308,7 +308,7 @@ class SecurityGroupDbMixin(ext_sg.SecurityGroupPluginBase):
                     port_range_max=rule['port_range_max'],
                     remote_ip_prefix=rule.get('remote_ip_prefix'))
                 context.session.add(db)
-            ret.append(self._make_security_group_rule_dict(db))
+                ret.append(self._make_security_group_rule_dict(db))
         return ret
 
     def create_security_group_rule(self, context, security_group_rule):

diff --git a/neutron/tests/unit/test_extension_security_group.py b/neutron/tests/unit/test_extension_security_group.py
index 58c049277bc0a96d0ee052ffb811ff83c02eddb7..4fac0e8868540160300c1cf8c8bcbba1d038604c 100644 (file)
--- a/neutron/tests/unit/test_extension_security_group.py
+++ b/neutron/tests/unit/test_extension_security_group.py
@@ -1238,8 +1238,9 @@ class TestSecurityGroups(SecurityGroupDBTestCase):
             rules = {'security_group_rules': [rule1['security_group_rule'],
                                               rule2['security_group_rule']]}
             res = self._create_security_group_rule(self.fmt, rules)
-            self.deserialize(self.fmt, res)
+            ret = self.deserialize(self.fmt, res)
             self.assertEqual(res.status_int, webob.exc.HTTPCreated.code)
+            self.assertEqual(2, len(ret['security_group_rules']))
 
     def test_create_security_group_rule_bulk_emulated(self):
         real_has_attr = hasattr