Tidy code
Dave Rolsky [Wed, 20 Jan 2010 21:48:30 +0000 (15:48 -0600)]
t/lib/SharedTests.pm

index b2f542b..3c83bee 100644 (file)
@@ -6,7 +6,6 @@ use warnings;
 use Scalar::Util qw( isweak );
 use Test::More;
 
-
 {
     package HasClassAttribute;
 
@@ -17,106 +16,106 @@ use Test::More;
     use vars qw($Lazy);
     $Lazy = 0;
 
-    class_has 'ObjectCount' =>
-        ( is        => 'rw',
-          isa       => 'Int',
-          default   => 0,
-        );
-
-    class_has 'WeakAttribute' =>
-        ( is        => 'rw',
-          isa       => 'Object',
-          weak_ref  => 1,
-        );
-
-    class_has 'LazyAttribute' =>
-        ( is      => 'rw',
-          isa     => 'Int',
-          lazy    => 1,
-          # The side effect is used to test that this was called
-          # lazily.
-          default => sub { $Lazy = 1 },
-        );
-
-    class_has 'ReadOnlyAttribute' =>
-        ( is      => 'ro',
-          isa     => 'Int',
-          default => 10,
-        );
-
-    class_has 'ManyNames' =>
-        ( is        => 'rw',
-          isa       => 'Int',
-          reader    => 'M',
-          writer    => 'SetM',
-          clearer   => 'ClearM',
-          predicate => 'HasM',
-        );
-
-    class_has 'Delegatee' =>
-        ( is      => 'rw',
-          isa     => 'Delegatee',
-          handles => [ 'units', 'color' ],
-          # if it's not lazy it makes a new object before we define
-          # Delegatee's attributes.
-          lazy    => 1,
-          default => sub { Delegatee->new() },
-        );
-
-    class_has 'Mapping' =>
-        ( metaclass => 'Collection::Hash',
-          is        => 'rw',
-          isa       => 'HashRef[Str]',
-          default   => sub { {} },
-          provides  =>
-          { exists => 'ExistsInMapping',
+    class_has 'ObjectCount' => (
+        is      => 'rw',
+        isa     => 'Int',
+        default => 0,
+    );
+
+    class_has 'WeakAttribute' => (
+        is       => 'rw',
+        isa      => 'Object',
+        weak_ref => 1,
+    );
+
+    class_has 'LazyAttribute' => (
+        is   => 'rw',
+        isa  => 'Int',
+        lazy => 1,
+
+        # The side effect is used to test that this was called
+        # lazily.
+        default => sub { $Lazy = 1 },
+    );
+
+    class_has 'ReadOnlyAttribute' => (
+        is      => 'ro',
+        isa     => 'Int',
+        default => 10,
+    );
+
+    class_has 'ManyNames' => (
+        is        => 'rw',
+        isa       => 'Int',
+        reader    => 'M',
+        writer    => 'SetM',
+        clearer   => 'ClearM',
+        predicate => 'HasM',
+    );
+
+    class_has 'Delegatee' => (
+        is      => 'rw',
+        isa     => 'Delegatee',
+        handles => [ 'units', 'color' ],
+
+        # if it's not lazy it makes a new object before we define
+        # Delegatee's attributes.
+        lazy    => 1,
+        default => sub { Delegatee->new() },
+    );
+
+    class_has 'Mapping' => (
+        metaclass => 'Collection::Hash',
+        is        => 'rw',
+        isa       => 'HashRef[Str]',
+        default   => sub { {} },
+        provides  => {
+            exists => 'ExistsInMapping',
             keys   => 'IdsInMapping',
             get    => 'GetMapping',
             set    => 'SetMapping',
-          },
-        );
-
-    class_has 'Built' =>
-        ( is      => 'ro',
-          builder => '_BuildIt',
-        );
-
-    class_has 'LazyBuilt' =>
-        ( is      => 'ro',
-          lazy    => 1,
-          builder => '_BuildIt',
-        );
-
-    class_has 'Triggerish' =>
-        ( is      => 'rw',
-          trigger => sub { shift->_CallTrigger(@_) },
-        );
-
-    has 'size' =>
-        ( is      => 'rw',
-          isa     => 'Int',
-          default => 5,
-        );
+        },
+    );
+
+    class_has 'Built' => (
+        is      => 'ro',
+        builder => '_BuildIt',
+    );
+
+    class_has 'LazyBuilt' => (
+        is      => 'ro',
+        lazy    => 1,
+        builder => '_BuildIt',
+    );
+
+    class_has 'Triggerish' => (
+        is      => 'rw',
+        trigger => sub { shift->_CallTrigger(@_) },
+    );
+
+    has 'size' => (
+        is      => 'rw',
+        isa     => 'Int',
+        default => 5,
+    );
 
     no Moose;
 
-    sub BUILD
-    {
+    sub BUILD {
         my $self = shift;
 
         $self->ObjectCount( $self->ObjectCount() + 1 );
     }
 
-    sub _BuildIt { 42 }
+    sub _BuildIt {42}
 
     our @Triggered;
-    sub _CallTrigger
-    {
+
+    sub _CallTrigger {
         push @Triggered, [@_];
     }
 
-    sub make_immutable
-    {
+    sub make_immutable {
         my $class = shift;
 
         $class->meta()->make_immutable();
@@ -129,15 +128,15 @@ use Test::More;
 
     use Moose;
 
-    has 'units' =>
-        ( is      => 'ro',
-          default => 5,
-        );
+    has 'units' => (
+        is      => 'ro',
+        default => 5,
+    );
 
-    has 'color' =>
-        ( is      => 'ro',
-          default => 'blue',
-        );
+    has 'color' => (
+        is      => 'ro',
+        default => 'blue',
+    );
 
     no Moose;
 }
@@ -150,48 +149,62 @@ use Test::More;
 
     extends 'HasClassAttribute';
 
-    class_has '+ReadOnlyAttribute' =>
-        ( default => 30 );
+    class_has '+ReadOnlyAttribute' => ( default => 30 );
 
-    class_has 'YetAnotherAttribute' =>
-        ( is      => 'ro',
-          default => 'thing',
-        );
+    class_has 'YetAnotherAttribute' => (
+        is      => 'ro',
+        default => 'thing',
+    );
 
     no Moose;
 }
 
-sub run_tests
-{
+sub run_tests {
     plan tests => 30;
 
     local $Test::Builder::Level = $Test::Builder::Level + 1;
 
     {
-        is( HasClassAttribute->ObjectCount(), 0,
-            'ObjectCount() is 0' );
+        is(
+            HasClassAttribute->ObjectCount(), 0,
+            'ObjectCount() is 0'
+        );
 
         my $hca1 = HasClassAttribute->new();
-        is( $hca1->size(), 5,
-            'size is 5 - object attribute works as expected' );
-        is( HasClassAttribute->ObjectCount(), 1,
-            'ObjectCount() is 1' );
+        is(
+            $hca1->size(), 5,
+            'size is 5 - object attribute works as expected'
+        );
+        is(
+            HasClassAttribute->ObjectCount(), 1,
+            'ObjectCount() is 1'
+        );
 
         my $hca2 = HasClassAttribute->new( size => 10 );
-        is( $hca2->size(), 10,
-            'size is 10 - object attribute can be set via constructor' );
-        is( HasClassAttribute->ObjectCount(), 2,
-            'ObjectCount() is 2' );
-        is( $hca2->ObjectCount(), 2,
-            'ObjectCount() is 2 - can call class attribute accessor on object' );
+        is(
+            $hca2->size(), 10,
+            'size is 10 - object attribute can be set via constructor'
+        );
+        is(
+            HasClassAttribute->ObjectCount(), 2,
+            'ObjectCount() is 2'
+        );
+        is(
+            $hca2->ObjectCount(), 2,
+            'ObjectCount() is 2 - can call class attribute accessor on object'
+        );
     }
 
     {
         my $hca3 = HasClassAttribute->new( ObjectCount => 20 );
-        is( $hca3->ObjectCount(), 3,
-            'class attributes passed to the constructor do not get set in the object' );
-        is( HasClassAttribute->ObjectCount(), 3,
-            'class attributes are not affected by constructor params' );
+        is(
+            $hca3->ObjectCount(), 3,
+            'class attributes passed to the constructor do not get set in the object'
+        );
+        is(
+            HasClassAttribute->ObjectCount(), 3,
+            'class attributes are not affected by constructor params'
+        );
     }
 
     {
@@ -201,79 +214,115 @@ sub run_tests
 
         undef $object;
 
-        ok( ! defined HasClassAttribute->WeakAttribute(),
-            'weak class attributes are weak' );
+        ok(
+            !defined HasClassAttribute->WeakAttribute(),
+            'weak class attributes are weak'
+        );
     }
 
     {
-        is( $HasClassAttribute::Lazy, 0,
-            '$HasClassAttribute::Lazy is 0' );
+        is(
+            $HasClassAttribute::Lazy, 0,
+            '$HasClassAttribute::Lazy is 0'
+        );
 
-        is( HasClassAttribute->LazyAttribute(), 1,
-            'HasClassAttribute->LazyAttribute() is 1' );
+        is(
+            HasClassAttribute->LazyAttribute(), 1,
+            'HasClassAttribute->LazyAttribute() is 1'
+        );
 
-        is( $HasClassAttribute::Lazy, 1,
-            '$HasClassAttribute::Lazy is 1 after calling LazyAttribute' );
+        is(
+            $HasClassAttribute::Lazy, 1,
+            '$HasClassAttribute::Lazy is 1 after calling LazyAttribute'
+        );
     }
 
     {
         eval { HasClassAttribute->ReadOnlyAttribute(20) };
-        like( $@, qr/\QCannot assign a value to a read-only accessor/,
-              'cannot set read-only class attribute' );
+        like(
+            $@, qr/\QCannot assign a value to a read-only accessor/,
+            'cannot set read-only class attribute'
+        );
     }
 
     {
-        is( Child->ReadOnlyAttribute(), 30,
-            q{Child class can extend parent's class attribute} );
+        is(
+            Child->ReadOnlyAttribute(), 30,
+            q{Child class can extend parent's class attribute}
+        );
     }
 
     {
-        ok( ! HasClassAttribute->HasM(),
-            'HasM() returns false before M is set' );
+        ok(
+            !HasClassAttribute->HasM(),
+            'HasM() returns false before M is set'
+        );
 
         HasClassAttribute->SetM(22);
 
-        ok( HasClassAttribute->HasM(),
-            'HasM() returns true after M is set' );
-        is( HasClassAttribute->M(), 22,
-            'M() returns 22' );
+        ok(
+            HasClassAttribute->HasM(),
+            'HasM() returns true after M is set'
+        );
+        is(
+            HasClassAttribute->M(), 22,
+            'M() returns 22'
+        );
 
         HasClassAttribute->ClearM();
 
-        ok( ! HasClassAttribute->HasM(),
-            'HasM() returns false after M is cleared' );
+        ok(
+            !HasClassAttribute->HasM(),
+            'HasM() returns false after M is cleared'
+        );
     }
 
     {
-        isa_ok( HasClassAttribute->Delegatee(), 'Delegatee',
-                'has a Delegetee object' );
-        is( HasClassAttribute->units(), 5,
-            'units() delegates to Delegatee and returns 5' );
+        isa_ok(
+            HasClassAttribute->Delegatee(), 'Delegatee',
+            'has a Delegetee object'
+        );
+        is(
+            HasClassAttribute->units(), 5,
+            'units() delegates to Delegatee and returns 5'
+        );
     }
 
     {
         my @ids = HasClassAttribute->IdsInMapping();
-        is( scalar @ids, 0,
-            'there are no keys in the mapping yet' );
+        is(
+            scalar @ids, 0,
+            'there are no keys in the mapping yet'
+        );
 
-        ok( ! HasClassAttribute->ExistsInMapping('a'),
-            'key does not exist in mapping' );
+        ok(
+            !HasClassAttribute->ExistsInMapping('a'),
+            'key does not exist in mapping'
+        );
 
         HasClassAttribute->SetMapping( a => 20 );
 
-        ok( HasClassAttribute->ExistsInMapping('a'),
-            'key does exist in mapping' );
+        ok(
+            HasClassAttribute->ExistsInMapping('a'),
+            'key does exist in mapping'
+        );
 
-        is( HasClassAttribute->GetMapping('a'), 20,
-            'value for a in mapping is 20' );
+        is(
+            HasClassAttribute->GetMapping('a'), 20,
+            'value for a in mapping is 20'
+        );
     }
 
     {
-        is( HasClassAttribute->Built(), 42,
-            'attribute with builder works' );
+        is(
+            HasClassAttribute->Built(), 42,
+            'attribute with builder works'
+        );
 
-        is( HasClassAttribute->LazyBuilt(), 42,
-            'attribute with lazy builder works' );
+        is(
+            HasClassAttribute->LazyBuilt(), 42,
+            'attribute with lazy builder works'
+        );
     }
 
     {
@@ -284,13 +333,15 @@ sub run_tests
         HasClassAttribute->Triggerish(84);
         is( HasClassAttribute->Triggerish(), 84, 'Triggerish is now 84' );
 
-        is_deeply( \@HasClassAttribute::Triggered,
-                   [ [ qw( HasClassAttribute 42 ) ],
-                     [ qw( HasClassAttribute 84 42 ) ],
-                   ],
-                   'trigger passes old value correctly' );
+        is_deeply(
+            \@HasClassAttribute::Triggered,
+            [
+                [qw( HasClassAttribute 42 )],
+                [qw( HasClassAttribute 84 42 )],
+            ],
+            'trigger passes old value correctly'
+        );
     }
 }
 
-
 1;