Details
-
Bug
-
Status: Closed
-
Major
-
Resolution: Won't Fix
-
5.3.2, 5.4
-
None
Description
In TAP5-1492 [1] "New features for SeleniumTestCase" one of the added methods to SeleniumTestCase is waitForElementToDisappear.
The commit [2] is by Howard on 31 Mar 2011.
The Prototype hide() documentation [3] briefly says "Hides and returns element".
I don't see it used in any of the tapestry projects and don't know if anyone has used it before.
Here is the code:
protected final void waitForElementToDisappear(String elementId)
{ String condition = String.format("window.$(\"%s\").hide()", elementId); waitForCondition(condition, PAGE_LOAD_TIMEOUT); }IMO this method does not what it is supposed to do.
A Patch is provided just with one minor change
protected final void waitForElementToDisappear(String elementId)
{ String condition = String.format("window.$(\"%s\") == undefined", elementId); waitForCondition(condition, PAGE_LOAD_TIMEOUT); }If there is a better approach please feel free to comment.
[1] TAP5-1492 https://issues.apache.org/jira/browse/TAP5-1492
[3] Prototype hide() function http://prototypejs.org/api/element/hide