[py] added type checks in setter methods of different browser options

py/selenium/webdriver/chromium/options.py

@@ -51,6 +51,8 @@ def binary_location(self, value: str) -> None:
         :Args:
          - value: path to the Chromium binary
         """
+        if not isinstance(value, str):
+            raise TypeError("Binary Location must be a string")
         self._binary_location = value
 
     @property
@@ -69,6 +71,8 @@ def debugger_address(self, value: str) -> None:
         :Args:
          - value: address of remote devtools instance if any (hostname[:port])
         """
+        if not isinstance(value, str):
+            raise TypeError("Debugger Address must be a string")
         self._debugger_address = value
 
     @property
@@ -162,6 +166,10 @@ def headless(self, value: bool) -> None:
             stacklevel=2,
         )
         args = {"--headless"}
+
+        if not isinstance(value, bool):
+            raise TypeError("value must be a boolean")
+
         if value:
             self._arguments.extend(args)
         else:

py/selenium/webdriver/firefox/options.py

@@ -67,6 +67,8 @@ def binary_location(self) -> str:
     @binary_location.setter  # noqa
     def binary_location(self, value: str) -> None:
         """Sets the location of the browser binary by string."""
+        if not isinstance(value, str):
+            raise TypeError("Binary Location must be a string")
         self.binary = value
 
     @property
@@ -122,6 +124,8 @@ def headless(self, value: bool) -> None:
         warnings.warn(
             "headless property is deprecated, instead use add_argument('-headless')", DeprecationWarning, stacklevel=2
         )
+        if not isinstance(value, bool):
+            raise TypeError("value must be a boolean")
         if value:
             self._arguments.append("-headless")
         elif "-headless" in self._arguments:

py/selenium/webdriver/safari/options.py

@@ -59,6 +59,8 @@ def binary_location(self, value: str) -> None:
         :Args:
          - value : path to the browser binary
         """
+        if not isinstance(value, str):
+            raise TypeError("Binary location must be a string")
         self._binary_location = value
 
     def to_capabilities(self) -> dict:
@@ -97,6 +99,8 @@ def automatic_inspection(self, value: bool) -> None:
         :Args:
          - value: boolean value
         """
+        if not isinstance(value, bool):
+            raise TypeError("Automatic Inspection must be a boolean")
         self.set_capability(self.AUTOMATIC_INSPECTION, value)
 
     @property
@@ -111,6 +115,8 @@ def automatic_profiling(self, value: bool) -> None:
         :Args:
          - value: boolean value
         """
+        if not isinstance(value, bool):
+            raise TypeError("Automatic Profiling must be a boolean")
         self.set_capability(self.AUTOMATIC_PROFILING, value)
 
     @property
@@ -126,4 +132,6 @@ def use_technology_preview(self, value: bool) -> None:
         :Args:
          - value: boolean value
         """
+        if not isinstance(value, bool):
+            raise TypeError("Use Technology Preview must be a boolean")
         self.set_capability("browserName", self.SAFARI_TECH_PREVIEW if value else "safari")

py/selenium/webdriver/safari/service.py

@@ -81,4 +81,6 @@ def reuse_service(self) -> bool:
 
     @reuse_service.setter
     def reuse_service(self, reuse: bool) -> None:
+        if not isinstance(reuse, bool):
+            raise TypeError("reuse must be a boolean")
         self._reuse_service = reuse

py/selenium/webdriver/wpewebkit/options.py

@@ -34,12 +34,14 @@ def binary_location(self) -> str:
         return self._binary_location
 
     @binary_location.setter
-    def binary_location(self, value) -> None:
+    def binary_location(self, value: str) -> None:
         """Allows you to set the browser binary to launch.
 
         :Args:
          - value : path to the browser binary
         """
+        if not isinstance(value, str):
+            raise TypeError("Binary Location must be a string")
         self._binary_location = value
 
     def to_capabilities(self):