httpd-apreq-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Stas Bekman <>
Subject Re: [patch] move Apache2 setup into a common file
Date Thu, 13 Jan 2005 19:34:16 GMT
Joe Schaefer wrote:
> Stas Bekman <> writes:
> [...]
>>As I've tweaked Makefile.PL and committed it, here it the latest
>>Index: glue/perl/t/lib/TestApReq/
>>--- glue/perl/t/lib/TestApReq/	(revision 0)
>>+++ glue/perl/t/lib/TestApReq/	(revision 0)
>>@@ -0,0 +1,7 @@
>>+package TestApReq::DocSetup;
>>+# common things needed to run inlined doc tests go here
>>+use Apache2; # XXX: automate this

>>Index: glue/perl/docs/Upload.pod
>>--- glue/perl/docs/Upload.pod	(revision 125036)
>>+++ glue/perl/docs/Upload.pod	(working copy)
>>@@ -3,7 +3,7 @@
>>  Apache::Upload - Methods for dealing with file uploads.
>>  =for testing
>>-    use Apache2;
>>+    use TestApReq::DocSetup;
> I'll withhold my -1 (veto) on this, at least for a few days.
> But the show-stopper problem with this particular patch is
> that you have introduced a new dependency into our doc tests.
> So unless you also plan to install TestApReq::DocSetup somewhere,
> users will not be able to run the doc tests post-installation.

Well, you've asked me to how move the setup part out of the tests, and I 
did it in this way. You've never said those tests are going to be run 
post-install. I never thought that was the idea about Test::Inline.

Any a bit more than trivial testing environment requires a setup. If you 
need to duplicate this setup in every test, then you ask yourself whether 
the testing environment/tools you've chosen are good enough for you. But 
as I've said enough, I'm not sure whether the purpose of Test::Inline is 
to run tests after install. I thought it was rather about documenting and 
testing at the same time, as you develop the tool and has nothing to do 
with the user.

In any case, there is no need to veto anything, since this is not my itch, 
so if you don't like it that's fine.

Stas Bekman            JAm_pH ------> Just Another mod_perl Hacker     mod_perl Guide --->

View raw message