On Sun, May 13, 2018 at 12:49:30PM +0200, Luca Toscano wrote: > Hi everybody, > > I came up with http://home.apache.org/~elukey/httpd-framework-pr61860.patch > to add some tests for PR 61860. The patch that I have in mind is for > http_protocol, but for the moment the "visible" use case is that an out of > range bytes request that leads to a 416 may include duplicate headers set > via Header always set. > > I can see several directories in the repo for the .t tests, like: > > - apache, that contains various tests, some of them named "prXXXX". > - modules, that contains specific tests for some modules. > - filter (that could be a subdir of the above, but not sure). > - and finally some generic dirs like "ssl", "php", etc.. > > Same thing for the supporting httpd.conf's snippets as well. > > Do we have some guidelines/conventions about where/how to add tests? I > haven't found anything in our docs up to know, but I might have missed > something.
Thanks a lot for adding the test! There are no conventions here and it doesn't really matter, more tests are good however they are named :) Where we have a reasonably small, self-contained regression test for a PR, I like the idea of having it at t/apache/prXXXXX.t and I did a bunch of tests like that at one time. Moving t/filter/*.t across to t/modules/*.t would probably make sense, we have several filter tests in t/modules already. Regards, Joe