move to github

[gitmo/MooseX-Storage.git] / t / 008_do_not_serialize.t

diff --git a/t/008_do_not_serialize.t b/t/008_do_not_serialize.t
index 527bc74..0748c5d 100644 (file)
--- a/t/008_do_not_serialize.t
+++ b/t/008_do_not_serialize.t
@@ -1,10 +1,9 @@
-#!/usr/bin/perl
-
 use strict;
 use warnings;
 
-use Test::More no_plan => 6;
-use Test::Exception;
+use Test::More tests => 13;
+use Test::Deep;
+use Test::Fatal;
 
 BEGIN {
     use_ok('MooseX::Storage');
@@ -20,38 +19,82 @@ BEGIN {
     has 'bar' => (
         metaclass => 'DoNotSerialize',
         is        => 'rw',
-        default   => sub { 'BAR' }        
+        default   => sub { 'BAR' }
     );
-    
+
     has 'baz' => (
         traits  => [ 'DoNotSerialize' ],
         is      => 'rw',
-        default => sub { 'BAZ' }        
-    );    
-    
+        default => sub { 'BAZ' }
+    );
+
     has 'gorch' => (
-        is      => 'rw', 
+        is      => 'rw',
         default => sub { 'GORCH' }
-    );    
+    );
 
     1;
 }
 
-my $foo = Foo->new;
-isa_ok($foo, 'Foo');
-
-is($foo->bar, 'BAR', '... got the value we expected');
-is($foo->baz, 'BAZ', '... got the value we expected');
-is($foo->gorch, 'GORCH', '... got the value we expected');
-
-is_deeply(
-    $foo->pack,
-    {
-        __CLASS__ => 'Foo',
-        gorch     => 'GORCH'
-    },
-    '... got the right packed class data'
-);
+{   my $foo = Foo->new;
+    isa_ok($foo, 'Foo');
+
+    is($foo->bar, 'BAR', '... got the value we expected');
+    is($foo->baz, 'BAZ', '... got the value we expected');
+    is($foo->gorch, 'GORCH', '... got the value we expected');
+
+    cmp_deeply(
+        $foo->pack,
+        {
+            __CLASS__ => 'Foo',
+            gorch     => 'GORCH'
+        },
+        '... got the right packed class data'
+    );
+}
+
+### more involved test; required attribute that's not serialized
+{   package Bar;
+    use Moose;
+    use MooseX::Storage;
+
+    with Storage;
+
+    has foo => (
+        metaclass   => 'DoNotSerialize',
+        required    => 1,
+        is          => 'rw',
+        isa         => 'Object',        # type constraint is important
+    );
+
+    has zot => (
+        default     => sub { $$ },
+        is          => 'rw',
+    );
+}
+
+{   my $obj = bless {};
+    my $bar = Bar->new( foo => $obj );
+
+    ok( $bar,                   "New object created" );
+    is( $bar->foo, $obj,        "   ->foo => $obj" );
+    is( $bar->zot, $$,          "   ->zot => $$" );
+
+    my $bpack = $bar->pack;
+    cmp_deeply(
+        $bpack,
+        {   __CLASS__   => 'Bar',
+            zot         => $$,
+        },                      "   Packed correctly" );
+
+    eval { Bar->unpack( $bpack ) };
+    ok( $@,                     "   Unpack without required attribute fails" );
+    like( $@, qr/foo/,          "       Proper error recorded" );
+
+    my $bar2 = Bar->unpack( $bpack, inject => { foo => bless {} } );
+    ok( $bar2,                  "   Unpacked correctly with foo => Object");
+}
+