This is a live mirror of the Perl 5 development currently hosted at https://github.com/perl/perl5
Update Test-Simple to alpha 076
authorChad Granum <chad.granum@dreamhost.com>
Sat, 22 Nov 2014 19:58:05 +0000 (11:58 -0800)
committerJames E Keenan <jkeenan@cpan.org>
Sat, 22 Nov 2014 21:18:39 +0000 (16:18 -0500)
For: RT #123277

15 files changed:
MANIFEST
cpan/Test-Simple/lib/Test/Builder.pm
cpan/Test-Simple/lib/Test/Builder/Module.pm
cpan/Test-Simple/lib/Test/Builder/Tester.pm
cpan/Test-Simple/lib/Test/Builder/Tester/Color.pm
cpan/Test-Simple/lib/Test/More.pm
cpan/Test-Simple/lib/Test/More/Tools.pm
cpan/Test-Simple/lib/Test/Simple.pm
cpan/Test-Simple/lib/Test/Stream.pm
cpan/Test-Simple/lib/Test/Stream/Context.pm
cpan/Test-Simple/lib/Test/Stream/IOSets.pm
cpan/Test-Simple/lib/Test/Tester.pm
cpan/Test-Simple/lib/Test/use/ok.pm
cpan/Test-Simple/lib/ok.pm
cpan/Test-Simple/t/Legacy/fork_die.t [new file with mode: 0644]

index d1b2c7c..bd4a00a 100644 (file)
--- a/MANIFEST
+++ b/MANIFEST
@@ -2371,6 +2371,7 @@ cpan/Test-Simple/t/Legacy/fail-more.t                     Test::Simple Test
 cpan/Test-Simple/t/Legacy/fail_one.t                   Test::Simple Test
 cpan/Test-Simple/t/Legacy/fail.t                       Test::Simple Test
 cpan/Test-Simple/t/Legacy/filehandles.t                        Test::Simple Test
 cpan/Test-Simple/t/Legacy/fail_one.t                   Test::Simple Test
 cpan/Test-Simple/t/Legacy/fail.t                       Test::Simple Test
 cpan/Test-Simple/t/Legacy/filehandles.t                        Test::Simple Test
+cpan/Test-Simple/t/Legacy/fork_die.t                   Test::Simple Test
 cpan/Test-Simple/t/Legacy/fork_in_subtest.t                    Test::Simple Test
 cpan/Test-Simple/t/Legacy/fork.t                       Test::Simple Test
 cpan/Test-Simple/t/Legacy/harness_active.t                     Test::Simple Test
 cpan/Test-Simple/t/Legacy/fork_in_subtest.t                    Test::Simple Test
 cpan/Test-Simple/t/Legacy/fork.t                       Test::Simple Test
 cpan/Test-Simple/t/Legacy/harness_active.t                     Test::Simple Test
index 1a28d72..aa9a417 100644 (file)
@@ -4,7 +4,7 @@ use 5.008001;
 use strict;
 use warnings;
 
 use strict;
 use warnings;
 
-our $VERSION = '1.301001_075';
+our $VERSION = '1.301001_076';
 $VERSION = eval $VERSION;    ## no critic (BuiltinFunctions::ProhibitStringyEval)
 
 
 $VERSION = eval $VERSION;    ## no critic (BuiltinFunctions::ProhibitStringyEval)
 
 
index 2ad2454..79340ed 100644 (file)
@@ -8,7 +8,7 @@ use Test::Builder 0.99;
 require Exporter;
 our @ISA = qw(Exporter);
 
 require Exporter;
 our @ISA = qw(Exporter);
 
-our $VERSION = '1.301001_075';
+our $VERSION = '1.301001_076';
 $VERSION = eval $VERSION;      ## no critic (BuiltinFunctions::ProhibitStringyEval)
 
 
 $VERSION = eval $VERSION;      ## no critic (BuiltinFunctions::ProhibitStringyEval)
 
 
index 28c0113..dfdfc5e 100644 (file)
@@ -1,7 +1,7 @@
 package Test::Builder::Tester;
 
 use strict;
 package Test::Builder::Tester;
 
 use strict;
-our $VERSION = '1.301001_075';
+our $VERSION = '1.301001_076';
 $VERSION = eval $VERSION;    ## no critic (BuiltinFunctions::ProhibitStringyEval)
 
 use Test::Stream 1.301001 '-internal';
 $VERSION = eval $VERSION;    ## no critic (BuiltinFunctions::ProhibitStringyEval)
 
 use Test::Stream 1.301001 '-internal';
index e8dfa85..6498c6a 100644 (file)
@@ -1,7 +1,7 @@
 package Test::Builder::Tester::Color;
 
 use strict;
 package Test::Builder::Tester::Color;
 
 use strict;
