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