[pve-devel] [PATCH qemu-server] test: migrate: correctly mock storage module

Fabian Ebner f.ebner at proxmox.com
Mon Feb 8 15:26:03 CET 2021


by fixing a typo. Since cfs_read_file within the storage module was not mocked,
the tests could fail on some setups. Now that get_bandwidth_limit is mocked,
cfs_read_file is not called anymore, but still mock it too for good measure and
to make it more future-proof.

Reported-by: Fabian Grünbichler <f.gruenbichler at proxmox.com>
Signed-off-by: Fabian Ebner <f.ebner at proxmox.com>
---
 test/MigrationTest/Shared.pm | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/test/MigrationTest/Shared.pm b/test/MigrationTest/Shared.pm
index d7aeb36..6552465 100644
--- a/test/MigrationTest/Shared.pm
+++ b/test/MigrationTest/Shared.pm
@@ -140,9 +140,10 @@ $storage_module->mock(
     config => sub {
 	return $storage_config;
     },
-    get_bandwitdth_limit => sub {
+    get_bandwidth_limit => sub {
 	return 123456;
     },
+    cfs_read_file => $mocked_cfs_read_file,
 );
 
 our $storage_plugin_module = Test::MockModule->new("PVE::Storage::Plugin");
-- 
2.20.1






More information about the pve-devel mailing list