support attributes without isa, better tests, release

[gitmo/MooseX-AlwaysCoerce.git] / t / 04-parameterized-roles.t

diff --git a/t/04-parameterized-roles.t b/t/04-parameterized-roles.t
index 1a296d6..d735cdd 100644 (file)
--- a/t/04-parameterized-roles.t
+++ b/t/04-parameterized-roles.t
@@ -1,18 +1,15 @@
 #!/usr/bin/env perl
+
 use strict;
 use warnings;
-
 use Test::More;
+use Test::Exception;
 
-BEGIN {
-    if (eval { require MooseX::Role::Parameterized }) {
-        plan tests => 8;
-    } else {
-        plan skip_all => 'This test needs MooseX::Role::Parameterized';
-    }
+unless (eval { require MooseX::Role::Parameterized }) {
+    plan skip_all => 'This test needs MooseX::Role::Parameterized';
 }
 
-{
+eval <<'EOF';
     package Role;
     use MooseX::Role::Parameterized;
     use MooseX::AlwaysCoerce;
@@ -20,7 +17,6 @@ BEGIN {
 
     # I do nothing!
     role {};
-    use Moose::Util::TypeConstraints;
 
     subtype 'MyType', as 'Int';
     coerce 'MyType', from 'Str', via { length $_ };
@@ -38,23 +34,40 @@ BEGIN {
     has uncoerced_attr => (is => 'rw', isa => 'Uncoerced');
 
     class_has uncoerced_class_attr => (is => 'rw', isa => 'Uncoerced');
-}
 
-{
+    has untyped_attr => (is => 'rw');
+
+    class_has untyped_class_attr => (is => 'rw');
+
     package Foo;
     use Moose;
     with 'Role';
+EOF
+
+if ($@) {
+    plan skip_all =>
+'MooseX::ClassAttribute is currently incompatible with MooseX::Role::Parameterized';
 }
 
-package main;
-use Test::Exception;
-use Test::NoWarnings;
+plan tests => 10;
+
+eval 'use Test::NoWarnings';
 
 ok( (my $instance = MyClass->new), 'instance' );
 
-lives_ok { $instance->foo('bar') } 'attribute coercion ran';
+{
+    local $TODO = 'waiting on Moose changes for role support, and ClassAttribute changes for paramterized role support';
 
-lives_ok { $instance->bar('baz') } 'class attribute coercion ran';
+    lives_and {
+        $instance->foo('bar');
+        is $instance->foo, 3;
+    } 'attribute coercion ran';
+}
+
+lives_and {
+    $instance->bar('baz');
+    is $instance->bar, 3;
+} 'class attribute coercion ran';
 
 dies_ok { $instance->baz('quux') }
     'class attribute coercion did not run with coerce => 0';
@@ -62,10 +75,22 @@ dies_ok { $instance->baz('quux') }
 dies_ok { $instance->quux('mtfnpy') }
     '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';
-
-lives_ok { $instance->uncoerced_class_attr(10) }
-    'set class attribute having type with no coercion and no coerce=0';
-
-
+lives_and {
+    $instance->uncoerced_attr(10);
+    is $instance->uncoerced_attr(10), 10;
+} 'set attribute having type with no coercion and no coerce=0';
+
+lives_and {
+    $instance->uncoerced_class_attr(10);
+    is $instance->uncoerced_class_attr(10), 10;
+} 'set class attribute having type with no coercion and no coerce=0';
+
+lives_and {
+    $instance->untyped_attr(10);
+    is $instance->untyped_attr, 10;
+} 'set untyped attribute';
+
+lives_and {
+    $instance->untyped_class_attr(10);
+    is $instance->untyped_class_attr, 10;
+} 'set untyped class attribute';