From 43d723eeb32041b612d9d1d2aa9bbdb4a170f4d0 Mon Sep 17 00:00:00 2001 From: Daniel Friesel Date: Sun, 15 Apr 2012 10:30:23 +0200 Subject: Use croak, fix tests --- Build.PL | 1 + bin/icli | 5 +++-- t/50-icli.t | 6 +++--- 3 files changed, 7 insertions(+), 5 deletions(-) diff --git a/Build.PL b/Build.PL index e7b8867..b314bd0 100644 --- a/Build.PL +++ b/Build.PL @@ -17,6 +17,7 @@ my $build = Module::Build->new( requires => { 'perl' => '5.10.0', 'autodie' => 0, + 'Carp' => 0, 'Date::Format' => 0, 'Getopt::Long' => 0, 'Term::ANSIColor' => 0, diff --git a/bin/icli b/bin/icli index 1e15305..3cceb7c 100755 --- a/bin/icli +++ b/bin/icli @@ -7,6 +7,7 @@ use strict; use warnings; use 5.010; +use Carp qw(croak); use Date::Format; use Getopt::Long qw/:config bundling/; use Term::ANSIColor; @@ -364,7 +365,7 @@ sub service_state { when (1) { return with_colour( ' WARNING', 'black on_yellow' ) } when (2) { return with_colour( 'CRITICAL', 'white on_red' ) } when (3) { return with_colour( ' UNKNOWN', 'white on_blue' ) } - default { die("Unknown service state: $digit\n") } + default { croak("Unknown service state: $digit\n") } } } @@ -379,7 +380,7 @@ sub host_state { when (0) { return with_colour( ' OK ', 'black on_green' ) } when (1) { return with_colour( ' DOWN ', 'white on_red' ) } when (2) { return with_colour( 'UNREACHABLE', 'white on_blue' ) } - default { die("Unknown host state: $digit\n") } + default { croak("Unknown host state: $digit\n") } } } diff --git a/t/50-icli.t b/t/50-icli.t index 62d7912..2e6f0c7 100644 --- a/t/50-icli.t +++ b/t/50-icli.t @@ -170,10 +170,10 @@ $icli = "bin/icli -f t/in/status.dat.weird.2 -c t/in/objects.cache"; $cmd = Test::Command->new(cmd => "$icli -lh -h alpha"); $cmd->exit_isnt_num(0); -$cmd->stdout_is_eq('alpha '); -$cmd->stderr_is_eq("Unknown host state: 23\n"); +$cmd->stdout_is_eq($EMPTY); +$cmd->stderr_like(qr{^Unknown host state: 23}s); $cmd = Test::Command->new(cmd => "$icli -ls -h aneurysm"); $cmd->exit_isnt_num(0); $cmd->stdout_is_eq('Disk: / '); -$cmd->stderr_is_eq("Unknown service state: 23\n"); +$cmd->stderr_like(qr{^Unknown service state: 23}s); -- cgit v1.2.3