X-Git-Url: http://git.shadowcat.co.uk/gitweb/gitweb.cgi?a=blobdiff_plain;f=t%2F020_attributes%2F004_attribute_triggers.t;h=e7993993db262a72744a4a576e15468cfd986e23;hb=d03bd989b97597428b460d7f9a021e2931893fa0;hp=b5cf34eb9b9b945f180b84e76e1aed380be6c4cc;hpb=65e14c863ae29b3fa9a54a84269984f5dad6a400;p=gitmo%2FMoose.git

diff --git a/t/020_attributes/004_attribute_triggers.t b/t/020_attributes/004_attribute_triggers.t
index b5cf34e..e799399 100644
--- a/t/020_attributes/004_attribute_triggers.t
+++ b/t/020_attributes/004_attribute_triggers.t
@@ -5,42 +5,40 @@ use warnings;
 
 use Scalar::Util 'isweak';
 
-use Test::More tests => 26;
+use Test::More tests => 40;
 use Test::Exception;
 
-BEGIN {
-    use_ok('Moose');           
-}
+
 
 {
     package Foo;
     use Moose;
-    
-    has 'bar' => (is      => 'rw', 
+
+    has 'bar' => (is      => 'rw',
                   isa     => 'Maybe[Bar]',
-                  trigger => sub { 
+                  trigger => sub {
                       my ($self, $bar) = @_;
                       $bar->foo($self) if defined $bar;
                   });
-                  
+
     has 'baz' => (writer => 'set_baz',
                   reader => 'get_baz',
                   isa    => 'Baz',
-                  trigger => sub { 
+                  trigger => sub {
                       my ($self, $baz) = @_;
                       $baz->foo($self);
-                  });              
-     
-                  
+                  });
+
+
     package Bar;
     use Moose;
-    
-    has 'foo' => (is => 'rw', isa => 'Foo', weak_ref => 1);           
-    
+
+    has 'foo' => (is => 'rw', isa => 'Foo', weak_ref => 1);
+
     package Baz;
     use Moose;
-    
-    has 'foo' => (is => 'rw', isa => 'Foo', weak_ref => 1);           
+
+    has 'foo' => (is => 'rw', isa => 'Foo', weak_ref => 1);
 }
 
 {
@@ -61,13 +59,13 @@ BEGIN {
     is($bar->foo, $foo, '... which in turn set the value bar.foo correctly');
 
     ok(isweak($bar->{foo}), '... bar.foo is a weak reference');
-    
+
     lives_ok {
         $foo->bar(undef);
     } '... did not die un-setting bar';
 
     is($foo->bar, undef, '... set the value foo.bar correctly');
-    is($bar->foo, $foo, '... which in turn set the value bar.foo correctly');    
+    is($bar->foo, $foo, '... which in turn set the value bar.foo correctly');
 
     # test the writer
 
@@ -87,9 +85,9 @@ BEGIN {
 
     my $baz = Baz->new;
     isa_ok($baz, 'Baz');
-    
+
     my $foo = Foo->new(bar => $bar, baz => $baz);
-    isa_ok($foo, 'Foo');    
+    isa_ok($foo, 'Foo');
 
     is($foo->bar, $bar, '... set the value foo.bar correctly');
     is($bar->foo, $foo, '... which in turn set the value bar.foo correctly');
@@ -107,14 +105,86 @@ BEGIN {
 {
     package Bling;
     use Moose;
-    
-    ::dies_ok { 
+
+    ::dies_ok {
         has('bling' => (is => 'rw', trigger => 'Fail'));
     } '... a trigger must be a CODE ref';
-    
-    ::dies_ok { 
+
+    ::dies_ok {
         has('bling' => (is => 'rw', trigger => []));
-    } '... a trigger must be a CODE ref';    
+    } '... a trigger must be a CODE ref';
+}
+
+# Triggers do not fire on built values
+
+{
+    package Blarg;
+    use Moose;
+
+    our %trigger_calls;
+    our %trigger_vals;
+    has foo => (is => 'rw', default => sub { 'default foo value' },
+                trigger => sub { my ($self, $val, $attr) = @_;
+                                 $trigger_calls{foo}++;
+                                 $trigger_vals{foo} = $val });
+    has bar => (is => 'rw', lazy_build => 1,
+                trigger => sub { my ($self, $val, $attr) = @_;
+                                 $trigger_calls{bar}++;
+                                 $trigger_vals{bar} = $val });
+    sub _build_bar { return 'default bar value' }
+    has baz => (is => 'rw', builder => '_build_baz',
+                trigger => sub { my ($self, $val, $attr) = @_;
+                                 $trigger_calls{baz}++;
+                                 $trigger_vals{baz} = $val });
+    sub _build_baz { return 'default baz value' }
+}
+
+{
+    my $blarg;
+    lives_ok { $blarg = Blarg->new; } 'Blarg->new() lives';
+    ok($blarg, 'Have a $blarg');
+    foreach my $attr (qw/foo bar baz/) {
+        is($blarg->$attr(), "default $attr value", "$attr has default value");
+    }
+    is_deeply(\%Blarg::trigger_calls, {}, 'No triggers fired');
+    foreach my $attr (qw/foo bar baz/) {
+        $blarg->$attr("Different $attr value");
+    }
+    is_deeply(\%Blarg::trigger_calls, { map { $_ => 1 } qw/foo bar baz/ }, 'All triggers fired once on assign');
+    is_deeply(\%Blarg::trigger_vals, { map { $_ => "Different $_ value" } qw/foo bar baz/ }, 'All triggers given assigned values');
+
+    lives_ok { $blarg => Blarg->new( map { $_ => "Yet another $_ value" } qw/foo bar baz/ ) } '->new() with parameters';
+    is_deeply(\%Blarg::trigger_calls, { map { $_ => 2 } qw/foo bar baz/ }, 'All triggers fired once on construct');
+    is_deeply(\%Blarg::trigger_vals, { map { $_ => "Yet another $_ value" } qw/foo bar baz/ }, 'All triggers given assigned values');
+}
+
+# Triggers do not receive the meta-attribute as an argument
+
+{
+    package Foo;
+    use Moose;
+    our @calls;
+    has foo => (is => 'rw', trigger => sub { push @calls, [@_] });
+}
+
+{
+    my $attr = Foo->meta->get_attribute('foo');
+    my $foo = Foo->new(foo => 2);
+    is_deeply(
+        \@Foo::calls,
+        [ [ $foo, 2 ] ],
+        'trigger called correctly on construction',
+    );
+    @Foo::calls = ();
+
+    $foo->foo(3);
+    is_deeply(
+        \@Foo::calls,
+        [ [ $foo, 3 ] ],
+        'trigger called correctly on set',
+    );
+    @Foo::calls = ();
+    Foo->meta->make_immutable, redo if Foo->meta->is_mutable;
 }