add test

Author: Delta456

py/selenium/webdriver/common/service.py

@@ -65,13 +65,13 @@ def __init__(
         else:
             self.log_output = log_output
 
-        self._path = executable_path
         self.port = port or utils.free_port()
         # Default value for every python subprocess: subprocess.Popen(..., creationflags=0)
         self.popen_kw = kwargs.pop("popen_kw", {})
         self.creation_flags = self.popen_kw.pop("creation_flags", 0)
         self.env = env or os.environ
         self.DRIVER_PATH_ENV_KEY = driver_path_env_key
+        self._path = self.env_path() or executable_path
 
     @property
     def service_url(self) -> str:

py/test/selenium/webdriver/firefox/firefox_service_tests.py

@@ -16,6 +16,7 @@
 # under the License.
 import os
 import subprocess
+import pytest
 
 from selenium.webdriver import Firefox
 from selenium.webdriver.firefox.service import Service
@@ -54,3 +55,27 @@ def test_log_output_as_stdout(capfd) -> None:
     out, err = capfd.readouterr()
     assert "geckodriver\tINFO\tListening" in out
     driver.quit()
+
+@pytest.fixture
+def service():
+    return Service()
+
+@pytest.mark.usefixtures("service")
+class TestGeckoDriverService:
+    service_path = "/path/to/geckodriver"
+
+    @pytest.fixture(autouse=True)
+    def setup_and_teardown(self):
+        os.environ['SE_GECKODRIVER'] = self.service_path
+        yield
+        os.environ.pop('SE_GECKODRIVER', None)
+
+    def test_uses_path_from_env_variable(self, service):
+        assert 'geckodriver' in service.path
+
+    def test_updates_path_after_setting_env_variable(self, service):
+        new_path = "/foo/bar"
+        os.environ['SE_GECKODRIVER'] = new_path
+        service.executable_path = self.service_path  # Simulating the update
+
+        assert 'geckodriver' in service.executable_path