[py] PEP 484 type hints for selenium.webdriver.chromium.options (#10099)

iampopovich · AutomatedTester · web-flow · commit 14c76671a5aa · 2021-11-29T12:47:20.000Z
Co-authored-by: David Burns &lt;david.burns@theautomatedtester.co.uk&gt;
diff --git a/py/selenium/webdriver/chromium/options.py b/py/selenium/webdriver/chromium/options.py
@@ -27,7 +27,7 @@
 class ChromiumOptions(ArgOptions):
     KEY = "goog:chromeOptions"
 
-    def __init__(self):
+    def __init__(self) -> None:
         super(ChromiumOptions, self).__init__()
         self._binary_location = ''
         self._extension_files = []
@@ -43,7 +43,7 @@ def binary_location(self) -> str:
         return self._binary_location
 
     @binary_location.setter
-    def binary_location(self, value: str):
+    def binary_location(self, value: str) -> NoReturn:
         """
         Allows you to set where the chromium binary lives
         :Args:
@@ -52,14 +52,14 @@ def binary_location(self, value: str):
         self._binary_location = value
 
     @property
-    def debugger_address(self: str):
+    def debugger_address(self: str) -> str:
         """
         :Returns: The address of the remote devtools instance
         """
         return self._debugger_address
 
     @debugger_address.setter
-    def debugger_address(self, value: str):
+    def debugger_address(self, value: str) -> NoReturn:
         """
         Allows you to set the address of the remote devtools instance
         that the ChromeDriver instance will try to connect to during an
@@ -122,7 +122,7 @@ def experimental_options(self) -> dict:
         """
         return self._experimental_options
 
-    def add_experimental_option(self, name: str, value: Union[str, int, dict, List[str]]):
+    def add_experimental_option(self, name: str, value: Union[str, int, dict, List[str]]) -> NoReturn:
         """
         Adds an experimental option which is passed to chromium.
 
@@ -142,7 +142,7 @@ def headless(self) -> bool:
         return '--headless' in self._arguments
 
     @headless.setter
-    def headless(self, value: bool):
+    def headless(self, value: bool) -> NoReturn:
         """
         Sets the headless argument
         :Args:
