Add test plan, tidy, and add test descriptions

[gitmo/Class-MOP.git] / t / 104_AttributesWithHistory_test.t

diff --git a/t/104_AttributesWithHistory_test.t b/t/104_AttributesWithHistory_test.t
index ada9d67..7950ade 100644 (file)
--- a/t/104_AttributesWithHistory_test.t
+++ b/t/104_AttributesWithHistory_test.t
@@ -1,20 +1,16 @@
-#!/usr/bin/perl
-
 use strict;
 use warnings;
 
-use Test::More tests => 19;
+use Test::More tests => 27;
 use File::Spec;
 
-BEGIN { 
-    use_ok('Class::MOP');    
-    require_ok(File::Spec->catdir('examples', 'AttributesWithHistory.pod'));
+BEGIN {use Class::MOP;    
+    require_ok(File::Spec->catfile('examples', 'AttributesWithHistory.pod'));
 }
 
 {
     package Foo;
-    
-    use Class::MOP 'meta';
+    use metaclass;
     
     Foo->meta->add_attribute(AttributesWithHistory->new('foo' => (
         accessor         => 'foo',
@@ -29,8 +25,8 @@ BEGIN {
     
     sub new  {
         my $class = shift;
-        bless $class->meta->construct_instance(@_) => $class;
-    }        
+        $class->meta->new_object(@_);
+    }   
 }
 
 my $foo = Foo->new();
@@ -42,11 +38,20 @@ can_ok($foo, 'set_bar');
 can_ok($foo, 'get_bar');
 can_ok($foo, 'get_bar_history');
 
+my $foo2 = Foo->new();
+isa_ok($foo2, 'Foo');
+
 is($foo->foo, undef, '... foo is not yet defined');
 is_deeply(
     [ $foo->get_foo_history() ],
     [ ],
     '... got correct empty history for foo');
+    
+is($foo2->foo, undef, '... foo2 is not yet defined');
+is_deeply(
+    [ $foo2->get_foo_history() ],
+    [ ],
+    '... got correct empty history for foo2');    
 
 $foo->foo(42);
 is($foo->foo, 42, '... foo == 42');
@@ -55,6 +60,25 @@ is_deeply(
     [ 42 ],
     '... got correct history for foo');
 
+is($foo2->foo, undef, '... foo2 is still not yet defined');
+is_deeply(
+    [ $foo2->get_foo_history() ],
+    [ ],
+    '... still got correct empty history for foo2');
+        
+$foo2->foo(100);
+is($foo->foo, 42, '... foo is still == 42');
+is_deeply(
+    [ $foo->get_foo_history() ],
+    [ 42 ],
+    '... still got correct history for foo');
+
+is($foo2->foo, 100, '... foo2 == 100');
+is_deeply(
+    [ $foo2->get_foo_history() ],
+    [ 100 ],
+    '... got correct empty history for foo2');
+
 $foo->foo(43);
 $foo->foo(44);
 $foo->foo(45);