Ok, I did another look at the testing stuff and I think we can for now leave most of the stuff as is, but move it out of commons - as it has dependencies on some Sling bundles.
But we have a problem in the slingtest module - this module uses internal classes of the jcr resource module - which it definitly should not; internal classes should imho not be used by any other library. And this stuff is exactly the stuff that creates the circular dependency. So I'm personally more in favour of removing this again! We have one place where it is used (in the i18n tests) and I think we can change the test easily by using mock objects. So my suggested actions are now: - remove dependency to internal stuff (and adapt the tests) - merge both testing modules into a single one - move the testing module out of commons Carsten -- Carsten Ziegeler cziege...@apache.org
