On Mon, Apr 6, 2009 at 9:37 AM, Felix Meschberger <[email protected]> wrote: > Hi Bertrand > > Bertrand Delacretaz schrieb: >> ...Why did we move the "sling" folders from target/sling (where mvn clean >> kills them) to siblings of src folders? >> Is that on purpose, or a side effect of other changes?...
> This started to surface when I redid the launchpad. One thing I did was > to remove support for system properties in the Sling web app, since > system properties have a lot of issues in web application scenario. Ok, in general this seems to make sense. > > ...Maybe we would have to start and stop Sling differently. Eg. by > launching a separate process with Sling standalone and shutdown that > instance after the tests. Such setup could be hooked into the > pre-integration-test and post-integration-test phases.... Can't we just use a specific system property to set the sling.home value for integration tests? We could use "sling.integration.testing.home" for example, to make it clear what that is, and this property would override any other setting, logging a warning when that happens. Seems reasonably foolproof and simple to implement, or am I missing something? Redoing our testing infrastructure just for this reason feels a bit like overkill to me. -Bertrand
