On Feb 28, 2010, at 5:46 PM, Jerome Velociter wrote:
On 2/28/10 5:32 PM, vmassol (SVN) wrote:
Author: vmassol
Date: 2010-02-28 17:32:49 +0100 (Sun, 28 Feb 2010)
New Revision: 27370
Added:
contrib/sandbox/webdriver-ui-tests/src/test/it/org/xwiki/it/ui/AbstractAdminAuthenticatedTest.java
Removed:
contrib/sandbox/webdriver-ui-tests/src/test/it/org/xwiki/it/ui/AbstractAdminAuthenticatedTest.java
Modified:
contrib/sandbox/webdriver-ui-tests/src/test/it/org/xwiki/it/ui/AbstractTest.java
contrib/sandbox/webdriver-ui-tests/src/test/it/org/xwiki/it/ui/ImportTest.java
contrib/sandbox/webdriver-ui-tests/src/test/it/org/xwiki/it/ui/LoginTest.java
contrib/sandbox/webdriver-ui-tests/src/test/it/org/xwiki/it/ui/elements/AdministrationPage.java
contrib/sandbox/webdriver-ui-tests/src/test/it/org/xwiki/it/ui/elements/BasePage.java
contrib/sandbox/webdriver-ui-tests/src/test/it/org/xwiki/it/ui/elements/HistoryPane.java
contrib/sandbox/webdriver-ui-tests/src/test/it/org/xwiki/it/ui/elements/HomePage.java
contrib/sandbox/webdriver-ui-tests/src/test/it/org/xwiki/it/ui/elements/ImportPage.java
contrib/sandbox/webdriver-ui-tests/src/test/it/org/xwiki/it/ui/elements/LoginPage.java
contrib/sandbox/webdriver-ui-tests/src/test/it/org/xwiki/it/ui/elements/RegisterPage.java
Log:
Testing Selenium 2.0 (merge with WebDriver).
* Make sure the package is deleted before the test if it's there
* Added license headers
* Minor refactorings
- public void selectOptionReplaceHistory()
+ public void toggleReplaceHistoryOption()
{
getDriver().findElement(By.xpath("//input[@name='historyStrategy'
and @value='replace']")).click();
}
I don't think toggle is the right word here. Toggle usually refers to a
2-states switch
Mabye "selectReplaceHistoryOption" would better fit.
The reason I've used toggle is because you had added a method called
unselectOptionReplaceHistory() and its implementation was the same as
selectOptionHistory.
Are you sure it's not a toggle, i.e if I click twice on it, it's not in the same
state as initially?
Thanks
-Vincent