openstack/cinder
Code Issues Proposed changes

Merge "Adding regression test for update_group"

Browse Source
This commit is contained in:
Zuul 2018-10-17 13:42:39 +00:00 committed by Gerrit Code Review
commit a9467860b5
1 changed files with 16 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

16
cinder/tests/unit/group/test_groups_manager.py
View File

@ -382,6 +382,22 @@ class GroupManagerTestCase(test.TestCase):
self.volume.db.volume_get.reset_mock() self.volume.db.volume_get.reset_mock()
self.volume.db.volume_get = volume_get_orig self.volume.db.volume_get = volume_get_orig
def test_update_group_vol_not_found(self):
"""Test add non existent volume to group"""
group = tests_utils.create_group(
self.context,
availability_zone=CONF.storage_availability_zone,
volume_type_ids=[fake.VOLUME_TYPE_ID],
group_type_id=fake.GROUP_TYPE_ID,
host=CONF.host)
self.volume.create_group(self.context, group)
self.assertRaises(exception.VolumeNotFound,
self.volume.update_group,
self.context, group,
add_volumes=fake.VOLUME_ID,
remove_volumes=fake.VOLUME2_ID)
@mock.patch('cinder.db.sqlalchemy.api.' @mock.patch('cinder.db.sqlalchemy.api.'
'volume_glance_metadata_copy_to_volume') 'volume_glance_metadata_copy_to_volume')
@mock.patch('cinder.db.sqlalchemy.api.' @mock.patch('cinder.db.sqlalchemy.api.'