X-Git-Url: http://git.shadowcat.co.uk/gitweb/gitweb.cgi?a=blobdiff_plain;f=t%2Fbasic.t;h=416e39b74fe2dd6f7e1f37cb0bd5e73328023118;hb=3cba7a2c0211121aabb5cda50db17a5d84de1773;hp=308c667ab9af11e4f0a290e698794e3142dc4877;hpb=b76217297a5f92982c29177d97eb655b52d164eb;p=p5sagit%2FDevel-Size.git diff --git a/t/basic.t b/t/basic.t index 308c667..416e39b 100644 --- a/t/basic.t +++ b/t/basic.t @@ -1,29 +1,16 @@ #!/usr/bin/perl -w -use Test::More; +use Test::More tests => 30; use strict; - -my $tests; - -BEGIN - { - chdir 't' if -d 't'; - plan tests => 12; - - 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.71', 'VERSION MATCHES'); + unless is ($Devel::Size::VERSION, '0.77_50', 'VERSION MATCHES'); ############################################################################# # some basic checks: @@ -34,9 +21,9 @@ $foo = "12"; %foo = (a => 1, b => 2); my $x = "A string"; -my $y = "A much much longer string"; # need to be at least 7 bytes longer for 64 bit -ok (size($x) < size($y), 'size() of strings'); -ok (total_size($x) < total_size($y), 'total_size() of strings'); +my $y = "A much much longer string"; # need to be at least 7 bytes longer for 64 bit +cmp_ok(size($x), '<', size($y), 'size() of strings'); +cmp_ok(total_size($x), '<', total_size($y), 'total_size() of strings'); my @x = (1..4); my @y = (1..200); @@ -44,7 +31,7 @@ my @y = (1..200); my $size_1 = total_size(\@x); my $size_2 = total_size(\@y); -ok ( $size_1 < $size_2, 'size() of array refs'); +cmp_ok($size_1, '<', $size_2, 'size() of array refs'); # the arrays alone shouldn't be the same size $size_1 = size(\@x); @@ -61,7 +48,7 @@ $y = 12; $y .= ''; $size_1 = size($x); $size_2 = size($y); -ok ($size_1 < $size_2, ' ."" makes string longer'); +cmp_ok($size_1, '<', $size_2, ' ."" makes string longer'); ############################################################################# # check that the tracking_hash is working @@ -70,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 @@ -80,14 +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?) +cmp_ok( total_size( sub{ do{ my $t=0 }; } ), '>', 0, + 'total_size( sub{ my $t=0 } ) > 0' ); + +# CPAN RT #58484 and #58485 +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'); + } +}