openstack/cinder
Code Issues Proposed changes

Merge "More thorough mock on a backup message test"

Browse Source
This commit is contained in:
Zuul 2024-08-07 19:32:55 +00:00 committed by Gerrit Code Review
commit 30cdc86784
1 changed files with 4 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
cinder/tests/unit/backup/test_backup_messages.py
View File

@ -244,6 +244,8 @@ class BackupUserMessagesTest(test.TestCase):
fake_context,
detail=message_field.Detail.BACKUP_CREATE_CLEANUP_ERROR)
@mock.patch('cinder.scheduler.host_manager.HostManager.'
'_choose_backend_filters')
@mock.patch('cinder.scheduler.host_manager.HostManager.'
'_get_available_backup_service_host')
@mock.patch('cinder.volume.volume_utils.update_backup_error')
@ -252,7 +254,8 @@ class BackupUserMessagesTest(test.TestCase):
@mock.patch('cinder.message.api.API.create')
def test_backup_create_scheduling_error(
self, mock_msg_create, mock_get_vol, mock_vol_update,
mock_update_error, mock_get_backup_host):
mock_update_error, mock_get_backup_host, mock_choose_filters):
mock_choose_filters.return_value = ['AvailabilityZoneFilter']
manager = sch_manager.SchedulerManager()
fake_context = mock.MagicMock()
fake_backup = mock.MagicMock(id=fake.BACKUP_ID,