Proxmox/pve-manager
5
0
Fork 0
Code Pull Requests Releases Activity

fix tests in 'vzdump_new_test.pl'

Browse Source 
commmit
b241deb7 ("close #3476: vzdump: add job-init hook")

introduces a 'cfs_update' in VZDump->new(), which overwrites our
carefully prepared storage/vm config which can break the tests

to prevent this, mock 'cfs_update' to do nothing, since we never
want to load the 'real' configs during tests anyway

fixes commit b241deb731

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>
This commit is contained in:
Dominik Csapak 2022-01-28 09:36:47 +01:00 committed by Fabian Grünbichler
parent d20113ef7e
commit 8a6681900a
1 changed files with 4 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
test/vzdump_new_test.pl
View File

@ -61,6 +61,10 @@ $pve_cluster_module->mock(
die "unexpected filename '$filename'\n" if $filename ne 'storage.cfg'; die "unexpected filename '$filename'\n" if $filename ne 'storage.cfg';
return $storage_config; return $storage_config;
}, },
# never update during the tests
cfs_update => sub {
return;
},
); );
my $pve_tools_module = Test::MockModule->new('PVE::Tools'); my $pve_tools_module = Test::MockModule->new('PVE::Tools');