Refactor docstrings in EdgeOptions and Service classes for clarity and consistency

Author: iampopovich

py/selenium/webdriver/edge/options.py

@@ -23,20 +23,32 @@ class Options(ChromiumOptions):
     KEY = "ms:edgeOptions"
 
     def __init__(self) -> None:
+        """Initialize EdgeOptions with default settings."""
         super().__init__()
         self._use_webview = False
 
     @property
     def use_webview(self) -> bool:
+        """Returns:
+            Whether WebView2 is enabled for Edge browser.
+        """
         return self._use_webview
 
     @use_webview.setter
     def use_webview(self, value: bool) -> None:
+        """Enables or disables WebView2 support for Edge browser.
+
+        Args:
+            value: True to enable WebView2 support, False to disable.
+        """
         self._use_webview = bool(value)
 
     def to_capabilities(self) -> dict:
-        """Creates a capabilities with all the options that have been set and
-        :Returns: A dictionary with everything."""
+        """Creates a capabilities with all the options that have been set.
+
+        Returns:
+            A dictionary with all set options for Edge browser.
+        """
         caps = super().to_capabilities()
         if self._use_webview:
             caps["browserName"] = "webview2"
@@ -45,4 +57,7 @@ def to_capabilities(self) -> dict:
 
     @property
     def default_capabilities(self) -> dict:
+        """Returns:
+            The default capabilities for Edge browser.
+        """
         return DesiredCapabilities.EDGE.copy()

py/selenium/webdriver/edge/service.py

@@ -26,12 +26,13 @@ class Service(service.ChromiumService):
     """A Service class that is responsible for the starting and stopping of
     `msedgedriver`.
 
-    :param executable_path: install path of the msedgedriver executable, defaults to `msedgedriver`.
-    :param port: Port for the service to run on, defaults to 0 where the operating system will decide.
-    :param log_output: (Optional) int representation of STDOUT/DEVNULL, any IO instance or String path to file.
-    :param service_args: (Optional) Sequence of args to be passed to the subprocess when launching the executable.
-    :param env: (Optional) Mapping of environment variables for the new process, defaults to `os.environ`.
-    :param driver_path_env_key: (Optional) Environment variable to use to get the path to the driver executable.
+    Args:
+        executable_path: Install path of the msedgedriver executable, defaults to `msedgedriver`.
+        port: Port for the service to run on, defaults to 0 where the operating system will decide.
+        log_output: (Optional) int representation of STDOUT/DEVNULL, any IO instance or String path to file.
+        service_args: (Optional) Sequence of args to be passed to the subprocess when launching the executable.
+        env: (Optional) Mapping of environment variables for the new process, defaults to `os.environ`.
+        driver_path_env_key: (Optional) Environment variable to use to get the path to the driver executable.
     """
 
     def __init__(
@@ -44,6 +45,7 @@ def __init__(
         driver_path_env_key: Optional[str] = None,
         **kwargs,
     ) -> None:
+        """Initialize Edge service with the specified parameters."""
         self._service_args = list(service_args or [])
         driver_path_env_key = driver_path_env_key or "SE_EDGEDRIVER"
 
@@ -59,10 +61,21 @@ def __init__(
 
     @property
     def service_args(self) -> Sequence[str]:
+        """Returns:
+            The sequence of service arguments.
+        """
         return self._service_args
 
     @service_args.setter
     def service_args(self, value: Sequence[str]):
+        """Sets the service arguments for the Edge driver.
+
+        Args:
+            value: A sequence of strings representing service arguments.
+
+        Raises:
+            TypeError: If value is not a sequence or is a string.
+        """
         if isinstance(value, str) or not isinstance(value, Sequence):
             raise TypeError("service_args must be a sequence")
         self._service_args = list(value)