server group unit testing a bit wrong

the function of create_test_server_group should be used instead
create_test_aggregate

Change-Id: I383bc563d369144a5f918064309b73e7984b61ab
Closes-Bug: #1736331
This commit is contained in:
liuluyang 2017-12-04 23:23:44 -08:00
parent fce0cb4c25
commit 1b2490bdf0
3 changed files with 5 additions and 2 deletions

View File

@ -1006,7 +1006,7 @@ class Connection(api.Connection):
try:
server_group_ref = models.ServerGroup()
server_group_ref.update(values)
server_group_ref.save(session)
server_group_ref.save(session=session)
except db_exc.DBDuplicateEntry:
raise exception.ServerGroupExists(group_uuid=values['uuid'])
if policies:

View File

@ -32,7 +32,7 @@ class DbServerGroupTestCase(base.DbTestCase):
members=['server1', 'server2'])
def test_server_group_create(self):
sg = utils.create_test_aggregate(name='testing')
sg = utils.create_test_server_group(name='testing')
self.assertEqual('testing', sg.name)
def test_server_group_get(self):

View File

@ -237,6 +237,9 @@ def create_test_server_group(context={}, **kw):
"""
server_fault = get_test_server_group(**kw)
# Let DB generate ID if it isn't specified explicitly
if 'id' not in kw:
del server_fault['id']
dbapi = db_api.get_instance()
members = server_fault.pop('members')
policies = server_fault.pop('policies')