Skip to content

[updated] Add new and old capabilities/properties support to InternetExplorerDrive...#50

Closed
alex-savchuk wants to merge 3 commits intoSeleniumHQ:masterfrom
alex-savchuk:ie_driver_java
Closed

[updated] Add new and old capabilities/properties support to InternetExplorerDrive...#50
alex-savchuk wants to merge 3 commits intoSeleniumHQ:masterfrom
alex-savchuk:ie_driver_java

Conversation

@alex-savchuk
Copy link
Copy Markdown
Contributor

Add new and old capabilities/properties support to InternetExplorerDriver.
Updated to support changes in arguments names etc.

What's done:

  • add all capabilities names into InternetExplorerDriver
  • unify all constructors of InternetExplorerDriver
    • no changes in method access and parameters for compatibility
    • add max customized constructor
    • add support for all IEDriverServer-related capabilities
    • add support for all IEDriverServer-related capabilities as properties
  • fix potential problems with file paths contains spaces

This PR fixes #4525 and partially #4287 issues.

the CLA as Alex Savchuk on behalf of Mail.Ru has been signed.

…iver.

Add new and old capabilities/properties support to
InternetExplorerDriver.
update changes according to commentaries
update changes according to commentaries
@alex-savchuk
Copy link
Copy Markdown
Contributor Author

Some comments for last commit:

  • delete _CAPABILITY postfix
  • remove guard from setupProxy() method
  • use this() in all "simple" constructors
  • add guards for proxy, service and capabilities in "complex" constructor
  • replace single quotes by double
  • [additional] apply ie-switches arg only if force-createprocess arg is set as it was done in dotnet driver
  • rename capability names as usually

@alex-savchuk
Copy link
Copy Markdown
Contributor Author

Hm. About removing guard from setupProx() (what is equal to parmanent instanciating of WindowsProxyManager) -
there is issue https://code.google.com/p/selenium/issues/detail?id=4135
which requests to disable such behaviour due in some configuration access to win registry is denied
and exceptions are raised (according to issue).

need to discuss will be moved proxy code into browser driver or not. depnd on it need to enable or disable this guard, i think.

@alex-savchuk
Copy link
Copy Markdown
Contributor Author

Implemented in c4d3038

corevo added a commit to applitools/selenium that referenced this pull request Dec 11, 2017
yiming-tang-cs pushed a commit to ponder-lab/selenium that referenced this pull request Jan 2, 2020
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.

org.openqa.selenium.safari.SafariDriver#toString fails with org.openqa.selenium.WebDriverException

1 participant