fixed

[gitmo/Class-MOP.git] / t / 022_attribute_duplication.t

diff --git a/t/022_attribute_duplication.t b/t/022_attribute_duplication.t
index f23d4a1..3fa68b1 100644 (file)
--- a/t/022_attribute_duplication.t
+++ b/t/022_attribute_duplication.t
@@ -3,7 +3,9 @@
 use strict;
 use warnings;
 
-use Test::More tests => 17;
+use Scalar::Util;
+
+use Test::More tests => 29;
 
 BEGIN {
     use_ok('Class::MOP');
@@ -36,6 +38,20 @@ one first.
     ::is($bar_attr->writer, 'set_bar', '... the bar attribute has the writer set_bar');    
     ::is($bar_attr->associated_class, Foo->meta, '... and the bar attribute is associated with Foo->meta');
     
+    ::is($bar_attr->get_read_method,  'get_bar', '... $attr does have an read method');
+    ::is($bar_attr->get_write_method, 'set_bar', '... $attr does have an write method');    
+    
+    {
+        my $reader = $bar_attr->get_read_method_ref;
+        my $writer = $bar_attr->get_write_method_ref;        
+        
+        ::isa_ok($reader, 'Class::MOP::Method');
+        ::isa_ok($writer, 'Class::MOP::Method');        
+        
+        ::is(Scalar::Util::reftype($reader->body), 'CODE', '... it is a plain old sub');
+        ::is(Scalar::Util::reftype($writer->body), 'CODE', '... it is a plain old sub');                
+    }    
+    
     Foo->meta->add_attribute('bar' => 
         reader => 'assign_bar'
     );    
@@ -47,6 +63,20 @@ one first.
     
     my $bar_attr2 = Foo->meta->get_attribute('bar');
     
+    ::is($bar_attr2->get_read_method,  'assign_bar', '... $attr does have an read method');
+    ::ok(!$bar_attr2->get_write_method, '... $attr does have an write method');    
+    
+    {
+        my $reader = $bar_attr2->get_read_method_ref;
+        my $writer = $bar_attr2->get_write_method_ref;        
+        
+        ::isa_ok($reader, 'Class::MOP::Method');
+        ::ok(!Scalar::Util::blessed($writer), '... the writer method is not blessed though');        
+        
+        ::is(Scalar::Util::reftype($reader->body), 'CODE', '... it is a plain old sub');
+        ::is(Scalar::Util::reftype($writer), 'CODE', '... it is a plain old sub');                
+    }    
+    
     ::isnt($bar_attr, $bar_attr2, '... this is a new bar attribute');
     ::isnt($bar_attr->associated_class, Foo->meta, '... and the old bar attribute is no longer associated with Foo->meta');