X-Git-Url: http://git.shadowcat.co.uk/gitweb/gitweb.cgi?a=blobdiff_plain;f=t%2F004_w_cycles.t;h=aa92ad76ac5f048a5d7f89efbd5cafc26a245d6d;hb=19d8a777247d4816be278da681de598526eaf1f2;hp=c816b62094aa1058b522bc3f2f441c75ace163d6;hpb=81a84db6ebe5ec540d0035fc6d1104cc49c81e25;p=gitmo%2FMooseX-Storage.git diff --git a/t/004_w_cycles.t b/t/004_w_cycles.t index c816b62..aa92ad7 100644 --- a/t/004_w_cycles.t +++ b/t/004_w_cycles.t @@ -9,15 +9,10 @@ BEGIN { use_ok('MooseX::Storage'); } -=pod - -This test demonstrates two things: - -- cycles will not work in the default engine -- you can use a special metaclass to tell - MooseX::Storage to skip an attribute - -=cut +# This test demonstrates two things: +# +# - cycles will not work in the default engine +# - you can use a special metaclass to tell MooseX::Storage to skip an attribute { @@ -33,9 +28,9 @@ This test demonstrates two things: { my $circular = Circular->new; isa_ok($circular, 'Circular'); - + $circular->cycle($circular); - + like(exception { $circular->pack; }, qr/^Basic Engine does not support cycles/, @@ -61,19 +56,19 @@ This test demonstrates two things: with Storage; has 'node' => (is => 'rw'); - + has 'children' => ( - is => 'ro', - isa => 'ArrayRef', + is => 'ro', + isa => 'ArrayRef', default => sub {[]} ); - + has 'parent' => ( metaclass => 'DoNotSerialize', - is => 'rw', + is => 'rw', isa => 'Tree', ); - + sub add_child { my ($self, $child) = @_; $child->parent($self); @@ -84,28 +79,28 @@ This test demonstrates two things: { my $t = Tree->new(node => 100); isa_ok($t, 'Tree'); - + cmp_deeply( - $t->pack, + $t->pack, { __CLASS__ => 'Tree', node => 100, children => [], }, '... got the right packed version'); - + my $t2 = Tree->new(node => 200); - isa_ok($t2, 'Tree'); - + isa_ok($t2, 'Tree'); + $t->add_child($t2); - + cmp_deeply($t->children, [ $t2 ], '... got the right children in $t'); - + is($t2->parent, $t, '... created the cycle correctly'); - isa_ok($t2->parent, 'Tree'); - + isa_ok($t2->parent, 'Tree'); + cmp_deeply( - $t->pack, + $t->pack, { __CLASS__ => 'Tree', node => 100, @@ -113,18 +108,18 @@ This test demonstrates two things: { __CLASS__ => 'Tree', node => 200, - children => [], - } + children => [], + } ], }, - '... got the right packed version (with parent attribute skipped in child)'); - + '... got the right packed version (with parent attribute skipped in child)'); + cmp_deeply( - $t2->pack, + $t2->pack, { __CLASS__ => 'Tree', node => 200, - children => [], + children => [], }, '... got the right packed version (with parent attribute skipped)'); } @@ -134,7 +129,7 @@ This test demonstrates two things: use Moose; use MooseX::Storage; with Storage; - + has 'x' => ( is => 'rw', isa => 'HashRef' ); has 'y' => ( is => 'rw', isa => 'HashRef' ); } @@ -153,13 +148,13 @@ This test demonstrates two things: ok( Double->unpack( $pack || {} ), " And unpacked again" ); } - + my $pack = $double->pack( engine_traits => [qw/DisableCycleDetection/] ); ok( $pack, " Object packs when cycle check is disabled"); ok( Double->unpack( $pack ), " And unpacked again" ); -} +} ### the same as above, but now done with a trait ### this fails with cycle detection on @@ -167,7 +162,7 @@ This test demonstrates two things: use Moose; use MooseX::Storage; with Storage( traits => ['DisableCycleDetection'] ); - + has 'x' => ( is => 'rw', isa => 'HashRef' ); has 'y' => ( is => 'rw', isa => 'HashRef' ); } @@ -179,4 +174,4 @@ This test demonstrates two things: ok( $pack, "Object packs with DisableCycleDetection trait"); ok( DoubleNoCycle->unpack( $pack ), " Unpacked again" ); -} +}