Beefy Boxes and Bandwidth Generously Provided by pair Networks
Think about Loose Coupling

Re^2: too much testing?

by geektron (Curate)
on Sep 30, 2005 at 19:19 UTC ( #496494=note: print w/replies, xml ) Need Help??

in reply to Re: too much testing?
in thread too much testing?

i was going to look at Test::Exception as another way of doing it. (i admin the box, so i can install what i want.)

the "standard" eval method came to mind also, but not using like with it. as always, There's More ....

Replies are listed 'Best First'.
Re^3: too much testing?
by leriksen (Curate) on Oct 03, 2005 at 01:25 UTC
    A comon pattern I find in my testing code is

    my @dieings; my @warnings; my $rc; eval { local $SIG{__WARN__} = sub {push @warnings, @_}; local $SIG{__DIE__ } = sub {push @dieings , @_}; $rc = $obj->method(@params); } is(@warnings, $expectedWarningCount, 'correct number of warnings raise +d'); like(shift(@warnings), qr/expected pattern of first warning/, 'first w +arning message correct'); like(shift(@warnings), qr/expected pattern of second warning/, 'second + warning message correct'); ... is(@dieings, $expectedDeathCount, 'correct number of deaths raised'); like(shift(@dieings), qr/expected pattern of first death/, 'first deat +h message correct'); ... is($rc, $expectReturnValue, 'method returned required value'); # or, if method returns an object isa_ok($rc, 'Expect::Class', 'received expected class'); ...

    ...reality must take precedence over public relations, for nature cannot be fooled. - R P Feynmann

      that's an interesting way of handling it.

      funny thing is, people seem to be gravitating to the part about catching $SIG{DIE} and not even noticing the title, or part of the real "meditation" ... though i guess not explicitly commenting on it is a commentary in itself.

      being the novice tester, i really am wondering when 'enough is enough' ...

        OK, so as to your question(s) - are you testing "too much"

        I have strong opinions on this, based on years of writing code that has experienced jut about every failure mode imaginable.

        I have a personal belief that you have tested enough when you feel you have exercised as much of the code as you reasonably can. I use Devel::Cover to tell me how much I haven't done, and use my experience and judgement to make informed decisions as to what tests I will add for the remains.

        For example, do you need to test what your code does if read() fails ? If so, writing test cases to do this is a bit difficult, but if you need it, do it.
        Otherwise, code and test on the assumption "read() will never fail" and move on.

        Devel::Cover isn't perfect, and sometimes it can be very difficult to work with - for example, it complains about my $class = ref($proto)||$proto; as having an untested path in most "reasonable" usages. You have to write a test that calls the constructor in a manner you would expect to see for a function-based module e.g. Module::new(), not Module->new()
        So you need to ask yourself "is testing for people calling my OO methods as normal functions something I want to do ?"

        If it is important to you, you help newbies who do "Module::new" by mistake.

        If not, ignore the report of the untested path from D::C and move on.

        As an aside, I wrote a meditation on what I had to go through to get a reasonably large body of modules to get to 100% coverage as reported by D::C - see Lessons learned from getting to 100% with Devel::Cover about my struggles and discoveries from doing a deep testing effort.

        ...reality must take precedence over public relations, for nature cannot be fooled. - R P Feynmann

Log In?

What's my password?
Create A New User
Domain Nodelet?
Node Status?
node history
Node Type: note [id://496494]
and the web crawler heard nothing...

How do I use this?Last hourOther CB clients
Other Users?
Others exploiting the Monastery: (9)
As of 2023-12-11 12:37 GMT
Find Nodes?
    Voting Booth?
    What's your preferred 'use VERSION' for new CPAN modules in 2023?

    Results (41 votes). Check out past polls.