Hi Ludovic, You should gather those 3 methods into one: * functional tests are slow to run and this will slow down the overall tests (the setup is long) * the naming of the test methods is not explanatory Thanks -Vincent On Jun 2, 2008, at 3:39 PM, ludovic (SVN) wrote:

Author: ludovic Date: 2008-06-02 15:39:10 +0200 (Mon, 02 Jun 2008) New Revision: 10078

Modified: xwiki-products/xwiki-enterprise/branches/xwiki-enterprise-1.4/ distribution-test/selenium-tests/src/test/it/com/xpn/xwiki/it/ selenium/WysiwygEditorFromWikiTest.java Log: XWIKI-2343 Wysiwyg editor breaks accents in links

Modified: xwiki-products/xwiki-enterprise/branches/xwiki- enterprise-1.4/distribution-test/selenium-tests/src/test/it/com/xpn/ xwiki/it/selenium/WysiwygEditorFromWikiTest.java =================================================================== --- xwiki-products/xwiki-enterprise/branches/xwiki-enterprise-1.4/ distribution-test/selenium-tests/src/test/it/com/xpn/xwiki/it/ selenium/WysiwygEditorFromWikiTest.java 2008-06-01 21:50:23 UTC (rev 10077) +++ xwiki-products/xwiki-enterprise/branches/xwiki-enterprise-1.4/ distribution-test/selenium-tests/src/test/it/com/xpn/xwiki/it/ selenium/WysiwygEditorFromWikiTest.java 2008-06-02 13:39:10 UTC (rev 10078) @@ -100,4 +100,29 @@ clickLinkWithText("Wiki"); assertEquals("## comment", getFieldValue("content")); } + + public void testAccentsInLinks() throws Exception + { + setFieldValue("content", "[\u00E9t\u00E9]"); + clickLinkWithText("WYSIWYG"); + clickLinkWithText("Wiki"); + assertEquals("[\u00E9t\u00E9]", getFieldValue("content")); + } + + public void testAccentsInLinks2() throws Exception + { + setFieldValue("content", "[test>\u00E9t\u00E9]"); + clickLinkWithText("WYSIWYG"); + clickLinkWithText("Wiki"); + assertEquals("[test>\u00E9t\u00E9]", getFieldValue("content")); + } + + public void testAccentsInLinks3() throws Exception + { + setFieldValue("content", "[\u00E9t\u00E9>test]"); + clickLinkWithText("WYSIWYG"); + clickLinkWithText("Wiki"); + assertEquals("[\u00E9t\u00E9>test]", getFieldValue("content")); + } + }