Include method name in immutable methods (fixes #49680)
[gitmo/Class-MOP.git] / t / 070_immutable_metaclass.t
index 5b1a1ca..e6f716e 100644 (file)
-#!/usr/bin/perl
-
 use strict;
 use warnings;
 
-use Test::More tests => 73;
+use Test::More tests => 76;
 use Test::Exception;
 
-BEGIN {
-    use_ok('Class::MOP');
-}
+use Class::MOP;
 
 {
     package Foo;
-    
+
     use strict;
     use warnings;
     use metaclass;
-    
+
     __PACKAGE__->meta->add_attribute('bar');
-    
+
     package Bar;
-    
+
     use strict;
     use warnings;
     use metaclass;
-    
+
     __PACKAGE__->meta->superclasses('Foo');
 
-    __PACKAGE__->meta->add_attribute('baz');    
-    
+    __PACKAGE__->meta->add_attribute('baz');
+
     package Baz;
-    
+
     use strict;
     use warnings;
     use metaclass;
-    
+
     __PACKAGE__->meta->superclasses('Bar');
 
-    __PACKAGE__->meta->add_attribute('bah');    
+    __PACKAGE__->meta->add_attribute('bah');
 }
 
 {
     my $meta = Foo->meta;
-    is($meta->name, 'Foo', '... checking the Foo metaclass');
-    
-    ok($meta->is_mutable, '... our class is mutable');
-    ok(!$meta->is_immutable, '... our class is not immutable');    
+    my $original_metaclass_name = ref $meta;
 
-    lives_ok {
-        $meta->make_immutable();
-    } '... changed Foo to be immutable';
-    
-    ok(!$meta->make_immutable, '... make immutable now returns nothing');
-    
-    ok(!$meta->is_mutable, '... our class is no longer mutable');
-    ok($meta->is_immutable, '... our class is now immutable');    
-
-    isa_ok($meta, 'Class::MOP::Class');
-    
-    dies_ok { $meta->add_method()    } '... exception thrown as expected';
-    dies_ok { $meta->alias_method()  } '... exception thrown as expected';
+    is_deeply(
+        { $meta->immutable_options }, {},
+        'immutable_options is empty before a class is made_immutable'
+    );
+
+    $meta->make_immutable;
+
+    my $immutable_metaclass = $meta->_immutable_metaclass->meta;
+
+    my $immutable_class_name = $immutable_metaclass->name;
+
+    ok( !$immutable_class_name->is_mutable,  '... immutable_metaclass is not mutable' );
+    ok( $immutable_class_name->is_immutable, '... immutable_metaclass is immutable' );
+    is( $immutable_class_name->meta, $immutable_metaclass,
+        '... immutable_metaclass meta hack works' );
+
+    is_deeply(
+        { $meta->immutable_options },
+        {
+            inline_accessors   => 1,
+            inline_constructor => 1,
+            inline_destructor  => 0,
+            debug              => 0,
+            immutable_trait    => 'Class::MOP::Class::Immutable::Trait',
+            constructor_name   => 'new',
+            constructor_class  => 'Class::MOP::Method::Constructor',
+            destructor_class   => undef,
+        },
+        'immutable_options is empty before a class is made_immutable'
+    );
+
+    isa_ok( $meta, "Class::MOP::Class" );
+}
+
+{
+    my $meta = Foo->meta;
+    is( $meta->name, 'Foo', '... checking the Foo metaclass' );
+
+    ok( !$meta->is_mutable,    '... our class is not mutable' );
+    ok( $meta->is_immutable, '... our class is immutable' );
+
+    isa_ok( $meta, 'Class::MOP::Class' );
+
+    dies_ok { $meta->add_method() } '... exception thrown as expected';
+    dies_ok { $meta->alias_method() } '... exception thrown as expected';
     dies_ok { $meta->remove_method() } '... exception thrown as expected';
-    
-    dies_ok { $meta->add_attribute()    } '... exception thrown as expected';
+
+    dies_ok { $meta->add_attribute() } '... exception thrown as expected';
     dies_ok { $meta->remove_attribute() } '... exception thrown as expected';
-                        
-    dies_ok { $meta->add_package_symbol()    } '... exception thrown as expected';
-    dies_ok { $meta->remove_package_symbol() } '... exception thrown as expected';
+
+    dies_ok { $meta->add_package_symbol() }
+    '... exception thrown as expected';
+    dies_ok { $meta->remove_package_symbol() }
+    '... exception thrown as expected';
+
+    lives_ok { $meta->identifier() }
+    '... no exception for get_package_symbol special case';
 
     my @supers;
     lives_ok {
         @supers = $meta->superclasses;
-    } '... got the superclasses okay';
+    }
+    '... got the superclasses okay';
+
+    dies_ok { $meta->superclasses( ['UNIVERSAL'] ) }
+    '... but could not set the superclasses okay';
 
-    dies_ok { $meta->superclasses([ 'UNIVERSAL' ]) } '... but could not set the superclasses okay';
-    
     my $meta_instance;
     lives_ok {
         $meta_instance = $meta->get_meta_instance;
-    } '... got the meta instance okay';
-    isa_ok($meta_instance, 'Class::MOP::Instance');
-    is($meta_instance, $meta->get_meta_instance, '... and we know it is cached');
-    
+    }
+    '... got the meta instance okay';
+    isa_ok( $meta_instance, 'Class::MOP::Instance' );
+    is( $meta_instance, $meta->get_meta_instance,
+        '... and we know it is cached' );
+
     my @cpl;
     lives_ok {
         @cpl = $meta->class_precedence_list;
-    } '... got the class precedence list okay';    
+    }
+    '... got the class precedence list okay';
     is_deeply(
-    \@cpl,
-    [ 'Foo' ],
-    '... we just have ourselves in the class precedence list');
-    
+        \@cpl,
+        ['Foo'],
+        '... we just have ourselves in the class precedence list'
+    );
+
     my @attributes;
     lives_ok {
-        @attributes = $meta->compute_all_applicable_attributes;
-    } '... got the attribute list okay';
+        @attributes = $meta->get_all_attributes;
+    }
+    '... got the attribute list okay';
     is_deeply(
-    \@attributes,
-    [ $meta->get_attribute('bar') ],
-    '... got the right list of attributes');
+        \@attributes,
+        [ $meta->get_attribute('bar') ],
+        '... got the right list of attributes'
+    );
 }
 
 {
     my $meta = Bar->meta;
-    is($meta->name, 'Bar', '... checking the Bar metaclass');    
-    
-    ok($meta->is_mutable, '... our class is mutable');
-    ok(!$meta->is_immutable, '... our class is not immutable');    
+    is( $meta->name, 'Bar', '... checking the Bar metaclass' );
+
+    ok( $meta->is_mutable,    '... our class is mutable' );
+    ok( !$meta->is_immutable, '... our class is not immutable' );
 
     lives_ok {
         $meta->make_immutable();
-    } '... changed Bar to be immutable';
-    
-    ok(!$meta->make_immutable, '... make immutable now returns nothing');
-    
-    ok(!$meta->is_mutable, '... our class is no longer mutable');
-    ok($meta->is_immutable, '... our class is now immutable');    
-
-    isa_ok($meta, 'Class::MOP::Class');
-    
-    dies_ok { $meta->add_method()    } '... exception thrown as expected';
-    dies_ok { $meta->alias_method()  } '... exception thrown as expected';
+    }
+    '... changed Bar to be immutable';
+
+    ok( !$meta->make_immutable, '... make immutable now returns nothing' );
+
+    ok( !$meta->is_mutable,  '... our class is no longer mutable' );
+    ok( $meta->is_immutable, '... our class is now immutable' );
+
+    isa_ok( $meta, 'Class::MOP::Class' );
+
+    dies_ok { $meta->add_method() } '... exception thrown as expected';
+    dies_ok { $meta->alias_method() } '... exception thrown as expected';
     dies_ok { $meta->remove_method() } '... exception thrown as expected';
