summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorTon Voon <tonvoon@users.sourceforge.net>2007-02-08 15:58:15 (GMT)
committerTon Voon <tonvoon@users.sourceforge.net>2007-02-08 15:58:15 (GMT)
commit489df48f9d1891b46ae4be911080ab4667058f2d (patch)
treefea89ac882fe294d9c73b36bb3133d75cfb89caf
parent0e41613e9f547ab97365329572555936e84f77b2 (diff)
downloadmonitoring-plugin-perl-489df48f9d1891b46ae4be911080ab4667058f2d.tar.gz
Fixed warnings when no uom specified for add_perfdata
git-svn-id: https://nagiosplug.svn.sourceforge.net/svnroot/nagiosplug/Nagios-Plugin/trunk@1618 f882894a-f735-0410-b71e-b25c423dba1c
-rw-r--r--Changes4
-rw-r--r--lib/Nagios/Plugin/Performance.pm2
-rw-r--r--t/Nagios-Plugin-01.t10
-rw-r--r--t/Nagios-Plugin-Performance.t2
4 files changed, 16 insertions, 2 deletions
diff --git a/Changes b/Changes
index 88c0fbe..384538b 100644
--- a/Changes
+++ b/Changes
@@ -1,5 +1,9 @@
1Revision history for Perl module Nagios::Plugin. 1Revision history for Perl module Nagios::Plugin.
2 2
3??? ??
4 - fixed warnings when no uom specified for add_perfdata
5 - added max_state function in N::P::Functions
6
30.15 19th December 2006 70.15 19th December 2006
4 - exposed Getopt and Threshold functionality from top level Nagios::Plugin 8 - exposed Getopt and Threshold functionality from top level Nagios::Plugin
5 - exchanged Class::Struct for Class::Accessor 9 - exchanged Class::Struct for Class::Accessor
diff --git a/lib/Nagios/Plugin/Performance.pm b/lib/Nagios/Plugin/Performance.pm
index 6e4cd03..fc1c0bc 100644
--- a/lib/Nagios/Plugin/Performance.pm
+++ b/lib/Nagios/Plugin/Performance.pm
@@ -22,7 +22,7 @@ struct "Nagios::Plugin::Performance" => {
22 22
23sub perfoutput { 23sub perfoutput {
24 my $self = shift; 24 my $self = shift;
25 my $output = $self->label."=".$self->value.$self->uom.";".$self->threshold->warning.";".$self->threshold->critical; 25 my $output = $self->label."=".$self->value. ($self->uom || "") .";".$self->threshold->warning.";".$self->threshold->critical;
26 return $output; 26 return $output;
27} 27}
28 28
diff --git a/t/Nagios-Plugin-01.t b/t/Nagios-Plugin-01.t
index db12c35..3ada472 100644
--- a/t/Nagios-Plugin-01.t
+++ b/t/Nagios-Plugin-01.t
@@ -1,7 +1,7 @@
1# Nagios::Plugin original test cases 1# Nagios::Plugin original test cases
2 2
3use strict; 3use strict;
4use Test::More tests => 12; 4use Test::More tests => 13;
5 5
6BEGIN { use_ok('Nagios::Plugin') }; 6BEGIN { use_ok('Nagios::Plugin') };
7 7
@@ -41,6 +41,14 @@ $p->add_perfdata(
41cmp_ok( $p->all_perfoutput, 'eq', "size=1kB;10:25;~:25", "Perfdata correct"); 41cmp_ok( $p->all_perfoutput, 'eq', "size=1kB;10:25;~:25", "Perfdata correct");
42#diag "dumping perfdata: ". Dumper ($p->perfdata); 42#diag "dumping perfdata: ". Dumper ($p->perfdata);
43 43
44$p->add_perfdata(
45 label => "time",
46 value => "3.52",
47 threshold => $t,
48 );
49
50is( $p->all_perfoutput, "size=1kB;10:25;~:25 time=3.52;10:25;~:25", "Perfdata correct when no uom specified");
51
44my $expected = {qw( 52my $expected = {qw(
45 -1 WARNING 53 -1 WARNING
46 1 WARNING 54 1 WARNING
diff --git a/t/Nagios-Plugin-Performance.t b/t/Nagios-Plugin-Performance.t
index e0eb2f6..1ae330a 100644
--- a/t/Nagios-Plugin-Performance.t
+++ b/t/Nagios-Plugin-Performance.t
@@ -80,3 +80,5 @@ cmp_ok( $p[0]->rrdlabel, "eq", "home_a_m", "changing / to _");
80cmp_ok( $p[1]->rrdlabel, "eq", "shared_folder_big", "replacing bad characters"); 80cmp_ok( $p[1]->rrdlabel, "eq", "shared_folder_big", "replacing bad characters");
81cmp_ok( $p[2]->rrdlabel, "eq", "1234567890123456789", "shortening rrd label"); 81cmp_ok( $p[2]->rrdlabel, "eq", "1234567890123456789", "shortening rrd label");
82 82
83
84# add_perfdata tests in t/Nagios-Plugin-01.t