X-Git-Url: http://git.shadowcat.co.uk/gitweb/gitweb.cgi?a=blobdiff_plain;f=t%2Flog.t;h=ea0734d05ac00207eeb7dba87d788bd20a5fa13d;hb=e7b182828c58ebc7f661fc7cd99e96bc69922228;hp=6fc26fe59f03a95bc1e8a31d9073b24add090ecd;hpb=73c0f681c7fd2f59bc85566fcca70e5b25ed1acb;p=p5sagit%2FLog-Contextual.git

diff --git a/t/log.t b/t/log.t
index 6fc26fe..ea0734d 100644
--- a/t/log.t
+++ b/t/log.t
@@ -1,13 +1,45 @@
 use strict;
 use warnings;
 
-use lib 't/lib';
-use VarLogger;
 use Log::Contextual qw{:log with_logger set_logger};
+use Log::Contextual::SimpleLogger;
 use Test::More qw(no_plan);
-my $var_logger1 = VarLogger->new;
-my $var_logger2 = VarLogger->new;
-my $var_logger3 = VarLogger->new;
+
+my @levels = qw(debug trace warn info error fatal);
+
+my $var1;
+my $var2;
+my $var3;
+my $var_logger1 = Log::Contextual::SimpleLogger->new({
+   levels  => [qw(trace debug info warn error fatal)],
+   coderef => sub { $var1 = shift },
+});
+
+my $var_logger2 = Log::Contextual::SimpleLogger->new({
+   levels  => [qw(trace debug info warn error fatal)],
+   coderef => sub { $var2 = shift },
+});
+
+my $var_logger3 = Log::Contextual::SimpleLogger->new({
+   levels  => [qw(trace debug info warn error fatal)],
+   coderef => sub { $var3 = shift },
+});
+
+SETLOGGER: {
+   set_logger(sub { $var_logger3 });
+   log_debug { 'set_logger' };
+   is( $var3, "[debug] set_logger\n", 'set logger works' );
+}
+
+SETLOGGERTWICE: {
+   my $foo;
+   local $SIG{__WARN__} = sub { $foo = shift };
+   set_logger(sub { $var_logger3 });
+   like(
+      $foo, qr/set_logger \(or -logger\) called more than once!  This is a bad idea! at/,
+      'set_logger twice warns correctly'
+   );
+}
 
 WITHLOGGER: {
    with_logger sub { $var_logger2 } => sub {
@@ -19,29 +51,47 @@ WITHLOGGER: {
 
    };
 
-   is( $var_logger1->var, 'dnothing!', 'inner scoped logger works' );
-   is( $var_logger2->var, 'dfrew!', 'outer scoped logger works' );
-}
-
-SETLOGGER: {
-   set_logger(sub { $var_logger3 });
-   log_debug { 'set_logger' };
-   is( $var_logger3->var, 'dset_logger', 'set logger works' );
+   is( $var1, "[debug] nothing!\n", 'inner scoped logger works' );
+   is( $var2, "[debug] frew!\n", 'outer scoped logger works' );
 }
 
 SETWITHLOGGER: {
    with_logger $var_logger1 => sub {
-      log_debug { 'nothing again!' }
+      log_debug { 'nothing again!' };
+      # do this just so the following set_logger won't warn
+      local $SIG{__WARN__} = sub {};
+      set_logger(sub { $var_logger3 });
+      log_debug { 'this is a set inside a with' };
    };
 
-   is( $var_logger1->var, 'dnothing again!',
+   is( $var1, "[debug] nothing again!\n",
       'inner scoped logger works after using set_logger'
    );
 
+   is( $var3, "[debug] this is a set inside a with\n",
+      'set inside with works'
+   );
+
    log_debug { 'frioux!' };
-   is( $var_logger3->var, 'dfrioux!',
+   is( $var3, "[debug] frioux!\n",
       q{set_logger's logger comes back after scoped logger}
    );
 }
 
+VANILLA: {
+   for (@levels) {
+      main->can("log_$_")->(sub { 'fiSMBoC' });
+      is( $var3, "[$_] fiSMBoC\n", "$_ works");
+
+      my @vars = main->can("log_$_")->(sub { 'fiSMBoC: ' . $_[1] }, qw{foo bar baz});
+      is( $var3, "[$_] fiSMBoC: bar\n", "log_$_ works with input");
+      ok( eq_array(\@vars, [qw{foo bar baz}]), "log_$_ passes data through correctly");
+
+      my $val = main->can("logS_$_")->(sub { 'fiSMBoC: ' . $_[0] }, 'foo');
+      is( $var3, "[$_] fiSMBoC: foo\n", "logS_$_ works with input");
+      is( $val, 'foo', "logS_$_ passes data through correctly");
+   }
+}
+
 ok(!eval { Log::Contextual->import; 1 }, 'Blank Log::Contextual import dies');
+