-    
-    dies_ok { $meta->add_attribute()    } '... exception thrown as expected';
+
+    dies_ok { $meta->add_attribute() } '... exception thrown as expected';
     dies_ok { $meta->remove_attribute() } '... exception thrown as expected';
-                        
-    dies_ok { $meta->add_package_symbol()    } '... exception thrown as expected';
-    dies_ok { $meta->remove_package_symbol() } '... exception thrown as expected';
+
+    dies_ok { $meta->add_package_symbol() }
+    '... exception thrown as expected';
+    dies_ok { $meta->remove_package_symbol() }
+    '... exception thrown as expected';
 
     my @supers;
     lives_ok {
         @supers = $meta->superclasses;
-    } '... got the superclasses okay';
+    }
+    '... got the superclasses okay';
+
+    dies_ok { $meta->superclasses( ['UNIVERSAL'] ) }
+    '... but could not set the superclasses okay';
 
-    dies_ok { $meta->superclasses([ 'UNIVERSAL' ]) } '... but could not set the superclasses okay';
-    
     my $meta_instance;
     lives_ok {
         $meta_instance = $meta->get_meta_instance;
-    } '... got the meta instance okay';
-    isa_ok($meta_instance, 'Class::MOP::Instance');
-    is($meta_instance, $meta->get_meta_instance, '... and we know it is cached');    
-    
+    }
+    '... got the meta instance okay';
+    isa_ok( $meta_instance, 'Class::MOP::Instance' );
+    is( $meta_instance, $meta->get_meta_instance,
+        '... and we know it is cached' );
+
     my @cpl;
     lives_ok {
         @cpl = $meta->class_precedence_list;
-    } '... got the class precedence list okay';    
+    }
+    '... got the class precedence list okay';
     is_deeply(
-    \@cpl,
-    [ 'Bar', 'Foo'],
-    '... we just have ourselves in the class precedence list');
-    
+        \@cpl,
+        [ 'Bar', 'Foo' ],
+        '... we just have ourselves in the class precedence list'
+    );
+
     my @attributes;
     lives_ok {
-        @attributes = $meta->compute_all_applicable_attributes;
-    } '... got the attribute list okay';
+        @attributes = $meta->get_all_attributes;
+    }
+    '... got the attribute list okay';
     is_deeply(
-    [ sort { $a->name cmp $b->name } @attributes ],
-    [ Foo->meta->get_attribute('bar'), $meta->get_attribute('baz') ],
-    '... got the right list of attributes');
+        [ sort { $a->name cmp $b->name } @attributes ],
+        [ Foo->meta->get_attribute('bar'), $meta->get_attribute('baz') ],
+        '... got the right list of attributes'
+    );
 }
 
 {
     my $meta = Baz->meta;
-    is($meta->name, 'Baz', '... checking the Baz metaclass');    
-    
-    ok($meta->is_mutable, '... our class is mutable');
-    ok(!$meta->is_immutable, '... our class is not immutable');    
+    is( $meta->name, 'Baz', '... checking the Baz metaclass' );
+
+    ok( $meta->is_mutable,    '... our class is mutable' );
+    ok( !$meta->is_immutable, '... our class is not immutable' );
 
     lives_ok {
         $meta->make_immutable();
-    } '... changed Baz to be immutable';
-    
-    ok(!$meta->make_immutable, '... make immutable now returns nothing');
-    
-    ok(!$meta->is_mutable, '... our class is no longer mutable');
-    ok($meta->is_immutable, '... our class is now immutable');    
-
-    isa_ok($meta, 'Class::MOP::Class');
-    
-    dies_ok { $meta->add_method()    } '... exception thrown as expected';
-    dies_ok { $meta->alias_method()  } '... exception thrown as expected';
+    }
+    '... changed Baz to be immutable';
+
+    ok( !$meta->make_immutable, '... make immutable now returns nothing' );
+
+    ok( !$meta->is_mutable,  '... our class is no longer mutable' );
+    ok( $meta->is_immutable, '... our class is now immutable' );
+
+    isa_ok( $meta, 'Class::MOP::Class' );
+
+    dies_ok { $meta->add_method() } '... exception thrown as expected';
+    dies_ok { $meta->alias_method() } '... exception thrown as expected';
     dies_ok { $meta->remove_method() } '... exception thrown as expected';
