[pve-devel] applied: [PATCH manager] fix tests in 'vzdump_new_test.pl'

Fabian Grünbichler f.gruenbichler at proxmox.com
Fri Jan 28 09:46:01 CET 2022


thanks! seems like it only broke tests/build as root, which I never do ;)

On January 28, 2022 9:36 am, Dominik Csapak wrote:
> commmit
> a55cfb53 ("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 a55cfb5359f86adc2d3981cab032bbf8a0411931
> 
> Signed-off-by: Dominik Csapak <d.csapak at proxmox.com>
> ---
>  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');
> -- 
> 2.30.2
> 
> 
> 
> _______________________________________________
> pve-devel mailing list
> pve-devel at lists.proxmox.com
> https://lists.proxmox.com/cgi-bin/mailman/listinfo/pve-devel
> 
> 
> 




More information about the pve-devel mailing list