X-Git-Url: http://git.shadowcat.co.uk/gitweb/gitweb.cgi?a=blobdiff_plain;f=t%2Fbasic.t;h=416e39b74fe2dd6f7e1f37cb0bd5e73328023118;hb=3cba7a2c0211121aabb5cda50db17a5d84de1773;hp=31e1810867a4c2fcc82a30ebe69b4076047dd8cb;hpb=87372f42ffeb807c18d53a4a2d2337ed6cbf6fa3;p=p5sagit%2FDevel-Size.git

diff --git a/t/basic.t b/t/basic.t
index 31e1810..416e39b 100644
--- a/t/basic.t
+++ b/t/basic.t
@@ -1,27 +1,16 @@
 #!/usr/bin/perl -w
 
-use Test::More;
+use Test::More tests => 30;
 use strict;
-
-BEGIN
-   {
-   chdir 't' if -d 't';
-   plan tests => 15;
-
-   use lib '../lib';
-   use lib '../blib/arch';
-   use_ok('Devel::Size');
-   }
+use Devel::Size qw(size total_size);
 
 can_ok ('Devel::Size', qw/
   size
   total_size
   /);
 
-Devel::Size->import( qw(size total_size) );
-
 die ("Uhoh, test uses an outdated version of Devel::Size")
-  unless is ($Devel::Size::VERSION, '0.72_50', 'VERSION MATCHES');
+    unless is ($Devel::Size::VERSION, '0.77_50', 'VERSION MATCHES');
 
 #############################################################################
 # some basic checks:
@@ -68,8 +57,8 @@ my($a,$b) = (1,2);
 my @ary1 = (\$a, \$a);
 my @ary2 = (\$a, \$b);
 
-isnt ( total_size(\@ary2) - total_size(\@ary1), 0,
-    'total_size(\@ary1) < total_size(\@ary2)');
+cmp_ok(total_size(\@ary1), '<', total_size(\@ary2),
+       'the tracking hash is working');
 
 #############################################################################
 # check that circular references don't mess things up
@@ -78,25 +67,107 @@ my($c1,$c2); $c2 = \$c1; $c1 = \$c2;
 
 is (total_size($c1), total_size($c2), 'circular references');
 
-#############################################################################
-# GLOBS
-
-isnt (total_size(*foo), 0, 'total_size(*foo) > 0');
-
-#############################################################################
-# CODE ref
-
-my $code = sub { '1' };
-
-isnt (total_size($code), 0, 'total_size($code) > 0');
-
 ##########################################################
 # RT#14849 (& RT#26781 and possibly RT#29238?)
-isnt( total_size( sub{ do{ my $t=0 }; } ), 0, 'total_size( sub{ my $t=0 } ) > 0' );
+cmp_ok( total_size( sub{ do{ my $t=0 }; } ), '>', 0,
+	'total_size( sub{ my $t=0 } ) > 0' );
 
 # CPAN RT #58484 and #58485
-isnt (total_size(\&total_size), 0, 'total_size(\&total_size) > 0');
-
-use constant LARGE => 'N' x 4096;
-
-isnt (total_size(\&LARGE), 0, 'total_size for a constant > 0');
+cmp_ok(total_size(\&total_size), '>', 0, 'total_size(\&total_size) > 0');
+
+use constant LARGE => 'N' x 8192;
+
+cmp_ok (total_size(\&LARGE), '>', 8192,
+        'total_size for a constant includes the constant');
+
+{
+    my $a = [];
+    my $b = \$a;
+    # 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');
+}
+
+# Must call direct - avoid all copying:
+foreach(['undef', total_size(undef)],
+	['no', total_size(1 == 0)],
+	['yes', total_size(1 == 1)],
+       ) {
+    my ($name, $size) = @$_;
+    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');
+}
+
+sub shared_hash_keys {
+    my %h = @_;
+    my $one = total_size([keys %h]);
+    cmp_ok($one, '>', 0, 'Size of one entry is sane');
+    my $two =  total_size([keys %h, keys %h]);
+    cmp_ok($two, '>', $one, 'Two take more space than one');
+    my $increment = $two - $one;
+    is(total_size([keys %h, keys %h, keys %h]), $one + 2 * $increment,
+		 'Linear size increase for three');
+    return $increment;
+}
+
+{
+    my $small = shared_hash_keys(Perl => 'Rules');
+    my $big = shared_hash_keys('x' x 1024, '');
+ SKIP: {
+	skip("[keys %h] doesn't copy as shared hash key scalars prior to 5.10.0",
+	     1) if $] < 5.010;
+	is ($small, $big, 'The "shared" part of shared hash keys is spotted');
+    }
+}
+
+{
+    use vars '%DANG_DANG_DANG_DANG_DANG_DANG_DANG_DANG_DANG';
+    my $hash_size = total_size(\%DANG_DANG_DANG_DANG_DANG_DANG_DANG_DANG_DANG);
+    cmp_ok($hash_size, '>', 0, 'Hash size is sane');
+    my $stash_size
+	= total_size(\%DANG_DANG_DANG_DANG_DANG_DANG_DANG_DANG_DANG::);
+    cmp_ok($stash_size, '>',
+	   $hash_size + length 'DANG_DANG_DANG_DANG_DANG_DANG_DANG_DANG_DANG',
+	   'Stash size is larger than hash size plus length of the name');
+}
+
+{
+    my %h = (Perl => 'Rules');
+    my $hash_size = total_size(\%h);
+    cmp_ok($hash_size, '>', 0, 'Hash size is sane');
+    my $a = keys %h;
+    if ($] < 5.010) {
+	is(total_size(\%h), $hash_size,
+	   "Creating iteration state doesn't need to allocate storage");
+	# because all hashes carry the overhead of this storage from creation
+    } else {
+	cmp_ok(total_size(\%h), '>', $hash_size,
+	       'Creating iteration state allocates storage');
+    }
+}