From: Dave Rolsky Date: Tue, 17 Feb 2009 15:14:44 +0000 (+0000) Subject: Tidy this code X-Git-Tag: 0.71~16 X-Git-Url: http://git.shadowcat.co.uk/gitweb/gitweb.cgi?a=commitdiff_plain;h=ca851c73b08284983e4f7c5a04ad6ebc61caebd4;p=gitmo%2FMoose.git Tidy this code --- diff --git a/t/200_examples/007_Child_Parent_attr_inherit.t b/t/200_examples/007_Child_Parent_attr_inherit.t index d7abef0..07f9426 100644 --- a/t/200_examples/007_Child_Parent_attr_inherit.t +++ b/t/200_examples/007_Child_Parent_attr_inherit.t @@ -13,97 +13,124 @@ be used to manage parent-child relationships. =cut { + package Parent; use Moose; - + has 'last_name' => ( - is => 'rw', + is => 'rw', isa => 'Str', trigger => sub { my $self = shift; + # if the parents last-name changes # then so do all the childrens - foreach my $child (@{$self->children}) { - $child->last_name($self->last_name); + foreach my $child ( @{ $self->children } ) { + $child->last_name( $self->last_name ); } } ); - - has 'children' => (is => 'rw', isa => 'ArrayRef', default => sub {[]}); + + has 'children' => + ( is => 'rw', isa => 'ArrayRef', default => sub { [] } ); } { + package Child; use Moose; - + has 'parent' => ( - is => 'rw', - isa => 'Parent', - required => 1, - trigger => sub { + is => 'rw', + isa => 'Parent', + required => 1, + trigger => sub { my $self = shift; - # if the parent is changed,.. - # make sure we update - $self->last_name($self->parent->last_name); + + # if the parent is changed,.. + # make sure we update + $self->last_name( $self->parent->last_name ); } ); - + has 'last_name' => ( - is => 'rw', - isa => 'Str', + is => 'rw', + isa => 'Str', lazy => 1, default => sub { (shift)->parent->last_name } ); - + } -my $parent = Parent->new(last_name => 'Smith'); -isa_ok($parent, 'Parent'); +my $parent = Parent->new( last_name => 'Smith' ); +isa_ok( $parent, 'Parent' ); -is($parent->last_name, 'Smith', '... the parent has the last name we expected'); +is( $parent->last_name, 'Smith', + '... the parent has the last name we expected' ); -$parent->children([ - map { Child->new(parent => $parent) } (0 .. 3) -]); +$parent->children( [ map { Child->new( parent => $parent ) } ( 0 .. 3 ) ] ); -foreach my $child (@{$parent->children}) { - is($child->last_name, $parent->last_name, '... parent and child have the same last name (' . $parent->last_name . ')'); +foreach my $child ( @{ $parent->children } ) { + is( $child->last_name, $parent->last_name, + '... parent and child have the same last name (' + . $parent->last_name + . ')' ); } $parent->last_name('Jones'); -is($parent->last_name, 'Jones', '... the parent has the new last name'); +is( $parent->last_name, 'Jones', '... the parent has the new last name' ); -foreach my $child (@{$parent->children}) { - is($child->last_name, $parent->last_name, '... parent and child have the same last name (' . $parent->last_name . ')'); +foreach my $child ( @{ $parent->children } ) { + is( $child->last_name, $parent->last_name, + '... parent and child have the same last name (' + . $parent->last_name + . ')' ); } # make a new parent -my $parent2 = Parent->new(last_name => 'Brown'); -isa_ok($parent2, 'Parent'); +my $parent2 = Parent->new( last_name => 'Brown' ); +isa_ok( $parent2, 'Parent' ); -# orphan the child +# orphan the child -my $orphan = pop @{$parent->children}; +my $orphan = pop @{ $parent->children }; -# and then the new parent adopts it +# and then the new parent adopts it $orphan->parent($parent2); -foreach my $child (@{$parent->children}) { - is($child->last_name, $parent->last_name, '... parent and child have the same last name (' . $parent->last_name . ')'); +foreach my $child ( @{ $parent->children } ) { + is( $child->last_name, $parent->last_name, + '... parent and child have the same last name (' + . $parent->last_name + . ')' ); } -isnt($orphan->last_name, $parent->last_name, '... the orphan child does not have the same last name anymore (' . $parent2->last_name . ')'); -is($orphan->last_name, $parent2->last_name, '... parent2 and orphan child have the same last name (' . $parent2->last_name . ')'); +isnt( $orphan->last_name, $parent->last_name, + '... the orphan child does not have the same last name anymore (' + . $parent2->last_name + . ')' ); +is( $orphan->last_name, $parent2->last_name, + '... parent2 and orphan child have the same last name (' + . $parent2->last_name + . ')' ); # make sure that changes still will not propagate $parent->last_name('Miller'); -is($parent->last_name, 'Miller', '... the parent has the new last name (again)'); - -foreach my $child (@{$parent->children}) { - is($child->last_name, $parent->last_name, '... parent and child have the same last name (' . $parent->last_name . ')'); +is( $parent->last_name, 'Miller', + '... the parent has the new last name (again)' ); + +foreach my $child ( @{ $parent->children } ) { + is( $child->last_name, $parent->last_name, + '... parent and child have the same last name (' + . $parent->last_name + . ')' ); } -isnt($orphan->last_name, $parent->last_name, '... the orphan child is not affected by changes in the parent anymore'); -is($orphan->last_name, $parent2->last_name, '... parent2 and orphan child have the same last name (' . $parent2->last_name . ')'); +isnt( $orphan->last_name, $parent->last_name, + '... the orphan child is not affected by changes in the parent anymore' ); +is( $orphan->last_name, $parent2->last_name, + '... parent2 and orphan child have the same last name (' + . $parent2->last_name + . ')' );