From 8a6681900ab2a53e324e74031017d83d814d79d8 Mon Sep 17 00:00:00 2001 From: Dominik Csapak Date: Fri, 28 Jan 2022 09:36:47 +0100 Subject: [PATCH] fix tests in 'vzdump_new_test.pl' 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 b241deb731369ff96b26bfa670c5c8d273aea02c Signed-off-by: Dominik Csapak --- test/vzdump_new_test.pl | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/test/vzdump_new_test.pl b/test/vzdump_new_test.pl index 23676239..8cd73075 100755 --- a/test/vzdump_new_test.pl +++ b/test/vzdump_new_test.pl @@ -61,6 +61,10 @@ $pve_cluster_module->mock( die "unexpected filename '$filename'\n" if $filename ne 'storage.cfg'; return $storage_config; }, + # never update during the tests + cfs_update => sub { + return; + }, ); my $pve_tools_module = Test::MockModule->new('PVE::Tools');