Author: vmassol
Date: 2009-09-23 23:45:22 +0200 (Wed, 23 Sep 2009)
New Revision: 23882
Modified:
enterprise/branches/xwiki-enterprise-2.0/distribution-test/
wysiwyg-tests/src/test/it/com/xpn/xwiki/it/selenium/LinkTest.java
Log:
Try to fix flickering test
Modified: enterprise/branches/xwiki-enterprise-2.0/distribution-
test/wysiwyg-tests/src/test/it/com/xpn/xwiki/it/selenium/
LinkTest.java
===================================================================
--- enterprise/branches/xwiki-enterprise-2.0/distribution-test/
wysiwyg-tests/src/test/it/com/xpn/xwiki/it/selenium/LinkTest.java
2009-09-23 21:22:39 UTC (rev 23881)
+++ enterprise/branches/xwiki-enterprise-2.0/distribution-test/
wysiwyg-tests/src/test/it/com/xpn/xwiki/it/selenium/LinkTest.java
2009-09-23 21:45:22 UTC (rev 23882)
@@ -167,7 +167,7 @@
/**
* Test the basic feature for adding a link to a new page in a
new space.
*
- * @see
http://jira.xwiki.org/jira/browse/XWIKI-3511
+ * @see <a
href="http://jira.xwiki.org/jira/browse/
XWIKI-3511">XWIKI-3511</a>
*/
public void testCreateLinkToNewPageInNewSpace()
{
@@ -931,7 +931,7 @@
/**
* Test that editing a link with custom parameters set from
wiki syntax preserves the parameters of the link.
*
- * @see
http://jira.xwiki.org/jira/browse/XWIKI-3568
+ * @see <a
href="http://jira.xwiki.org/jira/browse/
XWIKI-3568">XWIKI-3568</a>
*/
public void testEditLinkPreservesCustomParameters()
{
@@ -988,8 +988,8 @@
/**
* Test that quotes in link tooltips are correctly escaped.
*
- * @see
http://jira.xwiki.org/jira/browse/XWIKI-3569
- * @see
http://jira.xwiki.org/jira/browse/XWIKI-3575
+ * @see <a
href="http://jira.xwiki.org/jira/browse/
XWIKI-3569">XWIKI-3569</a>
+ * @see <a
href="http://jira.xwiki.org/jira/browse/
XWIKI-3569">XWIKI-3575</a>
*/
public void testQuoteInLinkTooltip()
{
@@ -1267,7 +1267,7 @@
/**
* Test that a relative link is correctly edited.
*
- * @see
http://jira.xwiki.org/jira/browse/XWIKI-3676
+ * @see <a
href="http://jira.xwiki.org/jira/browse/
XWIKI-3676">XWIKI-3676</a>
*/
public void testEditRelativeLink()
{
@@ -1851,7 +1851,7 @@
protected void waitForStepToLoad(String name)
{
- waitForCondition("selenium.isElementPresent('//
*[contains(@class, \"" + name + "\")]');");
+ assertAndWaitForElement("//*[contains(@class, '" + name +
"')]");
Is there any other reason for this change of waiting method besides
"let's try
something else"?