make the test style match the rest of the (modern) Moose tests

[gitmo/Moose.git] / t / 070_attribute_helpers / 203_trait_hash.t

diff --git a/t/070_attribute_helpers/203_trait_hash.t b/t/070_attribute_helpers/203_trait_hash.t
index b6d8553..f9b6fd3 100644 (file)
--- a/t/070_attribute_helpers/203_trait_hash.t
+++ b/t/070_attribute_helpers/203_trait_hash.t
@@ -17,11 +17,11 @@ BEGIN {
     use Moose::AttributeHelpers;
 
     has 'options' => (
-        traits    => [qw/Collection::Hash/],
-        is        => 'ro',
-        isa       => 'HashRef[Str]',
-        default   => sub { {} },
-        handles  => {
+        traits  => [qw/Collection::Hash/],
+        is      => 'ro',
+        isa     => 'HashRef[Str]',
+        default => sub { {} },
+        handles => {
             'set_option'       => 'set',
             'get_option'       => 'get',
             'has_options'      => 'empty',
@@ -33,15 +33,15 @@ BEGIN {
             'option_accessor'  => 'accessor',
             'key_value'        => 'kv',
             'options_elements' => 'elements',
-            'quantity' => [ accessor => ['quantity'] ],
+            'quantity'         => [ accessor => ['quantity'] ],
         },
     );
 }
 
 my $stuff = Stuff->new();
-isa_ok($stuff, 'Stuff');
+isa_ok( $stuff, 'Stuff' );
 
-can_ok($stuff, $_) for qw[
+can_ok( $stuff, $_ ) for qw[
     set_option
     get_option
     has_options
@@ -54,103 +54,121 @@ can_ok($stuff, $_) for qw[
     option_accessor
 ];
 
-ok(!$stuff->has_options, '... we have no options');
-is($stuff->num_options, 0, '... we have no options');
+ok( !$stuff->has_options, '... we have no options' );
+is( $stuff->num_options, 0, '... we have no options' );
 
-is_deeply($stuff->options, {}, '... no options yet');
-ok(!$stuff->has_option('foo'), '... we have no foo option');
+is_deeply( $stuff->options, {}, '... no options yet' );
+ok( !$stuff->has_option('foo'), '... we have no foo option' );
 
 lives_ok {
-    $stuff->set_option(foo => 'bar');
-} '... set the option okay';
+    $stuff->set_option( foo => 'bar' );
+}
+'... set the option okay';
 
-ok($stuff->is_defined('foo'), '... foo is defined');
+ok( $stuff->is_defined('foo'), '... foo is defined' );
 
-ok($stuff->has_options, '... we have options');
-is($stuff->num_options, 1, '... we have 1 option(s)');
-ok($stuff->has_option('foo'), '... we have a foo option');
-is_deeply($stuff->options, { foo => 'bar' }, '... got options now');
+ok( $stuff->has_options, '... we have options' );
+is( $stuff->num_options, 1, '... we have 1 option(s)' );
+ok( $stuff->has_option('foo'), '... we have a foo option' );
+is_deeply( $stuff->options, { foo => 'bar' }, '... got options now' );
 
 lives_ok {
-    $stuff->set_option(bar => 'baz');
-} '... set the option okay';
+    $stuff->set_option( bar => 'baz' );
+}
+'... set the option okay';
 
-is($stuff->num_options, 2, '... we have 2 option(s)');
-is_deeply($stuff->options, { foo => 'bar', bar => 'baz' }, '... got more options now');
+is( $stuff->num_options, 2, '... we have 2 option(s)' );
+is_deeply( $stuff->options, { foo => 'bar', bar => 'baz' },
+    '... got more options now' );
 
-is($stuff->get_option('foo'), 'bar', '... got the right option');
+is( $stuff->get_option('foo'), 'bar', '... got the right option' );
 
-is_deeply([ $stuff->get_option(qw(foo bar)) ], [qw(bar baz)], "get multiple options at once");
+is_deeply( [ $stuff->get_option(qw(foo bar)) ], [qw(bar baz)],
+    "get multiple options at once" );
 
 lives_ok {
-    $stuff->set_option(oink => "blah", xxy => "flop");
-} '... set the option okay';
+    $stuff->set_option( oink => "blah", xxy => "flop" );
+}
+'... set the option okay';
 
-is($stuff->num_options, 4, "4 options");
-is_deeply([ $stuff->get_option(qw(foo bar oink xxy)) ], [qw(bar baz blah flop)], "get multiple options at once");
+is( $stuff->num_options, 4, "4 options" );
+is_deeply( [ $stuff->get_option(qw(foo bar oink xxy)) ],
+    [qw(bar baz blah flop)], "get multiple options at once" );
 
 lives_ok {
     $stuff->delete_option('bar');
-} '... deleted the option okay';
+}
+'... deleted the option okay';
 
 lives_ok {
     $stuff->delete_option('oink');
-} '... deleted the option okay';
+}
+'... deleted the option okay';
 
 lives_ok {
     $stuff->delete_option('xxy');
-} '... deleted the option okay';
+}
+'... deleted the option okay';
 
-is($stuff->num_options, 1, '... we have 1 option(s)');
-is_deeply($stuff->options, { foo => 'bar' }, '... got more options now');
+is( $stuff->num_options, 1, '... we have 1 option(s)' );
+is_deeply( $stuff->options, { foo => 'bar' }, '... got more options now' );
 
 $stuff->clear_options;
 
-is_deeply($stuff->options, { }, "... cleared options" );
+is_deeply( $stuff->options, {}, "... cleared options" );
 
 lives_ok {
     $stuff->quantity(4);
-} '... options added okay with defaults';
+}
+'... options added okay with defaults';
 
-is($stuff->quantity, 4, 'reader part of curried accessor works');
+is( $stuff->quantity, 4, 'reader part of curried accessor works' );
 
-is_deeply($stuff->options, {quantity => 4}, '... returns what we expect');
+is_deeply( $stuff->options, { quantity => 4 }, '... returns what we expect' );
 
 lives_ok {
-    Stuff->new(options => { foo => 'BAR' });
-} '... good constructor params';
+    Stuff->new( options => { foo => 'BAR' } );
+}
+'... good constructor params';
 
 ## check some errors
 
 dies_ok {
-    $stuff->set_option(bar => {});
-} '... could not add a hash ref where an string is expected';
+    $stuff->set_option( bar => {} );
+}
+'... could not add a hash ref where an string is expected';
 
 dies_ok {
-    Stuff->new(options => { foo => [] });
-} '... bad constructor params';
+    Stuff->new( options => { foo => [] } );
+}
+'... bad constructor params';
 
 ## test the meta
 
 my $options = $stuff->meta->get_attribute('options');
-does_ok($options, 'Moose::AttributeHelpers::Trait::Collection::Hash');
-
-is_deeply($options->handles, {
-    'set_option'       => 'set',
-    'get_option'       => 'get',
-    'has_options'      => 'empty',
-    'num_options'      => 'count',
-    'clear_options'    => 'clear',
-    'delete_option'    => 'delete',
-    'has_option'       => 'exists',
-    'is_defined'       => 'defined',
-    'option_accessor'  => 'accessor',
-    'key_value'        => 'kv',
-    'options_elements' => 'elements',
-    'quantity' => [ accessor => ['quantity'] ],
-}, '... got the right handles mapping');
-
-is($options->type_constraint->type_parameter, 'Str', '... got the right container type');
+does_ok( $options, 'Moose::AttributeHelpers::Trait::Collection::Hash' );
+
+is_deeply(
+    $options->handles,
+    {
+        'set_option'       => 'set',
+        'get_option'       => 'get',
+        'has_options'      => 'empty',
+        'num_options'      => 'count',
+        'clear_options'    => 'clear',
+        'delete_option'    => 'delete',
+        'has_option'       => 'exists',
+        'is_defined'       => 'defined',
+        'option_accessor'  => 'accessor',
+        'key_value'        => 'kv',
+        'options_elements' => 'elements',
+        'quantity'         => [ accessor => ['quantity'] ],
+    },
+    '... got the right handles mapping'
+);
+
+is( $options->type_constraint->type_parameter, 'Str',
+    '... got the right container type' );
 
 $stuff->set_option( oink => "blah", xxy => "flop" );
 my @key_value = $stuff->key_value;