[python] provide element locator in the error when element is invisible

[php-coder]

When we're trying to interact with invisible element from Python library we got an exception with the following message: You may only interact with visible elements. When this message is showing to user it doesn't help him to debug this situation because it miss the information about element itself.

I'm suggesting to include element locator in the error message, for example: Element id=my-bth is invisible. You may only interact with visible elements.

(I was initially reported this to the Selenium2Library (https://github.com/robotframework/Selenium2Library/issues/769) but author has said that this is Selenium issue.)

[aneeshchandy]

Which browser are you using? If it is htmlunit driver, then you may want to file the bug here: https://github.com/SeleniumHQ/htmlunit-driver/issues.

[p0deje]

Isn't stacktrace enough to point to the element in code that is not visible?

I'm personally inclined to keep it as is, especially since it's rather easy to extend error message in frameworks (at least with Ruby), but let's hear from other committers.

[php-coder]

Which browser are you using? If it is htmlunit driver, then you may want to file the bug here:

Yes, it's htmlunit and I'm going to create issue in their bug tracker as you've suggested. Thanks!

[p0deje]

Yes, it's htmlunit and I'm going to create issue in their bug tracker as you've suggested. Thanks!

If you think this should no longer be an bug in Selenium, please close this issue.

[php-coder]

Isn't stacktrace enough to point to the element in code that is not visible?

I'm using RobotFramework with a plugin for Selenium and the only way to see this stacktrace is to enable debug level. Also I'm running tests on TravisCI and when test fails I couldn't get the report at all. A minor improvement that I'm asking makes debugging process much easier.

[p0deje]

A minor improvement that I'm asking makes debugging process much easier.

I'm not against that, however there are complications that are needed to be taken into account. For example, if we take id from element, what should we do for elements that have no id?

[php-coder]

For example, if we take id from element, what should we do for elements that have no id?

It's a good question. I suspect that it will be hard to construct something like a full path to the element (/html/body/div/ul/li[3]/span) in that case. In this case, an error message can be the same as now (without any details).