-    
-    dies_ok { $meta->add_attribute()    } '... exception thrown as expected';
+
+    dies_ok { $meta->add_attribute() } '... exception thrown as expected';
     dies_ok { $meta->remove_attribute() } '... exception thrown as expected';
-                        
-    dies_ok { $meta->add_package_symbol()    } '... exception thrown as expected';
-    dies_ok { $meta->remove_package_symbol() } '... exception thrown as expected';
+
+    dies_ok { $meta->add_package_symbol() }
+    '... exception thrown as expected';
+    dies_ok { $meta->remove_package_symbol() }
+    '... exception thrown as expected';
 
     my @supers;
     lives_ok {
         @supers = $meta->superclasses;
-    } '... got the superclasses okay';
+    }
+    '... got the superclasses okay';
+
+    dies_ok { $meta->superclasses( ['UNIVERSAL'] ) }
+    '... but could not set the superclasses okay';
 
-    dies_ok { $meta->superclasses([ 'UNIVERSAL' ]) } '... but could not set the superclasses okay';
-    
     my $meta_instance;
     lives_ok {
         $meta_instance = $meta->get_meta_instance;
-    } '... got the meta instance okay';
-    isa_ok($meta_instance, 'Class::MOP::Instance');
-    is($meta_instance, $meta->get_meta_instance, '... and we know it is cached');    
-    
+    }
+    '... got the meta instance okay';
+    isa_ok( $meta_instance, 'Class::MOP::Instance' );
+    is( $meta_instance, $meta->get_meta_instance,
+        '... and we know it is cached' );
+
     my @cpl;
     lives_ok {
         @cpl = $meta->class_precedence_list;
-    } '... got the class precedence list okay';    
+    }
+    '... got the class precedence list okay';
     is_deeply(
-    \@cpl,
-    [ 'Baz', 'Bar', 'Foo'],
-    '... we just have ourselves in the class precedence list');
-    
+        \@cpl,
+        [ 'Baz', 'Bar', 'Foo' ],
+        '... we just have ourselves in the class precedence list'
+    );
+
     my @attributes;
     lives_ok {
-        @attributes = $meta->compute_all_applicable_attributes;
-    } '... got the attribute list okay';
+        @attributes = $meta->get_all_attributes;
+    }
+    '... got the attribute list okay';
     is_deeply(
-    [ sort { $a->name cmp $b->name } @attributes ],
-    [ $meta->get_attribute('bah'), Foo->meta->get_attribute('bar'), Bar->meta->get_attribute('baz') ],
-    '... got the right list of attributes');
+        [ sort { $a->name cmp $b->name } @attributes ],
+        [
+            $meta->get_attribute('bah'), Foo->meta->get_attribute('bar'),
+            Bar->meta->get_attribute('baz')
+        ],
+        '... got the right list of attributes'
+    );
 }
 
+# This test probably needs to go last since it will muck up the Foo class
+{
+    my $meta = Foo->meta;
 
+    $meta->make_mutable;
+    $meta->make_immutable(
+        inline_accessors   => 0,
+        inline_constructor => 0,
+        constructor_name   => 'newer',
+    );
+
+    is_deeply(
+        { $meta->immutable_options },
+        {
+            inline_accessors   => 0,
+            inline_constructor => 0,
+            inline_destructor  => 0,
+            debug              => 0,
+            immutable_trait    => 'Class::MOP::Class::Immutable::Trait',
+            constructor_name   => 'newer',
+            constructor_class  => 'Class::MOP::Method::Constructor',
+            destructor_class   => undef,
+        },
+        'custom immutable_options are returned by immutable_options accessor'
+    );
+}