[pve-devel] [PATCH pve-manager 18/27] test: fix vzdump notification test

Lukas Wagner l.wagner at proxmox.com
Tue Nov 7 11:18:18 CET 2023


The signature of the PVE::Notify functions have changed, this commit
adapts the mocked functions so that the tests work again.

Signed-off-by: Lukas Wagner <l.wagner at proxmox.com>
---
 test/vzdump_notification_test.pl | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/test/vzdump_notification_test.pl b/test/vzdump_notification_test.pl
index 21c31651..631606bb 100755
--- a/test/vzdump_notification_test.pl
+++ b/test/vzdump_notification_test.pl
@@ -38,14 +38,14 @@ my $result_properties;
 
 my $mock_notification_module = Test::MockModule->new('PVE::Notify');
 my $mocked_notify = sub {
-    my ($channel, $severity, $title, $text, $properties) = @_;
+    my ($severity, $title, $text, $properties, $metadata) = @_;
 
     $result_text = $text;
     $result_properties = $properties;
 };
 my $mocked_notify_short = sub {
-    my ($channel, @rest) = @_;
-    return $mocked_notify->($channel, '<some severity>', @rest);
+    my (@params) = @_;
+    return $mocked_notify->('<some severity>', @params);
 };
 
 $mock_notification_module->mock(
-- 
2.39.2






More information about the pve-devel mailing list