Discussion:
mod_cache brokenness test
Colm MacCarthaigh
2005-09-27 09:55:17 UTC
Permalink
I've done up a test for mod_cache + mod_dir brokenness;

http://people.apache.org/~colm/cache-test.patch

I've added a virtualhost for mod_cache, because caching can easily
interfere with a lot of the other tests, and it's best kept out of
the way (and to be expanded upon!).

Anyway, as this is my first attempt at something in httpd/test/, I'm
sure I got something wrong, and I'm not too sure of the procedures.

Is it vaguely right? And can I commit it?
--
Colm MacCárthaigh Public Key: colm+***@stdlib.net
Joe Orton
2005-09-27 10:10:29 UTC
Permalink
Post by Colm MacCarthaigh
I've done up a test for mod_cache + mod_dir brokenness;
http://people.apache.org/~colm/cache-test.patch
I've added a virtualhost for mod_cache, because caching can easily
interfere with a lot of the other tests, and it's best kept out of
the way (and to be expanded upon!).
Anyway, as this is my first attempt at something in httpd/test/, I'm
sure I got something wrong, and I'm not too sure of the procedures.
Is it vaguely right? And can I commit it?
Some nits but otherwise go ahead, great!

1. use <VirtualHost mod_cache> - the _default_: is not necessary AFAIK
2. use just "need 'cache', 'disk_cache'" for the requirements

But this test is going to fail with current releases, is that right? I
aim to keep the tests all passing with each tip-of-branch
2.0.x/2.2.x/trunk. So an additional requirement:

need foo, bar, need_min_apache_version('2.3.0')

would be good.

joe
Colm MacCarthaigh
2005-09-27 10:54:52 UTC
Permalink
Post by Joe Orton
Post by Colm MacCarthaigh
Is it vaguely right? And can I commit it?
Some nits but otherwise go ahead, great!
1. use <VirtualHost mod_cache> - the _default_: is not necessary AFAIK
2. use just "need 'cache', 'disk_cache'" for the requirements
Excellent, even simpler.
Post by Joe Orton
But this test is going to fail with current releases, is that right?
Yes, and trunk too, it has yet to be fixed anywhere :/
Post by Joe Orton
I aim to keep the tests all passing with each tip-of-branch
need foo, bar, need_min_apache_version('2.3.0')
would be good.
I've added it all, and will commit shortly. I'll get back to working
on a fix.
--
Colm MacCárthaigh Public Key: colm+***@stdlib.net
Loading...