Change documention for get and get to reflect actual behavior.
2shortplanks [Mon, 7 Sep 2009 10:08:09 +0000 (11:08 +0100)]
Add test to ensure scalar usage of ->get remains the same now it's
documented.

lib/Moose/Meta/Attribute/Native/Trait/Hash.pm
t/070_native_traits/203_trait_hash.t

index 9d57157..999bbc8 100644 (file)
@@ -63,13 +63,16 @@ L<Moose::Meta::Attribute::Native::MethodProvider::Hash>.
 
 =over 4
 
-=item B<get($key)>
+=item B<get($key, $key2, $key3...)>
 
-Returns an element of the hash by its key.
+Returns values from the hash.
 
-=item B<set($key)>
+In list context return a list of values in the hash for the given keys.
+In scalar context returns the value for the last key specified.
 
-Sets the element in the hash at the given key to the given value.
+=item B<set($key => $value, $key2 => $value2 ...)>
+
+Sets the elements in the hash to the given values.
 
 =item B<delete($key)>
 
index ef93bc4..7d63286 100644 (file)
@@ -3,7 +3,7 @@
 use strict;
 use warnings;
 
-use Test::More tests => 46;
+use Test::More tests => 47;
 use Test::Exception;
 use Test::Moose 'does_ok';
 
@@ -81,6 +81,9 @@ 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( scalar($stuff->get_option(qw( foo bar) )), "baz",
+       '... got last option in scalar context');
+
 lives_ok {
     $stuff->set_option( oink => "blah", xxy => "flop" );
 }