blob: 1e7b5a4e82ad371a3ebebaa04d75205682803b0b [file] [log] [blame]
From f4bed5a7db90fa1a7dd5bf098f2f6652fe8b9c53 Mon Sep 17 00:00:00 2001
From: Peter Penchev <openstack-dev@storpool.com>
Date: Tue, 17 Jan 2023 01:20:59 +0200
Subject: [PATCH 4/9] StorPool: fix the "rename volume" unit test emulation
Also update the "name" field of the actual data structure in
the fake list of volumes.
Change-Id: Iee9e3feea1af4f3d0acdf05ea1b540ecf59e755b
---
cinder/tests/unit/volume/drivers/test_storpool.py | 7 ++++++-
1 file changed, 6 insertions(+), 1 deletion(-)
diff --git a/cinder/tests/unit/volume/drivers/test_storpool.py b/cinder/tests/unit/volume/drivers/test_storpool.py
index 0eac3d957..442bdbb88 100644
--- a/cinder/tests/unit/volume/drivers/test_storpool.py
+++ b/cinder/tests/unit/volume/drivers/test_storpool.py
@@ -134,7 +134,10 @@ class MockAPI(object):
volumes[name]['size'] = data['size']
if 'rename' in data and data['rename'] != name:
- volumes[data['rename']] = volumes[name]
+ new_name = data['rename']
+ volumes[new_name] = volumes[name]
+ if volumes[new_name]['name'] == name:
+ volumes[new_name]['name'] = new_name
del volumes[name]
@@ -258,6 +261,8 @@ class StorPoolTestCase(test.TestCase):
def assertVolumeNames(self, names):
self.assertListEqual(sorted([volumeName(n) for n in names]),
sorted(volumes.keys()))
+ self.assertListEqual(sorted([volumeName(n) for n in names]),
+ sorted(data['name'] for data in volumes.values()))
@mock_volume_types
def test_create_delete_volume(self):
--
2.43.0