Skip to content

Interface for Select component extracted - #2104

Merged
lukeis merged 5 commits into
SeleniumHQ:masterfrom
YevheniiaSilbernagl:master
May 15, 2016
Merged

Interface for Select component extracted#2104
lukeis merged 5 commits into
SeleniumHQ:masterfrom
YevheniiaSilbernagl:master

Conversation

@YevheniiaSilbernagl

@YevheniiaSilbernagl YevheniiaSilbernagl commented May 13, 2016

Copy link
Copy Markdown

This interface will allow to create custom realisation of like-select components, which actually do not have select tag inside.

This interface will allow to create custom realisation of like-select components, which actually do not have select tag inside.
@YevheniiaSilbernagl

Copy link
Copy Markdown
Author

could somebody help me to understand what is the problem with tests?

@lukeis

lukeis commented May 13, 2016

Copy link
Copy Markdown
Member

It's not the tests, you have compilation issues. First you need to add an entry in the BUCK and build.desc in the same folder. Next you should make the methods on the new interface public.

@YevheniiaSilbernagl

Copy link
Copy Markdown
Author

@lukeis Thank you for your support, I've fixed that issues, but is still fails. Sems like in a py module, but I didn't change py code, could you please take a look once more time?


java_library(name = "elements",
srcs = [
'ISelect.java',

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

double quotes required

* @return Whether this select element support selecting multiple options at the same time? This
* is done by checking the value of the "multiple" attribute.
*/
@Override

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

can you revert all the Overrides added? it's not required for implementing interfaces, only overriding methods extended.

@YevheniiaSilbernagl

Copy link
Copy Markdown
Author

@lukeis overrides removed, tests passed

@lukeis
lukeis merged commit f73c7c1 into SeleniumHQ:master May 15, 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

Successfully merging this pull request may close these issues.

2 participants