X-Git-Url: http://git.shadowcat.co.uk/gitweb/gitweb.cgi?a=blobdiff_plain;f=t%2F004_w_cycles.t;h=9664cba93d1080196c77d57effd38904216d8a28;hb=1d4669291edf1495a90d5b110a6334b05ca1cc78;hp=6dd816e0c6caaaa741cf38e799ef022d3cbed93c;hpb=9d3c60f5f35a8d28951049e8daccd9f67c22f9aa;p=gitmo%2FMooseX-Storage.git diff --git a/t/004_w_cycles.t b/t/004_w_cycles.t index 6dd816e..9664cba 100644 --- a/t/004_w_cycles.t +++ b/t/004_w_cycles.t @@ -1,9 +1,8 @@ -#!/usr/bin/perl - use strict; use warnings; use Test::More tests => 18; +use Test::Deep; use Test::Fatal; BEGIN { @@ -15,7 +14,7 @@ BEGIN { This test demonstrates two things: - cycles will not work in the default engine -- you can use a special metaclass to tell +- you can use a special metaclass to tell MooseX::Storage to skip an attribute =cut @@ -34,9 +33,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/, @@ -62,19 +61,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); @@ -85,28 +84,28 @@ This test demonstrates two things: { my $t = Tree->new(node => 100); isa_ok($t, 'Tree'); - - is_deeply( - $t->pack, + + cmp_deeply( + $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); - - is_deeply($t->children, [ $t2 ], '... got the right children in $t'); - + + cmp_deeply($t->children, [ $t2 ], '... got the right children in $t'); + is($t2->parent, $t, '... created the cycle correctly'); - isa_ok($t2->parent, 'Tree'); - - is_deeply( - $t->pack, + isa_ok($t2->parent, 'Tree'); + + cmp_deeply( + $t->pack, { __CLASS__ => 'Tree', node => 100, @@ -114,18 +113,18 @@ This test demonstrates two things: { __CLASS__ => 'Tree', node => 200, - children => [], - } + children => [], + } ], }, - '... got the right packed version (with parent attribute skipped in child)'); - - is_deeply( - $t2->pack, + '... got the right packed version (with parent attribute skipped in child)'); + + cmp_deeply( + $t2->pack, { __CLASS__ => 'Tree', node => 200, - children => [], + children => [], }, '... got the right packed version (with parent attribute skipped)'); } @@ -135,7 +134,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' ); } @@ -154,13 +153,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 @@ -168,7 +167,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' ); } @@ -180,4 +179,4 @@ This test demonstrates two things: ok( $pack, "Object packs with DisableCycleDetection trait"); ok( DoubleNoCycle->unpack( $pack ), " Unpacked again" ); -} +}