actually, there's no reason that overriding 'is' is unsafe
[gitmo/Moose.git] / t / 020_attributes / 022_illegal_options_for_inheritance.t
CommitLineData
7782e1da 1#!/usr/bin/perl
2
3use strict;
4use warnings;
5use Test::More;
ec835085 6use Test::Exception;
7782e1da 7
ec835085 8{
9 package Foo;
10 use Moose;
11
12 has foo => (
13 is => 'ro',
14 );
7782e1da 15
ec835085 16 has bar => (
17 clearer => 'clear_bar',
18 );
19}
7782e1da 20
21{
ec835085 22 package Foo::Sub;
7782e1da 23 use Moose;
24
ec835085 25 extends 'Foo';
7782e1da 26
d21262bd 27 ::lives_ok { has '+foo' => (is => 'rw') } "can override is";
ec835085 28 ::throws_ok { has '+foo' => (reader => 'bar') } qr/illegal/, "can't override reader";
29 ::lives_ok { has '+foo' => (clearer => 'baz') } "can override unspecified things";
30
31 ::throws_ok { has '+bar' => (clearer => 'quux') } qr/illegal/, "can't override clearer";
32 ::lives_ok { has '+bar' => (predicate => 'has_bar') } "can override unspecified things";
33}
34
35{
36 package Bar::Meta::Attribute;
37 use Moose::Role;
38
39 has my_illegal_option => (is => 'ro');
7782e1da 40
41 around illegal_options_for_inheritance => sub {
ec835085 42 return (shift->(@_), 'my_illegal_option');
7782e1da 43 };
ec835085 44}
7782e1da 45
ec835085 46{
7782e1da 47 package Bar;
48 use Moose;
49
ec835085 50 ::lives_ok {
51 has bar => (
52 traits => ['Bar::Meta::Attribute'],
53 my_illegal_option => 'FOO',
54 is => 'bare',
55 );
56 } "can use illegal options";
57
58 has baz => (
59 traits => ['Bar::Meta::Attribute'],
60 is => 'bare',
7782e1da 61 );
62}
63
ec835085 64{
65 package Bar::Sub;
66 use Moose;
67
68 extends 'Bar';
69
70 ::throws_ok { has '+bar' => (my_illegal_option => 'BAR') }
71 qr/illegal/,
72 "can't override illegal attribute";
73 ::lives_ok { has '+baz' => (my_illegal_option => 'BAR') }
74 "can add illegal option if superclass doesn't set it";
75}
76
7782e1da 77my $bar_attr = Bar->meta->get_attribute('bar');
ec835085 78ok((grep { $_ eq 'my_illegal_option' } $bar_attr->illegal_options_for_inheritance) > 0, '... added my_illegal_option as illegal option for inheritance');
7782e1da 79
80done_testing;