From 28f1f21ca25d7fb3cd5cbf88a4b9814b0861cde5 Mon Sep 17 00:00:00 2001 From: Thomas Lamprecht Date: Fri, 18 Oct 2019 20:29:20 +0200 Subject: [PATCH] add cleanup_pending tests Signed-off-by: Thomas Lamprecht --- tests/abstract-config-tests.pl | 36 +++++++++++++++++++++++++++++++++- 1 file changed, 35 insertions(+), 1 deletion(-) diff --git a/tests/abstract-config-tests.pl b/tests/abstract-config-tests.pl index 8f58e42..086edcb 100755 --- a/tests/abstract-config-tests.pl +++ b/tests/abstract-config-tests.pl @@ -107,7 +107,7 @@ my $tests = [ { method => 'remove_from_pending_delete', # $conf, $key subtests => [ - { # simple test addition to of a pending deletion to the empty config + { params => [ { pending => { delete => 'memory', } }, 'memory' ], expect => { pending => {} }, , @@ -124,6 +124,40 @@ my $tests = [ }, ] }, +{ + method => 'cleanup_pending', # $conf + subtests => [ + { # want to delete opt which is not in config? -> all done, cleanup! + params => [{ + pending => { delete => 'memory', } + }], + map_expect_to_param_id => 0, + expect => { pending => {} }, + }, + { # should do nothing, delete is pending and memory is set after all + params => [{ + memory => 128, + pending => { delete => 'memory', } + }], + map_expect_to_param_id => 0, + expect => { + memory => 128, + pending => { delete => 'memory', } + }, + }, + { # the pending change is the same as the currents config value, cleanup pending! + params => [{ + memory => 128, + pending => { memory => 128, } + }], + map_expect_to_param_id => 0, + expect => { + memory => 128, + pending => {} + }, + }, + ] +}, ]; # tests definition end -- 2.39.5