Skip to content

Commit

Permalink
removed redundant getter and setter method in wpewebkit/options.py
Browse files Browse the repository at this point in the history
  • Loading branch information
sandeepsuryaprasad committed Jun 7, 2023
1 parent 84f8eb2 commit 15feb5c
Show file tree
Hide file tree
Showing 3 changed files with 2 additions and 11 deletions.
2 changes: 1 addition & 1 deletion py/selenium/webdriver/remote/errorhandler.py
Original file line number Diff line number Diff line change
Expand Up @@ -242,4 +242,4 @@ def check_response(self, response: Dict[str, Any]) -> None:
elif "alert" in value:
alert_text = value["alert"].get("text")
raise exception_class(message, screen, stacktrace, alert_text) # type: ignore[call-arg] # mypy is not smart enough here
raise exception_class(message, screen, stacktrace)
raise exception_class(message, screen, stacktrace)
8 changes: 0 additions & 8 deletions py/selenium/webdriver/wpewebkit/options.py
Original file line number Diff line number Diff line change
Expand Up @@ -27,14 +27,6 @@ def __init__(self) -> None:
self._binary_location = ""
self._caps = DesiredCapabilities.WPEWEBKIT.copy()

@property
def capabilities(self):
return self._caps

def set_capability(self, name, value) -> None:
"""Sets a capability."""
self._caps[name] = value

@property
def binary_location(self) -> str:
"""Returns the location of the browser binary otherwise an empty
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -270,5 +270,4 @@ def test_handle_errors_better(handler):
}
with pytest.raises(exceptions.WebDriverException) as e:
handler.check_response(response)

assert "Could not start a new session." in e.value.msg
assert "Could not start a new session." in e.value.msg

0 comments on commit 15feb5c

Please sign in to comment.