Add a TODO test for the "pragma propagating into require" issue

[perl/modules/indirect.git] / t / 30-scope.t

diff --git a/t/30-scope.t b/t/30-scope.t
index f2a785f38b2d1830c6c5eca061100ade95e919c5..8eb7eccb975042c5a45a9e45f339ddaca0bc2015 100644 (file)
--- a/t/30-scope.t
+++ b/t/30-scope.t
@@ -4,36 +4,80 @@ use strict;
 use warnings;
 
 my $tests;
-BEGIN { $tests = 8 }
+BEGIN { $tests = 18 }
 
-use Test::More tests => $tests + 1;
+use Test::More tests => 1 + $tests + 1 + 2 + 2;
 
-my %wrong = map { $_ => 1 } 2, 3, 5, 7;
+use lib 't/lib';
+
+my %wrong = map { $_ => 1 } 2, 3, 5, 7, 9, 10, 14, 15, 17, 18;
+
+sub expect {
+ my ($pkg) = @_;
+ return qr/^warn:Indirect\s+call\s+of\s+method\s+"new"\s+on\s+object\s+"$pkg"\s+at\s+\(eval\s+\d+\)\s+line\s+\d+/;
+}
 
 {
  my $code = do { local $/; <DATA> };
- my @warns;
+ my (%res, $num, @left);
  {
-  local $SIG{__WARN__} = sub { push @warns, join '', 'warn:', @_ };
+  local $SIG{__WARN__} = sub {
+   ++$num;
+   my $w = join '', 'warn:', @_;
+   if ($w =~ /"P(\d+)"/ and not exists $res{$1}) {
+    $res{$1} = $w;
+   } else {
+    push @left, "[$num] $w";
+   }
+  };
   eval "die qq{ok\\n}; $code";
+  is($@, "ok\n", 'DATA compiled fine');
  }
- my $left = 0;
- my %res = map {
-  if (/"P(\d+)"/) {
-   $1 => $_
-  } else {
-   ++$left; ()
-  }
- } @warns;
  for (1 .. $tests) {
   my $w = $res{$_};
   if ($wrong{$_}) {
-   like($w, qr/^warn:Indirect\s+call\s+of\s+method\s+"new"\s+on\s+object\s+"P$_"/, "$_ should warn");
+   like($w, expect("P$_"), "$_ should warn");
   } else {
    is($w, undef, "$_ shouldn't warn");
   }
  }
- is($left, 0, 'nothing left');
+ is(@left, 0, 'nothing left');
+ diag "Extraneous warnings:\n", @left if @left;
+}
+
+{
+ my $w = '';
+ local $SIG{__WARN__} = sub {
+  $w = 'more than 2 warnings' if $w;
+  $w = join '', 'warn:', @_
+ };
+ {
+  eval 'no indirect; my $x = new Foo';
+  like($w, expect('Foo'), "eval 'no indirect; my \$x = new Foo'");
+ }
+ $w = '';
+ {
+  {
+   no indirect;
+   eval 'my $x = new Bar';
+  }
+  if ($] < 5.009005) {
+   is($w, '', "eval 'no indirect; my \$x = new Bar'");
+  } else {
+   like($w, expect('Bar'), "no indirect; eval 'my \$x = new Bar'");
+  }
+ }
+}
+
+{
+ local $TODO = 'Need a workaround for this' if $] < 5.010001;
+ my @w;
+ {
+  local $SIG{__WARN__} = sub { push @w, join '', @_ };
+  eval 'no indirect; use indirect::TestRequired';
+ }
+ is         $@, '',  'require test didn\'t croak';
+ is_deeply \@w, [ ], 'pragma didn\'t propagate into the required file';
 }
 
 __DATA__
@@ -61,3 +105,30 @@ my $g = new P7;
 use indirect;
 
 my $h = new P8;
+
+{
+ no indirect;
+ eval { my $i = new P9 };
+}
+
+eval { no indirect; my $j = new P10 };
+
+{
+ use indirect;
+ new P11 do { use indirect; new P12 };
+}
+
+{
+ use indirect;
+ new P13 do { no indirect; new P14 };
+}
+
+{
+ no indirect;
+ new P15 do { use indirect; new P16 };
+}
+
+{
+ no indirect;
+ new P17 do { no indirect; new P18 };
+}