throw an error on invalid builder names as well (RT#78479)

[gitmo/Moo.git] / t / method-generate-accessor.t

diff --git a/t/method-generate-accessor.t b/t/method-generate-accessor.t
index 9c16b72..256cc5c 100644 (file)
--- a/t/method-generate-accessor.t
+++ b/t/method-generate-accessor.t
@@ -11,6 +11,12 @@ my $gen = Method::Generate::Accessor->new;
   use Moo;
 }
 
+{
+  package WithOverload;
+  use overload '&{}' => sub { sub { 5 } };
+  sub new { bless {} }
+}
+
 $gen->generate_method('Foo' => 'one' => { is => 'ro' });
 
 $gen->generate_method('Foo' => 'two' => { is => 'rw' });
@@ -25,10 +31,67 @@ like(
   qr/Unknown is purple/, 'is purple rejected'
 );
 
+for my $setting (qw( default coerce )) {
+  like(
+    exception { $gen->generate_method('Foo' => 'four' => { is => 'ro', $setting => 5 }) },
+    qr/Invalid $setting/, "$setting - scalar rejected"
+  );
+
+  like(
+    exception { $gen->generate_method('Foo' => 'five' => { is => 'ro', $setting => [] }) },
+    qr/Invalid $setting/, "$setting - arrayref rejected"
+  );
+
+  like(
+    exception { $gen->generate_method('Foo' => 'five' => { is => 'ro', $setting => Foo->new }) },
+    qr/Invalid $setting/, "$setting - non-code-convertible object rejected"
+  );
+
+  is(
+    exception { $gen->generate_method('Foo' => 'six' => { is => 'ro', $setting => sub { 5 } }) },
+    undef, "$setting - coderef accepted"
+  );
+
+  is(
+    exception { $gen->generate_method('Foo' => 'seven' => { is => 'ro', $setting => bless sub { 5 } => 'Blah' }) },
+    undef, "$setting - blessed sub accepted"
+  );
+
+  is(
+    exception { $gen->generate_method('Foo' => 'eight' => { is => 'ro', $setting => WithOverload->new }) },
+    undef, "$setting - object with overloaded ->() accepted"
+  );
+
+  like(
+    exception { $gen->generate_method('Foo' => 'nine' => { is => 'ro', $setting => bless {} => 'Blah' }) },
+    qr/Invalid $setting/, "$setting - object rejected"
+  );
+}
+
+is(
+  exception { $gen->generate_method('Foo' => 'ten' => { is => 'ro', builder => '_build_ten' }) },
+  undef, 'builder - string accepted',
+);
+
+like(
+  exception { $gen->generate_method('Foo' => 'eleven' => { is => 'ro', builder => sub {} }) },
+  qr/Invalid builder/, 'builder - coderef rejected'
+);
+
+like(
+  exception { $gen->generate_method('Foo' => 'twelve' => { is => 'ro', builder => 'build:twelve' }) },
+  qr/Invalid builder/, 'builder - invalid name rejected',
+);
+
+is(
+  exception { $gen->generate_method('Foo' => 'thirteen' => { is => 'ro', builder => 'build::thirteen' }) },
+  undef, 'builder - fully-qualified name accepted',
+);
+
 my $foo = Foo->new(one => 1);
 
 is($foo->one, 1, 'ro reads');
-ok(!eval { $foo->one(-3); 1 }, 'ro dies on write attempt');
+ok(exception { $foo->one(-3) }, 'ro dies on write attempt');
 is($foo->one, 1, 'ro does not write');
 
 is($foo->two, undef, 'rw reads');