From: Michael G. Schwern <schwern@pobox.com>
Date: Sun, 27 Jun 2010 23:50:39 +0000 (-0700)
Subject: Only coerce if the accessor's type can coerce.
X-Git-Tag: 0.05~2
X-Git-Url: http://git.shadowcat.co.uk/gitweb/gitweb.cgi?a=commitdiff_plain;h=e930b397d88d9bbb04a4186c387485ca2aaf5e23;p=gitmo%2FMooseX-AlwaysCoerce.git

Only coerce if the accessor's type can coerce.

Otherwise the Moose will get angry.
---

diff --git a/t/01-basic.t.orig b/t/01-basic.t.orig
new file mode 100644
index 0000000..87ca3c0
--- /dev/null
+++ b/t/01-basic.t.orig
@@ -0,0 +1,51 @@
+#!/usr/bin/env perl
+use strict;
+use warnings;
+
+use Test::More tests => 7;
+
+{
+    package MyClass;
+    use Moose;
+    use MooseX::AlwaysCoerce;
+    use Moose::Util::TypeConstraints;
+
+    subtype 'MyType', as 'Int';
+    coerce 'MyType', from 'Str', via { length $_ };
+
+    subtype 'Uncoerced', as 'Int';
+
+    has foo => (is => 'rw', isa => 'MyType');
+
+    class_has bar => (is => 'rw', isa => 'MyType');
+
+    class_has baz => (is => 'rw', isa => 'MyType', coerce => 0);
+
+    has quux => (is => 'rw', isa => 'MyType', coerce => 0);
+
+    has uncoerced_attr => (is => 'rw', isa => 'Uncoerced');
+
+    class_has uncoerced_class_attr => (is => 'rw', isa => 'Uncoerced');
+}
+
+ok( (my $instance = MyClass->new), 'instance' );
+
+eval { $instance->foo('bar') };
+ok( (!$@), 'attribute coercion ran' );
+
+eval { $instance->bar('baz') };
+ok( (!$@), 'class attribute coercion ran' );
+
+eval { $instance->baz('quux') };
+ok( $@, 'class attribute coercion did not run with coerce => 0' );
+
+undef $@;
+
+eval { $instance->quux('mtfnpy') };
+ok( $@, 'attribute coercion did not run with 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';