Warn when an accessor for one attr overwrites another attr's accessor.

[gitmo/Moose.git] / t / 020_attributes / 001_attribute_reader_generation.t

diff --git a/t/020_attributes/001_attribute_reader_generation.t b/t/020_attributes/001_attribute_reader_generation.t
index 7ad1447..5c625e8 100644 (file)
--- a/t/020_attributes/001_attribute_reader_generation.t
+++ b/t/020_attributes/001_attribute_reader_generation.t
@@ -3,32 +3,39 @@
 use strict;
 use warnings;
 
-use Test::More tests => 14;
-use Test::Exception;
+use Test::More;
+use Test::Fatal;
 
-BEGIN {
-    use_ok('Moose');           
-}
 
 {
     package Foo;
     use Moose;
-    
+
     eval {
         has 'foo' => (
             reader => 'get_foo'
         );
     };
     ::ok(!$@, '... created the reader method okay');
-    
+
     eval {
         has 'lazy_foo' => (
-            reader => 'get_lazy_foo', 
-            lazy => 1, 
+            reader => 'get_lazy_foo',
+            lazy => 1,
             default => sub { 10 }
         );
     };
-    ::ok(!$@, '... created the lazy reader method okay') or warn $@;    
+    ::ok(!$@, '... created the lazy reader method okay') or warn $@;
+
+    my $warn;
+
+    eval {
+        local $SIG{__WARN__} = sub { $warn = $_[0] };
+        has 'mtfnpy' => (
+            reder => 'get_mftnpy'
+        );
+    };
+    ::ok($warn, '... got a warning for mispelled attribute argument');
 }
 
 {
@@ -37,26 +44,42 @@ BEGIN {
 
     can_ok($foo, 'get_foo');
     is($foo->get_foo(), undef, '... got an undefined value');
-    dies_ok {
+    isnt( exception {
         $foo->get_foo(100);
-    } '... get_foo is a read-only';
-    
+    }, undef, '... get_foo is a read-only' );
+
     ok(!exists($foo->{lazy_foo}), '... no value in get_lazy_foo slot');
-    
+
     can_ok($foo, 'get_lazy_foo');
     is($foo->get_lazy_foo(), 10, '... got an deferred value');
-    dies_ok {
+    isnt( exception {
         $foo->get_lazy_foo(100);
-    } '... get_lazy_foo is a read-only';    
+    }, undef, '... get_lazy_foo is a read-only' );
 }
 
 {
-    my $foo = Foo->new(foo => 10, lazy_foo => 100);
+    my $foo = Foo->new;
     isa_ok($foo, 'Foo');
 
-    is($foo->get_foo(), 10, '... got the correct value');
-    is($foo->get_lazy_foo(), 100, '... got the correct value');    
+    my $attr = $foo->meta->find_attribute_by_name("lazy_foo");
+
+    isa_ok( $attr, "Moose::Meta::Attribute" );
+
+    ok( $attr->is_lazy, "it's lazy" );
+
+    is( $attr->get_raw_value($foo), undef, "raw value" );
+
+    is( $attr->get_value($foo), 10, "lazy value" );
+
+    is( $attr->get_raw_value($foo), 10, "raw value" );
 }
 
+{
+    my $foo = Foo->new(foo => 10, lazy_foo => 100);
+    isa_ok($foo, 'Foo');
 
+    is($foo->get_foo(), 10, '... got the correct value');
+    is($foo->get_lazy_foo(), 100, '... got the correct value');
+}
 
+done_testing;