From 32ec6265a4161790bc3a548b0d2c1237e30b0d1a Mon Sep 17 00:00:00 2001 From: Sven Nierlein Date: Fri, 13 Sep 2013 13:54:47 +0200 Subject: unified nc syntax older netcats do not understand -p, they expect host and port as additional argument. This is backwards compatibel to newer clients. Also support netcat instead of nc. --- plugins/t/check_udp.t | 18 +++++++++++++++--- 1 file changed, 15 insertions(+), 3 deletions(-) (limited to 'plugins/t') diff --git a/plugins/t/check_udp.t b/plugins/t/check_udp.t index 6d24675a..ae1718c4 100644 --- a/plugins/t/check_udp.t +++ b/plugins/t/check_udp.t @@ -10,6 +10,8 @@ use NPTest; my $res; +alarm(120); # make sure tests don't hang + plan tests => 14; $res = NPTest->testCmd( "./check_udp -H localhost -p 3333" ); @@ -28,9 +30,17 @@ $res = NPTest->testCmd( "./check_udp -H localhost -p 3333 -s foo -e bar" ); cmp_ok( $res->return_code, '==', 2, "Errors correctly because no udp service running" ); like ( $res->output, '/No data received from host/', "Output OK"); +my $nc; +if(system("which netcat >/dev/null 2>&1") == 0) { + $nc = 'netcat -w 3 -l -u -p 3333'; +} +elsif(system("which nc >/dev/null 2>&1") == 0) { + $nc = 'nc -w 3 -l -u -4 localhost 3333'; +} + SKIP: { - skip "No netcat available", 6 unless (system("which nc > /dev/null") == 0); - open (NC, "echo 'barbar' | nc -l -p 3333 -u |"); + skip "No netcat available", 6 unless $nc; + open (NC, "echo 'barbar' | $nc |"); sleep 1; $res = NPTest->testCmd( "./check_udp -H localhost -p 3333 -s '' -e barbar -4" ); cmp_ok( $res->return_code, '==', 0, "Got barbar response back" ); @@ -39,7 +49,7 @@ SKIP: { # Start up a udp server listening on port 3333, quit after 3 seconds # Otherwise will hang at close - my $pid = open(NC, "nc -l -p 3333 -u -w 3