More control tests

diff --git a/t/17-ctl.t b/t/17-ctl.t
index b25c5ee6812fac6154cc727027cb88706415f944..bedfdb0428b440eeec3b80c8fb3279e6b0cb0f63 100644 (file)
--- a/t/17-ctl.t
+++ b/t/17-ctl.t
@@ -3,11 +3,9 @@
 use strict;
 use warnings;
 
-use Test::More tests => 14 + 1;
+use Test::More tests => 4 * 8 + 10 + 1 + 1;
 
-use Variable::Magic qw/wizard cast/;
-
-my $wiz;
+use Variable::Magic qw/wizard cast VMG_UVAR/;
 
 sub expect {
  my ($name, $where, $suffix) = @_;
@@ -16,33 +14,121 @@ sub expect {
  qr/^\Q$name\E at $where line \d+\.$end/
 }
 
-eval {
- $wiz = wizard data => sub { $_[1]->() };
- my $x;
- cast $x, $wiz, sub { die "carrot" };
-};
+my @scalar_tests = (
+ [ 'data', sub { \(my $x) },   sub { }                    ],
+ [ 'get',  sub { \(my $x) },   sub { my $y = ${$_[0]} }   ],
+ [ 'set',  sub { \(my $x) },   sub { ${$_[0]} = 1 }       ],
+ [ 'len',  sub { [ 1 .. 3 ] }, sub { my $res = @{$_[0]} } ],
+);
 
-like $@, expect('carrot', $0), 'die in data callback';
+# Data, get, set, len
 
-eval {
- $wiz = wizard data => sub { $_[1] },
-               set  => sub { $_[1]->(); () };
- my $x;
- cast $x, $wiz, sub { die "lettuce" };
- $x = 5;
-};
+for my $t (@scalar_tests) {
+ my ($name, $init, $code) = @$t;
 
-like $@, expect('lettuce', $0), 'die in set callback';
+ my $wiz = wizard $name => sub { die 'leek' };
 
-my $res = eval {
- $wiz = wizard data => sub { $_[1] },
-               len  => sub { $_[1]->(); () };
- my @a = (1 .. 3);
- cast @a, $wiz, sub { die "potato" };
- @a;
-};
+ {
+  local $@;
+  eval {
+   my $x = $init->();
+   &cast($x, $wiz);
+   $code->($x);
+  };
+  like $@, expect('leek', $0),
+                            "die in $name callback (direct, \$@ unset) in eval";
+ }
+
+ {
+  local $@;
+  eval {
+   my $x = $init->();
+   &cast($x, $wiz);
+   $@ = 'artichoke';
+   $code->($x);
+  };
+  like $@, expect('leek', $0),
+                              "die in $name callback (direct, \$@ set) in eval";
+ }
+
+ {
+  local $@;
+  eval q{BEGIN {
+   my $x = $init->();
+   &cast($x, $wiz);
+   $code->($x);
+  }};
+  like $@, expect('leek', $0, "\nBEGIN.*"),
+                           "die in $name callback (direct, \$@ unset) in BEGIN";
+ }
+
+ {
+  local $@;
+  eval q{BEGIN {
+   my $x = $init->();
+   &cast($x, $wiz);
+   $@ = 'artichoke';
+   $code->($x);
+  }};
+  like $@, expect('leek', $0, "\nBEGIN.*"),
+                             "die in $name callback (direct, \$@ set) in BEGIN";
+ }
 
-like $@, expect('potato', $0), 'die in len callback';
+ $wiz = wizard(
+  ($name eq 'data' ? () : (data  => sub { $_[1] })),
+   $name => sub { $_[1]->(); () },
+ );
+
+ {
+  local $@;
+  eval {
+   my $x = $init->();
+   &cast($x, $wiz, sub { die 'lettuce' });
+   $code->($x);
+  };
+  like $@, expect('lettuce', $0),
+                          "die in $name callback (indirect, \$@ unset) in eval";
+ }
+
+ {
+  local $@;
+  eval {
+   my $x = $init->();
+   &cast($x, $wiz, sub { die 'carrot' });
+   $@ = 'artichoke';
+   $code->($x);
+  };
+  like $@, expect('carrot', $0),
+                          "die in $name callback (indirect, \$@ unset) in eval";
+ }
+
+ {
+  local $@;
+  eval q{BEGIN {
+   my $x = $init->();
+   &cast($x, $wiz, sub { die "pumpkin" });
+   $code->($x);
+  }};
+  like $@, expect('pumpkin', undef, "\nBEGIN.*"),
+                         "die in $name callback (indirect, \$@ unset) in BEGIN";
+ }
+
+ {
+  local $@;
+  eval q{BEGIN {
+   my $x = $init->();
+   &cast($x, $wiz, sub { die "chard" });
+   $@ = 'artichoke';
+   $code->($x);
+  }};
+  like $@, expect('chard', undef, "\nBEGIN.*"),
+                           "die in $name callback (indirect, \$@ set) in BEGIN";
+ }
+}
+
+# Free
+
+my $wiz;
 
 eval {
  $wiz = wizard data => sub { $_[1] },
@@ -68,7 +154,7 @@ like $@, expect('zucchini', $0),
 
 eval {
  $wiz = wizard free => sub { die 'eggplant' };
- $@ = "vuvuzela";
+ $@ = "artichoke";
  {
   my $x;
   cast $x, $wiz;
@@ -82,48 +168,40 @@ like $@, expect('eggplant', $0),
 eval q{BEGIN {
  $wiz = wizard free => sub { die 'onion' };
  my $x;
- cast $x, $wiz;;
+ cast $x, $wiz;
 }};
 
 like $@, expect('onion', undef, "\nBEGIN.*"), 'die in free callback in BEGIN';
 
-# Inspired by B::Hooks::EndOfScope
-
 eval q{BEGIN {
- $wiz = wizard data => sub { $_[1]->() };
- my $x;
- cast $x, $wiz, sub { die "pumpkin" };
+ $wiz = wizard data => sub { $_[1] },
+               len  => sub { $_[1]->(); $_[2] },
+               free => sub { my $x = @{$_[0]}; () };
+ my @a = (1 .. 5);
+ cast @a, $wiz, sub { die "pepperoni" };
 }};
 
-like $@, expect('pumpkin', undef, "\nBEGIN.*"), 'die in data callback in BEGIN';
+like $@, expect('pepperoni', undef, "\nBEGIN.*"),
+                                'die in free callback in len callback in BEGIN';
+
+# Inspired by B::Hooks::EndOfScope
 
 eval q{BEGIN {
  $wiz = wizard data => sub { $_[1] },
                free => sub { $_[1]->(); () };
  $^H |= 0x020000;
- cast %^H, $wiz, sub { die "macaroni" };
+ cast %^H, $wiz, sub { die 'cabbage' };
 }};
 
-like $@, expect('macaroni'), 'die in free callback at end of scope';
-
-eval q{BEGIN {
- $wiz = wizard data => sub { $_[1] },
-               len  => sub { $_[1]->(); $_[2] },
-               free => sub { my $x = @{$_[0]}; () };
- my @a = (1 .. 5);
- cast @a, $wiz, sub { die "pepperoni" };
-}};
-
-like $@, expect('pepperoni', undef, "\nBEGIN.*"),'die in len callback in BEGIN';
+like $@, expect('cabbage'), 'die in free callback at end of scope';
 
 use lib 't/lib';
 
+my $vm_tse_file = 't/lib/Variable/Magic/TestScopeEnd.pm';
 
 eval "use Variable::Magic::TestScopeEnd";
-
-like $@,
-     expect('turnip', 't/lib/Variable/Magic/TestScopeEnd.pm', "\nBEGIN(?s:.*)"),
-     'die in BEGIN in require in eval string triggers hints hash destructor';
+like $@, expect('turnip', $vm_tse_file, "\nBEGIN(?s:.*)"),
+        'die in BEGIN in require in eval string triggers hints hash destructor';
 
 eval q{BEGIN {
  Variable::Magic::TestScopeEnd::hook {
@@ -145,13 +223,43 @@ sub run_perl {
  system { $^X } $^X, '-T', map("-I$_", @INC), '-e', $code;
 }
 
+my $has_capture_tiny = do { local $@; eval 'use Capture::Tiny 0.08 (); 1' };
+
 SKIP:
 {
- skip 'Capture::Tiny 0.08 is not installed' => 1
-                                     unless eval "use Capture::Tiny 0.08 (); 1";
- my $code = 'use Variable::Magic qw/wizard cast/; { BEGIN { $^H |= 0x020000; cast %^H, wizard free => sub { die q[cucumber] } } }';
- my $output = Capture::Tiny::capture_merged(sub { run_perl $code });
+ my $count = 1;
+
+ skip 'Capture::Tiny 0.08 is not installed' => $count unless $has_capture_tiny;
+
+ my $output = Capture::Tiny::capture_merged(sub { run_perl <<' CODE' });
+use Variable::Magic qw/wizard cast/; { BEGIN { $^H |= 0x020000; cast %^H, wizard free => sub { die q[cucumber] } } }
+ CODE
  skip 'Test code didn\'t run properly' => 1 unless defined $output;
  like $output, expect('cucumber', '-e', "\nExecution(?s:.*)"),
-                                   'die at compile time and not in eval string';
+                  'die in free callback at compile time and not in eval string';
+ --$count;
+}
+
+# Uvar
+
+SKIP:
+{
+ my $count = 1;
+
+ skip 'No nice uvar magic for this perl'    => $count unless VMG_UVAR;
+ skip 'Capture::Tiny 0.08 is not installed' => $count unless $has_capture_tiny;
+
+ my $output = Capture::Tiny::capture_merged(sub { run_perl <<' CODE' });
+use Variable::Magic qw/wizard cast/; BEGIN { cast %::, wizard fetch => sub { die q[salsify] } } hlagh()
+ CODE
+ skip 'Test code didn\'t run properly' => $count unless defined $output;
+ my $suffix = "\nExecution(?s:.*)";
+ if ($] >= 5.011005) {
+  $suffix = "(?:\nsalsify at -e line \\d+.){12}" . $suffix;
+ } elsif ($] >= 5.011) {
+  $suffix = "(?:\nsalsify at -e line \\d+.){3}" . $suffix;
+ }
+ like $output, expect('salsify', '-e', $suffix),
+                  'die in free callback at compile time and not in eval string';
+ --$count;
 }