test array delegation edge cases

[gitmo/Moose.git] / t / 030_roles / 004_role_composition_errors.t

diff --git a/t/030_roles/004_role_composition_errors.t b/t/030_roles/004_role_composition_errors.t
index 0325dc4..b2c58f4 100644 (file)
--- a/t/030_roles/004_role_composition_errors.t
+++ b/t/030_roles/004_role_composition_errors.t
@@ -3,86 +3,156 @@
 use strict;
 use warnings;
 
-use Test::More tests => 10;
+use Test::More;
 use Test::Exception;
 
 
-
 {
+
     package Foo::Role;
     use Moose::Role;
-    
+
     requires 'foo';
 }
 
 is_deeply(
     [ sort Foo::Role->meta->get_required_method_list ],
-    [ 'foo' ],
-    '... the Foo::Role has a required method (foo)');
+    ['foo'],
+    '... the Foo::Role has a required method (foo)'
+);
 
 # classes which does not implement required method
 {
+
     package Foo::Class;
     use Moose;
-    
-    ::dies_ok { with('Foo::Role') } '... no foo method implemented by Foo::Class';
+
+    ::dies_ok { with('Foo::Role') }
+        '... no foo method implemented by Foo::Class';
 }
 
 # class which does implement required method
 {
+
     package Bar::Class;
     use Moose;
-    
-    ::dies_ok  { with('Foo::Class') } '... cannot consume a class, it must be a role';
-    ::lives_ok { with('Foo::Role')  } '... has a foo method implemented by Bar::Class';
-    
-    sub foo { 'Bar::Class::foo' }
+
+    ::dies_ok { with('Foo::Class') }
+        '... cannot consume a class, it must be a role';
+    ::lives_ok { with('Foo::Role') }
+        '... has a foo method implemented by Bar::Class';
+
+    sub foo {'Bar::Class::foo'}
 }
 
 # role which does implement required method
 {
+
     package Bar::Role;
     use Moose::Role;
-    
-    ::lives_ok { with('Foo::Role') } '... has a foo method implemented by Bar::Role';
-    
-    sub foo { 'Bar::Role::foo' }
+
+    ::lives_ok { with('Foo::Role') }
+        '... has a foo method implemented by Bar::Role';
+
+    sub foo {'Bar::Role::foo'}
 }
 
 is_deeply(
     [ sort Bar::Role->meta->get_required_method_list ],
     [],
-    '... the Bar::Role has not inherited the required method from Foo::Role');
+    '... the Bar::Role has not inherited the required method from Foo::Role'
+);
 
 # role which does not implement required method
 {
+
     package Baz::Role;
     use Moose::Role;
-    
-    ::lives_ok { with('Foo::Role') } '... no foo method implemented by Baz::Role';
+
+    ::lives_ok { with('Foo::Role') }
+        '... no foo method implemented by Baz::Role';
 }
 
 is_deeply(
     [ sort Baz::Role->meta->get_required_method_list ],
-    [ 'foo' ],
-    '... the Baz::Role has inherited the required method from Foo::Role');
-    
+    ['foo'],
+    '... the Baz::Role has inherited the required method from Foo::Role'
+);
+
 # classes which does not implement required method
 {
+
     package Baz::Class;
     use Moose;
 
-    ::dies_ok { with('Baz::Role') } '... no foo method implemented by Baz::Class2';
+    ::dies_ok { with('Baz::Role') }
+        '... no foo method implemented by Baz::Class2';
 }
 
 # class which does implement required method
 {
+
     package Baz::Class2;
     use Moose;
 
-    ::lives_ok { with('Baz::Role') } '... has a foo method implemented by Baz::Class2';
+    ::lives_ok { with('Baz::Role') }
+        '... has a foo method implemented by Baz::Class2';
+
+    sub foo {'Baz::Class2::foo'}
+}
+
+
+{
+    package Quux::Role;
+    use Moose::Role;
+
+    requires qw( meth1 meth2 meth3 meth4 );
+}
 
-    sub foo { 'Baz::Class2::foo' }
-}    
-    
+# RT #41119
+{
+
+    package Quux::Class;
+    use Moose;
+
+    ::throws_ok { with('Quux::Role') }
+        qr/\Q'Quux::Role' requires the methods 'meth1', 'meth2', 'meth3', and 'meth4' to be implemented by 'Quux::Class'/,
+        'exception mentions all the missing required methods at once';
+}
+
+{
+    package Quux::Class2;
+    use Moose;
+
+    sub meth1 { }
+
+    ::throws_ok { with('Quux::Role') }
+        qr/'Quux::Role' requires the methods 'meth2', 'meth3', and 'meth4' to be implemented by 'Quux::Class2'/,
+        'exception mentions all the missing required methods at once, but not the one that exists';
+}
+
+{
+    package Quux::Class3;
+    use Moose;
+
+    has 'meth1' => ( is => 'ro' );
+    has 'meth2' => ( is => 'ro' );
+
+    ::throws_ok { with('Quux::Role') }
+        qr/'Quux::Role' requires the methods 'meth3' and 'meth4' to be implemented by 'Quux::Class3'/,
+        'exception mentions all the missing methods at once, but not the accessors';
+}
+
+{
+    package Quux::Class4;
+    use Moose;
+
+    sub meth1 { }
+    has 'meth2' => ( is => 'ro' );
+
+    ::throws_ok { with('Quux::Role') }
+        qr/'Quux::Role' requires the methods 'meth3' and 'meth4' to be implemented by 'Quux::Class4'/,
+        'exception mentions all the require methods that are accessors at once, as well as missing methods, but not the one that exists';
+}
 
+done_testing;