Check for service process before driver.quit() on Windows

Author: mdmintz

seleniumbase/fixtures/base_case.py

@@ -68,6 +68,9 @@ def test_anything(self):
 logging.getLogger("urllib3").setLevel(logging.ERROR)
 urllib3.disable_warnings()
 LOGGER.setLevel(logging.WARNING)
+is_windows = False
+if sys.platform in ["win32", "win64", "x64"]:
+    is_windows = True
 python3 = True
 if sys.version_info[0] < 3:
     python3 = False
@@ -9979,7 +9982,8 @@ def quit_extra_driver(self, driver=None):
                 "Use this method only if get_new_driver() has been called."
             )
         try:
-            driver.quit()
+            if not is_windows or driver.service.process:
+                driver.quit()
         except AttributeError:
             pass
         except Exception:
@@ -11666,7 +11670,8 @@ def __quit_all_drivers(self):
         self._drivers_list.reverse()  # Last In, First Out
         for driver in self._drivers_list:
             try:
-                driver.quit()
+                if not is_windows or driver.service.process:
+                    driver.quit()
             except AttributeError:
                 pass
             except Exception:

seleniumbase/plugins/pytest_plugin.py

@@ -10,6 +10,9 @@
 from seleniumbase.config import settings
 from seleniumbase.fixtures import constants
 
+is_windows = False
+if sys.platform in ["win32", "win64", "x64"]:
+    is_windows = True
 pytest_plugins = ["pytester"]  # Adds the "testdir" fixture
 
 
@@ -1349,7 +1352,8 @@ def pytest_runtest_teardown(item):
                 and self.driver
                 and "--pdb" not in sys.argv
             ):
-                self.driver.quit()
+                if not is_windows or self.driver.service.process:
+                    self.driver.quit()
         except Exception:
             pass
         try:
@@ -1402,7 +1406,8 @@ def _perform_pytest_unconfigure_():
         # Close the shared browser session
         if sb_config.shared_driver:
             try:
-                sb_config.shared_driver.quit()
+                if not is_windows or sb_config.shared_driver.service.process:
+                    sb_config.shared_driver.quit()
             except AttributeError:
                 pass
             except Exception:

seleniumbase/plugins/selenium_plugin.py

@@ -8,6 +8,10 @@
 from seleniumbase.core import proxy_helper
 from seleniumbase.fixtures import constants
 
+is_windows = False
+if sys.platform in ["win32", "win64", "x64"]:
+    is_windows = True
+
 
 class SeleniumBrowser(Plugin):
     """
@@ -744,7 +748,8 @@ def finalize(self, result):
     def afterTest(self, test):
         try:
             # If the browser window is still open, close it now.
-            self.driver.quit()
+            if not is_windows or self.driver.service.process:
+                self.driver.quit()
         except AttributeError:
             pass
         except Exception: