On Oct 5, 2004, at 11:08 AM, Geoffrey Young wrote:

I think that all Apache::TestMB would need to do is add a make target that
looks like this:


test-cover ::
        @cover -delete
        @HARNESS_PERL_SWITCHES=-MDevel::Cover=+inc,$(HOME)/.apache-test
APACHE_TEST_EXTRA_ARGS=-one-process $(MAKE) test
        @cover

broken down it does this

- HARNESS_PERL_SWITCHES gets Devel::Cover started

Module::Build's testcover target already does this.

- +inc,$(HOME)/.apache-test keeps coverage away from generated A-T files,
which isn't required

Ah, cool. But $(HOME) doesn't correspond to ~/ here, does it?

- -one-process puts httpd in -X mode, but is only suppored in A-T current cvs

That's an environment variable? I think I wouldn't worry about that, since the next version of TestMB wouldn't come out before the next version of A-T.


there is more to do if you want to generate coverage on mod_perl handlers,
but that needs to happen from httpd.conf land. however, this will get
Devel::Cover started for *.t files that test normal Perl modules.

Yes, as I said, Module::Build's testcover target already does this. But I'll add the code to prevent it from including .apache-test files, just as soon as I figure out where they are.


Regards,

David



Reply via email to