-our $VERSION = '1.301001_075';
+our $VERSION = '1.301001_076';
 $VERSION = eval $VERSION;    ## no critic (BuiltinFunctions::ProhibitStringyEval)
 
 use Test::Stream 1.301001 '-internal';
 $VERSION = eval $VERSION;    ## no critic (BuiltinFunctions::ProhibitStringyEval)
 
 use Test::Stream 1.301001 '-internal';
index fcbf4c5..d1b7e65 100644 (file)
@@ -4,7 +4,7 @@ use 5.008001;
 use strict;
 use warnings;
 
 use strict;
 use warnings;
 
-our $VERSION = '1.301001_075';
+our $VERSION = '1.301001_076';
 $VERSION = eval $VERSION;    ## no critic (BuiltinFunctions::ProhibitStringyEval)
 
 use Test::Stream 1.301001 '-internal';
 $VERSION = eval $VERSION;    ## no critic (BuiltinFunctions::ProhibitStringyEval)
 
 use Test::Stream 1.301001 '-internal';
index 7357f35..98027cc 100644 (file)
@@ -334,6 +334,8 @@ sub subtest {
     $ctx->clear;
     my $todo = $ctx->hide_todo;
 
     $ctx->clear;
     my $todo = $ctx->hide_todo;
 
+    my $pid = $$;
+
     my ($succ, $err) = try {
         {
             no warnings 'once';
     my ($succ, $err) = try {
         {
             no warnings 'once';
@@ -352,6 +354,19 @@ sub subtest {
         }
     };
 
         }
     };
 
+    if ($$ != $pid && !$ctx->stream->_use_fork) {
+        warn <<"        EOT";
+Subtest finished with a new PID ($$ vs $pid) while forking support was turned off!
+This is almost certainly not what you wanted. Did you fork and forget to exit?
+        EOT
+
+        # Did the forked process try to exit via die?
+        die $err unless $succ;
+    }
+
+    # If a subtest forked, then threw an exception, we need to propogate that right away.
+    die $err unless $succ || $$ == $pid || $err->isa('Test::Stream::Event');
+
     $ctx->set;
     $ctx->restore_todo($todo);
     # This sends the subtest event
     $ctx->set;
     $ctx->restore_todo($todo);
     # This sends the subtest event
index c5e6808..297c490 100644 (file)
@@ -5,10 +5,10 @@ use 5.008001;
 use strict;
 use warnings;
 
 use strict;
 use warnings;
 
-our $VERSION = '1.301001_075';
+our $VERSION = '1.301001_076';
 $VERSION = eval $VERSION;    ## no critic (BuiltinFunctions::ProhibitStringyEval)
 
 $VERSION = eval $VERSION;    ## no critic (BuiltinFunctions::ProhibitStringyEval)
 
-use Test::Stream 1.301001_075 '-internal';
+use Test::Stream 1.301001_076 '-internal';
 use Test::Stream::Toolset;
 
 use Test::Stream::Exporter;
 use Test::Stream::Toolset;
 
 use Test::Stream::Exporter;
index 789544d..6decda3 100644 (file)
@@ -2,7 +2,7 @@ package Test::Stream;
 use strict;
 use warnings;
 
 use strict;
 use warnings;
 
-our $VERSION = '1.301001_075';
+our $VERSION = '1.301001_076';
 $VERSION = eval $VERSION;    ## no critic (BuiltinFunctions::ProhibitStringyEval)
 
 use Test::Stream::Context qw/context/;
 $VERSION = eval $VERSION;    ## no critic (BuiltinFunctions::ProhibitStringyEval)
 
 use Test::Stream::Context qw/context/;
index 5b17d42..51b89e2 100644 (file)
@@ -164,10 +164,15 @@ sub _find_context {
     my $level = 2 + $add + $tb;
     my ($package, $file, $line, $subname) = caller($level);
 
     my $level = 2 + $add + $tb;
     my ($package, $file, $line, $subname) = caller($level);
 
-    return unless $package;
-
-    while ($package eq 'Test::Builder') {
-        ($package, $file, $line, $subname) = caller(++$level);
+    if ($package) {
+        while ($package eq 'Test::Builder') {
+            ($package, $file, $line, $subname) = caller(++$level);
+        }
+    }
+    else {
+        while (!$package) {
+            ($package, $file, $line, $subname) = caller(--$level);
+        }
     }
 
     return unless $package;
     }
 
     return unless $package;
index ae86277..e2352ef 100644 (file)
@@ -79,7 +79,7 @@ sub _copy_io_layers {
 }
 
 sub _autoflush {
 }
 
 sub _autoflush {
-    my($fh) = shift;
+    my($fh) = pop;
     my $old_fh = select $fh;
     $| = 1;
     select $old_fh;
     my $old_fh = select $fh;
     $| = 1;
     select $old_fh;
index c0a5cd9..48e6c7d 100644 (file)
@@ -16,7 +16,7 @@ require Exporter;
 
 use vars qw( @ISA @EXPORT $VERSION );
 
 
 use vars qw( @ISA @EXPORT $VERSION );
 
-our $VERSION = '1.301001_075';
+our $VERSION = '1.301001_076';
 $VERSION = eval $VERSION;    ## no critic (BuiltinFunctions::ProhibitStringyEval)
 
 @EXPORT  = qw( run_tests check_tests check_test cmp_results show_space );
 $VERSION = eval $VERSION;    ## no critic (BuiltinFunctions::ProhibitStringyEval)
 
 @EXPORT  = qw( run_tests check_tests check_test cmp_results show_space );
index 7e041dc..b1ac438 100644 (file)
@@ -3,7 +3,7 @@ use strict;
 use warnings;
 use 5.005;
 
 use warnings;
 use 5.005;
 
-our $VERSION = '1.301001_075';
+our $VERSION = '1.301001_076';
 $VERSION = eval $VERSION;    ## no critic (BuiltinFunctions::ProhibitStringyEval)
 
 use Test::Stream 1.301001 '-internal';
 $VERSION = eval $VERSION;    ## no critic (BuiltinFunctions::ProhibitStringyEval)
 
 use Test::Stream 1.301001 '-internal';
index b6b51e4..18c6d2c 100644 (file)
@@ -6,7 +6,7 @@ use Test::Stream 1.301001 '-internal';
 use Test::More 1.301001 ();
 use Test::Stream::Carp qw/croak/;
 
 use Test::More 1.301001 ();
 use Test::Stream::Carp qw/croak/;
 
-our $VERSION = '1.301001_075';
+our $VERSION = '1.301001_076';
 $VERSION = eval $VERSION;    ## no critic (BuiltinFunctions::ProhibitStringyEval)
 
 sub import {
 $VERSION = eval $VERSION;    ## no critic (BuiltinFunctions::ProhibitStringyEval)
 
 sub import {
diff --git a/cpan/Test-Simple/t/Legacy/fork_die.t b/cpan/Test-Simple/t/Legacy/fork_die.t
new file mode 100644 (file)
index 0000000..d649e1a
--- /dev/null
@@ -0,0 +1,79 @@
+use strict;
+use warnings;
+
+use Config;
+
+BEGIN {
+    my $Can_Fork = $Config{d_fork} ||
+                   (($^O eq 'MSWin32' || $^O eq 'NetWare') and
+                    $Config{useithreads} and
+                    $Config{ccflags} =~ /-DPERL_IMPLICIT_SYS/
+                   );
+
+    if( !$Can_Fork ) {
+        require Test::More;
+        Test::More::plan(skip_all => "This system cannot fork");
+        exit 0;
+    }
+    elsif ($^O eq 'MSWin32' && $] == 5.010000) {
+        require Test::More;
+        Test::More::plan('skip_all' => "5.10 has fork/threading issues that break fork on win32");
+        exit 0;
+    }
+}
+
+# The failure case for this test is producing 2 results, 1 pass and 1 fail,
+# both with the same test number. If this test file does anything other than 1
+# (non-indented) result that passes, it has failed in one way or another.
+use Test::More tests => 1;
+use Test::Stream qw/context/;
+
+my $line;
+
+subtest do_it => sub {
+    ok(1, "Pass!");
+
+    my ($read, $write);
+    pipe($read, $write) || die "Could not open pipe";
+
+    my $pid = fork();
+    die "Forking failed!" unless defined $pid;
+
+    unless($pid) {
+        close($read);
+        Test::Stream::IOSets->_autoflush($write);
+        my $ctx = context();
+        my $handles = $ctx->stream->io_sets->init_encoding('legacy');
+        $handles->[0] = $write;
+        $handles->[1] = $write;
+        $handles->[2] = $write;
+        *STDERR = $write;
+        *STDOUT = $write;
+
+        die "This process did something wrong!"; BEGIN { $line = __LINE__ };
+    }
+    close($write);
+
+    waitpid($pid, 0);
+    ok($?, "Process exited with failure");
+
+    {
+        local $SIG{ALRM} = sub { die "Read Timeout\n" };
+        alarm 2;
+        my @output = map {chomp($_); $_} <$read>;
+        alarm 0;
+        is_deeply(
+            \@output,
+            [
+                "Subtest finished with a new PID ($pid vs $$) while forking support was turned off!",
+                'This is almost certainly not what you wanted. Did you fork and forget to exit?',
+                "This process did something wrong! at t/Legacy/fork_die.t line $line.",
+            ],
+            "Got warning and exception, nothing else"
+       );
+    }
+
+    ok(1, "Pass After!");
+};
+
+done_testing;