On 4/19/13, Branko Čibej <[email protected]> wrote: > On 19.04.2013 16:38, Olemis Lang wrote: >> On 4/19/13, Gary Martin <[email protected]> wrote: >> [...] >>> I wonder if it would be wrong to ask for the XmlRpcPlugin maintainers to >>> resolve the issue by making sure that any required contrib files are >>> available. Another alternative would be to vendor branch the plugin >>> specifically to fix that. >> JFTR , the situation is not limited to XmlRpcPlugin . Indeed it reuses >> the fixture setup for functional tests implemented in core and adds >> nothing else. It is not possible to run Trac's functional tests either > > If Trac's test suite depends on scripts in contrib, all I can say is > that's clearly wrong-headed. >
I see it like if it was a testing dependency (e.g. test_requires) resolved inside the very same repository file hierarchy . Notice that these are functional tests , not unit tests . FWIW Trac core does not rely upon htpasswd.py at all . That's the responsibility of admins . However in order to run functional test suites and setting up the fixture the goal is to interact with the system just like users do , in a real working environment . Permission checks in that context will require real users with real passwords (not mocks et al.) , hence the need for the script in /contrib . -- Regards, Olemis. Apache™ Bloodhound contributor http://issues.apache.org/bloodhound Blog ES: http://simelo-es.blogspot.com/ Blog EN: http://simelo-en.blogspot.com/ Featured article:
