Stories
Slash Boxes
Comments
NOTE: use Perl; is on undef hiatus. You can read content, but you can't post it. More info will be forthcoming forthcomingly.

All the Perl that's Practical to Extract and Report

use Perl Log In

Log In

[ Create a new account ]

jk2addict (4946)

jk2addict
  (email not shown publicly)
http://today.icantfocus.com/blog/
AOL IM: chrislaco (Add Buddy, Send Message)
Yahoo! ID: chrislaco@sbcglobal.net (Add User, Send Message)
Jabber: laco@scribblewerks.com

Journal of jk2addict (4946)

Sunday November 05, 2006
09:59 PM

Handel Coverage Is a Go

[ #31523 ]

The reports can be found on the server.

That last 5% is painful. I didn't resort to too many tricks. I'm using Devel::Cover 0.59 on my laptop and on the server, and a version of 5.8.4 and 5.8.6. Somethings would cover on one version and not another, like

$foo ||= getstuff() || otherget() || '';

And the ever popular

my $class = ref $self || $self;

can't be covered in the 0/0 table without running them as functions instead of methods, which is totally pointless.

The only thing left is the mysterious eval lines in Handel::Base. In any case, it's a great starting point for 1.0. Just some Perl::Critic tings to wrap up, and it will be go time pretty soon.

The Fine Print: The following comments are owned by whoever posted them. We are not responsible for them in any way.
 Full
 Abbreviated
 Hidden
More | Login | Reply
Loading... please wait.
  • You could always write
        my $class = eval 'ref $self || $self';
    if you worry about not getting 100% coverage.
    • And then when he runs `perlcritic`, he'll get a failure...

      Expression form of "eval" at line 7, column 15. See page 161 of PBP. (Severity: 5)

      ...the joys of hacking code to pass tests meant to improve our code ;)

    • Eeewww, please don't! :)

      I'd much rather you optimize for readability than obsessive 100% coverage.