Re: [Apache-dispatch-devel] Fwd: SF.net SVN: apache-dispatch: [50]trunk

[Fred M. <fr...@ta...>]

On Wed, 9 Aug 2006, Geoffrey Young wrote:
> Fred Moyer wrote:
>
> I think I see a few things that could be contributing.  first of all,
> the results of the Makefile.PL are odd when A-T isn't present - it spits
> out "sorry, can't run tests" _and_ runs the tests.  we need to decide
> what we'll be requiring :)
>
> WS::CDN has both apache and non-apache tests, so I wanted to run _some_
> of the tests no matter what, but configure and run the apache tests if
> A-T was properly set up.  we probably only want to run tests if
>
>  o A-T is configured
>  o apache has mod_perl configured
>
> to that end I think we need a patch like the one attached.

Got it - my bad, I left the A::T dependency line in there.

> make sure that our enabled httpd has mod_perl available to it - if it
> doesn't then we can't run our tests at all.  but to find out httpd needs
> to be fully configured, which doesn't happen until we actually run 'make
> test'.  the solution to this little problem is all.t, a special test
> file with magical properties.  yum :)
>
> so, all.t would look something like this
>
>  http://search.cpan.org/src/GEOFF/Apache-SSLLookup-2.00_04/t/live/all.t

I was wondering what that did before, but if I had read the testing docs 
on perl.apache.org all the way through I would probably know :)

> you _also_ need to protect all of extra.last.conf.in in <IfModule>
> statements such that if we have httpd without mod_perl httpd still
> starts, so we can then run all.t, which will then skip over our tests.
> apache will then shut down.  yeah, I know, but it's just the way things
> work.

Not a problem - I actually hit a bug in libapreq because of this and 
submitted a patch so I grok the importance.

> the simple import form (:) is fine for *.t clients (like this one) while
> the action form (-) is a requirement for *.pm server side tests because
> it adds (yet more) magic.

> anyway, I hope this helps clear things up a bit.  I'm happy to work on
> and commit things, but I didn't want to step on your work.

don't worry about stepping on my work, looks like I've stepped on it in a 
few places :)  It looks like I haven't done my A::T homework thoroughly 
enough here.  I'll take another pass at this after reading the testing 
page all the way through, but feel free to commit here if so desired.