Re: Test coverage

Barrie Slaymaker <[email protected]> Wed, 8 Sep 2004 10:19:40 -0400
Newsgroups gmane.comp.version-control.revml
Message-ID <[email protected]>
On Wed, Sep 08, 2004 at 01:42:58PM +0100, Chia-liang Kao wrote:
> Updated at http://wagner.elixus.org/~clkao/vcp-cover/

Thanks.  For those playing along at home, please note that 

- no platform can run all tests in all modes,
    - for instance clkao is not testing VSS and the experimental (and
      disabled) p4 C client library support.
- most of the untested conditionals and statements are for result code
  checking because Perl follows C's lead in returning error codes
  instead of throwing exceptions (and people wonder why I like
  exceptions so much, sigh).
- we should definitely test some things better
    - *please* feel free to add to our test suite, it's one of the most
      important things we can do.  See clkao's coverage report for good
      places to start.

>   make test PERL5OPT=-MDevel::Cover=+ignore,TestUtil
> 
> This makes all child processes to run with devel::cover.

Seems much better now; what did you change?

I've removed some dead code as a result of this :).

> pjcj said there's already internal facility to mark uncoverable
> things, but there's no UI yet.  Hopefully in the next Devel::Cover
> release.

That'll be interesting to see.  Wonder if it's like XPath/XQuery so you
can do some fine-grained categorizing.

- Barrie