switch to Test::Exception, add failing test for MooseX::Method::Signatures

diff --git a/Makefile.PL b/Makefile.PL
index 0ee133f..5809596 100644 (file)
--- a/Makefile.PL
+++ b/Makefile.PL
@@ -5,7 +5,8 @@ all_from 'lib/MooseX/AlwaysCoerce.pm';
 author   'Rafael Kitover <rkitover@cpan.org>';
 license  'perl';
 
-test_requires 'Test::More';
+test_requires 'Test::More' => '0.94';
+test_requires 'Test::Exception';
 
 requires 'Moose';
 requires 'namespace::autoclean';

diff --git a/t/01-basic.t b/t/01-basic.t
index bfe227c..3d2e365 100644 (file)
--- a/t/01-basic.t
+++ b/t/01-basic.t
@@ -3,6 +3,7 @@ use strict;
 use warnings;
 
 use Test::More tests => 7;
+use Test::Exception;
 
 {
     package MyClass;
@@ -30,22 +31,18 @@ use Test::More tests => 7;
 
 ok( (my $instance = MyClass->new), 'instance' );
 
-eval { $instance->foo('bar') };
-is $@, "", 'attribute coercion ran';
+lives_ok { $instance->foo('bar') } 'attribute coercion ran';
 
-eval { $instance->bar('baz') };
-is $@, "", 'class attribute coercion ran';
+lives_ok { $instance->bar('baz') } 'class attribute coercion ran';
 
-eval { $instance->baz('quux') };
-ok( $@, 'class attribute coercion did not run with coerce => 0' );
+dies_ok { $instance->baz('quux') }
+    'class attribute coercion did not run with coerce => 0';
 
-undef $@;
+dies_ok { $instance->quux('mtfnpy') }
+    'attribute coercion did not run with coerce => 0';
 
-eval { $instance->quux('mtfnpy') };
-ok( $@, 'attribute coercion did not run with coerce => 0' );
+lives_ok { $instance->uncoerced_attr(10) }
+    'set attribute having type with no coercion and no coerce=0';
 
-eval { $instance->uncoerced_attr(10) };
-is $@, "", 'set attribute having type with no coercion and no coerce=0';
-
-eval { $instance->uncoerced_class_attr(10) };
-is $@, "", 'set class attribute having type with no coercion and no coerce=0';
+lives_ok { $instance->uncoerced_class_attr(10) }
+    'set class attribute having type with no coercion and no coerce=0';

diff --git a/t/02-mx-m-s.t b/t/02-mx-m-s.t
new file mode 100644 (file)
index 0000000..3501d34
--- /dev/null
+++ b/t/02-mx-m-s.t
@@ -0,0 +1,36 @@
+#!/usr/bin/env perl
+use strict;
+use warnings;
+
+use Test::More;
+use Test::Exception;
+
+eval { require MooseX::Method::Signatures };
+plan skip_all => "No MooseX::Method::Signatures" if $@;
+
+plan tests => 2;
+
+{
+    package MyClass;
+    use Moose;
+    use MooseX::Method::Signatures;
+    use MooseX::AlwaysCoerce;
+    use Moose::Util::TypeConstraints;
+
+    BEGIN {
+        subtype 'MyType', as 'Int';
+        coerce 'MyType', from 'Str', via { length $_ };
+
+        subtype 'Uncoerced', as 'Int';
+    }
+
+    method foo (MyType :$foo, Uncoerced :$bar) {
+        return "$foo $bar";
+    }
+}
+
+ok( (my $instance = MyClass->new), 'instance' );
+
+lives_and {
+    is $instance->foo(foo => "text", bar => 42), '4 42';
+} 'method called with coerced and uncoerced parameters';