use Test::More;
-use Variable::Magic qw/wizard cast dispell VMG_UVAR VMG_OP_INFO_NAME VMG_OP_INFO_OBJECT/;
+use Variable::Magic qw<
+ wizard cast dispell
+ VMG_UVAR VMG_OP_INFO_NAME VMG_OP_INFO_OBJECT
+>;
my $run;
if (VMG_UVAR) {
- plan tests => 33;
+ plan tests => 43;
$run = 1;
} else {
plan skip_all => 'uvar magic is required to test symbol table hooks';
()
}
CB
-} qw/fetch store exists delete/);
+} qw<fetch store exists delete>);
$code .= ', data => sub { +{ guard => 0 } }';
is $@, "ok\n", 'stash: variables compiled fine';
is_deeply \%mg, {
- fetch => [ qw/thing stuff/ ],
- store => [ qw/thing stuff/ ],
+ fetch => [ qw<thing stuff> ],
+ store => [ qw<thing stuff> ],
}, 'stash: variables';
}
is $@, "ok\n", 'stash: function definitions compiled fine';
is_deeply \%mg, {
- store => [ qw/eat shoot leave shoot/ ],
+ store => [ qw<eat shoot leave shoot> ],
}, 'stash: function definitions';
}
roam();
};
+ my @calls = qw<eat shoot leave roam yawn roam>;
+
is $@, "ok\n", 'stash: function calls compiled fine';
is_deeply \%mg, {
- fetch => [ qw/eat shoot leave roam yawn roam/ ],
- store => [ qw/eat shoot leave roam yawn roam/ ],
+ fetch => \@calls,
+ store => ("$]" < 5.011_002 ? \@calls : [ map { ($_) x 2 } @calls ]),
}, 'stash: function calls';
}
is $@, '', 'stash: valid method call ran fine';
is_deeply \%mg, {
- fetch => [ qw/shoot/ ],
+ fetch => [ qw<shoot> ],
}, 'stash: valid method call';
}
+{
+ local %mg;
+
+ eval q{ Hlagh->shoot() };
+
+ is $@, '', 'stash: second valid method call ran fine';
+ is_deeply \%mg, {
+ fetch => [ qw<shoot> ],
+ }, 'stash: second valid method call';
+}
+
{
local %mg;
is $@, '', 'stash: valid dynamic method call ran fine';
is_deeply \%mg, {
- store => [ qw/shoot/ ],
+ store => [ qw<shoot> ],
}, 'stash: valid dynamic method call';
}
package Hlagher;
our @ISA;
BEGIN { @ISA = 'Hlagh' }
- Hlagher->shoot()
+ Hlagher->leave()
];
is $@, '', 'inherited valid method call ran fine';
is_deeply \%mg, {
- fetch => [ qw/ISA shoot/ ],
+ fetch => [ qw<ISA leave> ],
}, 'stash: inherited valid method call';
}
+{
+ local %mg;
+
+ eval q{ Hlagher->leave() };
+
+ is $@, '', 'second inherited valid method call ran fine';
+ is_deeply \%mg, { }, 'stash: second inherited valid method call doesn\'t call magic';
+}
+
+{
+ local %mg;
+
+ eval q{ Hlagher->shoot() };
+
+ is $@, '', 'inherited previously called valid method call ran fine';
+ is_deeply \%mg, {
+ fetch => [ qw<shoot> ],
+ }, 'stash: inherited previously called valid method call';
+}
+
+{
+ local %mg;
+
+ eval q{ Hlagher->shoot() };
+
+ is $@, '', 'second inherited previously called valid method call ran fine';
+ is_deeply \%mg, { }, 'stash: second inherited previously called valid method call doesn\'t call magic';
+}
+
{
local %mg;
like $@, qr/^Can't locate object method "unknown" via package "Hlagh"/, 'stash: invalid method call croaked';
is_deeply \%mg, {
- fetch => [ qw/unknown/ ],
- store => [ qw/unknown AUTOLOAD/ ],
+ fetch => [ qw<unknown> ],
+ store => [ qw<unknown AUTOLOAD> ],
}, 'stash: invalid method call';
}
like $@, qr/^Can't locate object method "unknown_too" via package "Hlagh"/, 'stash: invalid dynamic method call croaked';
is_deeply \%mg, {
- store => [ qw/unknown_too AUTOLOAD/ ],
+ store => [ qw<unknown_too AUTOLOAD> ],
}, 'stash: invalid dynamic method call';
}
like $@, qr/^Can't locate object method "also_unknown" via package "Hlagher"/, 'stash: invalid inherited method call croaked';
is_deeply \%mg, {
- fetch => [ qw/also_unknown AUTOLOAD/ ],
+ fetch => [ qw<also_unknown AUTOLOAD> ],
}, 'stash: invalid method call';
}
{
local %mg;
+ my @expected_stores = qw<nevermentioned eat shoot>;
+ @expected_stores = map { ($_) x 2 } @expected_stores if "$]" < 5.017_004;
+ push @expected_stores, 'nevermentioned' if "$]" < 5.017_001;
+
eval q{
package Hlagh;
undef &nevermentioned;
};
is $@, '', 'stash: delete executed fine';
- is_deeply \%mg, {
- store => [
- qw/nevermentioned nevermentioned eat eat shoot shoot nevermentioned/
- ],
- }, 'stash: delete';
+ is_deeply \%mg, { store => \@expected_stores }, 'stash: delete';
}
END {
{
package AutoHlagh;
- use vars qw/$AUTOLOAD/;
+ use vars qw<$AUTOLOAD>;
sub AUTOLOAD { return $AUTOLOAD }
}
is $res, 'AutoHlagh::autoloaded',
'stash: autoloaded method call returned the right thing';
is_deeply \%mg, {
- fetch => [ qw/autoloaded/ ],
- store => [ qw/autoloaded AUTOLOAD AUTOLOAD/ ],
+ fetch => [ qw<autoloaded> ],
+ store => [ qw<autoloaded AUTOLOAD AUTOLOAD> ],
}, 'stash: autoloaded method call';
}
is $res, 'AutoHlagher::also_autoloaded',
'stash: inherited autoloaded method returned the right thing';
is_deeply \%mg, {
- fetch => [ qw/also_autoloaded AUTOLOAD/ ],
- store => [ qw/AUTOLOAD/ ],
+ fetch => [ qw<also_autoloaded AUTOLOAD> ],
+ store => [ qw<AUTOLOAD> ],
}, 'stash: inherited autoloaded method call';
}
dispell %AutoHlagh::, $wiz;
+my $uo = 0;
$code = 'wizard '
. join (', ', map { <<CB;
$_ => sub {
my \$d = \$_[1];
return 0 if \$d->{guard};
local \$d->{guard} = 1;
- is \$_[3], undef, 'stash: undef op';
+ ++\$uo;
()
}
CB
-} qw/fetch store exists delete/);
+} qw<fetch store exists delete>);
+
+my $uo_exp = "$]" < 5.011_002 ? 2 : 3;
$code .= ', data => sub { +{ guard => 0 } }';
cast %Hlagh::, $wiz;
+is $uo, 0, 'stash: no undef op before function call with op name';
eval q{
die "ok\n";
package Hlagh;
meh();
};
-
-is $@, "ok\n", 'stash: function call with op name compiled fine';
+is $@, "ok\n", 'stash: function call with op name compiled fine';
+is $uo, $uo_exp, 'stash: undef op after function call with op name';
dispell %Hlagh::, $wiz;
+is $uo, $uo_exp, 'stash: undef op after dispell for function call with op name';
+
+$uo = 0;
$wiz = eval $code . ', op_info => ' . VMG_OP_INFO_OBJECT;
diag $@ if $@;
cast %Hlagh::, $wiz;
+is $uo, 0, 'stash: no undef op before function call with op object';
eval q{
die "ok\n";
package Hlagh;
wat();
};
-
-is $@, "ok\n", 'stash: function call with op object compiled fine';
+is $@, "ok\n", 'stash: function call with op object compiled fine';
+is $uo, $uo_exp,
+ 'stash: undef op after dispell for function call with op object';
dispell %Hlagh::, $wiz;
+is $uo, $uo_exp,
+ 'stash: undef op after dispell for function call with op object';