Re: [htmltmpl] Phalanx and HTML::Template

[Sam T. <sa...@tr...>]

On Fri, 17 Oct 2003, Gabor Szabo wrote:

> Current status v2.6
> -------------------
> There are 57 (actually 60 but the script reports 57) tests.
> Based on the report of Devel::Cover the tests provide a 67.95% code
> coverage.

That seems pretty low.  Is that meant to suggest that 33% of lines of
Perl code aren't executed during the test run?  Maybe you ran the
tests without TEST_SHARED_MEMORY and TEST_FILE_CACHE set?

> 1) Start from the documentation and write tests based
>    on the documentation. (but keep the old tests)

Cool.

> 2) use Test::More for these new tests.

Why?  That would add a new dependency for the module.  I got enough
flak when I started using Test.pm!

> 3) Fix documentation based on your input.

Cool.

> 4) Change the code if necessary to make it easier (or possible)
>    to test.

I'd be surprised if this was necessary.  HTML::Template presents a
nice black box and it's usually pretty easy to feed it a set of inputs
and examine the output.

> I have packaged the changes so far and it is available here:
> http://www.pti.co.il/download/HTML-Template-2.61_01.tar.gz

Could you make this available as a patch with 'diff -Naur'?  That
would make it easier to evaluate the changes at a glance.

-sam