File Coverage

blib/lib/Test/Refcount.pm

Criterion	Covered	Total	%
statement	43	44	97.7
branch	5	6	83.3
condition			n/a
subroutine	8	8	100.0
pod	2	2	100.0
total	58	60	96.6

#  You may distribute under the terms of either the GNU General Public License
#  or the Artistic License (the same terms as Perl itself)
#
#  (C) Paul Evans, 2008-2014 -- [e-mail]

package Test::Refcount;

use strict;
use warnings;
use base qw( Test::Builder::Module );

use Scalar::Util qw( weaken refaddr );
use B qw( svref_2object );

our $VERSION = '0.08';

our @EXPORT = qw(
   is_refcount
   is_oneref
);

use constant HAVE_DEVEL_MAT_DUMPER => defined eval { require Devel::MAT::Dumper };

=head1 NAME

C - assert reference counts on objects

=head1 SYNOPSIS

 use Test::More tests => 2;
 use Test::Refcount;

 use Some::Class;

 my $object = Some::Class->new();

 is_oneref( $object, '$object has a refcount of 1' );

 my $otherref = $object;

 is_refcount( $object, 2, '$object now has 2 references' );

=head1 DESCRIPTION

The Perl garbage collector uses simple reference counting during the normal
execution of a program. This means that cycles or unweakened references in
other parts of code can keep an object around for longer than intended. To
help avoid this problem, the reference count of a new object from its class
constructor ought to be 1. This way, the caller can know the object will be
properly DESTROYed when it drops all of its references to it.

This module provides two test functions to help ensure this property holds
for an object class, so as to be polite to its callers.

If the assertion fails; that is, if the actual reference count is different to
what was expected, either of the following two modules may be used to assist
the developer in finding where the references are.

=over 4

=item *

If L module is installed, a reverse-references trace is
printed to the test output.

=item *

If L is installed, this test module will use it to dump the state
of the memory after a failure. It will create a F<.pmat> file named the same
as the unit test, but with the trailing F<.t> suffix replaced with
F<-TEST.pmat> where C is the number of the test that failed (in case
there was more than one).

=back

See the examples below for more information.

=cut

=head1 FUNCTIONS

=cut

=head2 is_refcount( $object, $count, $name )

Test that $object has $count references to it.

=cut

sub is_refcount($$;$)
{
   my ( $object, $count, $name ) = @_;
   @_ = ();

   my $tb = __PACKAGE__->builder;

   if( !ref $object ) {
      my $ok = $tb->ok( 0, $name );
      $tb->diag( "  expected a reference, was not given one" );
      return $ok;
   }

   weaken $object; # So this reference itself doesn't show up

   my $REFCNT = svref_2object($object)->REFCNT;

   my $ok = $tb->ok( $REFCNT == $count, $name );

   unless( $ok ) {
      $tb->diag( "  expected $count references, found $REFCNT" );

      if( eval { require Devel::FindRef } ) {
         $tb->diag( Devel::FindRef::track( $object ) );
      }
      elsif( HAVE_DEVEL_MAT_DUMPER ) {
         my $file = $0;
         my $num = $tb->current_test;

         # Trim the .t off first then append -$num.pmat, in case $0 wasn't a .t file
         $file =~ s/\.(?:t|pm|pl)$//;
         $file .= "-$num\.pmat";

         $tb->diag( sprintf "SV address is 0x%x", refaddr $object );
         $tb->diag( "Writing heap dump to $file" );
         Devel::MAT::Dumper::dump( $file );
      }
   }

   return $ok;
}

=head2 is_oneref( $object, $name )

Assert that the $object has only 1 reference to it.

=cut

sub is_oneref($;$)
{
   splice( @_, 1, 0, ( 1 ) );
   goto &is_refcount;
}

=head1 EXAMPLE

Suppose, having written a new class C, you now want to check that its
constructor and methods are well-behaved, and don't leak references. Consider
the following test script:

 use Test::More tests => 2;
 use Test::Refcount;

 use MyBall;

 my $ball = MyBall->new();
 is_oneref( $ball, 'One reference after construct' );

 $ball->bounce;

 # Any other code here that might be part of the test script

 is_oneref( $ball, 'One reference just before EOF' );

The first assertion is just after the constructor, to check that the reference
returned by it is the only reference to that object. This fact is important if
we ever want C to behave properly. The second call is right at the
end of the file, just before the main scope closes. At this stage we expect
the reference count also to be one, so that the object is properly cleaned up.

Suppose, when run, this produces the following output (presuming
C is available):

 1..2
 ok 1 - One reference after construct
 not ok 2 - One reference just before EOF
 #   Failed test 'One reference just before EOF'
 #   at demo.pl line 16.
 #   expected 1 references, found 2
 # MyBall=ARRAY(0x817f880) is
 # +- referenced by REF(0x82c1fd8), which is
 # |     in the member 'self' of HASH(0x82c1f68), which is
 # |        referenced by REF(0x81989d0), which is
 # |           in the member 'cycle' of HASH(0x82c1f68), which was seen before.
 # +- referenced by REF(0x82811d0), which is
 #       in the lexical '$ball' in CODE(0x817fa00), which is
 #          the main body of the program.
 # Looks like you failed 1 test of 2.

From this output, we can see that the constructor was well-behaved, but that a
reference was leaked by the end of the script - the reference count was 2,
when we expected just 1. Reading the trace output, we can see that there were
2 references that C could find - one stored in the $ball
lexical in the main program, and one stored in a HASH. Since we expected to
find the $ball lexical variable, we know we are now looking for a leak in a
hash somewhere in the code. From reading the test script, we can guess this
leak is likely to be in the bounce() method. Furthermore, we know that the
reference to the object will be stored in a HASH in a member called C.

By reading the code which implements the bounce() method, we can see this is
indeed the case:

 sub bounce
 {
    my $self = shift;
    my $cycle = { self => $self };
    $cycle->{cycle} = $cycle;
 }

From reading the C output, we find that the HASH this object
is referenced in also contains a reference to itself, in a member called
C. This comes from the last line in this function, a line that
purposely created a cycle, to demonstrate the point. While a real program
probably wouldn't do anything quite this obvious, the trace would still be
useful in finding the likely cause of the leak.

If C is unavailable, then these detailed traces will not be
produced. The basic reference count testing will still take place, but a
smaller message will be produced:

 1..2
 ok 1 - One reference after construct
 not ok 2 - One reference just before EOF
 #   Failed test 'One reference just before EOF'
 #   at demo.pl line 16.
 #   expected 1 references, found 2
 # Looks like you failed 1 test of 2.

=head1 BUGS

=over 4

=item * Temporaries created on the stack

Code which creates temporaries on the stack, to be released again when the
called function returns does not work correctly on perl 5.8 (and probably
before). Examples such as

 is_oneref( [] );

may fail and claim a reference count of 2 instead.

Passing a variable such as

 my $array = [];
 is_oneref( $array );

works fine. Because of the intention of this test module; that is, to assert
reference counts on some object stored in a variable during the lifetime of
the test script, this is unlikely to cause any problems.

=back

=head1 ACKNOWLEDGEMENTS

Peter Rabbitson  - for suggesting using core's C
instead of C to obtain refcounts

=head1 AUTHOR

Paul Evans

=cut

0x55AA;