That's great :), thanks a lot for updating the golden files. regards, Catalin
On Nov 16, 2007 8:25 PM, Matt Cooper <[EMAIL PROTECTED]> wrote: > FYI - I went ahead and commited the golden file changes r595766. > > Perhaps I lost the email but I am a little concerned that I never saw > a continuum build failure for this so there may be yet a bigger issue > here too with the build automation. > > Regards, > Matt > > On Nov 15, 2007 6:21 AM, Matthias Wessendorf <[EMAIL PROTECTED]> wrote: > > There is also a wiki page about it (see [1]) > > > > -M > > > > [1] http://wiki.apache.org/myfaces/Trinidad_RenderKit_test_framework > > > > > > On Nov 15, 2007 2:44 AM, Catalin Kormos <[EMAIL PROTECTED]> > wrote: > > > Hi Jeanne, > > > > > > That's great, many thanks for providing this information, as i wasn't > very > > > sure how to tackle it. > > > > > > regards, > > > Catalin > > > > > > > > > > > > On Nov 15, 2007 3:33 AM, Jeanne Waldman < [EMAIL PROTECTED]> > wrote: > > > > No objections. > > > > > > > > The normal practice is when I see test failures, I go to the > > > > test-failures directory. In your case, you see > > > > treeTable-minimal-golden.xml, treeTable-minimalIE-golden.xml, etc. > > > > > > > > Then what I do is diff the src with the test-failures. So you would > > > > look at the diff between the src treeTable-minimal-golden.xml and > the > > > > test-failure file of the same name. If the diff is what you expect, > then > > > > you copy your test-failure file into the src file. > > > > > > > > Then you rebuild. If it is successful, you check in, so you'll check > in > > > > the treeTable*golden.xml files and the build will run for everyone. > > > > > > > > The golden file tests are a great way to catch accidental changes to > > > > renderers. > > > > > > > > Thanks, > > > > Jeanne > > > > > > > > > > > > Catalin Kormos wrote: > > > > > Hi, > > > > > > > > > > Yes you're right, sorry about that; before, text icons were > supported, > > > > > and the patch introduced image icons, and the tests weren't > patched yet. > > > > > If nobody has any objections, we'll update the tests also, asap, > to test > > > > > for img instead of span in the ouput. > > > > > > > > > > regards, > > > > > Catalin > > > > > > > > > > On Nov 14, 2007 11:33 PM, Andrew Robinson < > [EMAIL PROTECTED] > > > > > > > > > > > > > > > > > <mailto:[EMAIL PROTECTED]>> wrote: > > > > > > > > > > The current trunk (1.0.5) is broken in SVN with the golden > files > > > test > > > > > failing for the treeTable. Looks like img tags are now being > used > > > > > instead of span elements. > > > > > > > > > > I think it was revision 595029 that broke it: > > > > > > > > > > r595029 | ckormos | 2007-11-14 13:14:19 -0700 (Wed, 14 Nov > 2007) | 1 > > > > > line > > > > > Changed paths: > > > > > M > > > /myfaces/trinidad/trunk/trinidad/src/site/xdoc/skin-selectors.xml > > > > > M > > > > > > > > > /myfaces/trinidad/trunk/trinidad/trinidad-examples/trinidad-demo/src/main/java/org/apache/myfaces/trinidaddemo/DemoTreeData.java > > > > > > > > > > M > > > > > > > > > /myfaces/trinidad/trunk/trinidad/trinidad-examples/trinidad-demo/src/main/java/org/apache/myfaces/trinidaddemo/InfiniteTree.java > > > > > M > > > > > > > > > /myfaces/trinidad/trunk/trinidad/trinidad-examples/trinidad-demo/src/main/java/org/apache/myfaces/trinidaddemo/TreeNodeImpl.java > > > > > > > > > > M > > > > > > > > > /myfaces/trinidad/trunk/trinidad/trinidad-examples/trinidad-demo/src/main/webapp/demos/panelPageSkinDemo.jspx > > > > > M > > > > > > > > > /myfaces/trinidad/trunk/trinidad/trinidad-examples/trinidad-demo/src/main/webapp/skins/beach/beach.css > > > > > > > > > > A > > > > > > > > > /myfaces/trinidad/trunk/trinidad/trinidad-examples/trinidad-demo/src/main/webapp/skins/beach/images/document.png > > > > > A > > > > > > > > > /myfaces/trinidad/trunk/trinidad/trinidad-examples/trinidad-demo/src/main/webapp/skins/beach/images/yellow- > > > > > folder-closed.png > > > > > A > > > > > > > > > /myfaces/trinidad/trunk/trinidad/trinidad-examples/trinidad-demo/src/main/webapp/skins/beach/images/yellow- > folder-open.png > > > > > M > > > > > > > > > /myfaces/trinidad/trunk/trinidad/trinidad-examples/trinidad-demo/src/main/webapp/skins/purple/purpleSkin.css > > > > > > > > > > M > > > > > > > > > /myfaces/trinidad/trunk/trinidad/trinidad-impl/src/main/java/org/apache/myfaces/trinidadinternal/renderkit/core/desktop/TreeTableRenderer.java > > > > > M > > > > > > > > > /myfaces/trinidad/trunk/trinidad/trinidad-impl/src/main/java/org/apache/myfaces/trinidadinternal/renderkit/core/skin/BaseDesktopSkin.java > > > > > > > > > > M > > > > > > > > > /myfaces/trinidad/trunk/trinidad/trinidad-impl/src/main/java/org/apache/myfaces/trinidadinternal/renderkit/core/xhtml/SkinProperties.java > > > > > M > > > > > > > > > /myfaces/trinidad/trunk/trinidad/trinidad-impl/src/main/java/org/apache/myfaces/trinidadinternal/renderkit/core/xhtml/SkinSelectors.java > > > > > > > > > > M > > > > > > > > > /myfaces/trinidad/trunk/trinidad/trinidad-impl/src/main/java/org/apache/myfaces/trinidadinternal/renderkit/core/xhtml/TreeRenderer.java > > > > > M > > > > > > > > > /myfaces/trinidad/trunk/trinidad/trinidad-impl/src/main/java/org/apache/myfaces/trinidadinternal/renderkit/core/xhtml/table/TreeNodeColumnRenderer.java > > > > > > > > > > A > > > > > > > > > /myfaces/trinidad/trunk/trinidad/trinidad-impl/src/main/resources/META-INF/adf/images/nav- > minus.gif > > > > > A > > > > > > > > > /myfaces/trinidad/trunk/trinidad/trinidad-impl/src/main/resources/META-INF/adf/images/nav- > > > plus.gif > > > > > A > > > > > > > > > /myfaces/trinidad/trunk/trinidad/trinidad-impl/src/main/resources/META-INF/adf/images/tree- > > > > > line-last.gif > > > > > A > > > > > > > > > /myfaces/trinidad/trunk/trinidad/trinidad-impl/src/main/resources/META-INF/adf/images/tree- > > > line-middle.gif > > > > > A > > > > > > > > > /myfaces/trinidad/trunk/trinidad/trinidad-impl/src/main/resources/META-INF/adf/images/tree- > > > > > line-trunk.gif > > > > > > > > > > fix for TRINIDAD-769 Tree / TreeTable skinning improvements > (thanks > > > to > > > > > Cristi Toth for provinding the patch). > > > > > > > > > ------------------------------------------------------------------------ > > > > > > > > > > > > > > > > > > > > > > > > > > > > -- > > Matthias Wessendorf > > > > further stuff: > > blog: http://matthiaswessendorf.wordpress.com/ > > mail: matzew-at-apache-dot-org > > >
