X-Git-Url: http://git.shadowcat.co.uk/gitweb/gitweb.cgi?a=blobdiff_plain;f=t%2Fbasic.t;h=127cb396c7dc0bfac66a55e5a766436d9bbb69ab;hb=d6158a765909946609d49f5e09122a73ec923def;hp=980899b7a3c7f0330819c1baf24eba30621882e9;hpb=6cba40961d44a6cd61f1cb8d3b9040e6879eadb0;p=p5sagit%2FDevel-Size.git

diff --git a/t/basic.t b/t/basic.t
index 980899b..127cb39 100644
--- a/t/basic.t
+++ b/t/basic.t
@@ -1,9 +1,8 @@
 #!/usr/bin/perl -w
 
-use Test::More tests => 16;
+use Test::More tests => 19;
 use strict;
 use Devel::Size qw(size total_size);
-use Scalar::Util qw(weaken);
 
 can_ok ('Devel::Size', qw/
   size
@@ -84,7 +83,18 @@ cmp_ok (total_size(\&LARGE), '>', 8192,
 {
     my $a = [];
     my $b = \$a;
-    # making a weakref upgrades the target to PVMG and adds magic
+    # Scalar::Util isn't in the core before 5.7.something.
+    # The test isn't really testing anything without the weaken(), but it
+    # isn't counter-productive or harmful to run it anyway.
+    unless (eval {
+	require Scalar::Util;
+	# making a weakref upgrades the target to PVMG and adds magic
+	Scalar::Util::weaken($b);
+	1;
+    }) {
+	die $@ if $] >= 5.008;
+    }
+
     is(total_size($a), total_size([]),
        'Any intial reference is dereferenced and discarded');
 }
@@ -98,3 +108,18 @@ foreach(['undef', total_size(undef)],
     is($size, 0,
        "PL_sv_$name is interpeter wide, so not counted as part of the structure's size");
 }
+
+{
+    # SvOOK stuff
+    my $uurk = "Perl Rules";
+    # This may upgrade the scalar:
+    $uurk =~ s/Perl//;
+    $uurk =~ s/^/Perl/;
+    my $before_size = total_size($uurk);
+    my $before_length = length $uurk;
+    cmp_ok($before_size, '>', $before_length, 'Size before is sane');
+    $uurk =~ s/Perl //;
+    is(total_size($uurk), $before_size,
+       "Size doesn't change because OOK is used");
+    cmp_ok(length $uurk, '<', $before_size, 'but string is shorter');
+}