Re: [xwiki-devs] [xwiki-notifications] r10078 - xwiki-products/xwiki-enterprise/branches/xwiki-enterprise-1.4/distribution-test/selenium-tests/src/test/it/com/xpn/xwiki/it/selenium - xwiki-devs@xwiki.org

2 Jun 2008

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"));
 +    }
 +
 } 