Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Implement class locator type #173

Closed
deki opened this issue Nov 27, 2015 · 2 comments
Closed

Implement class locator type #173

deki opened this issue Nov 27, 2015 · 2 comments
Assignees
Milestone

Comments

@deki
Copy link
Contributor

deki commented Nov 27, 2015

Class locator type currently results in UnsupportedOperationException.

I'm willing to implement this and create a pull request for it.

java.lang.UnsupportedOperationException: Unknown locator type: class=popup
at jp.vmi.selenium.selenese.locator.WebDriverElementFinder.findElementsInternal(WebDriverElementFinder.java:218)
at jp.vmi.selenium.selenese.locator.WebDriverElementFinder.findElements(WebDriverElementFinder.java:259)
at jp.vmi.selenium.selenese.locator.WebDriverElementFinder.findElements(WebDriverElementFinder.java:271)
at jp.vmi.selenium.selenese.locator.WebDriverElementFinder.findElement(WebDriverElementFinder.java:288)
at com.thoughtworks.selenium.webdriven.commands.IsElementPresent.handleSeleneseCommand(IsElementPresent.java:36)
at com.thoughtworks.selenium.webdriven.commands.IsElementPresent.handleSeleneseCommand(IsElementPresent.java:26)

@vmi vmi self-assigned this Nov 28, 2015
@vmi
Copy link
Owner

vmi commented Nov 29, 2015

Hi,

I welcome your pull request if you send it.

@vmi vmi added this to the undecided milestone Dec 1, 2015
deki added a commit to deki/selenese-runner-java that referenced this issue Dec 14, 2015
deki added a commit to deki/selenese-runner-java that referenced this issue Dec 14, 2015
deki added a commit to deki/selenese-runner-java that referenced this issue Dec 14, 2015
deki added a commit to deki/selenese-runner-java that referenced this issue Dec 14, 2015
vmi added a commit that referenced this issue Dec 14, 2015
@deki
Copy link
Contributor Author

deki commented Jan 27, 2016

Pull request is merged, so this issue can be closed.

@deki deki closed this as completed Jan 27, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants