Added default executable path in service module for (#10057)

* chrome
* edge
* firefox
* ie
* safari
* webkitgtk
* wpewebkit

Co-authored-by: dosas <dosas@github.com>
Co-authored-by: David Burns <david.burns@theautomatedtester.co.uk>

Author: 3 people

py/selenium/webdriver/chrome/service.py

@@ -19,12 +19,16 @@
 from selenium.webdriver.chromium import service
 
 
+DEFAULT_EXECUTEABLE_PATH = "chromedriver"
+
+
 class Service(service.ChromiumService):
     """
     Object that manages the starting and stopping of the ChromeDriver
     """
 
-    def __init__(self, executable_path: str, port: int = 0, service_args: List[str] = None,
+    def __init__(self, executable_path: str = DEFAULT_EXECUTEABLE_PATH,
+                 port: int = 0, service_args: List[str] = None,
                  log_path: str = None, env: str = None):
         """
         Creates a new instance of the Service

py/selenium/webdriver/chrome/webdriver.py

@@ -17,7 +17,7 @@
 import warnings
 from selenium.webdriver.chromium.webdriver import ChromiumDriver
 from .options import Options
-from .service import Service
+from .service import DEFAULT_EXECUTEABLE_PATH, Service
 from selenium.webdriver.common.desired_capabilities import DesiredCapabilities
 
 
@@ -33,7 +33,7 @@ class WebDriver(ChromiumDriver):
     http://chromedriver.storage.googleapis.com/index.html
     """
 
-    def __init__(self, executable_path="chromedriver", port=DEFAULT_PORT,
+    def __init__(self, executable_path=DEFAULT_EXECUTEABLE_PATH, port=DEFAULT_PORT,
                  options: Options = None, service_args=None,
                  desired_capabilities=None, service_log_path=DEFAULT_SERVICE_LOG_PATH,
                  chrome_options=None, service: Service = None, keep_alive=DEFAULT_KEEP_ALIVE):

py/selenium/webdriver/edge/service.py

@@ -19,9 +19,13 @@
 from selenium.webdriver.chromium import service
 
 
+DEFAULT_EXECUTEABLE_PATH = 'msedgedriver'
+
+
 class Service(service.ChromiumService):
 
-    def __init__(self, executable_path: str, port: int = 0, verbose: bool = False, log_path: str = None,
+    def __init__(self, executable_path: str = DEFAULT_EXECUTEABLE_PATH,
+                 port: int = 0, verbose: bool = False, log_path: str = None,
                  service_args: List[str] = None, env=None):
         """
         Creates a new instance of the EdgeDriver service.

py/selenium/webdriver/edge/webdriver.py

@@ -17,7 +17,7 @@
 import warnings
 from selenium.webdriver.chromium.webdriver import ChromiumDriver
 from .options import Options
-from .service import Service
+from .service import DEFAULT_EXECUTEABLE_PATH, Service
 from selenium.webdriver.common.desired_capabilities import DesiredCapabilities
 
 
@@ -32,7 +32,7 @@ class WebDriver(ChromiumDriver):
     https://developer.microsoft.com/en-us/microsoft-edge/tools/webdriver/
     """
 
-    def __init__(self, executable_path='msedgedriver', port=DEFAULT_PORT,
+    def __init__(self, executable_path=DEFAULT_EXECUTEABLE_PATH, port=DEFAULT_PORT,
                  options: Options = Options(), service_args=None,
                  capabilities=None, service_log_path=DEFAULT_SERVICE_LOG_PATH,
                  service: Service = None, keep_alive=False, verbose=False):

py/selenium/webdriver/firefox/service.py

@@ -20,11 +20,15 @@
 from selenium.webdriver.common import (service, utils)
 
 
+DEFAULT_EXECUTABLE_PATH = "geckodriver"
+
+
 class Service(service.Service):
     """Object that manages the starting and stopping of the
     GeckoDriver."""
 
-    def __init__(self, executable_path: str, port: int = 0, service_args: List[str] = None,
+    def __init__(self, executable_path: str = DEFAULT_EXECUTABLE_PATH,
+                 port: int = 0, service_args: List[str] = None,
                  log_path: str = "geckodriver.log", env: dict = None):
         """Creates a new instance of the GeckoDriver remote service proxy.
 

py/selenium/webdriver/firefox/webdriver.py

@@ -28,12 +28,11 @@
 from .firefox_profile import FirefoxProfile
 from .options import Options
 from .remote_connection import FirefoxRemoteConnection
-from .service import Service
+from .service import DEFAULT_EXECUTABLE_PATH, Service
 
 
 # Default for log_path variable. To be deleted when deprecations for arguments are removed.
 DEFAULT_LOG_PATH = None
-DEFAULT_EXECUTABLE_PATH = "geckodriver"
 DEFAULT_SERVICE_LOG_PATH = "geckodriver.log"
 
 

py/selenium/webdriver/ie/service.py

@@ -20,12 +20,15 @@
 from selenium.webdriver.common import service
 
 
+DEFAULT_EXECUTEABLE_PATH = 'IEDriverServer.exe'
+
+
 class Service(service.Service):
     """
     Object that manages the starting and stopping of the IEDriver
     """
 
-    def __init__(self, executable_path: str,
+    def __init__(self, executable_path: str = DEFAULT_EXECUTEABLE_PATH,
                  port: int = 0, host: str = None,
                  log_level: str = None, log_file: str = None):
         """

py/selenium/webdriver/ie/webdriver.py

@@ -19,7 +19,7 @@
 import warnings
 
 from selenium.webdriver.remote.webdriver import WebDriver as RemoteWebDriver
-from .service import Service
+from .service import DEFAULT_EXECUTEABLE_PATH, Service
 from .options import Options
 from selenium.webdriver.common import utils
 
@@ -34,7 +34,7 @@
 class WebDriver(RemoteWebDriver):
     """ Controls the IEServerDriver and allows you to drive Internet Explorer """
 
-    def __init__(self, executable_path='IEDriverServer.exe', capabilities=None,
+    def __init__(self, executable_path=DEFAULT_EXECUTEABLE_PATH, capabilities=None,
                  port=DEFAULT_PORT, timeout=DEFAULT_TIMEOUT, host=DEFAULT_HOST,
                  log_level=DEFAULT_LOG_LEVEL, service_log_path=DEFAULT_SERVICE_LOG_PATH,
                  options: Options = None, service: Service = None,

py/selenium/webdriver/safari/service.py

@@ -20,12 +20,16 @@
 from subprocess import PIPE
 
 
+DEFAULT_EXECUTABLE_PATH = "/usr/bin/safaridriver"
+
+
 class Service(service.Service):
     """
     Object that manages the starting and stopping of the SafariDriver
     """
 
-    def __init__(self, executable_path, port=0, quiet=False, service_args=None):
+    def __init__(self, executable_path: str = DEFAULT_EXECUTABLE_PATH,
+                 port=0, quiet=False, service_args=None):
         """
         Creates a new instance of the Service
 

py/selenium/webdriver/safari/webdriver.py

@@ -24,10 +24,10 @@
 from selenium.webdriver.common.desired_capabilities import DesiredCapabilities
 from selenium.webdriver.remote.webdriver import WebDriver as RemoteWebDriver
 from .options import Options
-from .service import Service
+from .service import DEFAULT_EXECUTABLE_PATH, Service
 from .remote_connection import SafariRemoteConnection
 
-DEFAULT_EXECUTABLE_PATH = "/usr/bin/safaridriver"
+
 DEFAULT_SAFARI_CAPS = DesiredCapabilities.SAFARI.copy()