On Feb 2, 2006, at 9:46, Nicholas Clark wrote:

I wasn't sure if Mock::* was the best name to use.

I think it's appropriate, since it really is just mocking a subset of the behaviour of the HTTP::* modules. The only alternative I can think of that might make sense is naming them Pod::Webserver::Daemon, Pod::Webserver::Request, etc...

Also, the patch as supplied was intended to give the minimum changes to the POD part of the code whilst keeping everything working, more as a proof of
concept than a final version. If the chosen path is to drop the use of
HTTP:Daemon and associated modules, then I believe that it's possible to clean the code still further by simplifying the API between the two halves.

I think it has the right level of abstraction now, and the break-down into separate packages makes it easy to test. But, send me a patch if you come up with a good way to simplify it further.

I've refactored and added tests for the module and for your additions, so sync up to the latest version:

  http://svn.lohutok.net/nam/trunk/perl5/modules/Pod/Webserver/

I'd also appreciate any bug (or success) reports from people who run the tests on unusual operating systems or versions of Perl.

Thanks!
Allison

Reply via email to