Hi Vidar,

On Fri, Feb 27, 2009 at 2:43 PM, Vidar Ramdal <vi...@idium.no> wrote:
> Any change of having the SLING-857 patch (reviewed and possibly)
> commited any day soon? Pretty-please-with-sugar-on-top?...

I had a look now that Felix committed that patch, and IIUC there are
no automated tests for that feature, could you add some?

The easiest might be to add some initial content that uses his XSLT
transform feature to the launchpad/testing module, and add some checks
in InitialContentTest [1]. Just some simple smoke tests would be good
enough IMHO, mainly to avoid regressions.

-Bertrand

[1] 
http://svn.apache.org/repos/asf/incubator/sling/trunk/launchpad/testing/src/test/java/org/apache/sling/launchpad/webapp/integrationtest/InitialContentTest.java

Reply via email to