TPF Devel::Cover grant report May 2013

No Comments

In accordance with the terms of my grant from TPF this is the monthly
report for my work on improving Devel::Cover covering May 2013.

In May Perl 5.18.0 was released. 5.18.0 introduced a new padrange operator
and I had failed to note the full implications for Devel::Cover. So, a
couple of days after 5.18.0 was released I released Devel::Cover 1.03 which
works correctly with the new operator, and contains a new test to ensure it
stays that way.

A new major release of Perl also means that I was able to clean out all the
tests for 5.17.x, and start testing for 5.19.0. I also updated cpancover.com
to use 5.18.0.

And speaking of cpancover, I put together the work which was done during the
QA Hackathon on perl5cover and got it up and running on cpancover.com.
Perl5cover is the project to provide useful Perl and C level code coverage for
the Perl core. There is a link at the bottom of the main page to
http://cpancover.com/blead/latest/coverage.html where you can see the results.
There is still work to be done in this area. In particular, some of the
results are lower than they should be. I think there is a problem with
collecting coverage for modules which Devel::Cover itself uses. This doesn't
usually show up because I am quite conservative in the CPAN modules I use, but
I haven't worried about the core modules.

There are also too many of the core tests failing under coverage. This is not
surprising, and it's hopefully something which we will be able to improve on
in the future. Jim Keenan is the driving force behind this work, much of
which is being built upon code from Abe Timmerman.

At the Satellite QA Hackathon in Tokyo, Kan Fushihara put together
Devel::Cover::Report::Coveralls. If you haven't tried it out yet, you should
have a look. In order to get the travis/coveralls integration correct he
needed some changes to the cover program, for which he sent me patches which
were applied this month.

To integrate coveralls into your CI process, register your project at
travis-ci.org, and add the following lines to .travis.yml (seasoned to taste):

before_install:
** cpanm -n Devel::Cover::Report::Coveralls**
script:
** perl Makefile.PL && make && cover -test -report coveralls**

I fixed a long-standing bug in pod coverage in which subroutines in different
packages but within the same file were not covered correctly. This turned out
to be one of those bugs where all the effort was in the search, and the fix
was trivial.

And finally I started investigating speeding up Devel::Cover. Devel::Cover
goes to some effort to have as minimal an overhead as possible. One way it
does this is to try to collect coverage only about the files in which you are
interested. But the checks to determine whether an op is in a file for which
coverage is being collected are also reasonably expensive. There is scope for
improvement here. In particular, the subroutine that does the ultimate check
is written in Perl. Rewriting that in XS would be advantageous. But reducing
the number of times the subroutine is called is also useful. More about this
in June's report though.

Merged pull requests:

  • 61 fix return code (need shift)
  • 57 cover -test exit code is always success, when test fails

Closed RT tickets:

  • #34888 Fix pod coverage for multiple packages in a file.

You can see the commits at https://github.com/pjcj/Devel--Cover/commits/master

Hours worked:

  • 18.05 2:45
  • 20.05 3:30
  • 27.05 1:00
  • 30.05 2:30
  • 31.05 2:15
  • Total 12:00
  • Total hours worked on grant: 269:35

Leave a comment

About TPF

The Perl Foundation - supporting the Perl community since 2000. Find out more at www.perlfoundation.org.

About this Entry

This page contains a single entry by mdk published on June 24, 2013 8:54 AM.

Improving Perl 5: Grant Report for Month 19 was the previous entry in this blog.

Tony Cook's Grant is Accepted is the next entry in this blog.

Find recent content on the main index or look in the archives to find all content.

OpenID accepted here Learn more about OpenID
Powered by Movable Type 4.38