More test fixing.
authorThiago H. de Paula Figueiredo <thiago@arsmachina.com.br>
Sat, 24 Nov 2018 18:33:55 +0000 (16:33 -0200)
committerThiago H. de Paula Figueiredo <thiago@arsmachina.com.br>
Sat, 24 Nov 2018 18:33:55 +0000 (16:33 -0200)
tapestry-core/src/test/groovy/org/apache/tapestry5/integration/app1/AlertsTests.groovy
tapestry-core/src/test/java/org/apache/tapestry5/integration/TapestryCoreTestCase.java
tapestry-core/src/test/java/org/apache/tapestry5/integration/app1/FormTests.java
tapestry-test/src/main/java/org/apache/tapestry5/test/SeleniumTestCase.java

index 92854f7..de1df6c 100644 (file)
@@ -34,13 +34,13 @@ class AlertsTests extends App1TestCase {
 
         clickAndWait "//input[@value='Traditional Update']"
 
-        waitForCSSSelectedElementToAppear "$CONTAINER .alert"
+        waitForCssSelectorToAppear "$CONTAINER .alert"
 
         assertTextPresent "trad warn until"
 
         clickAndWait BACK_TO_INDEX
 
-        waitForCSSSelectedElementToAppear "$CONTAINER .alert"
+        waitForCssSelectorToAppear "$CONTAINER .alert"
 
         assertTextPresent "trad warn until"
 
@@ -78,7 +78,7 @@ class AlertsTests extends App1TestCase {
 
         clickAndWait "//input[@value='Traditional Update']"
 
-        waitForCSSSelectedElementToAppear "$CONTAINER .alert"
+        waitForCssSelectorToAppear "$CONTAINER .alert"
 
         assertTextPresent "trad warn transient"
 
@@ -110,7 +110,7 @@ class AlertsTests extends App1TestCase {
 
         click "//input[@value='Ajax Update']"
 
-        waitForCSSSelectedElementToAppear "$CONTAINER .alert"
+        waitForCssSelectorToAppear "$CONTAINER .alert"
 
         assertTextPresent "ajax error until"
 
@@ -133,7 +133,7 @@ class AlertsTests extends App1TestCase {
         
         click "//input[@value='Ajax Update']"
         
-        waitForCSSSelectedElementToAppear "$CONTAINER .alert"
+        waitForCssSelectorToAppear "$CONTAINER .alert"
         
         assert isElementPresent("//div[@class='alert alert-dismissable alert-warning']/a/span[text()='Markup!']")
 
index 204f9ad..b9f71a9 100644 (file)
@@ -69,7 +69,7 @@ public abstract class TapestryCoreTestCase extends SeleniumTestCase
      */
     protected final void assertFirstAlert(String text)
     {
-        waitForCSSSelectedElementToAppear("[data-container-type=alerts] .alert");
+        waitForCssSelectorToAppear("*[data-container-type=alerts] .alert");
 
         // Add the special "x" for the close button to the text.
         assertText("css=[data-container-type=alerts] .alert span", text);
index fa792b6..5db2f79 100644 (file)
@@ -278,7 +278,7 @@ public class FormTests extends App1TestCase
 
         //make sure it's still selected if we navigate back...
         click("css=button.previousButton");
-        waitForCSSSelectedElementToAppear("td.selected");
+        waitForCssSelectorToAppear("td.selected");
 
         click("css=button.nextButton");
 
@@ -316,7 +316,7 @@ public class FormTests extends App1TestCase
         waitForSelectedToBeRemoved();
 
         click("css=div.datePicker .footerTable button");
-        waitForCSSSelectedElementToAppear("td.selected");
+        waitForCssSelectorToAppear("td.selected");
 
         //#1
         click("css=div.datePicker .footerTable button");
@@ -382,7 +382,7 @@ public class FormTests extends App1TestCase
         click("css=.x-impact .btn");
         waitForCondition(datePickerVisible);
         click("css=a.topLabel");
-        waitForCSSSelectedElementToAppear("div.labelPopup");
+        waitForCssSelectorToAppear("div.labelPopup");
         click("css=div.labelPopup a");
 
         waitForInvisible("css=div.labelPopup");
index b684895..d7ac8ab 100644 (file)
@@ -1654,26 +1654,6 @@ public abstract class SeleniumTestCase extends Assert implements Selenium
     }
 
     /**
-     * Waits, up to the page load limit for an element (identified by a CSS rule) to exist
-     * (it is not assured that the element will be visible).
-     *
-     * This implementation only works if the application provides a function onto the
-     * window object:  "testSupport.findCSSMatchCount()" which accepts a CSS rule and returns the number
-     * of matching elements.
-     *
-     * @param cssSelector
-     *         used to locate the element
-     * @since 5.3
-     * @deprecated Deprecated in 5.4 with no replacement
-     */
-    protected void waitForCSSSelectedElementToAppear(String cssSelector)
-    {
-        String condition = String.format("window.testSupport.findCSSMatchCount(\"%s\") > 0", cssSelector);
-
-        waitForCondition(condition, PAGE_LOAD_TIMEOUT);
-    }
-
-    /**
      * Waits for the element with the given client-side id to be present in the DOM (
      * does not assure that the element is visible).
      *