[py]: Remove redundant return clauses, use snake_case for some privat…

…e proxy methods
SeleniumHQ · Sep 3, 2022 · a7ddeca · a7ddeca
1 parent c21d220
commit a7ddeca
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 4 deletions.
diff --git a/py/selenium/webdriver/common/proxy.py b/py/selenium/webdriver/common/proxy.py
@@ -319,9 +319,9 @@ def socks_version(self, value) -> None:
         self.proxyType = ProxyType.MANUAL
         self.socksVersion = value
 
-    def _verify_proxy_type_compatibility(self, compatibleProxy):
-        if self.proxyType != ProxyType.UNSPECIFIED and self.proxyType != compatibleProxy:
-            raise Exception(f"Specified proxy type ({compatibleProxy}) not compatible with current setting ({self.proxyType})")
+    def _verify_proxy_type_compatibility(self, compatible_proxy):
+        if self.proxyType != ProxyType.UNSPECIFIED and self.proxyType != compatible_proxy:
+            raise Exception(f"Specified proxy type ({compatible_proxy}) not compatible with current setting ({self.proxyType})")
 
     def add_to_capabilities(self, capabilities):
         """

diff --git a/py/selenium/webdriver/remote/switch_to.py b/py/selenium/webdriver/remote/switch_to.py
@@ -132,7 +132,6 @@ def window(self, window_name) -> None:
                 driver.switch_to.window('main')
         """
         self._w3c_window(window_name)
-        return
 
     def _w3c_window(self, window_name):
         def send_handle(h):