Module: monitoring-plugins
 Branch: check_icmp_buffer_offerflow
 Commit: 44f19e798692e9e4787f5cdb5e3e340e75bb14e3
 Author: RincewindsHat <12514511+rincewinds...@users.noreply.github.com>
   Date: Fri Dec  3 09:56:24 2021 +0100
    URL: 
https://www.monitoring-plugins.org/repositories/monitoring-plugins/commit/?id=44f19e7

Fix tests (hopefully)

---

 plugins-root/t/check_icmp.t | 14 ++++++++++++--
 1 file changed, 12 insertions(+), 2 deletions(-)

diff --git a/plugins-root/t/check_icmp.t b/plugins-root/t/check_icmp.t
index b347a66..0f57438 100644
--- a/plugins-root/t/check_icmp.t
+++ b/plugins-root/t/check_icmp.t
@@ -12,7 +12,7 @@ my $allow_sudo = getTestParameter( "NP_ALLOW_SUDO",
        "no" );
 
 if ($allow_sudo eq "yes" or $> == 0) {
-       plan tests => 16;
+       plan tests => 20;
 } else {
        plan skip_all => "Need sudo to test check_icmp";
 }
@@ -35,60 +35,70 @@ my $hostname_invalid   = getTestParameter( 
"NP_HOSTNAME_INVALID",
 
 my $res;
 
+# 1
 $res = NPTest->testCmd(
        "$sudo ./check_icmp -H $host_responsive -w 10000ms,100% -c 10000ms,100%"
        );
 is( $res->return_code, 0, "Syntax ok" );
 like( $res->output, $successOutput, "Output OK" );
 
+# 2
 $res = NPTest->testCmd(
        "$sudo ./check_icmp -H $host_responsive -w 0ms,0% -c 10000ms,100%"
        );
 is( $res->return_code, 1, "Syntax ok, with forced warning" );
 like( $res->output, $failureOutput, "Output OK" );
 
+# 3
 $res = NPTest->testCmd(
        "$sudo ./check_icmp -H $host_responsive -w 0,0% -c 0,0%"
        );
 is( $res->return_code, 2, "Syntax ok, with forced critical" );
 like( $res->output, $failureOutput, "Output OK" );
 
+# 4
 $res = NPTest->testCmd(
        "$sudo ./check_icmp -H $host_nonresponsive -w 10000ms,100% -c 
10000ms,100%"
        );
 is( $res->return_code, 2, "Timeout - host nonresponsive" );
 like( $res->output, '/100%/', "Error contains '100%' string (for 100% packet 
loss)" );
 
+# 4
 $res = NPTest->testCmd(
        "$sudo ./check_icmp -w 10000ms,100% -c 10000ms,100%"
        );
 is( $res->return_code, 3, "No hostname" );
 like( $res->output, '/No hosts to check/', "Output with appropriate error 
message");
 
+# 5
 $res = NPTest->testCmd(
        "$sudo ./check_icmp -H $host_nonresponsive -w 10000ms,100% -c 
10000ms,100% -n 1 -m 0"
        );
 is( $res->return_code, 0, "One host nonresponsive - zero required" );
 like( $res->output, $successOutput, "Output OK" );
 
+# 6
 $res = NPTest->testCmd(
        "$sudo ./check_icmp -H $host_responsive -H $host_nonresponsive -w 
10000ms,100% -c 10000ms,100% -n 1 -m 1"
        );
 is( $res->return_code, 0, "One of two host nonresponsive - one required" );
 like( $res->output, $successOutput, "Output OK" );
 
+# 7
 $res = NPTest->testCmd(
        "$sudo ./check_icmp -H $host_responsive -H $host_nonresponsive -w 
10000ms,100% -c 10000ms,100% -n 1 -m 2"
        );
 is( $res->return_code, 2, "One of two host nonresponsive - two required" );
 like( $res->output, $failureOutput, "Output OK" );
 
+# 8
 $res = NPTest->testCmd(
-       "$sudo ./check_icmp -H ::1 -b 65507"
+       "$sudo ./check_icmp -H $host_responsive -b 65507"
        );
 is( $res->return_code, 0, "Try max paket size" );
 like( $res->output, $successOutput, "Output OK - Didn't overflow" );
 
+# 9
 $res = NPTest->testCmd(
        "$sudo ./check_icmp -H ::1 -vvv"
        );

Reply via email to