diff options
-rw-r--r-- | lib/Monitoring/Plugin/Getopt.pm | 4 | ||||
-rw-r--r-- | t/Monitoring-Plugin-Getopt-04.t | 30 |
2 files changed, 31 insertions, 3 deletions
diff --git a/lib/Monitoring/Plugin/Getopt.pm b/lib/Monitoring/Plugin/Getopt.pm index ebdd559..e09ff62 100644 --- a/lib/Monitoring/Plugin/Getopt.pm +++ b/lib/Monitoring/Plugin/Getopt.pm | |||
@@ -81,7 +81,7 @@ sub _spec_to_help | |||
81 | { | 81 | { |
82 | my ($self, $spec, $label) = @_; | 82 | my ($self, $spec, $label) = @_; |
83 | 83 | ||
84 | my ($opts, $type) = split /=/, $spec, 2; | 84 | my ($opts, $type) = split /=|:/, $spec, 2; |
85 | my (@short, @long); | 85 | my (@short, @long); |
86 | for (split /\|/, $opts) { | 86 | for (split /\|/, $opts) { |
87 | if (length $_ == 1) { | 87 | if (length $_ == 1) { |
@@ -97,7 +97,7 @@ sub _spec_to_help | |||
97 | $help .= '=' . $label; | 97 | $help .= '=' . $label; |
98 | } | 98 | } |
99 | else { | 99 | else { |
100 | $help .= $type eq 'i' ? '=INTEGER' : '=STRING'; | 100 | $help .= ($type eq 'i' || $type eq '+' || $type =~ /\d+/) ? '=INTEGER' : '=STRING'; |
101 | } | 101 | } |
102 | } | 102 | } |
103 | elsif ($label) { | 103 | elsif ($label) { |
diff --git a/t/Monitoring-Plugin-Getopt-04.t b/t/Monitoring-Plugin-Getopt-04.t index b6345d0..9b51883 100644 --- a/t/Monitoring-Plugin-Getopt-04.t +++ b/t/Monitoring-Plugin-Getopt-04.t | |||
@@ -2,7 +2,7 @@ | |||
2 | 2 | ||
3 | use strict; | 3 | use strict; |
4 | 4 | ||
5 | use Test::More tests => 11; | 5 | use Test::More tests => 15; |
6 | BEGIN { use_ok('Monitoring::Plugin::Getopt') }; | 6 | BEGIN { use_ok('Monitoring::Plugin::Getopt') }; |
7 | 7 | ||
8 | # Needed to get evals to work in testing | 8 | # Needed to get evals to work in testing |
@@ -78,6 +78,30 @@ sub setup | |||
78 | [ undef, 'PERCENT%' ], | 78 | [ undef, 'PERCENT%' ], |
79 | ); | 79 | ); |
80 | 80 | ||
81 | # Named args with *optional* but pre-set value | ||
82 | $ng->arg( | ||
83 | spec => 'dirport|d:9030', | ||
84 | help => 'dirport', | ||
85 | ); | ||
86 | |||
87 | # Named args with *optional* string value | ||
88 | $ng->arg( | ||
89 | spec => 'enablesomething|s:s', | ||
90 | help => 'something', | ||
91 | ); | ||
92 | |||
93 | # Named args with *optional* integer value (same as ":0") | ||
94 | $ng->arg( | ||
95 | spec => 'testtimeout|T:i', | ||
96 | help => 'testtimeout', | ||
97 | ); | ||
98 | |||
99 | # Named args with *optional* but increasing integer value | ||
100 | $ng->arg( | ||
101 | spec => 'verbosity|v:+', | ||
102 | help => 'verbosity', | ||
103 | ); | ||
104 | |||
81 | return $ng; | 105 | return $ng; |
82 | } | 106 | } |
83 | 107 | ||
@@ -94,4 +118,8 @@ like($@, qr/\n -H, --hostname=ADDRESS\n Hostname\n/, 'hostname ok'); | |||
94 | like($@, qr/\n --avatar=AVATAR\n Avatar\n/, 'avatar ok'); | 118 | like($@, qr/\n --avatar=AVATAR\n Avatar\n/, 'avatar ok'); |
95 | like($@, qr/\n --disk=BYTES\n Disk limit in BYTES\n --disk=PERCENT%\n Disk limit in PERCENT\n --disk=STRING\n Disk limit in FOOBARS \(Default: 1024\)\n/, 'disk multiline ok'); | 119 | like($@, qr/\n --disk=BYTES\n Disk limit in BYTES\n --disk=PERCENT%\n Disk limit in PERCENT\n --disk=STRING\n Disk limit in FOOBARS \(Default: 1024\)\n/, 'disk multiline ok'); |
96 | like($@, qr/\n --limit=STRING\n Limit in BYTES\n --limit=PERCENT%\n Limit in PERCENT\n/, 'limit multiline ok'); | 120 | like($@, qr/\n --limit=STRING\n Limit in BYTES\n --limit=PERCENT%\n Limit in PERCENT\n/, 'limit multiline ok'); |
121 | like($@, qr/\n -d, --dirport=INTEGER/, 'dirport ok'); | ||
122 | like($@, qr/\n -s, --enablesomething=STRING/, 'enablesomething ok'); | ||
123 | like($@, qr/\n -T, --testtimeout=INTEGER/, 'testtimeout ok'); | ||
124 | like($@, qr/\n -v, --verbosity=INTEGER/, 'verbosity ok'); | ||
97 | #print $@; | 125 | #print $@; |