You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@tapestry.apache.org by hl...@apache.org on 2012/12/11 22:32:48 UTC

[1/2] git commit: Keep fixing broken tests

Updated Branches:
  refs/heads/5.4-js-rewrite 65ae8fbe8 -> 92fcaa9fa


Keep fixing broken tests


Project: http://git-wip-us.apache.org/repos/asf/tapestry-5/repo
Commit: http://git-wip-us.apache.org/repos/asf/tapestry-5/commit/92fcaa9f
Tree: http://git-wip-us.apache.org/repos/asf/tapestry-5/tree/92fcaa9f
Diff: http://git-wip-us.apache.org/repos/asf/tapestry-5/diff/92fcaa9f

Branch: refs/heads/5.4-js-rewrite
Commit: 92fcaa9fafcc563848fd56c3fee5316d75bfc420
Parents: 319cbeb
Author: Howard M. Lewis Ship <hl...@apache.org>
Authored: Tue Dec 11 13:25:24 2012 -0800
Committer: Howard M. Lewis Ship <hl...@apache.org>
Committed: Tue Dec 11 13:25:24 2012 -0800

----------------------------------------------------------------------
 .../tapestry5/integration/app1/FormTests.java      |   14 ++++++++------
 1 files changed, 8 insertions(+), 6 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/tapestry-5/blob/92fcaa9f/tapestry-core/src/test/java/org/apache/tapestry5/integration/app1/FormTests.java
----------------------------------------------------------------------
diff --git a/tapestry-core/src/test/java/org/apache/tapestry5/integration/app1/FormTests.java b/tapestry-core/src/test/java/org/apache/tapestry5/integration/app1/FormTests.java
index 9903d75..86eb651 100644
--- a/tapestry-core/src/test/java/org/apache/tapestry5/integration/app1/FormTests.java
+++ b/tapestry-core/src/test/java/org/apache/tapestry5/integration/app1/FormTests.java
@@ -40,12 +40,12 @@ public class FormTests extends TapestryCoreTestCase
     {
         openLinks("Page Context in Form");
 
-        assertTextSeries("//li[%d]", 1, "betty", "wilma", "context with spaces", "context/with/slashes");
+        assertTextSeries("//div[@class='main']//li[%d]", 1, "betty", "wilma", "context with spaces", "context/with/slashes");
         assertFieldValue("t:ac", "betty/wilma/context$0020with$0020spaces/context$002fwith$002fslashes");
 
         clickAndWait(SUBMIT);
 
-        assertTextSeries("//li[%d]", 1, "betty", "wilma", "context with spaces", "context/with/slashes");
+        assertTextSeries("//div[@class='main']//li[%d]", 1, "betty", "wilma", "context with spaces", "context/with/slashes");
         assertFieldValue("t:ac", "betty/wilma/context$0020with$0020spaces/context$002fwith$002fslashes");
     }
 
@@ -222,7 +222,7 @@ public class FormTests extends TapestryCoreTestCase
 
         click("css=.x-birthday i.icon-calendar");
 
-        sleep(50);
+        sleep(200);
 
         assertText("//A[@class='topLabel']", "1966 d\u00e9cembre");
     }
@@ -317,18 +317,20 @@ public class FormTests extends TapestryCoreTestCase
 
         waitForCSSSelectedElementToAppear("div.datePicker");
 
-        click("//button[@text='None']");
+        String noneButton = "//button[text()='None']";
+
+        click(noneButton);
 
         waitForInvisible(pickerGoneSelector);
 
         assertFieldValue("asteroidImpact", "");
 
-        click("//button[@text='None']");
+        click(noneButton);
 
         waitForCSSSelectedElementToAppear("div.datePicker");
         assertFalse(isElementPresent("css=td.selected"));
 
-        click("//button[@text='None']");
+        click(noneButton);
 
         waitForInvisible(pickerGoneSelector);
         assertFieldValue("asteroidImpact", "");