X-Git-Url: http://git.vpit.fr/?a=blobdiff_plain;f=t%2F22-len.t;h=5d3ae92220c660e012ef1f9fcbfe8ed0f7354d40;hb=54a1b1e4382ad55a54cbd874d6ffd87bf58dffbc;hp=282fd37519314e63cb655eba4bf3f466564cd07f;hpb=77a84f75f33e3ee44e61182dec76699e23025375;p=perl%2Fmodules%2FVariable-Magic.git

diff --git a/t/22-len.t b/t/22-len.t
index 282fd37..5d3ae92 100644
--- a/t/22-len.t
+++ b/t/22-len.t
@@ -1,24 +1,65 @@
 #!perl -T
 
-use Test::More tests => 6;
+use strict;
+use warnings;
 
-use Variable::Magic qw/wizard cast/;
+use Test::More tests => 13;
+
+use Variable::Magic qw/wizard cast VMG_COMPAT_SCALAR_LENGTH_NOLEN/;
 
 my $c = 0;
-my $n = int rand 1000;
+
+my $n = 1 + int rand 1000;
 my $wiz = wizard len => sub { ++$c; return $n };
-ok($c == 0, 'len : create wizard');
+is $c, 0, 'len: wizard() doesn\'t trigger magic';
 
 my @a = qw/a b c/;
 
+$c = 0;
 cast @a, $wiz;
-ok($c == 0, 'len : cast');
+is $c, 0, 'len: cast on array doesn\'t trigger magic';
 
+$c = 0;
 my $b = scalar @a;
-ok($c == 1, 'len : get length');
-ok($b == $n, 'len : get length correctly');
+is $c, 1,  'len: get array length triggers magic correctly';
+is $b, $n, 'len: get array length correctly';
+
+$c = 0;
+$b = $#a;
+is $c, 1,      'len: get last array index triggers magic correctly';
+is $b, $n - 1, 'len: get last array index correctly';
 
 $n = 0;
+
+$c = 0;
 $b = scalar @a;
-ok($c == 2, 'len : get length 0');
-ok($b == 0, 'len : get length 0 correctly');
+is $c, 1, 'len: get array length 0 triggers magic correctly';
+is $b, 0, 'len: get array length 0 correctly';
+
+SKIP: {
+ skip 'length() no longer calls mg_len magic' => 5 if VMG_COMPAT_SCALAR_LENGTH_NOLEN;
+
+ $c = 0;
+ $n = 1 + int rand 1000;
+ # length magic on scalars needs also get magic to be triggered.
+ $wiz = wizard get => sub { return 'anything' },
+               len => sub { ++$c; return $n };
+
+ my $x = int rand 1000;
+
+ $c = 0;
+ cast $x, $wiz;
+ is $c, 0, 'len: cast on scalar doesn\'t trigger magic';
+
+ $c = 0;
+ $b = length $x;
+ is $c, 1,  'len: get scalar length triggers magic correctly';
+ is $b, $n, 'len: get scalar length correctly';
+
+ $n = 0;
+
+ $c = 0;
+ $b = length $x;
+ is $c, 1,  'len: get scalar length 0 triggers magic correctly';
+ is $b, $n, 'len: get scalar length 0 correctly';
+}