> On June 24, 2014, 8:23 p.m., Ryan Baxter wrote: > > I delivered the change but had to revert it because it is breaking the > > build, some unit tests are failing. Can you please look into this? Also > > it would be good to make sure this new code is covered by a unit test. > > Jiaqing Guo wrote: > I have fixed the problem in the test case and add test cases for my > changeset. > By the way, I found > DefaultTemplateProcessorTest#testSafeCrossDocumentCloning case is always > failed and it break the build process. I confirmed it is not related to my > changeset. > Are there anyone know it?
I did a build before applying your patch and it passed there was no failure. I just applied your patch and did another build and it passed as well. Where are you seeing a failure? - Ryan ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/22363/#review46570 ----------------------------------------------------------- On Oct. 18, 2014, 3:16 p.m., Jiaqing Guo wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/22363/ > ----------------------------------------------------------- > > (Updated Oct. 18, 2014, 3:16 p.m.) > > > Review request for shindig, Dan Dumont, Ryan Baxter, Rich Thompson, and > Stanton Sievers. > > > Bugs: SHINDIG-1978 > https://issues.apache.org/jira/browse/SHINDIG-1978 > > > Repository: shindig > > > Description > ------- > > If gadget spec contains a "ModulePrefs" tag with "title" attribute like below: > <ModulePrefs title="this is title" description="Hello World Embedded > Experience gadget" height="400" width="500"> > Then expect the loaded gadget should contain proper "title" element like > below: > <html> > <head> > <title>this is title</title> > ...... > But the actual result is the load gadget doesn't contain "title" element. > > > Diffs > ----- > > > http://svn.apache.org/repos/asf/shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/render/RenderingGadgetRewriter.java > 1605175 > > http://svn.apache.org/repos/asf/shindig/trunk/java/gadgets/src/test/java/org/apache/shindig/gadgets/render/RenderingGadgetRewriterTest.java > 1533559 > > http://svn.apache.org/repos/asf/shindig/trunk/java/server/src/test/java/org/apache/shindig/server/endtoend/EndToEndTest.java > 1620491 > > Diff: https://reviews.apache.org/r/22363/diff/ > > > Testing > ------- > > > Thanks, > > Jiaqing Guo > >