Put a watchdog on openpid.t: it has been found to hang in some Win32 smokes.

[p5sagit/p5-mst-13.2.git] / t / TEST

diff --git a/t/TEST b/t/TEST
index 6fb6201..19f8fb1 100755 (executable)
--- a/t/TEST
+++ b/t/TEST
@@ -23,16 +23,31 @@ my %dir_to_switch =
      '../ext/File-Glob/t' => '-I.. -MTestInit', # FIXME - tests assume t/
      );
 
+# I think in the end I'd like "not absolute" to be the default", as it saves
+# some fakery within TestInit which can peturb tests, and takes CPU.
+my %no_abs =
+    ('../ext/Pod-Parser' => 1,
+    );
+             
 my %temp_no_core =
     ('../ext/B-Debug' => 1,
      '../ext/Compress-Raw-Bzip2' => 1,
      '../ext/Compress-Raw-Zlib' => 1,
-     '../ext/Devel-PPPort' => 1,
-     '../ext/Encode' => 1,
+     '../cpan/Devel-PPPort' => 1,
+     '../cpan/Getopt-Long' => 1,
      '../ext/IO-Compress' => 1,
-     '../ext/IPC-SysV' => 1,
+     '../ext/Math-BigInt' => 1,
+     '../ext/Math-BigRat' => 1,
      '../ext/MIME-Base64' => 1,
+     '../cpan/NEXT' => 1,
+     '../ext/parent' => 1,
+     '../cpan/Parse-CPAN-Meta' => 1,
+     '../ext/Pod-Simple' => 1,
+     '../cpan/podlators' => 1,
+     '../ext/Test-Simple' => 1,
+     '../cpan/Tie-RefHash' => 1,
      '../ext/Time-HiRes' => 1,
+     '../ext/Unicode-Collate' => 1,
      '../ext/Unicode-Normalize' => 1,
     );
 
@@ -80,6 +95,9 @@ our $show_elapsed_time = $ENV{HARNESS_TIMER} || 0;
 }
 
 chdir 't' if -f 't/TEST';
+if (-f 'TEST' && -f 'harness' && -d '../lib') {
+    @INC = '../lib';
+}
 
 die "You need to run \"make test\" first to set things up.\n"
   unless -e 'perl' or -e 'perl.exe' or -e 'perl.pm';
@@ -175,12 +193,15 @@ sub _scan_test {
     my $dir = $1;
     my $testswitch = $dir_to_switch{$dir};
     if (!defined $testswitch) {
-       if ($test =~ s!^(\.\./ext/[^/]+)/t!t!) {
+       if ($test =~ s!^(\.\./(?:cpan|dist|ext)/[^/]+)/t!t!) {
            $run_dir = $1;
            $return_dir = '../../t';
            $lib = '../../lib';
            $perl = '../../t/perl';
-           $testswitch = "-I../.. -MTestInit=U2T,A";
+           $testswitch = "-I../.. -MTestInit=U2T";
+           if (!$no_abs{$run_dir}) {
+               $testswitch = $testswitch . ',A';
+           }
            if ($temp_no_core{$run_dir}) {
                $testswitch = $testswitch . ',NC';
            }
@@ -189,7 +210,7 @@ sub _scan_test {
        }
     }
 
-    my $utf8 = $::with_utf8 ? '-I$lib -Mutf8' : '';
+    my $utf8 = $::with_utf8 ? "-I$lib -Mutf8" : '';
 
     my %options = (
        perl => $perl,
@@ -268,15 +289,11 @@ sub _after_fork {
 }
 
 sub _run_test {
-    my($harness, $test, $type) = @_;
-    if (!defined $type) {
-       # To conform to the interface expected by exec in TAP::Harness
-       $type = 'perl';
-    }
+    my ($test, $type) = @_;
 
     my $options = _scan_test($test, $type);
-
-    $test = $options->{test}; # Might have changed if we're in ext/Foo
+    # $test might have changed if we're in ext/Foo, so don't use it anymore
+    # from now on. Use $options->{test} instead.
 
     _before_fork($options);
 
@@ -327,7 +344,7 @@ sub _tests_from_manifest {
     my $mani = '../MANIFEST';
     if (open(MANI, $mani)) {
        while (<MANI>) {
-           if (m!^(ext/(\S+)/+(?:[^/\s]+\.t|test\.pl)|lib/\S+?(?:\.t|test\.pl))\s!) {
+           if (m!^((?:cpan|dist|ext)/(\S+)/+(?:[^/\s]+\.t|test\.pl)|lib/\S+?(?:\.t|test\.pl))\s!) {
                my $t = $1;
                my $extension = $2;
                if (!$::core || $t =~ m!^lib/[a-z]!) {
@@ -357,7 +374,7 @@ unless (@ARGV) {
     # then comp, to validate that require works
     # then run, to validate that -M works
     # then we know we can -MTestInit for everything else, making life simpler
-    foreach my $dir (qw(base comp run cmd io op uni mro)) {
+    foreach my $dir (qw(base comp run cmd io re op uni mro)) {
        _find_tests($dir);
     }
     _find_tests("lib") unless $::core;
@@ -486,7 +503,7 @@ EOT
            $te = '';
        }
 
-        my $results = _run_test(undef, $test, $type);
+       my $results = _run_test($test, $type);
 
        my $failure;
        my $next = 0;
@@ -552,7 +569,7 @@ EOT
 
                            # SKIP is essentially the same as TODO for t/TEST
                            # this still conforms to TAP:
-                           # http://search.cpan.org/dist/TAP/TAP.pod
+                           # http://search.cpan.org/dist/TAP/TAP.pm
                            $extra and $istodo = $extra =~ /#\s*(?:TODO|SKIP)\b/;
                            $istodo = 1 if $todo{$num};
 
@@ -572,7 +589,7 @@ EOT
                    else {
                        # module tests are allowed extra output,
                        # because Test::Harness allows it
-                       next if $test =~ /^\W*(ext|lib)\b/;
+                       next if $test =~ /^\W*(cpan|dist|ext|lib)\b/;
                        $failure = "FAILED--unexpected output at test $next";
                        last;
                    }