Fix the timing triggers are invoked

[gitmo/Mouse.git] / Makefile.PL

diff --git a/Makefile.PL b/Makefile.PL
index 095fb2e..20396d5 100755 (executable)
--- a/Makefile.PL
+++ b/Makefile.PL
@@ -1,4 +1,7 @@
+use strict;
+use warnings;
 use inc::Module::Install;
+use 5.008;
 
 name     'Mouse';
 all_from 'lib/Mouse.pm';
@@ -9,11 +12,15 @@ tests 't/*.t t/*/*.t';
 # > Fixed looks_like_number(undef) to return false for perl >= 5.009002
 requires 'Scalar::Util' => 1.14;
 
-build_requires 'Test::Exception';
-build_requires 'Test::More';
+test_requires 'Test::Exception' => 0.21;
+test_requires 'Test::More'      => 0.80;
 
 if ($Module::Install::AUTHOR) {
-    if (eval "package foo; use Moose; 1;") {
+    local @INC = ('lib', @INC);
+    require 'lib/Mouse.pm'; # for moose_version()
+    my $require_version = Mouse->moose_version;
+
+    if (eval{ require Moose; Moose->VERSION($require_version) }) {
         if (eval 'use Module::Install::AuthorTests; 1') {
             create_moose_compatibility_test();
             recursive_author_tests('xt');
@@ -21,8 +28,9 @@ if ($Module::Install::AUTHOR) {
             print "you don't have a M::I::AuthorTests.\n";
         }
     } else {
-        print "you don't have a moose. skipping moose compatibility test\n";
+        print "you don't have Moose $require_version. skipping moose compatibility test\n";
     }
+    system("author/generate-mouse-tiny.pl");
 }
 
 auto_include;
@@ -33,37 +41,46 @@ sub create_moose_compatibility_test {
     require File::Spec;
     require File::Basename;
 
+    print "Creating compatibility tests in xt/compatibility/* ...\n";
+
+    File::Path::rmtree(File::Spec->catfile('xt', 'compatibility'));
+
     # some test does not pass... currently skip it.
     my %SKIP_TEST = (
         '016-trigger.t'    => "trigger's argument is incompatble :(",
-        '020-load-class.t' => "&Moose::is_class_loaded doesn't exists",
-        '019-handles.t'    => 'incompatible',
-        '025-more-isa.t'   => 'Class::MOP::is_class_loaded is not compatible with Mouse::is_class_loaded',
         '029-new.t'        => 'Class->new(undef) incompatible',
         '010-isa-or.t'     => 'Mouse has a [BUG]',
         '044-attribute-metaclass.t' => 'Moose::Meta::Attribute does not have a "create"',
         '047-attribute-metaclass-role.t' => 'Moose::Meta::Attribute does not have a "create"',
         '201-squirrel.t'      => 'skip Squirrel',
         '202-squirrel-role.t' => 'Squirrel is ...',
-        '400-define-role.t'   => 'incompatibility',
         '600-tiny-tiny.t'     => "Moose doesn't support ::Tiny",
         '601-tiny-mouse.t'    => "Moose doesn't support ::Tiny",
         '602-mouse-tiny.t'    => "Moose doesn't support ::Tiny",
-        '031_roles_applied_in_create.t' => 'wtf?',
+        '031_roles_applied_in_create.t' => 't/lib/* classes are not Moose classes/roles',
     );
 
+    my @compat_tests;
+
     File::Find::find(
         {
             wanted => sub {
                 return unless -f $_;
+
+                return if /failing/; # skip tests in failing/ directories
+
                 my $basename = File::Basename::basename($_);
                 return if $basename =~ /^\./;
-                return if $SKIP_TEST{$basename};
-                
+
+                if(exists $SKIP_TEST{$basename}){
+                    print "# skip $basename because: $SKIP_TEST{$basename}\n";
+                    return;
+                }
+
                 my $dirname = File::Basename::dirname($_);
 
                 my $tmpdir = File::Spec->catfile('xt', 'compatibility', $dirname);
-                File::Path::make_path($tmpdir);
+                File::Path::mkpath($tmpdir);
 
                 my $tmpfile = File::Spec->catfile($tmpdir, $basename);
                 open my $wfh, '>', $tmpfile or die $!;
@@ -74,14 +91,20 @@ sub create_moose_compatibility_test {
                         close $rfh;
                         $s;
                     };
+                    $src =~ s/Mouse::(?:Util::)?is_class_loaded/Class::MOP::is_class_loaded/g;
+                    $src =~ s/Mouse::(?:Util::)?load_class/Class::MOP::load_class/g;
                     $src =~ s/Mouse/Moose/g;
                     $src;
                 };
                 close $wfh;
+                push @compat_tests, $tmpfile;
             },
             no_chdir => 1
         },
         't',
     );
+    print "Compatibility tests created.\n";
+
+    clean_files "@compat_tests";
 }