Missing test cases - Githubissues

modcluster / mod_proxy_cluster

mod_cluster is an intelligent native Apache httpd-based and pure-Java Undertow-based load-balancer

https://www.modcluster.io

Apache License 2.0

7 stars 15 forks source link

Missing test cases #93

Open jajik opened 1 year ago

jajik commented 1 year ago

Currently, tests are missing:

[ ] multiple balancers
[ ] issue 63 tests
[x] MODCLUSTER-204
[x] #236
[x] #271
[x] ModCluster Management protocol (e.g. #273)

rhusar commented 1 year ago

@jajik Is this regarding https://github.com/modcluster/mod_proxy_cluster/tree/main/test ? Do we have instructions how to run these?

jajik commented 1 year ago

Yes. And more or less yes. There is a README within test/native/ that describes it a bit. It might be worth it to mention it in the main README.

rhusar commented 1 year ago

Just for the context, when I created (split) the mod_proxy_cluster repository I did not cleanup the test infrastructure to get everything running again, so this has been on the back burner for quite a while now. Regarding the readme, I am also considering the idea of having this as part of the https://docs.modcluster.io/ - it's flexible, convenient to edit, doesn't pollute the source code SCM repo, etc. Moreover, the other goal is to get these tests running regularly in CI. Similarly, on the mod_cluster repo, the goal is to run real integration tests with real containers.

jajik commented 1 week ago

Several testcases were added in https://github.com/modcluster/mod_proxy_cluster/pull/283

edit: Jut to clarify, MODCLUSTER-204 was superseded by https://github.com/modcluster/mod_proxy_cluster/pull/271 so now the truncation does not happen and corresponding request are treated as erroneous.