| Peter Pentchev | 844a5d9 | 2023-01-17 03:07:56 +0200 | [diff] [blame^] | 1 | StorPool: fix the "rename volume" unit test emulation |
| 2 | |
| 3 | Also update the "name" field of the actual data structure in |
| 4 | the fake list of volumes. |
| 5 | |
| 6 | Change-Id: Iee9e3feea1af4f3d0acdf05ea1b540ecf59e755b |
| 7 | --- a/cinder/tests/unit/volume/drivers/test_storpool.py |
| 8 | +++ b/cinder/tests/unit/volume/drivers/test_storpool.py |
| 9 | @@ -135,7 +135,10 @@ |
| 10 | volumes[name]['size'] = data['size'] |
| 11 | |
| 12 | if 'rename' in data and data['rename'] != name: |
| 13 | - volumes[data['rename']] = volumes[name] |
| 14 | + new_name = data['rename'] |
| 15 | + volumes[new_name] = volumes[name] |
| 16 | + if volumes[new_name]['name'] == name: |
| 17 | + volumes[new_name]['name'] = new_name |
| 18 | del volumes[name] |
| 19 | |
| 20 | |
| 21 | @@ -259,6 +262,8 @@ |
| 22 | def assertVolumeNames(self, names): |
| 23 | self.assertListEqual(sorted([volumeName(n) for n in names]), |
| 24 | sorted(volumes.keys())) |
| 25 | + self.assertListEqual(sorted([volumeName(n) for n in names]), |
| 26 | + sorted(data['name'] for data in volumes.values())) |
| 27 | |
| 28 | @mock_volume_types |
| 29 | def test_create_delete_volume(self): |